Döngü çöküyor ... segFault C++

Question

Günlerin bir dizi listesi arasında dolaşan ve ilk 7 gün içinde geçiş yaptığımda, program çöküyor, yeniden başlatılmasını istediğimde, değişken num'te bir int depolayan bir for döngüsüm var 1. günde ve '1' maymununa dönüşür ... ancak ilk döngüden sonra çöker.

Neyi yanlış yapıyorum? beklenen büyüklükte

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

Sen hesaplama kullandığınız bağlamda doğru değil. Basit bir C-tipi dizi (içinde bulunduğunuz türden bağımsız olarak sahip olduğunuz şey), basit bir parametre olarak geçtiğinde işaretçi-türü olarak ifade edilir.

Böylece

bu:

void collectFood(string *days) 

ve sizeof(days) gerçekten işaretçi string için boyutu olduğunu açıkça ortaya çıkmaktadır:

void collectFood(string days[]) 

bu eşdeğerdir. Platformunuzda, bu bağlamda bir işaretçi sekiz bayt olacağını düşündüğünüzde, muhtemelen 64 bit kodu derlediğinizden emin olabilirsiniz ve bu nedenle dizinin gerçek boyutunu bir öğe ile aşacaksınız.

Bunu yapmanın birçok yolu vardır. Dizinin büyüklüğünü açıklayan collectFood işlevinize ekstra bir parametre bildirmeniz yeterlidir. main() bunun gibi collectFood çağırma

void collectFood(string days[], size_t N) 

ve For döngüsü koşulu içinde N kullanın:

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

alternatif dizi çıkarılabilir olmayan bir daktilo parametre alır bir şablon olacağını referans geçirilir: tekrar

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

ve u, Sayım sınırı olarak koşul içinde se N. Bu durumda, main()'dan gelen çağrı, şimdi sahip olduğunuz gibi kalır. N çağrı tarafından çıkarılacaktır. Ek bir avantaj olarak, bir ham işaretçiyi yanlışlıkla bu işleve iletemezsiniz; , bildirilmiş bir dizi türü olmalıdır, yoksa büyüklük çıkarılabilir ve derleyici size bir hata bildirir.

Başka yollar da var, ancak bunların ilkini, sadece uzunluğu geçip resmi bir parametre olarak ilan etmek, muhtemelen sizin için en kolay olanıdır.

Şansın en iyisi.

Answer 2

Sanırım sizeof (günler) kullanırken iç döngüdür. Diziyi bir işleve ilettiğinizde, dizinin ilk öğesine yalnızca bir işaretçi iletir, dizinin tamamını değil.

Bunu, dizinin boyutlarını bildiren bir işlev parametresi ekleyerek veya size size size bir size() yöntemi olan std :: vektörünü kullanarak geçiş yapabilirsiniz.Kişisel olarak std :: vector ile giderdim.