Um kurz zu zeigen, was passieren soll, hier ein kleiner Screenshot, wie das ganze aussieht:
http://i.imgur.com/Hf6bH.jpg
(später kommt dann ein Punkt hinzu, den man durch die zufällig generierten Balken durchsteuern muss)
Das Problem ist beim Generieren der Balken. Die werden in einem int-array gecacht (die y-Position von oben). Das funktioniert alles so weit, nur die ersten zwei haben komplett andere Werte, so als würde ich auf eine uninitialisierte Variable zugreifen.

Der relevante Code ist hier (Balken habe ich im Code "pipes" genannt):

WorldProvider.h:

class WorldProvider {
private:
	int* heightMap;
        // ...
public:
	static const int pipeWidth = 30;
	static const int pipeHeight = 300;
	WorldProvider(int displayWidth, int displayHeight);
	~WorldProvider();
	void getPipeBounds(int i, RECT& rect);
};

WorldProvider.cpp:

WorldProvider::WorldProvider(int displayWidth, int displayHeight) {
        // ...
	// int variable, die nicht so wichtig ist
	visiblePipes = ceil(displayWidth/pipeWidth);

	heightMap = new int[visiblePipes];
	for(int i=0; i<visiblePipes; i++){
		heightMap[i] = -1;
	}
        // ...
}

WorldProvider::~WorldProvider(){
	delete[] heightMap;
}

void WorldProvider::getPipeBounds(int i, RECT& rect) {
	if(i >= visiblePipes)
		return;

        // Wenn Standardwert -> zufälligen Wert setzen
	if(heightMap[i] == -1){
		cout << "Setting pipe " << i << endl;
		heightMap[i] = rand() % (displayHeight-pipeHeight);
	}

        // das übergebene RECT mit den Werten füllen
	int offset = heightMap[i];
	rect.left = i*pipeWidth;
	rect.right = rect.left + pipeWidth;
	rect.top = offset;
	rect.bottom = rect.top + pipeHeight;

}

Kurze Erläuterung:
Im Konstruktor erstelle ich das array und fülle es mit dem default-Wert -1.
Später, wenn beim Zeichnen des Bildschirms nach den Abmessungen gefragt wird (getPipeBounds) und der Wert des Balkens noch -1 ist, wird er mit einem Zufallswert belegt (und eine Debug-message ausgegeben).

Das Problem ist jetzt nur generell, dass bei jedem Programmstart immer die Werte von heightMap[0] und heightMap[1] in getPipeBounds irgendwelche sind, und nicht -1, dadurch werden sie auch nicht gefüllt und der Balken nicht angezeigt (wie im Screenshot zu sehen).

Wenn ich mir die Werte mit cout ausgeben lasse, sind sie nach dem initialisieren im Konstruktor wie erwartet -1, aber in getPipeBounds eben jene unerwarteten Werte.
Ich kann mir das beim besten Willen nicht erklären, da nirgends sonst auf die Variable zugegriffen wird (ist ja auch private), und ich sie eigentlich (meines Wissens) konform zugewiesen habe.

Ich hoffe, irgendwer hat eine Idee. Danke schon mal

lg
Bone

Ergebnisse:
Direkt nach dem Initialisieren im Konstruktor wird für "*heightMap" wie erwartet -1 angezeigt (der erste Wert im Array).

Ganz am Anfang von getPipeBounds ist er schon "korrupt", in diesem Beispiel hatte er den Wert 9765060. Nach der if-Abfrage (i >= visiblePipes) genau dasselbe.

Also im Prinzip die gleichen Ergebnisse, die ich durch Ausgeben festgestellt hatte.

Ansonsten könntest du mal zeigen wie du den WorldProvider in deinem restlichen code verwendest?

private:
  WorldProvider(WorldProvider const&);
  WorldProvider& operator=(WorldProvider const&);

in der Klassendefinition hinzu, ohne diese Dinger zu definieren. Lässt sich das Programm dann noch kompilieren und linken?

@firefly
In einer anderen Klasse (die die Game-Ticks übernimmt), wird zuerst ein (privater) Pointer auf WorldProvider deklariert:

WorldProvider* world;

Wenn das Game gestartet wird (der User drückt Enter), wird die Methode GameTicker::initGame() aufgerufen, die folgendes macht:

void GameTicker::initGame() {
	if(phase == PHASE_GAME)
		return;
	phase = PHASE_GAME;
	RECT winRect;
	GetWindowRect(hWnd, &winRect);
	world = new WorldProvider(winRect.right - winRect.left,	winRect.bottom - winRect.top);
}

Im Dekonstruktor von GameTicker wird einfach "delete world" aufgerufen.

@krümelkacker
Nein, beim Compilen kommt folgender Fehler:

..\src\/WorldProvider.h:8:2: error: 'WorldProvider::WorldProvider(const WorldProvider&)' is private
..\src\GameTicker.cpp:59:10: error: within this context

