Lagerverwaltung Kisten

  • C++

Es gibt 8 Antworten in diesem Thema. Der letzte Beitrag () ist von Sebastian.Pape.

    Lagerverwaltung Kisten

    Hallo,
    ich soll für mein Fernstudium einen Lagerverwaltung mit max. 50 Kisten erstellen, die ich eingeben, ändern, löschen und mir anzeigen kann. Leider kann ich meinen Code nicht ausführen und weis wirklich nicht woran es liegt.
    Hier der Code:

    C-Quellcode

    1. #include <iostream>
    3. using namespace std;
    5. struct Lager
    6. {
    7. int nummer;
    8. double breite;
    9. double laenge;
    10. double hoehe;
    11. double volumen;
    12. int aktiv=0;
    14. }aKiste;
    16. Lager eingeben(Lager aKiste[50])
    17. {
    18. int i=0;
    19. cout << "Welche Kiste soll Angelegt werden?: \n" << "Es dürfen max. 50 Kisten angelegt werden.";
    20. cin >> i;
    22. if(((i<50)&&(i!=0)&&(aKiste[i].aktiv!=1)))
    23. {
    24. cout << "Bitte geben Sie die Hoehe der " << i << " Kiste: " << " ein.";
    25. cin >> aKiste[i].hoehe;
    26. cout << "Bitte geben Sie die Breite der " << i << " Kiste: " << " ein.";
    27. cin >> aKiste[i].breite;
    28. cout << "Bitte geben Sie die Laenge der " << i << " Kiste: " << " ein.";
    29. cin >> aKiste[i].laenge;
    30. aKiste[i].aktiv=1;
    31. aKiste[i].volumen=aKiste[i].hoehe*aKiste[i].breite*aKiste[i].laenge;
    32. cout << '\n';
    33. }
    34. else if(i>=50)
    35. {
    36. cout << "Es sind max. 50 Kisten erlaubt!\n";
    37. }
    38. else
    39. {
    40. cout << "Diese Kiste ist bereits belegt!\n";
    41. }
    42. return aKiste[i];
    43. }
    45. Lager Kisteanzeigen(Lager aKiste[50])
    46. {
    47. int i=0;
    48. cout << "Welche Kiste soll angezeigt werden?: ";
    49. cin >> i;
    51. if((aKiste[i].nummer <50) && (aKiste[i].aktiv==1))
    52. {
    53. cout << "Kiste Nr.:\t " << aKiste[i].nummer << '\n';
    54. cout << "Volumen: \t" << aKiste[i].volumen << '\n';
    55. cout << "Aktiv: \t\t" << aKiste[i].aktiv << '\n';
    56. }
    57. else if((aKiste[i].nummer <50) && (aKiste[i].aktiv!=1))
    58. {
    59. cout << "Die Kiste nr.: \t" << aKiste[i].nummer << " ist nicht belegt!\n";
    60. }
    61. else cout << "Falsche eingabe? Es sind max. 50 Kisten erlaubt\n";
    63. return aKiste[i];
    64. }
    66. Lager aendern(Lager aKiste[50])
    67. {
    68. int i=0;
    70. cout << "Bitte geben Sie die Nummer der Kiste ein die Sie aendern wollen: ";
    71. cin >> i;
    73. if((i<50)&&(aKiste[i].aktiv==1))
    74. {
    75. cout << "Bitte geben Sie die Hoehe der " << i << " Kiste: " << " ein.";
    76. cin >> aKiste[i].hoehe;
    77. cout << "Bitte geben Sie die Breite der " << i << " Kiste: " << " ein.";
    78. cin >> aKiste[i].breite;
    79. cout << "Bitte geben Sie die Laenge der " << i << " Kiste: " << " ein.";
    80. cin >> aKiste[i].laenge;
    81. aKiste[i].aktiv=1;
    82. aKiste[i].volumen=aKiste[i].hoehe*aKiste[i].breite*aKiste[i].laenge;
    83. cout << '\n';
    84. }
    85. else if(i>=50)
    86. {
    87. cout << "Es sind max. 50 Kisten erlaubt!\n";
    88. }
    89. return aKiste[i];
    90. }
    92. Lager loeschen(Lager aKiste[50])
    93. {
    94. int i=0;
    96. cout << "Bitte geben Sie die Nummer der zu loeschenden Kiste ein: ";
    97. cin >> i;
    98. if ((i<50)&&(aKiste[i].aktiv==1))
    99. {
    100. aKiste[i].aktiv=0;
    101. cout << '\n';
    102. cout << "Kiste nr.: " << i << " wurde geloescht!\n";
    103. }
    104. return aKiste[i];
    105. }
    107. Lager alleAnzeigen(Lager alleKisten[50])
    108. {
    109. cout <<"Kistennr.:\tVolumen:\tAktiv:\n";
    110. for (int i=0;i<50;i++)
    111. {
    112. cout << alleKisten[i].nummer << "\t\t" << alleKisten[i].volumen << "\t\t" << alleKisten[i].aktiv <<'\n';
    113. }
    114. return alleKisten[50];
    115. }
    117. int main()
    118. {
    119. Lager alleKisten[50];
    120. int auswahl;
    122. for(int i=0;i<50;i++)
    123. {
    124. alleKisten[i].volumen=0;
    125. alleKisten[i].aktiv=0;
    126. alleKisten[i].nummer=i+1;
    127. }
    129. do
    130. {
    131. cout << "Kistenlager\n";
    132. cout << '\n';
    133. cout << "Drücke (1) um eine Kiste anzulegen\n";
    134. cout << "Drücke (2) um eine Kiste anzuzeigen\n";
    135. cout << "Drücke (3) um eine Kiste zu aendern\n";
    136. cout << "Drücke (4) um eine Kiste zu loeschen\n";
    137. cout << "Drücke (5) um alle Kisten anzuzeigen\n";
    138. cout << "Drücke (6) um das Programm zu beenden\n";
    139. cin >> auswahl;
    141. switch(auswahl)
    142. {
    143. case 1:
    144. eingeben(alleKisten);
    145. break;
    146. case 2:
    147. KisteAnzeigen(alleKisten);
    148. break;
    149. case 3:
    150. aendern(alleKisten);
    151. break;
    152. case 4:
    153. loeschen(alleKisten);
    154. break;
    155. case 5:
    156. alleAnzeigen(alleKisten);
    157. break;
    158. case 6:
    159. cout << "Das Programm Kistenlager wird beendet.";
    160. exit(0);
    161. }
    163. }while (auswahl !=6);
    165. return 0;
    166. }


    Gleich beim ersten "cin" eingeben der Kiste bekomme ich den Fehler:no match for 'operator>>' in 'std::cin >> "i"'. Und ich kann mir nicht erklären woran es liegen könnte.
    Vielen Dank schon mal für eure Hilfe.

    Gruß
    Sebastian

    Edit @Sebastian.Pape

    Lager Kisteanzeigen(Lager aKiste[50])
    KisteAnzeigen(alleKisten);

    C++ ist case sensitive;


    Cloud Computer? Nein Danke! Das ist nur ein weiterer Schritt zur totalen Überwachung.
    „Wer die Freiheit aufgibt, um Sicherheit zu gewinnen, wird am Ende beides verlieren.“
    Benjamin Franklin

    Dieser Beitrag wurde bereits 4 mal editiert, zuletzt von „NoIde“ ()

    Sebastian.Pape schrieb:

    Leider kann ich meinen Code nicht ausführen und weis wirklich nicht woran es liegt.
    Was passiert?
    Was soll passieren?
    Gibt der Compiler möglicherweise Hinweise, die Du uns vorenthältst?
    Wenn ja, welche?
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    @RodFromGermany
    Das ist es ja wenn ich das Programm Testen/ausführen will passiert nix. Mir wird in Zeile 20 sofort der Rote Punkt angezeigt mit:
    Warnung:non-static data member initializers only available with -std=c++11 or -std=gnu++11 [enabled by default]
    und dem Fehler:
    Fehler:no match for 'operator>>' in 'std::cin >> "i"

    Ich stehe komplett auf dem schlauch und denk mir das die Lösung eigentlich ganz einfach ist.

    @Sebastian.Pape Dein Quellcode compiliert nicht, weil da noch Fehler drinne sind, die Du zunächst beheben musst.
    Ich nehme mal an, dass Du über die Console der Variable i einen Wert zuweisen willst.
    Probierma zunächst, die Variable direkt ohne cin zu belegen, damit Du Dich vom Laufen des Rest-Codes überzeugen kannst.
    Dann fragst Du mal Frau Google nach c++ cin und bekommst reichlich Lektüre angeboten.
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    Dein C++ ist sehr C lastig. Hier hab ich dir mal ein Beispiel gemacht mit Hinzufügen und Auflisten von Kisten. Guck dir das mal an und wenn etwas unverständlich ist dann nachfragen.
    Spoiler anzeigen

    C-Quellcode

    1. #include <iostream>
    2. #include <limits>
    3. #include <vector>
    4. #include <string>
    6. struct box
    7. {
    8. public:
    9. box() noexcept : id(++max_id), length(0), height(0), width(0) {}
    10. box(double _length, double _height, double _width) noexcept : id(++max_id), length(_length),
    11. height(_height), width(_width) {}
    12. int get_id() const noexcept { return id; }
    13. double get_length() const noexcept { return length; }
    14. double get_height() const noexcept { return height; }
    15. double get_width() const noexcept { return width; }
    16. double get_volume() const noexcept { return width * height * length; }
    17. void set_length(double l)
    18. {
    19. if (l <= 0)
    20. throw std::invalid_argument("length must be > 0");
    21. length = l;
    22. }
    23. void set_height(double l)
    24. {
    25. if (l <= 0)
    26. throw std::invalid_argument("height must be > 0");
    27. height = l;
    28. }
    29. void set_width(double l)
    30. {
    31. if (l <= 0)
    32. throw std::invalid_argument("width must be > 0");
    33. width = l;
    34. }
    35. static box box_from_cin()
    36. {
    37. double w, h, l;
    38. l = get_number("Enter length: ");
    39. w = get_number("Enter width: ");
    40. h = get_number("Enter height: ");
    41. return box(l, h, w);
    42. }
    43. private:
    44. static int max_id;
    45. int id;
    46. double width;
    47. double length;
    48. double height;
    49. static double get_number(const std::string& text)
    50. {
    51. double d;
    52. while (std::cout << text && !(std::cin >> d) || d <= 0)
    53. {
    54. std::cin.clear();
    55. std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
    56. std::cout << "Invalid input" << std::endl;
    57. }
    58. return d;
    59. }
    60. };
    61. int box::max_id = 0;
    62. std::ostream& operator<<(std::ostream& str, const box& b)
    63. {
    64. return str << b.get_id() << "\t" << b.get_volume() << "\t";
    65. }
    66. int main()
    67. {
    68. const size_t max_boxes = 10;
    69. std::vector<box> boxes;
    70. int option = 0;
    71. while (option != 3)
    72. {
    73. std::cout << "Press (1) to add a box\n";
    74. if (boxes.size() != 0)
    75. std::cout << "Press (2) to list all boxes\n";
    76. std::cout << "Press (3) to exit\n";
    77. std::cin >> option;
    78. if (option == 1) // Bei mehr optionen ruhig den switch nehmen
    79. {
    80. if (boxes.size() >= max_boxes)
    81. {
    82. std::cout << "Maximum boxes reached" << std::endl;
    83. continue;
    84. }
    85. boxes.push_back(box::box_from_cin());
    86. std::cout << "Box has been added" << std::endl;
    87. }
    88. else if (option == 2 && boxes.size() != 0)
    89. {
    90. std::cout << "ID\tVolume" << std::endl;
    91. for (auto& b : boxes)
    92. std::cout << b << std::endl;
    93. }
    94. else if(option != 3)
    95. std::cout << "Unknown option" << std::endl;
    96. }
    97. return EXIT_SUCCESS;
    98. }

    Felder von structs/classes solltest du nach Möglichkeit private oder protected lassen. Public macht nur Sinn wenn's total egal ist was, wann in das Feld geschrieben wird. Arrays sind ziemlich über. Nimm std::array<> oder std::vector<> dafür.

    /PS
    Weiß einer warum der Quellcode so komisch formatiert ist ?( Ich hab den direkt aus VS eingefügt.

    @RodFromGermany
    Danke werde ich mir anschauen^^
    @Gonger96
    ich darf/soll nur das verwenden was im Studienheft vorkommt. Vector etc. kommt erst später dran. Habe schon ein paar mal festgestellt das es bei dieses Aufgaben viel einfache und schnellere Methoden gibt wenn man nicht das nimmt was im Studienheft steht sondern das was im über oder überübernächsten steht...

    @Sebastian.Pape Was ist denn das für ein Unterrichts- bzw. Studienfach?
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

Facebook
Telefonspende