Software: - C++ - Probleme

[Der_Mosch]

Ich Idiot hab meine Konstanten falsch initialisiert. Ich kanns nicht glauben, dass mir diese diese OFFENSICHTLICHE SCHEISSE nicht aufgefallen ist. Und ich hab mir das extra noch danebenkommentiert!

Kann den Thread mal einer löschen, ist ja peinlich

[Gonzo]

lol, mach dir nix draus, passiert jedem mal

[MyersGer]

Nur wegen offensichtlicher Peinlichkeiten, die der Unterhaltung der Allgemeinheit dienen, werden keine Threads gelöscht. Sonst müsste man hier ja viel zu viel löschen

Außerdem hast du deine Konstanten nicht falsch initialisiert. Man sollte einfach nur < statt <= verwenden. Dann muss man weniger nachdenken Sonst muss du bei der Array Definition immer noch +1 rechnen. Sieht auch doof aus und man weiß unten nicht, was man oben geschrieben hat.

[Der_Mosch]

Zitat von MyersGer

Außerdem hast du deine Konstanten nicht falsch initialisiert. Man sollte einfach nur < statt <= verwenden. Dann muss man weniger nachdenken

Ich weiss, dass das ne Alternative wäre, nur stört es mich, dass die Spielfeldgrösse dann ein Punkt niedriger aussieht, als sie es eigentlich ist (äh... du weisst, was ich meine).

[MyersGer]

Ähm ne nicht ganz Auf jeden Fall würde das Programm genauso abstürzen, wenn du das machst, was du in deinem vorherigen Post geschrieben hast, also das Ändern der Konstanten.

Du müssteste ja

const int GAME_ARRAY_WIDTH = 8;
const int GAME_ARRAY_HEIGTH = 5;

int gameArray[GAME_ARRAY_WIDTH+1][GAME_ARRAY_HEIGTH+1];

for (int y = 0; y <= GAME_ARRAY_HEIGTH; ++y)

oder

const int GAME_ARRAY_WIDTH = 9; //Jeweils +1, da es sich um
const int GAME_ARRAY_HEIGTH = 6;//Angaben für ein Array handelt

int gameArray[GAME_ARRAY_WIDTH][GAME_ARRAY_HEIGTH];

for (int y = 0; y <= GAME_ARRAY_HEIGTH-1; ++y)

(hier haben die beiden Kommentare dann den Sinn, den sie eigentlich beinhalten)
schreiben, damit es zu keinem Speicherzugriffsfehler kommt.

Da sieht

const int GAME_ARRAY_WIDTH = 8;
const int GAME_ARRAY_HEIGTH = 5;

int gameArray[GAME_ARRAY_WIDTH][GAME_ARRAY_HEIGTH];

for (int y = 0; y < GAME_ARRAY_HEIGTH; ++y)

irgendwie am sinnvollsten aus. Oben schreibt man hin, wie groß das Spielfeld sein soll und so groß isses dann auch. Quasi WYSIWYG


Oder ist es gerade einfach noch zu früh für mich und ich habe deinen Post nicht ganz verstanden?!

[Der_Mosch]

Äh... nein. Das Spielfeld ist 10*7 Felder gross Ich glaube, dass dich mein Kommentar verwirrt hat.

Und ich arbeite nicht in Wysiwyg. Ich arbeite mit Wygiwyd.

[Gonzo]

Wenn es 10*7 sein soll, dann sollte man auch 10*7 benutzen

const int GAME_ARRAY_WIDTH = 10;
const int GAME_ARRAY_HEIGTH = 7;

int gameArray[GAME_ARRAY_WIDTH][GAME_ARRAY_HEIGTH];

for (int y = 0; y < GAME_ARRAY_HEIGTH; ++y)
...

einfach und sauber und letztendlich nicht verwirrend wenn du dass in nen halben Jahr noch mal liest
Warum benutzt du eigentlich const int ? Wäre ein #define GAME_ARRAY_WIDTH 10 im Headerfile nicht praktischer ? Ist dann global bekannt und lässt sich dann auch in anderen Projektdateien leichter weiterverwenden ( z.B. #include "main.h" )
Nebenbei gesagt finde ich die Schreibweise ++y blöd, y++ liest sich irgendwie besser

[MyersGer]

In diesem Fall ja, bei überladenen Operatoren nicht. Da gibt es dann sogar Performance Unterschiede. Mit dem Postfix-Operator kann man aber bei Rechenoperationen auf die Nase fallen.
Bei For-Schleifen liest sich ein Postfix-Operator aber sicherlich besser, gerade wenn man an seine Mitmenschen denkt . Ansonsten: klarer Fall von Geschmackssache.

[Der_Mosch]

Zitat von Gonzo

Warum benutzt du eigentlich const int ? Wäre ein #define GAME_ARRAY_WIDTH 10 im Headerfile nicht praktischer ? Ist dann global bekannt und lässt sich dann auch in anderen Projektdateien leichter weiterverwenden ( z.B. #include "main.h" )

Erstens kommt mir das nicht wie guter Stil vor, zweitens ist das keine Konstante, sondern, wie du sicherlich weisst, eine Compilerdirektive.

Zitat von Gonzo

Nebenbei gesagt finde ich die Schreibweise ++y blöd, y++ liest sich irgendwie besser

Macht in diesem Fall keinen Unterschied.

gers, schöne Werbung. Hat funktioniert

[AMD-Powered]

Zitat von Der_Mosch

Kann den Thread mal einer löschen, ist ja peinlich

NÖ!!!

Will ich doch nicht dafür verantwortlich sein, dass du nochmal nach der Lösung suchst und nen Thread erstellst...

ergo:

... er bleibt.

Hier sind schon ganz andere Fragen gestellt worden und peinlich brauchts dir auch nicht sein. Zudem waren bestimmt schon 150.000 Google Bots da und haben alles gespeichert.


EDIT: MyersGer auch mal im Lande? Schon lange nichts mehr von dir gelesen.

[MyersGer]

Kommt drauf an, wo man liest. Allgemein habe ich allerdings eher eine lesende Rolle eingenommen. Und Aceton und was sonst so abgeht verschlingt auch einiges an Zeit. Da bleibt halt zum Schreiben nicht mehr viel Zeit.

[AMD-Powered]

Ist schon ok, hat mich ja auch nur so gefreut mal wieder was von dir zu lesen, dass ich dich irgendwie aus der Reserve locken musste um dir noch nen Post zu entlocken.

Hat ja auch geklappt. ^^


Ich lese im Moment auch mehr als ich schreibe.