Skip to content

Harry#1

Open
hsweet wants to merge 2 commits intomasterfrom
Harry1
Open

Harry#1
hsweet wants to merge 2 commits intomasterfrom
Harry1

Conversation

@hsweet
Copy link
Member

@hsweet hsweet commented Oct 15, 2016

The basic game engine seems to work. It still needs a lot of fleshing out. Might be a good idea to have the room descriptions in a text file that the game loads into the dictionary.

Removed room class, added a few simple methods to the player class.  The basic structure of the game is in place, but it needs a working game loop, some more working methods for the player, and a more interesting house.
Working game loop, and a few simple methods for the player.  Also made sure to use player.room everywhere instead of the room variable.  The game could work using room, but only for a single player.
@hsweet hsweet changed the title Harry1 Harry Oct 22, 2016
@TheArle
Copy link

TheArle commented Oct 26, 2016

i dont have write access to roomgame.py

@hsweet
Copy link
Member Author

hsweet commented Oct 27, 2016

Hey Ar..

You were right, access wise. I had everyone set up with read only. You
all should have write access now.

Remember to write some rooms with Rita, and Blaze 2 if he is willing..

And keep your eyes on the goal for the class, which is not to write your
game, but to give you some more (Python) tools to use to write your
version.

If we can make a solid game engine with a few rooms, you fork it off and
build on it.

Finally, walking the dog I remembered there is a thingie called a lambda
function that might or might not work inside a structure like a dictionary.
Interesting question. But more advanced, even if it could work. One
function per room will work and is easy. Only problem is that def 1():
won't work, buit def room1(): is fine, or def r1():

On Wed, Oct 26, 2016 at 12:12 PM, TheArle notifications@github.com wrote:

i dont have write access to roomgame.py


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFK8k6KyEkbAoWsc_qnSvCZCJrWDmph8ks5q33vggaJpZM4KXsay
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants