S
Nexus schrieb:
Mir sind noch einige kleinere Dinge aufgefallen, die nicht allzu tragisch sind, aber ich möchte sie trotzdem mal erwähnen:
#ifndef _MIETER_H_
#define _MIETER_H_
Du solltest Bezeichner nicht mit Unterstrich und Grossbuchstaben beginnen, da diese für den Compiler und die Implementierung reserviert sind. Gleiches gilt für Bezeichner, die zwei aufeinandergefolgte Unterstriche enthalten. Nimm also zum Beispiel MIETER_H , oder noch besser ein längerer, spezifischerer Name, damit es auch ja nie Namenskonflikte gibt.
#include <iostream>
Brauchst du das wirklich im Header? Ich versuche bei mir immer, Includes möglichst nur dort zu verwenden, wo sie auch benötigt werden.
~Mieter();
Ein einzelner Destruktor ist etwas merkwürdig; nach der Regel der Grossen Drei solltest du Kopierkonstruktor und Zuweisungsoperator ebenfalls überladen oder verbieten. Wenn dein Destruktor nichts tut, kannst du ihn weglassen, dann reicht der standardgenerierte aus.
Mieter MietArray[30];
for(int i = 0; i < 30; ++i)
MietArray[i];
Hier bist du wahrscheinlich noch nicht fertig, denn der aktuelle Code ist etwas fragwürdig...
Dravere schrieb:
@skullyan,
Ehm und weil Vorname und Name nicht gefunden wurden, hast du ein [i+1] vorne eingefügt? Oder wie soll man dies Logik verstehen?
Und wieso hast du uns nicht gleich deine eigentliche Sorge mitgeteilt? Jetzt bin ich noch verwirrter über diesen Thread ...
Siehst du seine Signatur?
kompilierbar != logisch
Er hat hier wohl versucht, Priorität auf kompilierbar zu setzen...
Das mit dem Destruktor stimmt, er wurde automatisch generiert als ich eine Klasse erstellt habe (VC++ 2008). Die includes sollte ich tatsächlich nur da verwenden, wo sie benötigt sind, danke! Die Bezeichner sind in meinem Buch genauso definiert, aber ich denke, da schlägt mich mal wieder eure Erfahrung.
