wie kann man am Besten eine Funktion erstellen, die dazu dient zu überprüfen ob alle Variablen einer Klasse anhand von Set-Methoden gesetzt wurden?

Vielen Dank,

Nadine

Man programmiert so, dass dieser Fall gar nicht vorkommen kann. Dazu dienen, unter anderem, Konstruktoren. Ein Objekt hat nach seiner Erstellung betriebsbereit zu sein. Falls dies nicht direkt möglich sein sollte (kann bei sehr komplexen Programmen manchmal der Fall sein, aber bei Anfängerprogrammen eigentlich eher nicht), dann gibt es auch Designpattern für eine stufenweise Erstellung.

wie kann man am Besten eine Funktion erstellen, die dazu dient zu überprüfen ob alle Variablen einer Klasse anhand von Set-Methoden gesetzt wurden?

Hi,
am besten gar nicht. Klingt als bräuchtest du einen Initialzustand für deine Instanzen? Dann initialisier deine Member doch beim Erstellen einer Instanz gleich korrekt.

Dass man Getter/Setter in C++ besser möglichst vermeidet (siehe vorherige Posts) sollte dir allerdings klar sein.

vielleicht sowas?

if(var1 = 0 || var2 = 0 || ... ){
  return;
}

vielleicht sowas?

Das ist mindestens auf drei Ebenen falsch.

Die Invariante kannst Du ggf. auf Gültigkeit überprüfen. Ist u.U. sinnvoll - und schadet nicht.

Fuchs aus dem Wald schrieb:

if(var1 = 0 || var2 = 0 || ... ){
  return;
}

vielleicht sowas?

Das ist mindestens auf drei Ebenen falsch.

Hab ich die richtig?

• Falsche bzw. keine Fehlerbehandlung.
• Typ der Variablen missachtet - bei chrono::duration wäre das nicht 0 sondern duration<...>::zero()
• Wert der Variablen kann auch initialisiert bzw. "gesetzt" 0 sein.

Und dann der Größte! Er nutzt ein compound-statement obwohl das gar nicht nötig ist (nur eine Zeile)!

nadine.wolken schrieb:

wie kann man am Besten eine Funktion erstellen, die dazu dient zu überprüfen ob alle Variablen einer Klasse anhand von Set-Methoden gesetzt wurden?

Das klingt nach dem XY-Problem (siehe Signatur).
Daher beschreibe mal deine Klasse, und wieso genau du das (angeblich!) brauchst.

SeppJ schrieb:

Fuchs aus dem Wald schrieb:

if(var1 = 0 || var2 = 0 || ... ){
  return;
}

vielleicht sowas?

Das ist mindestens auf drei Ebenen falsch.

Hab ich die richtig?

• Falsche bzw. keine Fehlerbehandlung.
• Typ der Variablen missachtet - bei chrono::duration wäre das nicht 0 sondern duration<...>::zero()
• Wert der Variablen kann auch initialisiert bzw. "gesetzt" 0 sein.

Und dann der Größte! Er nutzt ein compound-statement obwohl das gar nicht nötig ist (nur eine Zeile)!

Ich vermute, SeppJs Ebenen sind:

• Ganz unten: = passt hier nicht, es sollte == sein.
• Dazwischen: Wert der Variablen kann auch initialisiert bzw. "gesetzt" 0 sein.
• Oben: Das ganze Vorhaben ist unsinnig.

(Ja, ich habe schon verstanden, dass Sones Antwort Ironie war )

volkard 1:0 Sone

(Ja, ich habe schon verstanden, dass Sones Antwort Ironie war )

Gut. Es wäre etwas zu offensichtlich gewesen hinzuschreiben dass er den falschen Operator nutzt

P.S.: Mal ganz nebenbei, Volkard, ***?

Vielen Dank für eure Antworten erstmal. Ich dachte eher an eine Methode, die einen boolschen Wert zurückgibt, der besagt ob diese bestimmten variablen durch die Set-Methode gesetzt worden sind.

