handling the mousePressed and some other things...

Programming Questions

ofirofir85

handling the mousePressed and some other things...

in Programming Questions • 1 year ago

Hi:)

this is my code:

**Edit - I tried to change to color of the block in the display function , and it stays black... I cant find the problem ..

Block[][] grid;
int cols = 16;
int rows =10;
void setup()
{
size(800,500);
grid = new Block[cols][rows];
for(int x = 0;x<cols;x++)
{
for(int y = 0;y<rows;y++)
{
grid[x][y] = new Block(x*50,y*50,50,50);
}
}
}
void draw()
{
for(int x = 0;x<cols;x++)
{
for(int y = 0 ;y<rows;y++)
{
grid[x][y].display();
}
}
}
class Block
{
int x,y,w,h,xEnd,yEnd;
boolean clicked;
Block(int xx,int yy,int ww,int hh)
{
clicked = false;
x = xx;
y = yy;
w = ww;
h =hh;
xEnd = x+50;
yEnd = y+50;
}
void display()
{
if(clicked = false)
{
fill(color(255));
rect(x,y,w,h);
}
else if(clicked = true)
{
fill(color(0));
rect(x,y,w,h);
}
}
void changeColor()
{
if(clicked = false)
{
clicked = true;
}
else if(clicked = true)
{
clicked = false;
}
}
}
void mouseClicked()
{
moveToBlock(mouseX,mouseY);
}
void moveToBlock(int mX,int mY)
{
for(int x = 0;x<cols;x++)
{
for(int y = 0 ;y<rows;y++)
{
if(mX > grid[x][y].x && mX < grid[x][y].xEnd)
{
if(mY > grid[x][y].y && mY<grid[x][y].yEnd)
{
grid[x][y].changeColor();
}
}
}
}
}

My problem is that the block is not changing the color...

please help me

(Sorry for my English ...)

Replies(3)

tomschofi..

Re: handling the mousePressed and some other things...

1 year ago

Heya

change

//this means ASSIGN a false value to your variable whereas you want to

clicked = false

//compare clicked and false!

clicked == false

same with else if(clicked = true)

there may be other stuff to but that's a start!

benja

Re: handling the mousePressed and some other things...

1 year ago

You make a mistake when comparing values in your if-clauses. You need two "==".

if(clicked = false)

This means you assign "false" to the variable "clicked".
You would have to write one of the following:

if(clicked == false)
if(!clicked)

Then in you display()-function you might want to remove lines 53 and 59. Otherwise you change "clicked" on every frame causing the blocks to flicker.

Then another simplification, you could reduce your changeColor-function to this:

void changeColor()
{
clicked= !clicked;
}

ofirofir85

Re: handling the mousePressed and some other things...

1 year ago

Thanks for both of you...

I dont know how I didnt saw that.. I am stupid

Thanks Its working now!

Top Reply