В определенный момент массив почему-то обнуляется

@fingolfin_ · Регистрация: 19.08.2013

Студворк — интернет-сервис помощи студентам

Вот листинг:

Кликните здесь для просмотра всего текста

C++

#include <fstream> 
#include <iostream> 
using namespace std;
double _AverageGain[7776][45];
 
void main()
{
    int Digit[5];
    ifstream fin("d:\\File.txt"); // В файле дробные числа через пробел
    for (Digit[0] = 0; Digit[0] < 6; Digit[0]++)
        for (Digit[1] = 0; Digit[1] < 6; Digit[1]++)
            for (Digit[2] = 0; Digit[2] < 6; Digit[2]++)
            {
                for (Digit[3] = 0; Digit[3] < 6; Digit[3]++)
                    for (Digit[4] = 0; Digit[4] < 6; Digit[4]++)
                        for (int j = 0; j < 45; j++)
                            fin >> _AverageGain[Digit[0] * 6 * 6 * 6 + Digit[1] * 6 * 6 + Digit[2] * 6 + Digit[3]][j];
                cout << _AverageGain[Digit[0] * 6 * 6 * 6 + Digit[1] * 6 * 6 + Digit[2] * 6 + Digit[3]][0] << endl; // Вместо
                                                                       // элемента массива, полученного из файла, выводит ноль
            }
    system("pause");
}

Но если переместить вывод на один уровень вложенности ниже, но всё работает:

Кликните здесь для просмотра всего текста

C++

#include <fstream> 
#include <iostream> 
using namespace std;
double _AverageGain[7776][45];
 
void main()
{
    int Digit[5];
    ifstream fin("d:\\File.txt"); // В файле дробные числа через пробел
    for (Digit[0] = 0; Digit[0] < 6; Digit[0]++)
        for (Digit[1] = 0; Digit[1] < 6; Digit[1]++)
            for (Digit[2] = 0; Digit[2] < 6; Digit[2]++)
                for (Digit[3] = 0; Digit[3] < 6; Digit[3]++)
                {
                    for (Digit[4] = 0; Digit[4] < 6; Digit[4]++)
                        for (int j = 0; j < 45; j++)
                            fin >> _AverageGain[Digit[0] * 6 * 6 * 6 + Digit[1] * 6 * 6 + Digit[2] * 6 + Digit[3]][j];
                    cout << _AverageGain[Digit[0] * 6 * 6 * 6 + Digit[1] * 6 * 6 + Digit[2] * 6 + Digit[3]][0] << endl;
                                                                   // А так - всё прекрасно, выводит число из файла
                }
    system("pause");
}

Почему это происходит и как это победить? Если что, компиляю в VS 2017.

@nonedark2008 · 14.10.2017, 23:25

fingolfin_, ты уверен, что в файле не менее 349920 чисел?

@fingolfin_ · 14.10.2017, 23:34 **[ТС]**

nonedark2008, да, ровно 349920. Не думаю, что проблема в файле, так как он заполнялся таким же циклом с помощью ofstream.

Добавлено через 5 минут
В принципе, я бы мог работать напрямую с массивом и не использовать файл, но мне интересно. Лично для меня это очень странная и необычная проблема.

@DrOffset · 15.10.2017, 00:08

Сообщение от fingolfin_

C++

            for (Digit[2] = 0; Digit[2] < 6; Digit[2]++)
            {
                for (Digit[3] = 0; Digit[3] < 6; Digit[3]++)
                    for (Digit[4] = 0; Digit[4] < 6; Digit[4]++)
                        for (int j = 0; j < 45; j++)
                            fin >> _AverageGain[Digit[0] * 6 * 6 * 6 + Digit[1] * 6 * 6 + Digit[2] * 6 + Digit[3]][j];
                cout << _AverageGain[Digit[0] * 6 * 6 * 6 + Digit[1] * 6 * 6 + Digit[2] * 6 + Digit[3]][0] << endl; 
            }

Наводящий вопрос: чему на момент вывода в cout равно Digit[3]?

@fingolfin_ · 15.10.2017, 00:52 **[ТС]**

Сообщение от DrOffset

Наводящий вопрос: чему на момент вывода в cout равно Digit[3]?

Я правильно понимаю, что будет равен 6? Оказывается, всё так банально

. Но ведь я пробовал разные варианты...
Но теперь у меня возникло еще 3 вопроса:
1) Почему VS не ругается на использование неинициализированного элемента массива?
2) Почему расположение cout влияет на результат? Ведь если положить, что еще неинициализированный элемент будет выводиться как 0, то 0 должен быть в обоих случаях.
3) Насколько я понимаю, этот код

C++

1 2	for (int i = 0; i < 45; i++) cout << _AverageGain[0][i] << " ";

должен вывести первые 45 чисел из файла, но на деле выводятся какие-то другие числа.

@DrOffset · 15.10.2017, 01:06

Сообщение от fingolfin_

Почему VS не ругается на использование неинициализированного элемента массива?

Потому что он не обязан ругаться. Язык с возможностью прямой работы с памятью требует дисциплины и ответственности от программиста.

Сообщение от fingolfin_

Почему расположение cout влияет на результат?

Потому что во втором твоем примере корректный результат всех трех Digit[0], Digit[1], Digit[2] и Digit[3] контролируется условиями циклов.

Сообщение от fingolfin_

если положить, что неинициализированный элемент будет выводиться как 0, то 0 должен быть в обоих случаях.

Нет, так нельзя положить. Вывод неициализированного значения - это UB.

Сообщение от fingolfin_

