Run-Time Check Failure #2 - Stack around the variable 'Karte' was corrupted. - Seite 2 - Programmierung und Informatik

Treue Seele

Wohnort: Emsdetten

Beruf: Softwareentwickler

16.01.2011, 14:13

Howdi.

ich habs nicht als Angriff gesehen

Also wenn du sowieso nach jeder Funktion die ein false zurück liefert eine Fehlerausgabe machst wäre es da nicht besser ein vernünftiges ExceptionSystem zu basteln oder gar gleich eine direktere Fehlerbehandlung?

Deine Variante erscheint mir etwas.. seltsam, zb würde ich dies an deinem Beispiel verdeutlichen:

C-/C++-Quelltext

bool CCharakter::Func_Erhöhe_Koerperkraft()
{
  if (i_AttributPunkte)
  {
    i_Koerperkraft++;
    fl_TragKraftMax = i_Koerperkraft * 2000.0f;
    i_AttributPunkte--;
    return true;
  }
  else
  {
    // Keine Punkte zum Erhöhen
    *Fehler = St_FehlerCode::FC_CHAR_KEINEATTRIBUTSPUNKTE;
    return false;
  }
}

Würde ich da eher so schreiben:

C-/C++-Quelltext

void CCharakter::Func_Erhöhe_Koerperkraft()
{
  if (i_AttributPunkte == 0)
    throw NoAttributePointsException();

  i_Koerperkraft++;
  fl_TragKraftMax = i_Koerperkraft * 2000.0f;
  i_AttributPunkte--;
}

oder so:

C-/C++-Quelltext

void CCharakter::Func_Erhöhe_Koerperkraft()
 {
   if (i_AttributPunkte == 0)
   {
    OnError(St_FehlerCode::FC_CHAR_KEINEATTRIBUTSPUNKTE);
    return;
  }
 
   i_Koerperkraft++;
   fl_TragKraftMax = i_Koerperkraft * 2000.0f;
   i_AttributPunkte--;
 }

Es ist natürlich eine grosse Portion persönlicher Geschmack dabei, aber diese Varianten hätten den Vorteil das nicht extra mit FehlerCode Pointern gearbeitet werden muss und auch Mechaniken verwendet werden die genau für die Fehlerbehandlung erzeugt wurden. Auch ist die Fehlercode Behandlung nicht wirklich ein Teil deiner Karte, bzw es sollte nicht die Aufgabe der Karte sein die Fehlerbehandlung zu übernehmen. Aus diesem grund würde ich auch eher die 2. variante bevorzugen wenn diese Fehler an den Benutzer gemeldet werden müssen.

Zum Seitenanfang

Alter Hase

16.01.2011, 14:43

Also der Klassenname Karte ist bissi Irreführend. Anfangs war sie gedacht um wirklich nur die Karte zu laden und anzuzeigen ect.
Aber derzeit hat sich die Klasse als meine Eigentliche Gameengine entwickelt sprich in dieser Klasse laufen alle Fäden zusammen Deshalb wird dort auch der Fehler Ausgewertet.

Also von Exception war ich nicht so angetan, da mir da in Erinnerung ist das sie zwar zum Fehler abfangen da sind aber das Programm beenden. Kann mich natürlich auch täuschen so tief bin ich in das Thema (Exception) nicht eingestiegen.

Ich will dem Anwender ja nur Sagen: Das z.B. Er einen Gegenstand nicht Anlegen kann weil der Platz Belegt ist oder das er Einen Wert nicht erhöhen kann weil ihm die Punkte dafür fehlen.

Ich muß das noch mal Anschau mit den Exeption.

Wer aufhört besser werden zu wollen hört auf gut zu sein!

aktuelles Projekt:Rickety Racquet

Zum Seitenanfang

Treue Seele

Wohnort: Emsdetten

Beruf: Softwareentwickler

16.01.2011, 14:53

Dann baue doch für sowas Kontrollfunktionen ein:

	C-/C++-Quelltext
1 2 3 4	if (!CanUseAttributePoints()) Fehlerbehandlung(); else BenutzeAttributspunkt();

Zum Seitenanfang

Alter Hase

16.01.2011, 14:57

Naja irgendwie kommt mir das bekannt vor

	C-/C++-Quelltext
1 2 3 4 5	if(!Spieler.Func_Erhöhe_Koerperkraft()) { CKarte::Func_Fehler_Auswerten(); }

So habe ich es in meinem Programm bisher.

Aber ich danke dir für dein Vorschlag ich werde mich da nochmal einlesen und dann entscheiden wie ich es mache.

Wer aufhört besser werden zu wollen hört auf gut zu sein!

aktuelles Projekt:Rickety Racquet

Zum Seitenanfang

Sylence

Community-Fossil

Beiträge: 1 663

Beruf: Softwareentwickler

17.01.2011, 10:17

Zitat von »Koschi«

Also von Exception war ich nicht so angetan, da mir da in Erinnerung ist das sie zwar zum Fehler abfangen da sind aber das Programm beenden.

Unbehandelte Ausnahmen führen zum beenden des Programms.

Zum Seitenanfang

Treue Seele

Wohnort: Emsdetten

Beruf: Softwareentwickler

17.01.2011, 10:20

Vom Design her wäre es natürlich viel Sinnbehafteter wenn man dem user gar nicht erst die Möglichkeit gibt, dumme Dinge anzustellen. Sprich: Wenn der Knopf zum erhöhen der Körperkraft nicht angezeigt wird oder nicht anklickbar ist (disabled). Zusätzlich könnte ein Label anzeigen das nicht genügend Attributspunkte verfügbar sind, damit der Benutzer eine Erklärung hat, wieso denn dieser Knopf nicht benutzbar ist.

mfg Mark

Zum Seitenanfang

Alter Hase

17.01.2011, 20:33

Daran hatte ich noch gar nicht gedacht. Danke.
Muß ich mir durch den Kopf gehen lassen wie ich das umsetzen kann.

Gruß Koschi

Wer aufhört besser werden zu wollen hört auf gut zu sein!

aktuelles Projekt:Rickety Racquet

Zum Seitenanfang

MCP

Alter Hase

Beiträge: 513

Wohnort: Paderborn

Beruf: Software-Entwickler

18.01.2011, 22:14

Was mir hier in der Diskussion noch fehlt:
Wenn der Pointer Fehler auf 0 gesetzt wird, ohne das vorher das delete ausgeführt wird, hat man ein schönes Speicherleck. Also sollte man diese Stelle auf jeden Fall überarbeiten. Ein delete auf einen 0-Pointer hat keinen Effekt, der Speicher der mit new reserviert wurde wird dabei jedoch nicht gelöscht.

Zum Seitenanfang

Alter Hase

19.01.2011, 12:30

Also erst delete den pointer (Der mit new erzeugt wurde) und dann auf 0 Setzen bzw neue Adresse zuweisen oder ?

Merke ich mir mal für später

Danke

Wer aufhört besser werden zu wollen hört auf gut zu sein!

aktuelles Projekt:Rickety Racquet

Zum Seitenanfang

Treue Seele