Scanf и две строки

Question

Моя задача считывается двумя строками цифр и сохраняет их в разных массивах. Я решил использовать функцию scanf, но программа может читать только первую строку. Это мой плохой код.

int main() 
{ 

    int firstArray[50], secondArray[50], i, j; 

    /* fill an array with 0 */ 
    for(i=0; i<50; ++i) 
    { 
     firstArray[i]=secondArray[i]=0; 
    } 

    i=j=0; 

    while((scanf("%d", &firstArray[i]))== 1) { ++i; } 
    while((scanf("%d", &secondArray[j]))== 1) { ++j; } 

    /* Print this. */ 
    for(i = 0; i < 20; ++i) 
    { 
     printf("%d ", firstArray[i]); 
    } 
    putchar('\n'); 

    for(j = 0; j < 20; ++j) 
    { 
     printf("%d ", secondArray[j]); 
    } 

    return 0; 
} 

Я просто не понимаю, как работает функция scanf. Может кто-нибудь объяснить?

Answer 1

Вы не можете использовать scanf для этого.

Прочитать documentation.

Наблюдения:

1. с scanf, если ввести цифру ваш цикл работает вечно
2. нет никакой проверки на размер 50 предел ваших массивов
3. если нажать return то он игнорирует эту линию, потому что не соответствует вашему шаблону
4. , если вы введете письмо, образец не соответствует, и разрывы цикла

Поэтому используйте некоторые другие функции, возможно gets, atoi или strtol. И не забудьте проверить размер 50 ваших массивов.

Answer 2

scanf игнорирует пустые символы (включая новую строку). Таким образом, ваше сканирование будет считывать весь ввод в firstArray, если у вас нет разделителя «non blank».

Если файл/данные имеют ; в конце первой строки это не остановит чтение в firstArray там, и никогда не читал что-нибудь в secondArray - как вы никогда не потребляете ;.

/* This will never be 1 as ; is blocking */ 
while((scanf("%d", &secondArray[i])) == 1) { 

Итак: если вы разделите с т ; вы должны прочитать/проверить это, прежде чем читать в secondArray.

Можно также добавить что-то вроде:

char c; 

/* this can be done more tidy, but only as concept */ 
while((scanf("%d", &firstArray[i])) == 1 && i < max) { 
    ++i; 
    if ((c = getchar()) == '\n' || c == ';') 
     break; 
} 

Также вместо инициализации массива в 0 петли вы можете сказать:

int firstArray[50] = {0}; /* This set every item to 0 */ 

принять Также уведомление, чтобы вы не идете над 50 предел.

Answer 3

На самом деле, есть одна специальная точка в массивах C.

Хотя вы объявляете размер массива. скажем int arr[5]; Вы можете хранить значения, превышающие размер . Он не показывает никаких ошибок, но приводит к неопределенному поведению (может перезаписать другие переменные).

Пожалуйста, обратитесь на этот вопрос: Array size less than the no. of elements stored in it

В вас случае, это ваша проблема. Компилятор никогда не выходил за рамки первых заявлений. Таким образом, вы не получили никакой информации. На самом деле, он даже не скомпилировал весь код!

while((scanf("%d", &firstArray[i]))== 1) { ++i; }

Таким образом, вы могли бы написать это время заявление, как это:

while( scanf("%d", &firstArray[i]) ==1 && i<50) 
    i++; 

или еще:

while(i<50) 
{ 
     scanf("%d", &firstArray[i]);    
     i++; 
} 

или еще:

for (i=0; i<50; i++) 
    scanf("%d", &firstArray[i]); 

Answer 4

Вы говорите стри ngs цифр, и вы читаете %d. Формат сканирует вход для самой длинной последовательности, представляющей целочисленное (подписанное) значение. Две «разрядные строки» потребляются первым циклом while.

EDIT Вместо «строк цифр» вы должны указать «строки целых чисел». В этом случае он немного более тонкий, поскольку первый может потреблять все целые числа, если они не разделены чем-то, что не возможное целое число (например, ;).

Итак, чтобы сделать следующее, вы должны разделить две «линии» на то, что не может быть проанализировано как целое число и, которое не считается «белым символом». Не лучшее решение, но одно возможное.

#include <stdio.h> 
#include <ctype.h> 

int main() 
{ 

    int firstArray[50] = {0}; 
    int secondArray[50] = {0}; 
    int i, j, l1, l2; 
    int tmp; 

    i = j = 0; 

    // read integers, but not more than size of array 
    while(scanf("%d", &firstArray[i]) == 1 && i < sizeof(firstArray)) { 
    ++i; 
    } 

    // consume non digits 
    for(tmp = getchar(); tmp != EOF && !isdigit(tmp); tmp = getchar()); 
    // on EOF you should exit and stop processing; 
    // we read one more char, push it back if it was a digit 
    if (isdigit(tmp)) ungetc(tmp, stdin); 

    while(scanf("%d", &secondArray[j]) == 1 && j < sizeof(secondArray)) { 
    ++j; 
    } 

    l1 = i; // preserve how many ints were read 
    l2 = j; 

/* Print this. */ 
    for(i = 0; i < l1; ++i) 
    { 
    printf("%d ", firstArray[i]); 
    } 
    putchar('\n'); 

    for(j=0; j < l2; ++j) 
    { 
    printf("%d ", secondArray[j]); 
    } 

    return 0; 
} 

EDIT Решение, которое, возможно, подходит вашим потребностям лучше читать строки (один за раз) в буфер и sscanf буфера.