GameTicker.cpp Linie 59 ist der getter für die Welt und dereferenziert die Welt:

WorldProvider GameTicker::getWorld() {
	return *world;
}

(ist das eigentlich eine gute Idee? Anscheinend wird der Copy-Konstruktor aufgerufen?)

[...]/GameTicker.cpp:59: undefined reference to `WorldProvider::WorldProvider(WorldProvider const&)'

(selbe Linie wie gerade eben, wo "*world" aufgerufen wird)

Danke auf jeden Fall schon mal für eure Hilfe!

....
Ich glaube, ich hab das Problem gelöst! Ich hab jetzt die Methode getWorld() dementsprechend umgeändert, dass sie nur den Pointer zurückgibt, und nicht die Welt selbst. (WorldProvider* als Rückgabetyp)
Jetzt scheint alles zu funktionieren, ich hab nur keine Ahnung warum

Könnte mir das jemand erklären?

Abhilfe: Die naive Lösung wäre natürlich, keine flache Kopie zu machen. Aber das eigentliche Problem ist, warum du überhaupt ein dynamisches Feld selber verwaltest. new[] und delete[] kommen wirklich nie in C++ vor (Eine einzige Stelle (eigener Allokator) fällt mir ein, aber das ist sehr exotisch). new[] und delete[] sind die deutlichsten Anzeichen von allen, dass man einen std::vector möchte.

@krümelkacker
Nein, beim Compilen kommt folgender Fehler:

..\src\/WorldProvider.h:8:2: error: 'WorldProvider::WorldProvider(const WorldProvider&)' is private
..\src\GameTicker.cpp:59:10: error: within this context

GameTicker.cpp Linie 59 ist der getter für die Welt und dereferenziert die Welt:

WorldProvider GameTicker::getWorld() {
	return *world;
}

Da haben wir das problem schon, getWorld liefert laut methoden signatur eine kopie des Objektes WorldProvider zurück. Ist das von dir gewollt?
Da du keinen Kopier bzw. Zuweisungs Operator für die Klasse erstellt hast, erstellt der Kompiler diese. Und in diesen generierten Versionen werden die einzelnen Member kopiert (bei deinem array nur der pointer wert und nicht das Array als solches).

new[] und delete[] scheint ja nicht das Problem gewesen zu sein, aber ich sollte wohl mir mal den std::vector ansehen.

Danke für eure Hilfe, das Problem ist gelöst

new[] und delete[] scheint ja nicht das Problem gewesen zu sein, aber ich sollte wohl mir mal den std::vector ansehen.

Wie man's nimmt. Es war ein direkter Folgefehler, weil du mit rohen Zeigern rumhantierst. Gerade wenn du von Java kommst, solltest du in C++ vorsichtig sein mit Java-Techniken. Als Java-Programmierer solltest du zum Beispiel auch normales new und delete und alle Zeiger als Alarmsignal ansehen, dass du gerade Java in C++ machen möchtest (was keine gute Idee ist).

Als Java-Programmierer solltest du zum Beispiel auch normales new und delete und alle Zeiger als Alarmsignal ansehen, dass du gerade Java in C++ machen möchtest (was keine gute Idee ist).

Ich hatte keine andere Lösung gefunden, um world nicht schon im Konstruktor zu initialisieren, als einen Pointer (und damit new/delete) zu verwenden.

Die Welt soll ja erst erstellt werden, wenn initGame() aufgerufen wird, allerdings muss sie auch außerhalb von initGame verfügbar sein (kann also keine statische Variable sein). Wie funktioniert das sonst außer mit new?

Wobei es etwas ungewöhnlich ist, zu einem Objekt nochmal eine weitere Initialisierungsmethode anzubieten. Sollte das nicht der Konstruktor machen?

Ich hab mich jetzt auf das Erstellen von WorldProvider mit new bezogen. Das ist ja nur ein einzelnes Element, wieso bräuchte ich dann da einen vector?

world = new WorldProvider(winRect.right - winRect.left,    winRect.bottom - winRect.top);

Warum ich eine initialisierungs-methode verwende? Weil die GameTicker Klasse schon ganz am Anfang erstellt wird, da wird allerdings erst noch der Splash-Screen angezeigt. Erst wenn man Enter drückt, wird das Game gestartet und der WorldProvider soll erstellt werden (initGame).

In GameTicker ist gespeichert, welche Phase aktuell gerendert werden soll (PHASE_SPLASH für den Splash-Screen oder PHASE_GAME für die eigentliche Spieloberfläche).

Wenn gerade das Spiel läuft, wird über WorldProvider der Bildschirm gerendert.
game.getWorld()->getPipeBounds(i, pipeRect);

"getWorld" muss ja irgendwo seine WorldProvider-Instanz herbekommen. Und die ist an das GameTicker Objekt gebunden - also ein Member davon.

Allerdings darf die ja erst da sein, wenn das Game gestartet wird.

Wie soll ich "world" NICHT im Vorraus "erstellen", wenn es eine Member-Variable ist?