Do while loop with Character input

Question

私はこの単純なプログラムを書いています。これはユーザーが入力した数値の階乗を計算することになっています。プログラムは、新しい番号の階乗を見つけるために、プログラムを停止または続行するようユーザーに尋ねる必要があります。

ほとんどの場合、ユーザーはCapsLockに注意を払わないため、プログラムははいの答えとしてYまたはyを受け入れる必要があります。しかし、私がこのプログラムを実行するたびに、私がY/Yに入っても、それは終了します！

私はGoogleで検索して、問題が原因new line文字が私の文字入力を受け入れてもらうには可能性が出てたので、私は改行文字に対応するためにscanf("%c ", &choice);へscanf("%c", &choice);からscanfのコードを変更しますが、私のプログラムはまだ終了しつつありますY/yを入力として受け入れた後

ここにコードがあります。可能であれば、これらの問題に対処するためのベストプラクティスと方法を必要な修正とともにお知らせください。

#include<stdio.h> 
#include"Disablewarning.h" // header file to disable s_secure warning in visual studio contains #pragma warning (disable : 4996) 

void main() { 
    int factorial=1;//Stores the factorial value 
    int i; //Counter 
    char choice;//stores user choice to continue or terminte the program 

     do {//Makes sure the loop isn't terminated until the user decides 
      do{ 
       printf("Enter the no whose factorial you want to calculate:\t"); 
       scanf("%d", &i); 
      } while (i<0); 

     if (i == 0) //calculates 0! 
      factorial = 1; 
     else {//Calculates factorial for No greater than 1; 
      while (i > 0) { 
       factorial = factorial*i; 
       i--; 
      } 
     } 

     printf("\nThe factorialof entered no is :\t%d", factorial);//prints the final result 

     printf("\nDo you want to continue (Y/N)?"); 
     scanf("%c ", &choice); 

    } while (choice =="y" || choice =="Y"); // Checks if user wants to continue 

} 

Answer 1

：それは改行や他のスペースを無視するために%cの前にスペースを追加

} while (choice == 'y' || choice == 'Y'); // Checks if user wants to continue 

注： 簡単な見積もり'は文字と二重引用符に使用されます"は文字列に使用されます

Answer 2

あなたの最後から2番目の行は、文字リテラルすなわち'y'あるべきリテラル"y"文字列を、持っているVisual Studioで、私はプログラミング初心者だと私はこのコードを実行しています：

} while (choice =="y" || choice =="Y"); 

これは次のようになります。

} while (choice =='y' || choice =='Y'); 

また、scanf()は空白を消費しません。

も

printf("\nDo you want to continue (Y/N)? "); 
scanf(" %c", &choice); //You should add the space before %c, not after 

あなたが使用する必要があります：ちょうど、次のようなあなたのscanfを変更

scanf(" %c", &choice); 

Answer 3

いくつかのバグがコードにまだ存在していても、補正後、次のコードで
あなたは「Y」を入力する場合はやってみて、階乗それが

int factorial is already loaded with the previous value

#include "stdafx.h" 
#include <stdio.h> 
#include <iostream> 

using namespace System; 
using namespace std; 

int calculateFactorial(int i); 

int main() 
{ 
    int i; 
    char choice; 

    do{ 
     printf("Enter the no whose factorial you want to calculate:\t"); 
     scanf("%d", &i); 
     printf("\n The factorial of entered no is :\t %d", calculateFactorial(i)); 
     printf("\n Do you want to continue (Y/N)?"); 
     scanf(" %c", &choice); 
    } while (choice == 'y' || choice == 'Y'); 
    return 0; 
} 

int calculateFactorial(int i) { 
    int factorial = 1; 
    if (i == 0){ 
     factorial = 1; 
    }else { 
     while (i > 0){ 
      factorial = factorial*i; 
      i--; 
     } 
    } 
    return factorial; 
}