vérifier pseudocode. A besoin de validation pour l'attribution

voix
5

Je l'ai déjà tourné cela en sorte que vous ne contribuerez me triche. Je me demandais si cela ressemble à droite:

L'affectation: entrée une liste des noms des employés et des salaires, et déterminer la moyenne (moyenne) salaire ainsi que le nombre de salaires au-dessus et en dessous de la moyenne.

Le plan: Permettre l'entrée des noms et des salaires Calculer les valeurs moyennes Trier comptage des valeurs supérieures Numération moyenne des valeurs inférieures à moyenne

//This program will allow a user to input an employee name and salary
//The output will contain the mean salary 
//as well as the number of salaries above and below the mean
//
//Arrays Used:
//Name(K) = Array for employee names
//Salary(K) = Array for salaries
//
//Variables Used:
//Mean = Mean of all employees Salaries
//UpMean = Number of Employees making more than the mean
//DwnMean = Number of Employees making less than the mean
//Sum = Sum of all salaries
//CountM = Counter for Mean
//CountUp = Counter for # of salaries above mean
//CountDwn = Counter for # of salaries below mean

Main
    Call WelcomeMessage
    Call InputData
    Call Calculate
    Call OutputData
End Program

WelcomeMessage
    Write, “Beginning the Salary Program” 
End WelcomeMessage

InputData
    Declare Name(100) Of Strings
    Declare Salary(100) Of Real
    Declare Mean, UpMean, DwnMean As Real
    Set Sum = 0
    Set CountM = 0
    Set CountUp = 0
    Set CountDwn = 0
    Write, Enter Employee name and Salary.
    Write, Enter *,0 when done.
    Input Name(K), Salary(K)
    While Name(K) <> *
        Set CountM = CountM + 1
        Set Sum = Sum + Salary
        Write, Enter Employee name and Salary.
        Write, Enter *,0 when done.
        Input Name(K), Salary(K)
    End While
End InputData

Calculation
    //Here Mean is found
    Set Mean = Sum / CountM
    //Here Number of Employees making more than the mean is found
    For K = Step 1 to CountM
        If Salary(K) > Mean Then
            Set CountUp = CountUp + 1
        End If
    //Here Number of Employees making more than the mean is found
    Set CountDwn = CountM - CountUp
    //The above algorythm doesn't account for the possibility 
    //of someone making exactly the average so subtract 1 to reconcile
    If Salary(K) = Mean Then
            Set CountDwn = CountDwn - 1
    End If
End Calculation

OutputData
    Write, There were,  CountM, salaries entered.
    Write, The mean salary is:, Mean
    Write, There are, CountUp, employees who make more than the average
    Write, There are, CountDwn, employees who make less than the average
End OutputData
Créé 12/06/2009 à 00:02
source utilisateur
Dans d'autres langues...                            


3 réponses

voix
5

Semble bien. La seule chose que je suggère, est d'utiliser une structure do-tout en lisant le nom entrée / Salery. Comme vous pouvez le voir, vous avez la même logique avant que la boucle commence, et dans la boucle:

Write, "Enter Employee name and Salary."
Write, "Enter *,0 when done."
Input Name(K), Salary(K)

En outre, le code pseudo ne compile pas, puisque vous appelez Calculer, mais la routine est appelée calcul;)

Merci pour les suggestions. Pas encore vraiment familier avec Do-While. Qu'est-ce que cela ressemblerait? Je pensais que peut-être quelque chose à propos de l'entrée devrait changer dans la boucle, mais ne savais pas comment.

Il pourrait ressembler à ceci:

Do 
    Write, "Enter Employee name and Salary."
    Write, "Enter *,0 when done."
    Input Name(K), Salary(K)
    If Name(K) <> "*"
        Set CountM = CountM + 1
        Set Sum = Sum + Salary
    Else
        BreakLoop
    End If
End While (true)

Ce n'est pas vraiment une grande différence, mais plus une question de goût vraiment. Personnellement, je pense qu'il est plus facile à lire, puisque le code est écrit de telle manière que vous réalisez facilement que vous êtes censé quelque chose d'entrée, vérifiez l'entrée et faire quelque chose en fonction de l'entrée.

Dans votre boucle while Set CountM etc viennent après (dans le flux de texte) la première entrée, mais avant que le reste de l'entrée, ce qui signifie que vous devez regarder en arrière dans la partie supérieure de la boucle pour comprendre qu'il fait quelque chose après la précédente « rond » de la boucle. Maintenant, c'est juste une petite boucle, mais si elle était de 30 longues rangées (à Dieu ne plaise) que vous auriez à défiler vers le haut pour voir ce qui se passe. Si tu vois ce que je veux dire :)

Créé 12/06/2009 à 00:12
source utilisateur

voix
1

Une note sur le calcul CountDwn:

Votre « soustraire 1 à concilier » sera, selon la façon dont exactement l' Forerreur boucle fonctionne dans la langue de mise en œuvre, (a) générer une erreur de type « variable non déclarée », (b) générer un « indice hors de portée », ou (c ) soustraire un IFF le dernier salaire était exactement égal à la moyenne. ( En outre, votre code ne comprend pas End Fordans Calculation, mais je suppose qu'il devrait être immédiatement après la première End Ifdans cette fonction.)

Au lieu de calculer à CountDwnpartir CountUp(après tout, chaque salaire unique pourrait être égal à la moyenne), je vous suggère de l' inclure dans la boucle:

Calculation
    //Here Mean is found
    Set Mean = Sum / CountM

    For K = Step 1 to CountM
        //Here Number of Employees making more than the mean is found
        If Salary(K) > Mean Then
            Set CountUp = CountUp + 1
        End If

        //Here Number of Employees making less than the mean is found
        If Salary(K) < Mean Then
            Set CountDwn = CountDwn + 1
        End If
    End For
End Calculation

Notez que CountUp + CountDwnn'est pas nécessairement égale à CountM.

Créé 12/06/2009 à 00:50
source utilisateur

voix
0
FINAL ALGORITHM

START
OUTPUT "Enter the number of parcels"
INPUT NUMBEROFPARCELS
INTEGER PRICE = 0
INTEGER PARCELWEIGHT [1:NUMBEROFPARCELS]
INTEGER TOTALPRICE = 0

FOR PARCELLOOP = 1 TO NUMBEROFPARCELS
    INTEGER REJECT = 0
    INTEGER ACCEPT = 0
    INTEGER ACCEPTWEIGHT = 0
    INTEGER REJECTEDPARCELS = 0

    OUTPUT "Enter the weight of the parcel in kg"
    INPUT WEIGHT
    IF (WEIGHT < 1) THEN
        REJECT = REJECT + 1
        OUTPUT "The weight of the parcel should be atleast 1kg"
    ELSE
        IF (WEIGHT > 10) THEN
            REJECT = REJECT + 1
            OUTPUT "The weight of the parcel should be less than 10kg"
    ENDIF
    IF (WEIGHT > 1) THEN
        IF (WEIGHT < 10) THEN
            PARCELWEIGHT[PARCELLOOP] = WEIGHT
        ENDIF
    ENDIF


    OUTPUT "Enter the first dimension of the parcel in cm"
    INPUT DIMENSION1
    IF (DIMENSION1 > 80 ) THEN
        REJECT = REJECT + 1
        OUTPUT "Each dimension of the parcel should be less than 80"
    ENDIF

    OUTPUT "Enter the second dimension of the parcel in cm"
    INPUT DIMENSION2
    IF (DIMENSION2 > 80 ) THEN
        REJECT = REJECT + 1
        OUTPUT "Each dimension of the parcel should be less than 80"
    ENDIF

    OUTPUT "Enter the third dimension of the parcel in cm"
    INPUT DIMENSION3
    IF (DIMENSION3 > 80 ) THEN
        REJECT = REJECT + 1
        OUTPUT "Each dimension of the parcel should be less than 80"
    ENDIF

    TOTALDIMENSION = DIMENSION1 + DIMENSION2 + DIMENSION3
    IF (TOTALDIMENSION > 200 ) THEN
        REJECT = REJECT + 1
        OUTPUT "The size of the parcel should be less than 200cm"
    ENDIF

    IF (REJECT > 0 ) THEN
        OUTPUT "Your parcel has been rejected for the reasons above"
        REJECTEDPARCELS = REJECTEDPARCELS + 1
    ENDIF

    IF (REJECT = 0)THEN
        OUTPUT "Your parcel has been accepted"
        ACCEPT = ACCEPT + 1 
        ACCEPTWEIGHT = ACCEPTWEIGHT + WEIGHT
    END IF

    INTEGER PARCELSACCEPTED = ACCEPT
    INTEGER TOTALWEIGHT = ACCEPTWEIGHT
    INTEGER PARCELSREJECTED = REJECTEDPARCELS

    OUTPUT "The number of parcels accepted is " PARCELSACCEPTED " and the total weight of the parcels is " TOTALWEIGHT
    OUTPUT "The number of parcels rejected is " PARCELSREJECTED
NEXT PARCELLOOP

FOR PRICELOOP = 1 TO NUMBEROFPARCELS
    IF (PARCELWEIGHT[PARCELLOOP] < 5) THEN
        PRICE = PRICE + 10
        TOTALPRICE = TOTALPRICE +PRICE
    END IF

    IF (PARCELWEIGHT[PARCELLOOP] > 5) THEN
        PRICE = ((PARCELWEIGHT[PARCELLOOP] - 5)*0.10)/100
        TOTALPRICE = TOTALPRICE +PRICE
    END IF

    OUTPUT "The price of the parcel is " PRICE
NEXT PRICELOOP

OUTPUT "The total price of all the parcels is " TOTALPRICE
STOP
Créé 13/11/2016 à 04:29
source utilisateur

Cookies help us deliver our services. By using our services, you agree to our use of cookies. Learn more