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

Assignment 9 Commit 1 #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

froschs
Copy link

@froschs froschs commented Nov 9, 2015

First commit. It isn't pretty but I think it works.

First commit. It isn't pretty but I think it works.
System.out.println(diceReader.getLines().toString().equals("[]"));
String[] fileDice = diceReader.getLines().toString().split(" ");
for(String dieValue : fileDice){
int firstColon = dieValue.indexOf(":");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Split could be employed again on the : to help organize these inputs

@ninjapanzer
Copy link
Member

👍

@curtiskunkle
Copy link

drop some comments on your dice reader. It's supa confusing.

@froschs
Copy link
Author

froschs commented Nov 9, 2015

I'll clean this up and add some comments when I get a chance. Just relieved to get it working.

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.

3 participants