Skip to content
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

Code Review #1

Open
michaelcarrano opened this issue Mar 28, 2015 · 0 comments
Open

Code Review #1

michaelcarrano opened this issue Mar 28, 2015 · 0 comments

Comments

@michaelcarrano
Copy link

  1. Good job on the code style, I was able to follow things easily. One tip to automatically format your code is to press option + command + L (That should be the key combo to reformat code in IntelliJ)
  2. You do have some extra line spacing in the code, it would be best if everything was consistent. Some comments in the code would also be nice to make it even easier for me to follow along.
  3. I noticed you used single quotes when printing out text, you can have double quotes shown by escaping the quote.

You could do something like:

System.out.println("The voice on the other end says \"if you walk out of the booth a sniper will shoot you.\"");

Which would result in this being printed out:

The voice on the other end says "if you walk out of the booth a sniper will shoot you."
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

No branches or pull requests

1 participant