1

入力をscanf受け付けない があります。変数が初期化されていない場合でも、値は自動的にゼロになります。はscanfスキップされます:

printf("\nEnter the number of the student to be dropped: ");
fflush(stdin);
scanf(" %d ",&choice);
printf("choice is %d", choice);

プログラムを実行すると、すぐに「choice is 0」と表示されます。

上記のスニペットは、次のコードの drop() 関数から取得したものです。

    #include <stdio.h>
#include <stdlib.h>
#include <string.h>

struct student{
       char name[50];
       char* course;
};

main()
{
      char repeat;
      do{
      system("cls");
      int mainchoice;
      printf("Student Enrollment System");
      printf("\n");
      printf("\n");
      printf("1. View\n");
      printf("2. Enroll\n");
      printf("3. Drop enrollment\n");
      printf("Select choice: ");
      fflush(stdin);
      scanf("%d",&mainchoice);
      system("cls");
      switch(mainchoice){
             case 1:
                  view();
                  break;
             case 2:
                  enroll();
                  break;
             case 3:
                  drop();
                   break;
             default:
                     printf("Please enter a valid number.");
                     getch();
                     fflush(stdin);
                     break;
             }

      printf("\nWould you like to make another transaction? [Y/N]: ");
      fflush(stdin);
      scanf("%c",&repeat);
      }while(repeat=='Y'||repeat=='y');     
}

view(){
       int ctr = count();
       printf("Enrolled Students:\n\n");
       system("type records.txt");
       printf("\n\nNumber of students enrolled: %d", ctr);
       getch();
       fflush(stdin);

}

enroll(){
         int choice;
         char validate;
         printf("1. Information Technology\n");
         printf("2. Computer Science\n");
         printf("3. Computer Engineering\n");
         printf("4. Information Systems\n");
         struct student news;
         printf("Name: ");
         fflush(stdin);
         gets(news.name);
         printf("Course Number: ");
         fflush(stdin);
         scanf("%d", &choice);
         switch(choice){
                        case 1:
                             news.course = "BSIT";
                             break;
                        case 2:
                             news.course= "BSCS";
                             break;
                        case 3:
                             news.course = "BSCpE";
                             break;
                        case 4:
                             news.course = "BSIS";
                             break;
                        default:
                                printf("Please enter a valid number\n");
                                break;
                        }
         printf("Enroll %s to %s? [Y/N]:",news.name,news.course);
         fflush(stdin);
         scanf("%c", &choice);
         if(choice=='Y' || choice=='y')
         {
          FILE * records;
          records = fopen("records.txt", "a+");
          fprintf(records, "%s, %s\n",news.name,news.course);
          fclose(records);
          printf("%s has been enrolled to %s\n",news.name, news.course);

         }
         else
         {
             printf("You have chosen to cancel your transaction");
         }
}

drop(){
       printf("Drop Student:\n\n");
       int ctr = 0;
       int choice; //which student to delete
       char c;
       FILE * record; // original records.txt
       FILE* repo;    //temporary data storage


       record = freopen("records.txt", "r", stdin);
       while((c = fgetchar())!=EOF){
                if(c == '\n'){

                }
                else{
                     ctr=ctr+1;
                     printf("%d.  ", ctr);
                     while(1){       
                                     printf("%c",c);
                                     c= fgetchar();
                                     if(c=='\n'){
                                                 printf("%c",c);
                                                 break;
                                                 }

                     }  

                }              
       }
       fclose(record);
       fflush(stdin);
       fflush(stdin);
       printf("\nEnter the number of the student to be dropped: ");
       fflush(stdin);
       scanf(" %d ",&choice);
       getch();
       getch();
       fflush(stdin);
       ctr = 1;
       fflush(stdin);

       repo = fopen("temp.txt","w");
       record = freopen("records.txt","r",stdin);
       while((c = getchar()) != EOF){
                if(c == '\n'){

                }
                else{

                     while(ctr!=choice){       
                                     fprintf(repo,"%c",c);
                                     c= fgetchar();
                                     if(c=='\n'){
                                                 fprintf(repo,"%c",c);
                                                 ctr = ctr + 1;
                                                 break;
                                                 }
                                     }
                     }
                }
       fclose(record);      
       fclose(repo);

       getch();
}

//counts the number of rows in the record
int count(){
       int ctr=0;
       char c;
       FILE * records;
       records = freopen("records.txt","r", stdin);
       if(records!=NULL){
            while((c=fgetchar()) !=EOF){
                if(c=='\n'){
                    ctr = ctr+1;
                    }
                }                      
       }
       fclose(records);   
       return ctr; 
}

することは役に立たfflushないようです。何か案は?

4

1 に答える 1

6

の動作はfflush入力ストリームに対して定義されていません。fflush(stdin)はコーディング エラーであるため、これらの呼び出しをコードから削除する必要があります。

%c個々の文字をスキャンするときは、変換指定子の前に空白を追加します。これscanfにより、先頭の空白をスキップして、次の非空白文字を読み取るように指示されます。

scanf(" %c", &choice);

%dand変換指定子は%s、先頭の空白をスキップします。

編集

暗黙の型付けは C99 でサポートされなくなりました。これは悪い習慣です。関数を明示的に入力voidし、パラメーター リストとして使用して、引数を取らないことを指定します。

main() => int main(void)
view() => void view(void) // void since it isn't returning a value
drop() => void drop(void)

同様に、getsC99 で廃止され、2011 標準の時点で完全になくなりました。それ使用すると、プログラムに障害点/主要なセキュリティ ホールが発生します。fgets代わりに 使用してください。

于 2013-02-21T04:03:39.607 に答える