-
Notifications
You must be signed in to change notification settings - Fork 49
Add a Text-Based Casino Game for FogOS #81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
redoing from the repo i use for labs to here
add main, and some includes
uses pointers to update the values without having to set them after
will need some fixes before it can be used
combining shouldbuy and should sell next
will replace them with new function now
some improvements
|
I can code review this. |
|
Code Review
UX/UI Recommendations
High Level Checks
Code Checks
|
|
I'll review this :) |
|
This is a cool idea, it's always fun to play around with unique projects like this. I had to add the game to your Makefile so it would build. After playing the game, I had many questions that could only be answered by reading the source code. As Nino stated, it might be good to give the user an idea of what's going on behind the scenes... Say the user knows that they need to roll a 1 or 2 to win anything, otherwise they will lose, then you can also display the number that was rolled. As it is currently, it feels like maybe the casino is just scamming you (I lost every time, and then when I sold the all the stuff I bought for $50 each I made $26 back... :-)) This definitely needs a formatting pass, better documentation for the user (I'm not as worried about the javadoc, it would be good, but many of the functions are fairly self explanatory) and some UI improvements so the user knows what's going on and the bugs Nino pointed out are fixed. => +0.5 to project score. |


This change introduces a simple text-based casino game where users can bet on dice rolls and win or lose with coins and stuff as currencies.
changes:
/user/textGame.cgame mechanics
Gameplay
User Interaction
Game Summary
Running the Game
to run this game:
$U/_textGametoUPROGSin the makefiletextGamefrom the consoleThis program is intended to be self-contained, while adding an engaging feature for user entertainment to FogOS.