En moyenne, max et min programme en C

Donc je code en C, et j'ai besoin de venir avec un code qui va prendre la n des nombres de l'utilisateur, et à trouver leur minimum, maximum, moyenne, et la somme des carrés de leurs valeurs. Pour l'instant j'ai la moyenne et la somme des carrés de la partie, mais le minimum et le maximum est de mordre.

Garder à l'esprit que je suis très rudimentaire, et je n'ai pas atteint les tableaux encore. Tout ce que je sais sont des opérateurs logiques, fonctions, boucles, et l'utilisation de la stdlib.h, math.h et stdio.h les bibliothèques.

C'est ce que j'ai jusqu'à présent.
La fonction moyenne m'a donné beaucoup de problèmes quand j'ai essayé de mettre float et double lors de la compilation, donc, de le multiplier par un 1.0 fixe. J'ai tout juste le minimum et le maximum. Je reçois la dernière entrée que mon maximum, et un 0 pour mon minimum.

#include<stdio.h>
int main()
{
    float average;
    int i, n, count=0, sum=0, squaresum=0, num, min, max;


    printf("Please enter the number of numbers you wish to evaluate\n");
    scanf_s("%d",&n);

    printf("Please enter %d numbers\n",n);

    while(count<n)
    {
        min=0;
        max=0;

            if(num>max)
               max=num;
            if(num<min)
               min=num;

            scanf_s("%d",&num);

        sum = sum+num;
        squaresum = squaresum + (num*num);

        count++;
    }
        average = 1.0*sum/n;

    printf("Your average is %.2f\n",average);
    printf("The sum of your squares is %d\n",squaresum);    



    printf("Your maximum number is %d\n",max);
    printf("Your minimum number is %d\n",min);

return(0);
}

OriginalL'auteur user3133973 | 2013-12-25

  1. 3

    Votre algorithme est pas tout à fait droit. Ci-dessous est la mise en œuvre correcte:

    #include <stdio.h>
#include <stdlib.h>
#include <limits.h>
int main(void)
{
float average;
int n, num, count = 0, sum = 0, squaresum = 0;
int min = INT_MAX, max = INT_MIN;
bool gotAnswer = false;
/* Don't Let User Enter Wrong Input */
while(!gotAnswer)
{
printf("Please enter the number of numbers you wish to evaluate: ");
if(scanf_s("%d", &n) != 1)
{
/* User Entered Wrong Input; Clean Up stdin Stream*/
while(getchar() != '\n')
{
continue;
}
}
else
{
/* User Input Was Good */
gotAnswer = true;
}
}
/* Clear stdin Stream Just In Case */
while(getchar() != '\n')
continue;
while(count < n)
{
/* Don't Let User Enter Wrong Input */
gotAnswer = false;
printf("Enter number %d: ", count + 1);
if(scanf_s("%d", &num) != 1)
{
/* User Entered Wrong Input; Clean Up stdin Stream */
while(getchar() != '\n')
continue;
/* Let User Try Again */
continue;
}
else
{
/* User Input Was Correct */
gotAnswer = true;
/* Clear stdin Stream Just In Case */
while(getchar() != '\n')
continue;
}
if(num > max)
max = num;
if(num < min)
min = num;
sum += num;
squaresum += num * num;
count++;
}
average = 1.0 * sum / n;
printf("Your average is %.2f\n", average);
printf("The sum of your squares is %d\n", squaresum);    
printf("Your maximum number is %d\n", max);
printf("Your minimum number is %d\n", min);
system("pause");
return 0;
}

    J'ai ajouté la vérification des erreurs et de récupération. Veuillez vous demander si vous avez des questions au sujet de la logique.

    +1 pour la vérification des résultats de scanf() et à l'aide de min = INT_MAX, max = INT_MIN.
    Seulement parce que je savais que vous feriez sûr que je n'ai droit. 😀
    "Ho! ho! ho!"
    Vous n'obtenez ma liste de cadeaux de Noël! Merci Santa Chux!

    OriginalL'auteur Fiddling Bits

  2. 2

    La façon dont votre code est actuellement écrit, min doit commencer à une valeur élevée (pas de 0), ou le code ne fonctionne pas. La meilleure valeur est la valeur maximale possible pour une int.

    Vous devriez également considérer si oui ou non vous souhaitez réinitialiser ces variable à chaque passage dans la boucle.

    OriginalL'auteur Robert Harvey

  3. 1

    Entrez le premier num en dehors de la boucle et affectez-le à max min 

    scanf("%d",&num);
