Skip to content

Conversation

@manasser99
Copy link

New practice problem for Java Intermediate Lesson 10.2

@phrdang phrdang requested a review from JJ27 June 21, 2021 20:08
Copy link
Contributor

@JJ27 JJ27 left a comment

Choose a reason for hiding this comment

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

Left some comments. Thanks for the PR! I hope to see more problems in the future :)

Also, try your best to pass the style checks (if they're just getting you on annoying stuff just let me know and I can override them).

for(int j = 0; j < arr[0].length; j++)
{
//Check if current element is set to true
if(arr[i][j] == true)
Copy link
Contributor

Choose a reason for hiding this comment

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

You can actually change this line to

Suggested change
if(arr[i][j] == true)
if(arr[i][j])

Comment on lines +4 to +7
Create a program called BinaryReader in which a 4 x 4
array of booleans is created. Have the computer loop
through the array, printing a 0 if the current element is
set to false and a 1 if it set to true.
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of having them create an array themselves, you might want to consider having this be a method instead (the 4x4 array is the parameter and the method either prints or returns the string)

Copy link

@DelosReyesJordan DelosReyesJordan left a comment

Choose a reason for hiding this comment

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

Looks good!

@phrdang phrdang requested a review from JJ27 July 1, 2021 20:21
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