Почему вывод этой программы C неверен

Question

Я написал такую ​​программу c и ожидаю получить правильные выходы.

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

int main() 
{ 
    int t; 
    scanf("%d", &t); 
    int a[t-1], b[t-1]; 
    //printf("%d\n",sizeof(a)); 
    do{ 
    scanf("%d %d", &a[t-1], &b[t-1]); 
    }while(--t); 

    do{ 
     printf("%d\n",a[t-1] + b[t-1]); 
    }while(--t); 

    return 0; 
} 

Но я получаю случайные значения. Что не так с этой программой? Попытка ввести количество попыток, затем ввести значения, а затем распечатать добавление этих значений.

P.S Инициализация массива здесь динамична. Действительно ли это? Если нет, то какой должен быть правильный путь?

Answer 1

Когда вы объявляете массив длины 10, этот показатель от 0 до 9. Для этого массива нет индекса 10.

В вашем коде, ваш массив имеет размер t-1

int a[t-1], b[t-1]; 

Таким образом, в первой итерации, оператор scanf() и printf() доступ a[t-1], который не присутствует. И, поскольку его цикл do while, он гарантированно будет работать по крайней мере один раз.

Доступ к массиву за пределами его границ - Undefined behavior. Итак, как заметил Кук в комментариях, все может случиться.

Answer 2

Между прочтением всех значений и печатью всех значений, вы никогда не возвращаете t к его первоначальному значению. Вам нужно будет это сделать, возможно, используя другую переменную, чтобы сохранить исходное значение.

Answer 3

Размер массива должен быть постоянным, его невозможно установить во время выполнения. Для распределения памяти времени выполнения вам нужно использовать переменную указателя с функцией malloc. Поэтому в первую очередь вам нужно назначить постоянное значение переменной t во время объявления.

Просто подскажите, всегда пытайтесь объявить и инициализировать переменную перед любыми утверждениями. Инициализация массива должна предшествовать scanf, делая это, вы можете четко видеть проблему, то есть вы выделяете память (некоторые случайные), а затем получаете значение от пользователя. Компилятор должен делать аналогичную вещь, выделяя некоторую память произвольного размера, а затем получая значение t, теперь это значение t будет использоваться только в ваших итерациях, а не для распределения массива, поскольку массивы распределяются раньше с некоторым размером rand (что когда-либо мусор значение t указывает на момент его объявления). Иногда программа будет отображать некоторые случайные значения, и когда-нибудь она может потерпеть крах при доступе к памяти, которая не привязана.