Why wont this work?

Programming Questions

nicholas68

in Programming Questions • 11 months ago

So I am fairly new to processing. But I though I could have and inside an IF condition, so I am confused why this isn't working?

int colour = (int) random(3);
int rectcolour;
void setup() {
size(600, 600);
rectMode(CENTER);
smooth();
}
void draw() {
switch(colour) {
case 0:
fill(255, 0, 0);
break;
case 1:
fill(0, 255, 0);
break;
case 2:
fill(0, 0, 255);
break;
}
//rect(width/2, height/2, 200, 100);
fill(255, 0, 0);
rect(125, 525, 100, 50);
fill(0, 255, 0);
rect(300, 525, 100, 50);
fill(0, 0, 255);
rect(475, 525, 100, 50);
rectColour;
rect(width/2, height/2, 200, 100);
}
void mousePressed() {
if(mouseX > 0 && < 300) {
rectColour = fill(0, 255, 0);
}
if(mouseX > 300 && < 600) {
rectColour = fill(0, 0, 255);
}
}

Replies(2)

PhiLho

Some errors or potential gotchas, reading code from top to bottom...
- int colour = (int) random(3); is OK, but some people wonder why they have always the same value... If that's just an init condition, there is no problem.
- rectColour; at line 40 serves no purpose.
- if(mouseX > 0 && < 300) { That's probably the gist of your question... Well, in most programming languages I know, it just won't work: if statement is designed to work with various conditions, testing various variables, so even if you test the same one, you have to repeat it. Correct line is: if (mouseX > 0 && mouseX < 300) {
Note that to my knowledge, mouseX is never below zero, so you can simplify to if (mouseX < 300)
Of course, the second test must be: if (mouseX > 300 && mouseX < 600) {

nicholas68

Re: Why wont this work?

11 months ago

Thank you PhiLho, worked like a charm. I later saw that rectColour was doing nothing besides causing more problems.

With the first gotcha if you are referring to (int) before random that is because I was having int-float conversion issues.

And once again thanks, never considered that mouseX had to be repeated.

And if you where curious to my end goal I'm making a game where a timer starts to count down and a random colour displays on the screen, then before the time runs out you have to click the correct coloured box. trying to get buttons working was too much grief so I am trying to do a false button by allowing the action if the mouse is in the same area that the coloured box takes up. And then If I get time I'm going to make it so each round you get correct causes the next rounds timer to be one second lower, so the player will need a faster reaction time. (hoping I can do that with a case, and then at the end of each level increment up the level by one, moving it onto the next case. Think that should work.

Top Reply