Segmentation fault σε C++ σε Linux με string cin

[Simpleton]

Γεια σας, έχω τον εξής κώδικα:

void NewDoc() {
    system("clear"); // Εκκαθάριση οθόνης
    cout << "Εισαγωγή γιατρού\n--------------------\n" << endl;
    cout << "Όνομα:     ";
    string inp;
    cin >> inp;
    int i=0;
    while (Doctors[i]!=NULL)
    {if (!Doctors[i]->name.compare(inp)) 
    {cout << "Υπάρχει ήδη γιατρός με όνομα """ << inp << """. Δεν γίνεται εισαγωγή"; sleep(2000); menu();}
    else i++;
    }
    i=0; while (Doctors[i]!=NULL) {i++;}
    Doctors[i] = new Doc;
    Doctors[i]->name = inp;
    if (!Doctors[i]->name.compare(inp)) {cout << "Η εισαγωγή ολοκληρώθηκε."; sleep(2000); menu();} 
    else {cout << "Η εισαγωγή δεν ολοκληρώθηκε."; sleep(2000); NewDoc();}
}

Μετά το cin >> inp; μου πετάει segmentation fault. Το compile γίνεται χωρίς σφάλματα, σε KDevelop (Kubuntu 8.04). Τι μπορεί να φταίει; Ώς αρχικές δηλώσεις του προγράμματος έχω τις εξής:

#include <iostream>

using namespace std;

Ο υπόλοιπος κώδικας ίσως είναι λίγο μπακαλίστικος, αλλά αυτή τη στιγμή με διαολίζει αυτό το segfault.

Ευχαριστώ εκ των προτέρων για κάθε βοήθεια

[nikospara]

Καλημέρα,

Σου πετάει ακριβώς μετά το "cin >> inp" ή απλά κάπου μετά; Τρέξε το σε έναν debugger ή βάλε cout μετά από κάθε εντολή να δείς που ακριβώς έχει πρόβλημα (πχ Doctors != NULL).

Το έτρεξα ως και το "cin >> inp" και δεν παρουσίασε πρόβλημα...

[Simpleton]

Ευχαριστώ για την απάντηση

Εχτές μου το πέταγε αμέσως μόλις γράψω κάτι και πατήσω Enter στο prompt του "cin >> inp".

Σήμερα που το δοκίμασα πάλι, δεχόταν ό,τι έγραφα στο prompt, αλλά μετά δεν έκανε τίποτα. Ένα

cout << "OK";

μετά το "cin >> inp;" δεν έβγαζε το αντίστοιχο μήνυμα.

Τροποποίησα την ενότητα ως εξής:

void NewDoc() {
	system("clear"); // Εκκαθάριση οθόνης
	cout << "Εισαγωγή γιατρού\n--------------------\n" << endl;
	cout << "Όνομα:     ";
	string inp;
	cin >> inp;
	cout << inp;
}

Τώρα δέχεται κανονικά το όνομα που βάζω και μου το εμφανίζει μέσω της cout. Δεν καταλαβαίνω αφενός πώς άλλαξε συμπεριφορά από τη μία μέρα στην άλλη, αφετέρου τι μπορεί να το ενοχλεί στον υπόλοιπο κώδικα, ώστε να μην εκτελεί καν την cin.

Τα αντικείμενα string δεν χρειάζονται κάποια αρχικοποίηση, έτσι;

[nikospara]

Φίλε skaf το πρόβλημά σου είναι εδώ:

    while (Doctors[i]!=NULL)
    {if (!Doctors[i]->name.compare(inp)) 
    {cout << "Υπάρχει ήδη γιατρός με όνομα """ << inp << """. Δεν γίνεται εισαγωγή"; sleep(2000); menu();}
    else i++;
    }

Συγκεκριμένα το while δε σταματάει ποτέ (οπότε όταν φτάσει σε μνήμη που δεν κατέχεις, πετάει segfault).

Επιπλέον 99% χρησιμοποιείς με λάθος τρόπο την έξοδο "menu()": Η NewDoc() δεν επιστρέφει ποτέ, αλλά καλεί μιαν άλλη συνάρτηση. Αν κληθεί αρκούντως πολλές φορές θα μπουκώσει η στοίβα (stack). Εδώ αυτό δεν πρόκειται να γίνει, αλλά αν δεν το ξέρεις θα το κάνεις και αλλού. Πιο σωστό θα ήταν κάτι του στύλ:

int entryValidates = 1;
while (Doctors[i]!=NULL && i < lengthOfDoctorsArray)
{
  if (!Doctors[i]->name.compare(inp)) {
    cout << "Υπάρχει ήδη γιατρός με όνομα """ << inp << """. Δεν γίνεται εισαγωγή";
    sleep(2000);
    entryValidates = 0;
    break;
  }
  i++;
}
if( !entryValidates ) {
  return -1;
}

Εδώ με επιστροφή -1 σημειώνεται το λάθος στο υπόλοιπο πρόγραμμα, το οποίο και είναι υπεύθυνο για την κλήση της menu().

Αλλά γιατί δεν τύπωνε το ΟΚ;;; Κλασικό: Τα standard streams είναι buffered, δηλαδή δε γράφουν αμέσως αυτό που τους δίνεις στην έξοδό τους. Όταν λοιπόν πεταγόταν το segfault, το "ΟΚ" ήταν στο buffer και δεν είχε προλάβει να πάει στην οθόνη. Το πρόγραμμα τερμάτιζε βεβιασμένα και οι buffers πετάγονταν, με αποτέλεσμα το μήνυμα να χάνεται. Θα το έβλεπες αν είχες κάνει κάτι του στύλ: "cout << inp; cout.flush();"

Καλή συνέχεια

[Simpleton]

Για να μην παραθέτω επιλεκτικά και ψαχνόμαστε, ιδού ολόκληρος ο κώδικας του προγράμματος:

Spoiler:

#include <iostream>

using namespace std;

void menu();

class Doc {
    public:
        string name;
        int appt[30][8];
        Doc() {
            int i, j; for(i=0; i<30; i++) {for(j=0; j<8; j++) appt[i][j]=0;} // Αρχικοποίηση ραντεβού
        }
}*Doctors[100]; //Πίνακας δεικτών σε αντικείμενα Doc

class Pat {
    public:
        string name;
        int year, month, day, appt;
};

void NewDoc() {
    system("clear"); // Εκκαθάριση οθόνης
    cout << "Εισαγωγή γιατρού\n--------------------\n" << endl;
    cout << "Όνομα:     ";
    string inp;
    cin >> inp;
    int i=0;
    while (Doctors[i]!=NULL)
    {if (!Doctors[i]->name.compare(inp)) 
    {cout << "Υπάρχει ήδη γιατρός με όνομα """ << inp << """. Δεν γίνεται εισαγωγή"; sleep(2000); menu();}
    else i++;
    }
    i=0; while (Doctors[i]!=NULL) {i++;}
    Doctors[i] = new Doc;
    Doctors[i]->name = inp;
    if (!Doctors[i]->name.compare(inp)) {cout << "Η εισαγωγή ολοκληρώθηκε."; sleep(2000); menu();} 
    else {cout << "Η εισαγωγή δεν ολοκληρώθηκε."; sleep(2000); NewDoc();}
}

void menu() {
    int sel;
    system("clear");
    do {
        cout << "Μενού επιλογών:\n" << endl;
        cout << "1. Εισαγωγή γιατρού\n2. Εισαγωγή ασθενή\n\n3. Εισαγωγή ραντεβού\n4. Εμφάνιση υπαρχόντων ραντεβού\n\n5. Τερματισμός\n\n\nΕπιλογή:     ";
        cin >> sel;
    } while (sel < 1 || sel > 5);
    
    switch(sel) {
        case 1: NewDoc(); break;
        case 2:break;
        case 3:break;
        case 4:break;
        case 5:break;
    }
}    

main() {
    
    int i; for(i=0; i<100; i++) {Doctors[i]=NULL;} //Αρχικοποίηση πίνακα δεικτών
    menu();
}

Όπως βλέπεις, στην main αρχικοποιώ τον πίνακα δεικτών Doctors. Με την λογική (; ) μου, την πρώτη φορά που θα έκανα εισαγωγή ονόματος, δεν θα έμπαινε σε κανέναν από τους δύο βρόχους while, καθώς όλοι οι δείκτες θα ήταν NULL.

Όσο για την άτσαλη έξοδο από την NewDoc(), το ξέρω, άλλα έκανα αρχικά ένα πρόχειρο πρόγραμμα και σε αργότερο στάδιο φυσικα και θα το άλλαζα. Εκτός αν υπάρχει ενδεχόμενο να φταίει κι αυτό για την cin που δεν λειτουργεί.

Εννοείται ότι σε ευχαριστώ πολύ που ασχολήθηκες με το πρόβλημα μου

[turboirc]

Συγγνώμη, αλλα γιατι δεν το περνάς από τον debugger για να σου πει ακριβώς το πρόβλημα?

*Doctors[100]

Μήπως εννοούσες χωρίς το * ?

[Simpleton]

Συγγνώμη, αλλα γιατι δεν το περνάς από τον debugger για να σου πει ακριβώς το πρόβλημα?

Δεν ξέρω ακόμα να τον χειρίζομαι

*Doctors[100]

Μήπως εννοούσες χωρίς το * ?

Για να μην δεσμεύω εξαρχής χώρο για 100 αντικείμενα Doc, ήθελα να δηλώσω έναν πίνακα 100 θέσεων με δείκτες προς αντικείμενα Doc. Λάθος είναι;

[nikospara]

skaf αυτό που έστειλες μου δουλεύει, με μια μετατροπή: το όρισμα της sleep() είναι σε δευτερόλεπτα, οπότε το 2000 είναι κανα 40λεπτο

Κάντο σκέτο 2 για 2 δευτερόπλεπτα που μάλλον θές...

Πάντως segfault δεν κάνει με 2 εισαγωγές.

[turboirc]

1. Το βασικότερο είναι να χειρίζεσαι το debugger. Το debugging είναι στα πρωτα 10 βήματα στον προγραμματισμό και ειδικά σε ένα δύσκολο περιβάλλον σαν το Linux. Κάτσε και ψάξε ολα τα κόλπα που έχει ο debugger σου, το εννοώ όταν λέω όλα.

2. Οχι δεν είναι λάθος, αλλά σήμερα τέτοιας μορφής κώδικας θεωρείται επικίνδυνος και απαρχαιωμένος. Καλύτερα να χρησιμοποιήσεις std::vector. Εαν ακόμα δεν ξέρεις από STL, διάβασε και χρησιμοποίησέ τη.

3. Ο κώδικάς σου στα Windows δεν crasharei και επομένως δεν μπορώ να σε βοηθήσω περισσότερο από τον debugger σου. Γενικά πάντως ο G++ , ενώ είναι καλός C compiler, δεν είναι αξιόπιστος C++ compiler.

[Simpleton]

ΟΚ, το γεγονός ότι δεν κρασάρει σε άλλη πλατφόρμα είναι πολύ θετικό. Οπότε έχει τώρα να πέσει γερό διάβασμα για debugging και STL...

Ευχαριστώ και πάλι όλους που ασχοληθήκατε με την περίπτωση μου

[turboirc]

Καθόλου θετικό δεν είναι.
Δείχνει οτι ο compiler έχει πρόβλημα.

Αντιθέτως αμα κράσαρε και στα Windows θα σου λέγαμε και τον λόγο.
Και μια τελευταία συμβουλή. Υποψιαζομαι οτι τον κωδικα αυτόν δεν τον έγραψες εσύ ολόκληρο. Μην χρησιμοποιείς κώδικα άλλου χωρίς να έχεις μάθει τη γλώσσα τέλεια.

[Simpleton]

Καθόλου θετικό δεν είναι.
Δείχνει οτι ο compiler έχει πρόβλημα.

Αντιθέτως αμα κράσαρε και στα Windows θα σου λέγαμε και τον λόγο.
Και μια τελευταία συμβουλή. Υποψιαζομαι οτι τον κωδικα αυτόν δεν τον έγραψες εσύ ολόκληρο. Μην χρησιμοποιείς κώδικα άλλου χωρίς να έχεις μάθει τη γλώσσα τέλεια.

Ο compiler μπορεί να έχει πρόβλημα, αλλά το ότι δούλεψε σε εσάς σημαίνει ότι τουλάχιστον δεν έφταιγε ο κώδικας. Τελικά τον κατάφερα και δούλεψε και στο δικό μου σύστημα.

Ο κώδικας που παρέθεσα είναι όλος δικός μου.

[turboirc]

Σε αυτήν την περίπτωση έχεις προχωρήσει πολύ σε ασκήσεις, πιο γρήγορα από ότι έπρεπε.
Συνεχισε το διάβασμα και, κατα την γνώμη μου, δούλεψε με το Visual Studio.