Pro Variable eine Zusatzvariable vom Typ bool. Und beim Aufruf einer Set-Methode wird die Variable gesetzt und begleitend der bool auf true gesetzt.
(Aber das riecht echt nach Designfehler, sorry.)

Danke vielmals,

Nadine

@volkard: Danke für die Antwort erstmal. Wie meinst du pro Variable eine Zusatzvariable? Kannst du mir vllt. ein beispiel geben oder so?

Danke vielmals,

Nadine

class AutoDesignfehler
{
private:
   double masse;
   bool masseGesetzt;
   int anzahlSitzplautze;
   bool anzahlSitzplautzeGesetzt;
public:
   Auto(){
      masseGesetzt=false;
      anzahlSitzplautzeGesetzt=false;
   }
   bool istMasseGesetzt(){
      return masseGesetzt;
   }
   void setzeMasse(double masse){
      this->masse=masse;
      masseGesetzt=true;
   }
   double getMasse(){
      if(!masseIstGesetzt)
         throw logic_error("Och nööö!");
      return masse;
   }
   //selbes Spiel mit Sitzplätzen
}

nadine.wolken schrieb:

Vielen Dank für eure Antworten erstmal. Ich dachte eher an eine Methode, die einen boolschen Wert zurückgibt, der besagt ob diese bestimmten variablen durch die Set-Methode gesetzt worden sind.

Pro Variable eine Zusatzvariable vom Typ bool. Und beim Aufruf einer Set-Methode wird die Variable gesetzt und begleitend der bool auf true gesetzt.
(Aber das riecht echt nach Designfehler, sorry.)

Gute Idee, aber wie fragt man ab, ob die bool-Variablen korrekt gesetzt worden sind?

Gute Idee, aber wie fragt man ab, ob die bool-Variablen korrekt gesetzt worden sind?

Man erzwingt einfach, daß sie korrekzt gesetzt wurden. Dafür ist der Konstruktor da.
Aber warum erzwingt man nicht einfach auch, daß die Hauptvariablen korrekt gesetzt wurden? Wäre viel besser, als irgendwo zur Laufzeit Exceptions zu werfen.

void setzeSitzplaetze(int sitzplaetze){
      this->anzahlSitzplautze = sitzplaetze;
      anzahlSitzplautzeGesetzt = true;
   }

   bool istSitzplaetzeGesetzt(){
      return anzahlSitzplautzeGesetzt;
   }

Wäre das so ok für die Sitzplaetze?

Das heisst also die daten werdem im Konstruktor übergeben? Nur wenn die Übergabe erfolgt kann ein solches Objekt erst generiert werden?

Ja, das wäre die saubere Lösung.
Objekte versprechen, daß sie korrekt initialisiert wurden.

Dann gibt es auch keine Frage, ob die Daten alle da sind. Sie sind da.
Man darf alle Methoden aufrufen, weil das Objekt vollständig lebt.

Entsprechend fragt man auch nicht, ob ein Klassenobjekt lebt. Wenn es existiert, dann lebt es. Dann wurde es korrekt initialisiert.

Im Zusammenhang mit Zeigern kann man fragen, ob der Zeiger auf ein existierenden Objekt zeigt oder auf NULL, wo kein Objekt wohnen darf.

Jester schrieb:

Gute Idee, aber wie fragt man ab, ob die bool-Variablen korrekt gesetzt worden sind?

Man erzwingt einfach, daß sie korrekzt gesetzt wurden. Dafür ist der Konstruktor da.

oder eben doch ein "bool masseGesetztGesetzt"?

@Sone: aber man kann die Abfrage != NULL mit structs nicht machen.

Ähm, Nadine,
Diese Abfrage sollst du eigentlich nur bei Zeigern anwenden*.
Und technisch gesehen kannst du das auch nur bei skalaren Typen ( int , float , char , Zeiger, .....), solange NULL eben einfach 0 ist.

Das geht also bei Klassen, egal mit welchem class-key deklariert, nicht.

Wie bereits alle anderen gesagt haben, muss man das ganz anders angehen.

~*In C++11 ist es auch nicht NULL sondern nullptr.~