scanf ve strtok benim için düzgün çalışmıyor

Question

Kayıtları korumak için küçük bir menü tabanlı program yazmaya çalışıyorum. Kullanıcı, saklanacak toplam kişi sayısı için kullanılacak bir sayı girer (ad, soyadı, skor). Kullanıcı tek bir satıra tümüyle girer, boşluklarla ayrılır ve onları 3 sütuna (ilk ad, soyadı, skor) bölerim, daha sonra presler girer ve maksimum kişi çarpana kadar daha fazla bilgi girmeye devam eder.

Sorunum, çalıştırdığımda düzgün çalışmıyor; kullanıcı girdisini çalıştırıyor ve kabul ediyor, ancak sadece iki öğrenci için (test durumları için 5'ten büyük rakamlar kullanmam rağmen), program hemen biter (hata kodları yok, sadece biter ...) hatta menüye getirin. Birisi benim kodumda neyin yanlış olduğunu söyleyebilir mi?

int i, j, count, numberPeople, temp, choice; 
char people[15][3], tempArr[20]; 
char *token; 

printf("Please indicate number of records you want to enter (min 5, max 15): "); 
scanf("%d", &temp); 

while ((temp > 15) || (temp < 5)) { 
    printf("\nNumber not in specified range, try again.\n"); 
    printf("Please indicate number of records you want to enter (min 5, max 15): "); 
    scanf("%d", &temp); 
} 


numberPeople = temp; 


printf("\nEnter the first name, last name, and grade (put a space in between each): "); 
    for (i = 0; i < numberPeople; i++) { 
     fgets(tempArr, 20, stdin); 

    token = strtok(tempArr, " "); 

    for (j = 0; j < 3; j++) { 
     while (token != NULL) { 
      people[i][j] = *token; 
      printf("%s\n", token); // this is here to as a test case to see if my data was being stored. 
      token = strtok(NULL, " "); 
     } 
    } 

} 

EDITED: değişmiş scanf scanf()

Ardından, (j=0; j<3; j++) bölümü için gerekmez değil fgets(),

ÇIKIŞ

Please indicate number of records you want to enter (min 5, max 15): 5 

Enter the first name, last name, and grade (put a space in between each): firstname1 lastname1 85 
firstname1 
lastname1 
85 

firstname2 lastname2 84 
firstname2 
lastname2 

Program ended with exit code: 0 

Answer 1

Bir problem ilk isim arasında boşluk, soyadı ve notu vardır tüm giriş satırına okumak için scanf() kullandığınız edilir:

for (i = 0; i < numberPeople; i++) { 
    scanf("%s", tempArr); 
} 

scanf("%s", tempArr) okuma kapanıyor yerine aşağıdaki kullanmak İlk boşluğa ulaştığında.

for (i = 0; i < numberPeople; i++) { 
    fgets(tempArr, 20, stdin); 
} 

Ama @Pooya notlar gibi, bu dize boyutu ne yaptığınızı için çok küçük: Bu döngü için, fgets() kullanmak istiyorum.

char people[15][3] 

Eğer yığın yapıyoruz, bu kavramsal olarak üçüncü bir boyut haline gelir: Eğer öğrenci ve bilgi alanlarının iki boyutlu dizi tahsis rağmen, isimlerini ve notları tutmak için dize alanı tahsis asla:

scanf("%d", &temp); 

muhtemelen temizlenmesi gerekir ou: Bu scanf() sonra

char people[15][3][24] 

hala tampon bırakılan bir dönüş karakteri var t. Burada bir probleme @KevinDTimm ve @bruceg ipucu:

for (j = 0; j < 3; j++) { 
    while (token != NULL) { 
     people[i][j] = *token; 

Ama Kevin'in öneri endeksi j hesapları sanmıyorum. @Weather_Vane strtok() ayırıcı dizeye \r\n eklemenizi öneriyor:

token = strtok(NULL, " ") 

Aksi takdirde notu dize (son alan) bir sarkan yeni satır olacaktır. Ayrıca, strtok() tarafından döndürülen jetonun bir kopyasını oluşturmanız gerekir, doğrudan saklamamalısınız.girişinizi yaklaşık 24 karakter alır ama tempArr[20] tanımlı senin ölçümlere göre

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

#define MINIMUM_STUDENTS 5 
#define MAXIMUM_STUDENTS 15 
#define DATA_FIELDS 3 

#define MAXIMUM_LINE_LENGTH 100 
#define MAXIMUM_DATA_LENGTH 50 

int main(int argc, char **argv) { 

    int numberPeople; 
    char people[MAXIMUM_STUDENTS][DATA_FIELDS][MAXIMUM_DATA_LENGTH]; 

    printf("Please indicate number of records you want to enter (min %d, max %d): ", MINIMUM_STUDENTS, MAXIMUM_STUDENTS); 

    scanf("%d", &numberPeople); 

    while ((numberPeople < MINIMUM_STUDENTS) || (numberPeople > MAXIMUM_STUDENTS)) { 
     printf("\nNumber not in specified range, try again.\n"); 
     printf("Please indicate number of records you want to enter (min %d, max %d): ", MINIMUM_STUDENTS, MAXIMUM_STUDENTS); 
     scanf("%d", &numberPeople); 
    } 

    printf("\n"); 

    while ((getchar()) != '\n'); // flush the return (and anything else) after the number input above 

    printf("Enter the first name, last name, and grade (put a space in between each): \n"); 

    for (int i = 0; i < numberPeople; i++) { 
     char tempArr[MAXIMUM_LINE_LENGTH]; 

     fgets(tempArr, MAXIMUM_LINE_LENGTH, stdin); 

     char *token = strtok(tempArr, " "); 

     for (int j = 0; j < DATA_FIELDS && token != NULL; j++) { 
      strncpy(people[i][j], token, MAXIMUM_DATA_LENGTH); 

      // this is here to as a test case to see if my data was being stored. 
      printf("%s\n", people[i][j]); 

      token = strtok(NULL, " \r\n"); 
     } 

    } 

    // do what you need to do with the data here! 

    return 0; 
} 

Answer 2

Öncelikle fgets kullanmak, Bu strtok tarafından ele alınmıştır.

token = strtok(tempArr, " "); 
while (token != NULL) { 
    people[i][j] = *token; 
    printf("%s\n", token); // this is here to as a test case to see if my data was being stored. 
    token = strtok(NULL, " "); 
} 

Answer 3

:

hep birlikte bu önerileri koymak ve ben koştu başka bir şey sabitleme/temizlik, aşağıdaki rework sunuyoruz. 20 karakterden daha fazla bir şey tararsanız bellekte başka bir şey üzerine yazdığınızdan, tempArr[100] veya testleriniz için anlamlı olan herhangi bir sayıya sahip olmak daha iyidir.