Timeline for Hunt the Wumpus Game (C++ with QT)
Current License: CC BY-SA 4.0
7 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Dec 10, 2019 at 19:31 | vote | accept | Sandro4912 | ||
| Dec 10, 2019 at 16:51 | comment | added | Sandro4912 | I merged the commit. thanks for many good advice's. | |
| Dec 10, 2019 at 11:46 | comment | added | Edward |
I've sent you a pull request to show how to turn Room * into Room as per my suggestion above. As for the memory leak, I prefer your solution.
|
|
| Dec 9, 2019 at 9:23 | comment | added | Sandro4912 |
and good idea with the temporary array. I thought i could not use random_shuffle because the rooms are already connected. I updated the repo with the suggested changes.
|
|
| Dec 9, 2019 at 9:19 | comment | added | Sandro4912 |
for the memory leak. I think it would be more QT like to assign the QActions Room as a parent in the constructor. like mGuessWumpusAction = new QAction{tr("Has &Wumpus"), this};
|
|
| Dec 9, 2019 at 9:04 | comment | added | Sandro4912 |
Can it really work with std::array<Room, mCountOfRooms> mRooms; ? If I do that I cannot do auto room = qobject_cast<Room *>(sender()); in showLinesToNeigboursOfRoom. I modified it to use std::array<Room *, mCountOfRooms> mRooms It looks like it works but in the constructor of the array I have to type 20 times new Room
|
|
| Dec 7, 2019 at 22:49 | history | answered | Edward | CC BY-SA 4.0 |