max = min = num;

    Changer votre lors de la boucle pour boucle infinie 

    while(1) {...}

    et maintenant vérifier la condition que si votre compteur count est égal à n est ou de ne pas break de la boucle infinie 

    if(count == n)
break;

    Code complet après modification: 

    #include<stdio.h>
int main()
{
float average;
int i, n, count=0, sum=0, squaresum=0, num, min, max;
printf("Please enter the number of numbers you wish to evaluate\n");
scanf_s("%d",&n);
printf("Please enter %d numbers\n",n);
scanf_s("%d",&num);
max = min = num;
while(1)
{
if(num>max)
max=num;
if(num<min)
min=num;
sum = sum+num;
squaresum = squaresum + (num*num);
count++;
if(count == n)
break;
scanf_s("%d",&num);
}
average = 1.0*sum/n;
printf("Your average is %.2f\n",average);
printf("The sum of your squares is %d\n",squaresum);
printf("Your maximum number is %d\n",max);
printf("Your minimum number is %d\n",min);
return(0);
}

    OriginalL'auteur haccks

  4. 1

    Assumer votre premier nombre de la liste que le minimum et le maximum.
    Comparer chaque caractère suivant avec le minimum et le maximum actuel et de mettre à jour en conséquence.

    OriginalL'auteur udit bansal

  5. 0

    votre boucle while doit ressembler à

       min=3;
max=0;
while(count<n)
{ 
scanf("%d",&num);
if(num>max)
max=num;
if(num<min)
min=num;
sum = sum+num;
squaresum = squaresum + (num*num);
count++;
}

    Et je suis d'accord avec Robert Harvey♦.. Vous devez définir min

    Qu'avez-vous changé?
    et max sont hors de la boucle

    OriginalL'auteur AJ.

  6. 0

    Ajouter un booléen, déplacé de donner les valeurs min, max 0 sont le début de la boucle de la

    #include<stdio.h>
int main()
{
float average;
int i, n, count=0, sum=0, squaresum=0, num, min, max;
bool first = true;
printf("Please enter the number of numbers you wish to evaluate\n");
scanf_s("%d",&n);
printf("Please enter %d numbers\n",n);
min=0;
max=0;
while(count<n)
{
scanf_s("%d",&num);
if (first) {
first = false;
min = max = num;
}
if(num>max)
max=num;
if(num<min)
min=num;
sum = sum+num;
squaresum = squaresum + (num*num);
count++;
}
average = 1.0*sum/n;
printf("Your average is %.2f\n",average);
printf("The sum of your squares is %d\n",squaresum);    
printf("Your maximum number is %d\n",max);
printf("Your minimum number is %d\n",min);
return(0);
}

    Devrait également envisager de vérifier la valeur de retour de scanf

    OriginalL'auteur Ed Heal

  7. 0

    Il y a quelques problèmes dans votre code:

    1. Où num est lire? Vous devriez le faire avant min et max
    2. Quand boucle while exécute première fois vous devriez juste attribuer num à max et min.

    Quelque chose comme ça:

      int min = 0;
int max = 0;
//If your compiler supports C99 standard you can put
//bool first_time = true;
int first_time = 1;
while (count < n) {
printf("Please, enter the next number\n");
scanf_s("%d", &num); 
//If your compiler supports C99 you can put it easier:
//if (first_time) { 
if (first_time == 1) {
first_time = 0;
max = num;
min = num;
}
else {
if(num > max)
max = num;
if(num < min)
min = num;
}
...
    Pourquoi ne pas boolean?
    Guérir: booléen (bool) est apparu en C99 seulement; car il n'y a pas d'information dans la question qui C standard est utilisé, j'ai mis en int
    Nous sommes cogner en 2014 si l'on suppose que l'on a un peu plus moderne de compilateur que ça!
    Dire à tous les gens à l'aide de Microsoft compilateur pour passer les compilateurs si elles veulent apprendre quelque chose au-delà de C89! Vite! 😛

    OriginalL'auteur Dmitry Bychenko