La programmation en C - Boucle jusqu'à ce que les entrées de l'utilisateur numéro de scanf

J'ai besoin d'aide avec la vérification des erreurs pour mon programme. Je demande à l'utilisateur de saisir un entier et je voudrais vérifier si les utilisateurs d'entrée est un nombre entier. Sinon, répétez le scanf.

Mon code:

int main(void){

  int number1, number2;
  int sum;

  //asks user for integers to add
  printf("Please enter the first integer to add.");
  scanf("%d",&number1);

  printf("Please enter the second integer to add.");
  scanf("%d",&number2);
  //adds integers
  sum = number1 + number2;

  //prints sum
  printf("Sum of %d and %d = %d \n",number1, number2, sum);

  //checks if sum is divisable by 3
  if(sum%3 == 0){
    printf("The sum of these two integers is a multiple of 3!\n");
  }else {
    printf("The sum of these two integers is not a multiple of 3...\n");
  }
  return 0;
}

Lisez à propos de for, while et do-while boucle.
stackoverflow.com/questions/14099473/...
Aussi, renseignez-vous sur scanf(3). Il revient sur le succès, le nombre de succès éléments numérisés!
Lire aussi à propos de scanf, surtout de quoi il en retourne.
scanf() le type de retour peut être vérifiée. plz reportez-vous cela en détail

OriginalL'auteur Asia x3 | 2014-09-10

c if-statement scanf

7

scanf retourne le nombre d'éléments qu'elle a réussi à lire en fonction de votre format. Vous pouvez créer une boucle qui se ferme que lorsque scanf("%d", &number2); retourne 1. Le truc, cependant, est d'ignorer des données non valides lorsque scanf renvoie zéro, de sorte que le code devrait ressembler à ceci:
```
while (scanf("%d",&number2) != 1) {
    //Tell the user that the entry was invalid
    printf("You did not enter a valid number\n");
    //Asterisk * tells scanf to read and ignore the value
    scanf("%*s");
}
```
Puisque vous lire un nombre plus d'une fois dans votre code, pensez à faire une fonction pour cacher cette boucle, et d'appeler cette fonction deux fois dans votre main pour éviter la duplication.

OriginalL'auteur dasblinkenlight

Ici est une solution de votre problème. J'ai juste modifié quelques-uns de votre code.
Lire les commentaires de toutes les explications.

#include<stdio.h>
#include<stdlib.h>      //included to use atoi()
#include<ctype.h>       //included to use isalpha()
#define LEN 3   //for two digit numbers
int main(void)
{
char *num1=malloc(LEN);
char *num2=malloc(LEN);
int i,flag=0;
int number1,number2;
int sum;
do
{
printf("Please enter the first integer to add = ");
scanf("%s",num1);
for (i=0; i<LEN; i++)   //check for every letter of num1
{
if (isalpha(num1[i]))   //isalpha(num1[i]) returns true if num1[i] is alphabet
{                       //isalpha() is defined in ctype.h
flag=1;             //set flag to 1 if num1[i] is a alphabet
}
}
if(flag)
{
printf("Not a valid Integer\n");
flag=0;
continue;
}
else
{
break;
}
} while(1);
do
{
printf("Please enter the second integer to add = ");
scanf("%s",num2);
for (i=0; i<LEN; i++)
{
if (isalpha(num2[i]))
{
flag=1;
}
}
if(flag)
{
printf("Not a valid Integer\n");
flag=0;
continue;
}
else
{
break;
}
} while(1);
//strings to integers
number1= atoi(num1);    //atoi() is defined in stdlib.h
number2= atoi(num2);
//adds integers
sum = number1 + number2;
//prints sum
printf("Sum of %d and %d = %d \n",number1, number2, sum);
//checks if sum is divisable by 3
if(sum%3 == 0)
{
printf("The sum of these two integers is a multiple of 3!\n");
}
else
{
printf("The sum of these two integers is not a multiple of 3...\n");
}
return 0;
}

J'ai conçu ce que pour les numéros à deux chiffres, mais il fonctionne très bien pour plus de numéros à deux chiffres pour moi.
S'il vous plaît laissez-moi savoir que la même chose se passe dans votre cas.

Et si vous trouvez pourquoi ce qui se passe s'il vous plaît commentaire.

Et vous pouvez également utiliser strtol() au lieu de atoi(). Je ne suis pas à l'utiliser car de petites valeurs.

Différence entre atoi() et strtol()

atoi()

Pro: Simple.

Pro: Convertir un int.

Pro: Dans la bibliothèque C standard.

Pro: Rapide.

Con: Pas d'erreur de manipulation.

Con: Poignée ni hexadécimal ni octal.

strtol()

Pro: Simple.

Pro: Dans la bibliothèque C standard.

Pro: une Bonne gestion des erreurs.

Pro: Rapide.

Con: Convertir un long, pas int qui peuvent varier en taille.

OriginalL'auteur Sachin

Je tiens à dire que vous avez à faire un peu de validation personnalisée pour vérifier si scanf lire en entier ou pas.Je suis habitué fgets pas intéressé dans scanf.

#include <stdio.h> 
#include <ctype.h> 
#include <stdlib.h> 
#include <string.h> 
int validate ( char *a )
{
unsigned x;
for ( x = 0; x < strlen ( a ); x++ )
if ( !isdigit ( a[x] ) ) return 1;
return 0;
}
int main ( void )
{
int i;
char buffer[BUFSIZ];
printf ( "Enter a number: " );
if ( fgets ( buffer, sizeof buffer, stdin ) != NULL ) {
buffer[strlen ( buffer ) - 1] = '\0';
if ( validate ( buffer ) == 0 ) {
i = atoi ( buffer );
printf ( "%d\n", i );
}
else
printf ( "Error: Input validation\n" );
}
else
printf ( "Error reading input\n" );
return 0;
}

L'argument de isdigit doit être jeté à la unsigned char

OriginalL'auteur Jayesh Bhoi

0

Propre approche de ce problème peut être
1. lire à partir de stdin à l'aide de fgets().
2. utilisation strtol() pour convertir et stocker la valeur dans une int. Ensuite, vérifiez la char **endptr pour déterminer si la conversion est un succès [integer] ou pas.
3. Effectuer la tâche restante.
+1 pour strtol()
À l'aide de fgets() est une bonne approche, mais il n'y a pas besoin d'utiliser isdigit() pour "valider"; juste essayer avec strtol() et examiner sa "fin" pointeur pour voir si elle a consommé de l'entrée.
ouais, vous avez raison. Modifié la réponse.

OriginalL'auteur Sourav Ghosh

Vous devez vous connecter pour publier un commentaire.