C++ Programm stoppt bei Funktionsaufruf

Question

Ich bin neu in der Arbeit mit C++, so habe ich Probleme zu finden, wo mein Fehler liegt. Wenn mein Code den Anruf assignGate erreicht, funktioniert es nicht mehr. Ich habe versucht, mein Programm mit Zeile entfernt und alles hat gut funktioniert. Also ist das Problem wahrscheinlich in dieser Funktion, aber ich kann nichts falsch daran sehen.

#include "stdafx.h" 
#include <stdio.h> 
#include <iostream> 
#include <string> 
#include <fstream> 
using namespace std; 

struct node { 
    string code; 
    int flight; 
    int time; 
    string gate; 
    node *next; 
}; 

bool isEmpty(node *head); 
void insert(node *&head, node *&last, string code, int flight, int time); 
void remove(node *&head, node *&last); 
void assignGate(node *&head, int gates); 
void print(node *&head); 

int main() 
{ 
    int gates; 
    bool x = false; 
    string code; 
    int flight, time, gate; 
    node *head = NULL; 
    node *last = NULL; 
    cout << "Welcome to Gate Scheduler.\n"; 
    ifstream file("FlightList.txt"); 
    if (!file) { 
     cout << "Unable to open text file. \n"; 
     cout << "Make sure file is in correct location then restart program. \n"; 
    } 
    while (file >> code >> flight >> time) { 
     insert(head, last, code, flight, time); 
    } 
    cout << "Please enter the max number of gates avaliable:"; 
     cin >> gates; 
     assignGate(head, gates); 
     cout << "\n"; 
    print(head); 
    return 0; 
} 

bool isEmpty(node *head) { 
    if (head == NULL) { 
     return true; 
    } 
    else { 
     return false; 
    } 
} 

void insert(node *&head, node *&last, string code, int flight, int time) { 
    node *temp = new node; 
    temp->flight = flight; 
    temp->code = code; 
    temp->time = time; 
    temp->next = NULL; 
    if (isEmpty(head)) { 
     head = temp; 
     last = temp; 
    } 
    else { 
     last->next = temp; 
     last = temp; 
    } 
} 

void remove(node *&head, node *&last) { 
    if (isEmpty(head)) { 
     cout << "The list is already empty \n"; 
    } 
    else if (head == last) { 
     delete head; 
     head = NULL; 
     last = NULL; 
    } 
    else { 
     node *temp = head; 
     head = head->next; 
     delete temp; 
    } 
} 

void assignGate(node *&head, int gates) { 
    int y = 0; 
    int gate[6]; 
    node* temp = head; 
    while (temp->next != NULL) { 
     if (temp->time > 2300 || temp->time < 600) { 
      temp->gate = "N/A: Plane lands outside of airport operating hours."; 
     } 
     else { 
      for (y = 0; y <= gates; ++y) { 
       if (gate[y] == NULL) { 
        temp->gate = y; 
       } 
       else if (gate[y] + 100 < temp->time) { 
        temp->gate = y; 
       } 
      } 
      if (temp->gate != "0" || "1" || "2" || "3" || "4") { 
       temp->gate == "All gate are full at this time"; 
      } 
     } 
    } 
} 

void print(node *&head) { 
    node* temp = head; 
    while (temp->next != NULL) { 
     cout << "Flight " << temp->code << " " << temp->flight << " will be at gate " << temp->gate << "\n"; 
     temp = temp->next; 
    } 

} 

Answer 1

Als Erstes nehmen Sie an, dass Array Null initialisiert ist. Es ist nicht. Du hast keine Nullen in Gate []. Sie sollten es initialisieren, um sie auf Null zu setzen.

int gate[6] = {}; 

Sie sollten wirklich überprüfen, ob Gates kleiner als die Größe der Array sind.

Als nächstes Ihr Problem ist in diesem Teil:

 else if (gate[y] + 100 < temp->time) { 
       temp->gate = y; 
      } 
     } 
     if (temp->gate != "0" || "1" || "2" || "3" || "4") { 
      temp->gate == "All gate are full at this time"; 

temp->gate = y nicht tut, was Sie denken. Er weist einem Zeichen einen String zu, dessen Code in Integer gespeichert ist. Natürlich, wenn() würde fehlschlagen. Aber es würde auch scheitern, weil Sie eine falsche Operation geschrieben haben. (temp->gate != "0" || "1" || "2" || "3" || "4") entspricht ... (temp->gate != true) Sie sollten sich die Tabelle der Bedienerpriorität ansehen, um zu verstehen, warum.

Und es ist algorithmischen Fehler .. Sie nie mit Temp fortfahren. Es bleibt gleich, Sie werden nie erreichen Ausgang von der While-Schleife.

Es könnte andere Fehler geben, die ich weggelassen habe, weil sie für das aktuelle Problem nicht wesentlich sind. Ich konzentrierte mich auf die Funktion assignGate

PS. Dies ist kein Fehler, aber ist sehr sehr schlecht Stil:

bool isEmpty(node *head) { 
    if (head == NULL) { 
     return true; 
    } 
    else { 
     return false; 
    } 
} 

'Kopf == NULL' ist ein bool Ausdruck. Sie können es zurückgeben.

inline bool isEmpty(node *head) { 
    return head == NULL; 
}