Skip to content

did homework#3

Open
sbleier wants to merge 6 commits intomainfrom
bucket
Open

did homework#3
sbleier wants to merge 6 commits intomainfrom
bucket

Conversation

@sbleier
Copy link
Copy Markdown
Owner

@sbleier sbleier commented Apr 30, 2025

No description provided.

//constructor to set oval size;
public EraserTool(int size) {
this.size = size;
stroke = new BasicStroke (size);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.MethodParamPadCheck> reported by reviewdog 🐶
'(' is preceded with whitespace.

@sbleier
Copy link
Copy Markdown
Owner Author

sbleier commented Apr 30, 2025

I don't know why it's saying the build is failing... it's not giving me error messages in IntelliJ as far as I can tell. it seems to work

@sbleier
Copy link
Copy Markdown
Owner Author

sbleier commented Apr 30, 2025

oh I see the issue but I don't see how to solve it while still using the recursions? it's a stack overflow error

}

public void fillBucket(BufferedImage image, int x, int y, int target, int replace) {
if (x < 0 || x >= image.getWidth() || y < 0 || y >= image.getHeight() || image.getRGB(x, y) != target || fillCount > 1000) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> reported by reviewdog 🐶
Line is longer than 120 characters (found 132).

}

public void fillBucket(BufferedImage image, int x, int y, int target, int replace) {
if (x < 0 || x >= image.getWidth() || y < 0 ||
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck> reported by reviewdog 🐶
'||' should be on a new line.

@sbleier
Copy link
Copy Markdown
Owner Author

sbleier commented Apr 30, 2025

Screenshot 2025-04-30 at 3 47 15 PM

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.

1 participant