cin und char-array-pointer in der struct

Als c++ - Anfänger habe ich geschrieben das folgenden code:

int main(void){
struct car{
    char * make[200];
    int manfYear;
};

int num=0;

cout << "How many cars do you wish to catalogue? ";
cin >> num;
car * Cars = new car [num]; 

for (int i=1;i<=num;i++){
    cout << "Car #" << i << ":" << endl << "Please enter the make: ";
    cin.getline(*Cars->make,200);
    cout << "Please enter the year made: ";
    cin >> Cars->manfYear;
}

Mein problem ist, dass ich nicht bekommen kann meinen Kopf Runde ein problem, wo bekomme ich einen segmentation Fault, wenn das Programm läuft an der Stelle muss ich zu geben Sie die Modell des Autos. Kann mir bitte jemand erklären, was ich falsch mache?

Soweit ich es verstehe, ich bin vorbei ein Zeiger auf das array "machen", das sollte damit es funktioniert. Ist mein Verständnis Weg?

Vielen Dank im Voraus
Dan

InformationsquelleAutor Dan | 2012-04-09
  1. 4

    Vier Fragen, die ich sofort sehen:

    Ausgabe 1

    In Ihrem struct Sie haben:

    char * make[200];

    In Englisch, das sagt, "erstellen Sie ein array von 200 Zeiger auf Zeichen", wenn ich denke, Sie wollen sagen, "erstellen Sie ein array von 200 Zeichen." So sollten Sie statt:

    char make[200].

    Ausgabe 2

    Sind Sie Fehlverhalten ab 1. Dies überspringt das erste Auto in der array - erinnern-arrays sind null-indiziert. So sollten Sie statt:

    for (int i = 0 ; i < num ; i++)

    und für die Anzeige, man könnte sagen:

    cout << "Car #" << (i+1) << ":" << endl << "Please enter the make: ";

    Ausgabe 3

    Wo Sie sagen:

    cin.getline(*Cars->make,200);

    und

    cin >> Cars->manfYear;

    Dort, wo in diesen Zeilen, die Sie angeben, welches Auto der Benutzer ist das Auffüllen? Nirgends. Wenn man looping mit i, dann müssen Sie eigentlich erwähnt i. Diese arbeiten sollten:

    cin.getline(Cars[i].make,200);

    und

    cin >> Cars[i].manfYear;

    Feststellen, dass wir mit ., nicht ->. Dies ist, weil die Elemente in der Cars array sind die aktuellen Instanzen, keine Zeiger. Die Cars array selbst ist ein Zeiger, aber nicht seinen Inhalt.

    Ausgabe 4

    Alle Kredit @Ben C wer wies dies ersten: mischen der >> Betreiber mit getline() Funktion auf cin können führen zu seltsamen Verhalten, mit übrig gebliebenen CR's aus >> geht in die getline() nennen. Sie könnten entweder alle >> (Nachteil: Sie haben nicht die 200 limit erzwungen, wenn das Lesen der machen) oder alle cin.getline() (Nachteil: Sie haben die Verwendung von string-Puffer und dann konvertieren Sie Sie für die Anzahl der Fahrzeuge und Jahr), oder setzen cin.ignore() nach jedem Aufruf von >> etwa so:

    cin >> num;
cin.ignore();

    und

    cin >> Cars[i].manfYear;
cin.ignore();

    Wieder, alle Kredit, um @Ben C für bemerkt es zuerst.

    Nicht Zuletzt

    Durch Konvention, Klassen/structs haben Kapital-Namen und Variablen klein - /camelcase Namen. Betrachten Sie die Umbenennung der struct aus car zu Car), und das array von Cars zu cars. In anderen Worten, die Umkehrung der Kapitalisierung, die Sie jetzt haben.

    Schließlich, ich Stimme mit all die anderen Poster hier: sollten Sie in Betracht ziehen string statt char arrays.

    InformationsquelleAutor sparc_spread
  2. 2

    Verwenden Sie zuerst string statt der Arme alte C char[].

    Weiter: Sie wollen nicht char * make[200];. Sie möchten char make[200];. char * make[200] ist ein array von 200 Zeiger auf chars, die verwendet werden können, wie die 200 null-terminierte Zeichenfolgen - aber dann müssten Sie new[] jeden von Ihnen. Verwenden Sie einfach char make[200]; und cin.getline(Cars->make, 200);.

    • Genial. Alles macht nun Sinn. Sehr geschätzt wird.
    • Jetzt bin ich verwirrt mit dem, was geschieht, ich habe jetzt den folgenden code: int main(void){ struct Auto{ char machen[200]; int manfYear; }; int num=0; cout << "Wie viele Autos möchten Sie den Katalog? "; cin >> num; Auto * Autos = new Auto [num]; for (int i=0;i<num;i++){ cout << "Auto #" << i+1 << ":" << endl << "Bitte geben Sie das machen: "; cin.getline(Autos[i].machen,200); cin.get(); cout << "Bitte geben Sie das Jahr aus: "; cin >> Autos[i].manfYear; //Autos++; } Jetzt bin ich nicht einmal die chance zu geben Sie die manfyear, das Programm läuft aber druckt zufällige Daten.
    InformationsquelleAutor Griwes
  3. 2

    char * make[200] deklariert ein array von 200 Zeigern; ich vermute, dies ist nicht, was Sie suchen.

    Wenn Sie einfach nur suchen, um zu speichern eine Zeichenfolge, würde ich empfehlen, einen Blick in die C++ - string Art statt. 

    #include <iostream>
#include <string> 

int main()
{
    using namespace std;

    struct car
    {
        string make;
        int manfYear;
    };

    int num=0;

    cout << "How many cars do you wish to catalogue? ";
    cin >> num;
    car * Cars = new car [num]; 

    for (int i=1;i<=num;i++)
    {
        cout << "Car #" << i << ":" << endl << "Please enter the make: ";
        std::cin.ignore();
        getline(cin, Cars[i-1].make);
        cout << "Please enter the year made: ";
        cin >> Cars[i-1].manfYear;
    }
}

    Gab es auch ein paar andere kleinere Problemchen mit Ihrem code.

    1) Sie war mit Cars->manfYear - dies wird immer nur verweisen Sie auf das erste element des array. Ich nehme an, Sie wollen das nicht; die subscripting-syntax als pro Cars[i-1].manfYear wird Zugriff auf ein einzelnes Auto-Objekt in das array. (Denken Sie daran, dass array-Indizes beginnen bei null! - würde es eigentlich besser treffen idiomatischen Sinn für Ihr für loop variable bei null anfangen wie gut wirklich)

    2) vorsichtig Sein, die Art und Weise std::getline und die >> symbol zusammen zu arbeiten. die >> (stream-extraction operators) oft Blätter alle neue-Zeile-Zeichen, was bedeutet, dass Sie möglicherweise finden Sie unter "seltsam" Verhalten bei Ihren anrufen zu getline. Wenn Sie mischen die beiden zusammen, dann mit so etwas wie std::cin.ignore() wird Ihnen helfen, entsorgen Sie die neue-Zeile-Zeichen.

    InformationsquelleAutor Ben Cottrell
  4. 1

    Zunächst arrays in C++ sind indiziert von 0..n-1, so dass die Schleife ausgeführt werden muss

    for (int i = 0; i < num; i++) { ... }

    Zweitens, Sie haben erklärt make als 200-element-array von Zeiger zu char; dies ist wahrscheinlich nicht, was Sie wollen. Wenn make soll zum speichern einer Zeichenfolge, deklarieren Sie es als eine einfache Reihe von char: 

    struct car{
      char make[200];
      int manfYear;      
};

    Schließlich schreiben Sie Ihre getline Ruf als

    cin.getline(Cars[i].make, sizeof Cars[i].make); //fixed per comment below

    Obwohl Cars ist deklariert als pointer verwenden, können Sie den subskript-operator auf es , als ob es ein array wäre; dadurch werden Sie implizit zu dereferenzieren Cars, da a[i] wird interpretiert als *(a + i). Dies bedeutet auch, dass Sie die . Komponentenauswahl-operator im Gegensatz zu den -> Betreiber, da die Art der Cars[i] ist car, nicht car *.

    InformationsquelleAutor John Bode
  5. 1

    Zunächst char *make[200] ist nicht ein string von max 200 Zeichen, aber 200 Zeiger auf char.

    Zweite Sache: Sie sind die Dereferenzierung der Zeiger in cin.getline mit *: was passiert, ist, dass Sie erhalten den enthaltenen Wert in der ersten Zelle der 200 char* Zeiger. Aber Sie hat nicht initialisieren die einzelnen Zeiger, nur die höhere Ebene ein, so bekommt man einen segmentation Fault.

    Nur ändern char* make[200] zu char make[200] und *Cars->make zu Cars[i].make.

    • s/->/./. Cars ist ein car*, nicht ein car**.
    InformationsquelleAutor Jack
Schreibe einen Kommentar