für die Schleife Absturz ... segFault C++

Question

Ich habe eine for-Schleife, die durch eine Array-Liste von Tagen zykliert und speichert ein int in Variable num, wenn ich die ersten 7 Tage durchlaufen, stürzt mein Programm ab, ich möchte es neu starten am Tag 1 und Wechsel zu Affe '1' ... aber nach der ersten Schleife stürzt es ab.

Was mache ich falsch?

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

Sie Berechnung der erwarteten Größenordnung ist nicht korrekt im Kontext Sie verwenden. Ein einfaches Array vom C-Typ (das ist das, was Sie haben, unabhängig von dem darin enthaltenen Typ) drückt sich als Zeiger auf Typ aus, wenn es als einfacher Parameter übergeben wird.

Somit folgt aus:

void collectFood(string days[]) 

ist dies äquivalent:

void collectFood(string *days) 

und es wird deutlich, dass in der Tat sizeof(days) die Größe eines zustringZeiger ist. Auf Ihrer Plattform kann ich wahrscheinlich mit Sicherheit davon ausgehen, dass Sie 64-Bit-Code kompilieren, da ein Zeiger in diesem Kontext acht Bytes wäre und Sie somit die tatsächliche Größe Ihres Arrays um ein Element überschreiten würden.

Es gibt eine Vielzahl von Möglichkeiten, dies zu tun. Sie könnten einfach einen zusätzlichen Parameter für Ihre collectFood-Funktion deklarieren, der die Größe der Sequenz beschreibt.

void collectFood(string days[], size_t N) 

und verwenden N innerhalb Ihrer for-Schleife Bedingung, Aufrufen collectFood wie dies aus main():

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

Eine Alternative wäre eine Vorlage, die eine nicht typisierte Parameter aus dem Array abgeleitet erfolgt, die wird durch Referenz bestanden:

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

und u, wieder se N innerhalb der for-Bedingung als Zählgrenze. In diesem Fall würde der Anruf von main() so bleiben, wie Sie ihn jetzt haben. N würde durch den Aufruf abgeleitet werden. Als zusätzlichen Vorteil können Sie nicht versehentlich einen rohen Zeiger auf diese Funktion übergeben. es muss ein deklarierter Array-Typ sein, oder die Größe kann nicht abgeleitet werden und der Compiler wird einen Fehler ausspucken, der Ihnen das sagt.

Es gibt andere Wege, aber die ersteren, einfach die Länge zu überschreiten und es als formalen Parameter zu deklarieren, ist wahrscheinlich das einfachste für Sie zu verstehen.

Viel Glück.

Answer 2

Ich denke, es ist Ihre innere for-Schleife, wenn Sie sizeof (Tage) verwenden. Wenn Sie das Array an eine Funktion übergeben, übergibt es nur einen Zeiger an das erste Element des Arrays, nicht das gesamte Array.

Sie können dies beheben, indem Sie einen Funktionsparameter hinzufügen, der Ihnen die Größe des Arrays mitteilt, oder Sie können zur Verwendung von std :: vector wechseln, das eine size() -Methode hat.Ich würde persönlich mit st :: vector gehen.