undefined_name/School_Project
3
1
Fork 0
Code Issues 9 Pull Requests Projects 1 Releases 4 Wiki Activity

Add a condition to avoid white piece #65

Closed
Mat_02 wants to merge 3 commits from delete_white_piece into master
pull from: delete_white_piece
merge into: undefined_name:master
Conversation 6 Commits 3 Files Changed 1 +5 -2
Mat_02 commented 2023-05-17 23:52:29 +02:00
Member
Copy Link

A simple while to avoid white piece
i didn't make "if" in case we have two times a white piece idk if it can happen but who knows

A simple while to avoid white piece i didn't make "if" in case we have two times a white piece idk if it can happen but who knows
Mat_02 added 1 commit 2023-05-17 23:52:30 +02:00
Add a condition to avoid white piece
Some checks failed
continuous-integration/drone/push Build is failing
Details
continuous-integration/drone/pr Build is failing
Details
d37059b7de
Mat_02 commented 2023-05-17 23:56:27 +02:00
Author
Member
Copy Link

never mind it doesn't work

never mind it doesn't work
tonitch commented 2023-05-17 23:58:45 +02:00
Owner
Copy Link

C'est pas une mauvaise idée mais le problème est que tu ne vise qu'une seule couleur très précise là ^^

En gros il faut travailler avec la "distance" entre les couleurs pour s'assurer que les couleurs sont bien différentes, il y a des bons articles sur wikipedia qui parlent de ca si tu veux. ^^

C'est pas une mauvaise idée mais le problème est que tu ne vise qu'une seule couleur très précise là ^^ En gros il faut travailler avec la "distance" entre les couleurs pour s'assurer que les couleurs sont bien différentes, il y a des bons articles sur wikipedia qui parlent de ca si tu veux. ^^
Mat_02 added 1 commit 2023-05-17 23:59:46 +02:00
New condition for the while
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
fbd214bc23
Mat_02 commented 2023-05-18 00:01:23 +02:00
Author
Member
Copy Link

C'est pas une mauvaise idée mais le problème est que tu ne vise qu'une seule couleur très précise là ^^

En gros il faut travailler avec la "distance" entre les couleurs pour s'assurer que les couleurs sont bien différentes, il y a des bons articles sur wikipedia qui parlent de ca si tu veux. ^^

C'est vrai j'ai pris que le blanc mais si ca fait une color(254,254,254) c'est pas ouf

> C'est pas une mauvaise idée mais le problème est que tu ne vise qu'une seule couleur très précise là ^^ > > En gros il faut travailler avec la "distance" entre les couleurs pour s'assurer que les couleurs sont bien différentes, il y a des bons articles sur wikipedia qui parlent de ca si tu veux. ^^ > C'est vrai j'ai pris que le blanc mais si ca fait une color(254,254,254) c'est pas ouf
Mat_02 commented 2023-05-18 00:01:58 +02:00
Author
Member
Copy Link

Je modifie ca

Je modifie ca
Mat_02 added 1 commit 2023-05-18 00:32:55 +02:00
An other method to avoid all type of white
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
ecdff3cbd0 
Antho free to change this one I'm a little to lazy at this time
tonitch reviewed 2023-05-18 09:59:10 +02:00
app/src/main/java/school_project/Piece.java
@ -23,3 +26,2 @@
Random rand = new Random();
color = new Color(rand.nextDouble(), rand.nextDouble(), rand.nextDouble(), 1);
}
color = new Color((rand.nextDouble()%190), (rand.nextDouble()%190), (rand.nextDouble()%190), 1);
tonitch commented 2023-05-18 09:59:10 +02:00
Owner
Copy Link

https://docs.oracle.com/javase/8/docs/api/java/util/Random.html#nextDouble--

Returns the next pseudorandom, uniformly distributed double value between 0.0 and 1.0 from this random number generator's sequence.

https://docs.oracle.com/javase/8/javafx/api/javafx/scene/paint/Color.html#color-double-double-double-

Creates an opaque sRGB color with the specified red, green and blue values in the range 0.0-1.0.

Les valeurs sont entre 0 et 1 et pas entre 0 et 255

https://docs.oracle.com/javase/8/docs/api/java/util/Random.html#nextDouble-- >Returns the next pseudorandom, uniformly distributed double value between 0.0 and 1.0 from this random number generator's sequence. https://docs.oracle.com/javase/8/javafx/api/javafx/scene/paint/Color.html#color-double-double-double- > Creates an opaque sRGB color with the specified red, green and blue values in the range 0.0-1.0. Les valeurs sont entre 0 et 1 et pas entre 0 et 255
tonitch commented 2023-05-18 19:47:18 +02:00
Owner
Copy Link

Replaced by #74

Replaced by #74
tonitch closed this pull request 2023-05-18 19:47:18 +02:00
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Important
Doit absolument être fait
Moins important
Pas important
bug
Something is not working
done
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
suppléments
à faire si on a le temps
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: undefined_name/School_Project#65
No description provided.
Delete Branch "delete_white_piece"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?