anyone? :( 

Oh Dear...


     

yeah, that's what I told him yesterday but he keeps forgetting it...

anyone can make it a little bit clear for me? :( 

Hello,

those lines are not in use.

1. void Update_Position() {
2.   
3.   ellipse(q, 100,55 ,55);
4.   
5. }

you can delete them. I told you.

The function draw is called again and again automatically.
Therefore it is like a loop.
Therefore all ellipses are painted one on another. Again and again.

To avoid this, put the line

1. background(255);

in your function draw().
Then the screen is deleted before the ellipse is drawn at the new position.
That line was already in the Code I gave you.

The Code you posted is not complete, because with

1. stroke(Green_C);

you got an error: Green_C is not defined.
Please post the complete Code. I told you.

"Don't worry, keep up the work!"

Greetings, Chrisir  

i got it already,

the code doesn't run

why don't you post the full Code!? 

here you go

1. final color Green_C = color (2, 222, 2);
2. int q=300; // x
3. int w=100;  // y
4. int s = 110;  // radius
5. // ===========================================================
6. void setup () {
7.   size (800, 880);
8.   //strokeWeight(3);
9.   strokeWeight(1);
10. }
11. void draw() {
12.   // delete screen
13.   background(111);
14.   // line color black
15.   stroke (0);
16.   fill(222);
17.   rect(60, 300, 60, 60); 
18.   fill(22); 
19.   text("UP", 70, 320);
20.   fill(222);
21.   rect(120, 300, 60, 60);
22.   fill(22); 
23.   text(">", 130, 320); 
24.   fill(222);
25.   rect(180, 300, 60, 60); 
26.   fill(22); 
27.   text("Dn", 190, 320);   
28.   fill(222);
29.   rect(240, 300, 60, 60);     
30.   fill(22); 
31.   text("<", 250, 320); 
32.   stroke(Green_C);
33.   fill(Green_C);
34.   ellipse(q, w, s, s);
35. }
36. // ===========================================================
37. void Update_Position() {
38.   // no use!
39.   fill(2, 222, 222);                      
40.   strokeWeight(1);                      
41.   //draw(); 
42.   //Draw_Squares();
43.   //Draw_Grid();
44. }
45. void mousePressed() {
46.   if ((mouseX>60) && (mouseY>300) && (mouseX<120) && (mouseY<360))
47.   {
48.     w = w-60;
49.   }
50.   if ((mouseX>120) && (mouseY>300) && (mouseX<180) && (mouseY<360))
51.   {
52.     q = q+60;
53.   }
54.   if ((mouseX>180) && (mouseY>300) && (mouseX<240) && (mouseY<360)) 
55.   {
56.     w = w+60;
57.   }
58.   if ((mouseX>240) && (mouseY>300) && (mouseX<300) && (mouseY<360))
59.   {
60.     q = q-60;
61.   }
62.   Update_Position();
63. }

Chrisir  

Oh dear...

you still don't have background() in draw. That's deleting the scene.

And you call draw() in Update_Position()  - don't

you still don't have background() in draw. That's deleting the scene.

congratulations, man!

what do you mean by "lock"?

yeah, check your w and q similar as you did with your MouseX

Bye for now.

I am afraid so.

Hello,

you know, my actual Code is pretty bad.

E.g.: When you want to change the position of  a button, you have to change it in three places :
Here (where Mouse gets checked):

1. if ((mouseX>60) && (mouseY>300) && (mouseX<120) && (mouseY<360))

here (the rect)

1. rect(60, 300, 60, 60); 

and here (the text in it)

1.   text("UP", 70, 320);

So, it's double information. That is called redundant. Bad.

It should better be in one place only. So you would have a class Button and store all Buttons in a list of that class with position and color and text in it.

See the idea here:
http://www.openprocessing.org/visuals/?visualID=29501
file: Buttons.pde.

Greetings, Chrisir