spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Meine C++ Zeit liegt zwar schon eine Weile zurück, aber für mich sieht der Iterator eigentlich richtig aus.
Was du aber mal machen könntest: QRect hat eine contains() Methode, die genau das machst, was du da versuchst.

Was ich mich aber gerade frage: Warum übergibst du den Iterator als Parameter an die Funktion?

Debug mal und schau ob da wirklich mehrere Elemente in der Map sind.
Sind die Mauskoordinaten korrekt?
Bei deiner if-Bedingung ist ein Tile jedoch nicht 32 Einheiten, sondern 62 Einheiten groß!
Dank C++11 kannst du das Ganze übrigens auch auf folgendes kürzen:

Der Name der Fuktion irritiert mich im Übrigen, wenn ich auf ihren Typen gucke.

MfG
Check

Du weißt, wie groß deine Tiles sind. Warum errechnest du nicht einfach die Koordinaten und darüber den Index?
das würde dann ungefähr so aussehen:
tileX = mouseX / tileWidth
tileY = mouseY / tileHeight
Sollte der Klick auch für Bereiche abgefangen werden, auf dem keine Tiles dargestellt werden, muss außerdem noch eine Prüfung durchgeführt werden, ob die Tilekoordinaten sich im gültigen Bereich befinden.
Auch wäre es sinnvoll zu gucken, ob man nicht Events für das Klicken auf einen bestimmten Tile bekommen könnte, statt für den Klick auf die Fläche, auf der alle dargestellt werden. (Dann müsste diese Umrechnung oben nicht durchgeführt werden.)

Da bei dir die Tiles in einer Datenstruktur mit einem int-Index liegen, sollten alle Tiles auch geordnet hintereinander liegen (ansonsten solltest du ohnehin überdenken, was du bisher gemacht hast...)
An den Index würdest du dann entweder über
index = tileX + tileY * columns
oder
index = tileX * rows + tileY
kommen.

Die Methode "GetTileId" sollte als Parameter die Koordinaten des Tiles (nicht der Maus o. ä.) erwarten und dann die ID zurückliefern (wobei es sich wohl anbieten würde, die Informationen über ein Tile in bspw. einer struct oder einer Klasse zusammenzufassen und diese zurückzuliefern).
Die Umrechnung der Mauskoordinate in die Tilekoordinate sollte von der Oberfläche durchgeführt werden. Die Datenstrukturen im Hintergrund haben rein gar nichts mit der Visualisierung zu tun. Sollte an der Oberfläche mal ein Zoom eingebaut werden, muss so auch wirklich nur die Oberfläche (in dem Fall die Umrechnung der Mauskoordinate) angepasst werden und der Rest würde weiterhin funktionieren.

Und was das zu faul sein im Zusammenhang mit dem Rückgabetyp angeht:
Ich fange auch oft an, eine Methode zu schreiben, ohne eine sinnvolle Rückgabe zu haben.
ABER: es ist bereits die Rückgabe eines "Standardwerts" vom richtigen Typ vorhanden (null, 0, -1, Leerstring, ...)

Vielleicht nicht ganz zum Problem passend, aber: Warum übergibts du den Iterator als Parameter?
Er wird eh überschrieben und zum Ende hochgezählt. Die aufrufende Funktion hat dann einen Iterator
der aufs Ende zeigt bei sich rumfliegen. Darin sehe ich gerade keinen potenziellen Sinn.

Klingt nicht so gut, weil du damit Logik und Darstellung vermischst.

Zitat von »abcd00186«

Ich habe es jetzt anders gelöst und die TileMap in ein QTableWidget geladen, dadurch kann ich dann die aktuelle zeile und spalte herausfinden und damit dann den Index ausrechnen.
Danke für die ganzen Antworten

Zitat von »David Scherfgen«

Klingt nicht so gut, weil du damit Logik und Darstellung vermischst.

Das wiederum kann ich nicht ganz nachvollziehen. Liegt es an dem QTableWidget und irgendwelcher Magie, die dieses im Hintergrund anwendet? (Ich habe noch nicht mit Qt gearbeitet also könnte ich nur anhand vergleichbarer Konstrukte herleiten, was das Verhalten ist.) Oder an der Ermittlung des Index?

Wenn es die Tabelle sein soll, dann wäre es auch ganz gut zu wissen, was eine bessere Alternative darstellen soll. Wenn man sich um das Zeichnen gänzlich selbst kümmert und ein Element nimmt, welches beim Anklicken nicht zwischen Unterelementen unterscheidet, muss man sich wieder selbst um die Umrechnung der Mausposition zu Tile-Koordinate kümmern. (Bei einer Tabelle gehe ich auch davon aus, dass man den Inhalt der Zellen separat zeichnet und nicht die Tiles als Tabellenhintergrund, um dann die Zellen nur für den Input zu verwenden.)

Sollte es die Umrechnung (Tilekoordinate zu Index) sein, so sehe ich nicht, dass die Beschreibung seinerseits auf eine Vermischung schließen lässt. Das Ding, was die ganzen Tiles verwaltet, sollte nur über die Tilekoordinaten angesprochen werden, die Formulierung lässt aber nicht eindeutig darauf schließen, dass die Ermittlung des intern verwendeten Index nach außen verlagert werden soll.