for loop crashing ... segFault C++

Question

У меня есть цикл for, который циклически проходит через список массивов дней и сохраняет int в переменной num, когда я переключаюсь через первые 7 дней, моя программа вылетает, я хочу, чтобы она перезапустилась на 1-й день и перейти к обезьяне «1» ... но после первого цикла он сработает.

Что я делаю неправильно?

C++

//********************************************************************************************* 
//        function prototype 
//********************************************************************************************* 
void collectFood(string days[]); 



int main(){ 

    //array to hold week days 
    string days[7] = {"Mon", "Tue", "Wen", "Thur", "Fri", "Sat", "Sun"}; 

    collectFood(days); 

} 
//********************************************************************************************* 
//     function to collect foods amounts 
//********************************************************************************************* 
void collectFood(string days[]){  
    int num = 0; 

    // for loop for each monkey 
    for (int monkey=0; monkey < 3; monkey++){ 
     // for loop to take 7 days worth of food per monkey 
     for (int day=0; day < sizeof(days); day++){ 
      cout << "Enter the pounds of food eaten by monkey " << monkey 
      << " on " << days[day] << ": "; 
      cin >> num ; 
     } 
    } 
} 

Answer 1

Вы расчет ожидаемой величины не является правильным в контексте вы используете. Простой массив C-типа (который у вас есть, независимо от типа внутри) выражается как указатель на тип при передаче в качестве простого параметра.

Таким образом, это:

void collectFood(string days[]) 

эквивалентно следующему:

void collectFood(string *days) 

и становится совершенно очевидно, что sizeof(days) действительно размер указателя к string. На вашей платформе я могу с уверенностью предположить, что вы компилируете 64-битный код, поскольку указатель в этом контексте будет восемь байтов, и, таким образом, вы превысите фактический размер вашего массива на один элемент.

Существует множество способов сделать это. Вы можете просто объявить дополнительный параметр своей функции collectFood, которая описывает величину последовательности.

void collectFood(string days[], size_t N) 

и использовать N в вашем для цикла состояния, вызывая collectFood как это из main():

collectFood(days, sizeof(days)/sizeof(*days)); 

Альтернатива может быть шаблоном, который принимает не-напечатал параметр, выведенный из массива, который передается по ссылки:

template<size_t N> 
void collectFood(string (&days)[N]) 
{ 
    ... 
} 

и снова, U se N в качестве условия для ограничения срока. В этом случае звонок от main() остался бы таким же, как у вас сейчас. N будет выведено по вызову. В качестве дополнительного преимущества вы не можете случайно передать необработанный указатель на эту функцию; это должно быть быть объявленным типом массива, или величина не может быть выведена, и компилятор будет извергать ошибку, сообщающую вам об этом.

Есть и другие способы, но первый из них, просто передавая длину и объявляя его формальным параметром, скорее всего, будет самым легким для вас.

Удачи.

Answer 2

Я думаю, что это ваш внутренний цикл, когда вы используете sizeof (дни). Когда вы передаете массив функции, он передает только первый элемент массива, а не весь массив.

Вы можете исправить это, добавив параметр функции, который сообщает вам размер массива, или вы можете переключиться на использование std :: vector, который имеет метод size().Я бы лично пошел с std :: vector.