должен вывести первые 45 чисел из файла, но на деле выводятся какие-то другие числа.

Это потому, что у тебя есть еще один цикл

Сообщение от fingolfin_

C++

1	for (Digit[4] = 0; Digit[4] < 6; Digit[4]++)

Сообщение от fingolfin_

Но ведь я пробовал разные варианты...

Нельзя программировать методом тыка. Не можешь удержать в голове работу с таким большим массивом (что вполне резонно), сделай массив обозримо меньше и распиши на бумажке итерации.

@fingolfin_ · 15.10.2017, 14:59 **[ТС]**

Спасибо, теперь все проблемы решены!

Сообщение от DrOffset

Нельзя программировать методом тыка.

Почему же нельзя? Написать код, попробовать собрать, запустить, потом исправить ошибки и по новой, нет? У меня пока очень редко получается, что бы написанный код сразу работал без ошибок. Обычно я такие ошибки по невнимательности сразу сам исправляю, но здесь две ошибки (неучитывание Digit[4] в выражении и вывод неинициализированного значения) наложились и совсем сбили меня с толку. Спасибо за разъяснения!

@nonedark2008 · 15.10.2017, 15:13

Сообщение от fingolfin_

потом исправить ошибки и по новой

Метод тыка - это когда ты не знаешь, где находится ошибка, и начинаешь исправлять случайные места.
Как раз твой случай:

Сообщение от fingolfin_

Но если переместить вывод на один уровень вложенности ниже, но всё работает

Ты что-то поменял в случайном месте, а потом уверовал, что "все" стало работать. Такой подход приводит к еще большему непониманию и еще большему количеству ошибок.

@DrOffset · 15.10.2017, 15:14

Сообщение от fingolfin_

Почему же нельзя? Написать код, попробовать собрать, запустить, потом исправить ошибки и по новой, нет? У меня пока очень редко получается, что бы написанный код сразу работал без ошибок.

Потому что это контрпродуктивно. И делает тебя зависимым от случая. Верно работающий код не гарантия, что в нем нет ошибок. UB может и совершенно ожидаемым образом проявляться, создавать видимость работы (до поры). Поэтому ты должен понимать от и до что у тебя в программе происходит, а не проверять корректность методом тыка. И дело не в возможности писать код сразу без ошибок, речь не об этом, а в возможности понять, что у тебя в коде не так не только на основе многочисленных проб, а на основе понимания того как программа работает и должна работать. Поэтому, если ты не понимаешь как она работает, надо сесть и разобраться, а не мучить компилятор запусками в надежде, что сработает.

@nonedark2008 1392 / 1021 / 325 Регистрация: 28.07.2012 Сообщений: 2,810
	14.10.2017, 23:25	2
	fingolfin_, ты уверен, что в файле не менее 349920 чисел? 0

@fingolfin_ 4 / 4 / 0 Регистрация: 19.08.2013 Сообщений: 138
	14.10.2017, 23:34 [ТС]	3
	nonedark2008, да, ровно 349920. Не думаю, что проблема в файле, так как он заполнялся таким же циклом с помощью ofstream. Добавлено через 5 минут В принципе, я бы мог работать напрямую с массивом и не использовать файл, но мне интересно. Лично для меня это очень странная и необычная проблема. 0

@fingolfin_ 4 / 4 / 0 Регистрация: 19.08.2013 Сообщений: 138
	15.10.2017, 14:59 [ТС]	7
	Спасибо, теперь все проблемы решены! Сообщение от DrOffset Нельзя программировать методом тыка. Почему же нельзя? Написать код, попробовать собрать, запустить, потом исправить ошибки и по новой, нет? У меня пока очень редко получается, что бы написанный код сразу работал без ошибок. Обычно я такие ошибки по невнимательности сразу сам исправляю, но здесь две ошибки (неучитывание Digit[4] в выражении и вывод неинициализированного значения) наложились и совсем сбили меня с толку. Спасибо за разъяснения! 0

@nonedark2008 1392 / 1021 / 325 Регистрация: 28.07.2012 Сообщений: 2,810
	15.10.2017, 15:13	8
	Сообщение от fingolfin_ потом исправить ошибки и по новой Метод тыка - это когда ты не знаешь, где находится ошибка, и начинаешь исправлять случайные места. Как раз твой случай: Сообщение от fingolfin_ Но если переместить вывод на один уровень вложенности ниже, но всё работает Ты что-то поменял в случайном месте, а потом уверовал, что "все" стало работать. Такой подход приводит к еще большему непониманию и еще большему количеству ошибок. 1

@DrOffset 18473 / 9630 / 2356 Регистрация: 30.01.2014 Сообщений: 16,883
	15.10.2017, 15:14	9
	Сообщение от fingolfin_ Почему же нельзя? Написать код, попробовать собрать, запустить, потом исправить ошибки и по новой, нет? У меня пока очень редко получается, что бы написанный код сразу работал без ошибок. Потому что это контрпродуктивно. И делает тебя зависимым от случая. Верно работающий код не гарантия, что в нем нет ошибок. UB может и совершенно ожидаемым образом проявляться, создавать видимость работы (до поры). Поэтому ты должен понимать от и до что у тебя в программе происходит, а не проверять корректность методом тыка. И дело не в возможности писать код сразу без ошибок, речь не об этом, а в возможности понять, что у тебя в коде не так не только на основе многочисленных проб, а на основе понимания того как программа работает и должна работать. Поэтому, если ты не понимаешь как она работает, надо сесть и разобраться, а не мучить компилятор запусками в надежде, что сработает. 1

Опции темы

В определенный момент массив почему-то обнуляется

Решение