Loading...
Logo
Processing Forum
Programming Questions
elijah-boumasims
Elijah Bo..

Button detection not working for some reason

in Programming Questions  •  2 months ago  
I'm making a calculator as a beginning project. Everything has been going fine until I got to programming the buttons. When I tried to implement button hover detection instead of detecting over the button it detected over the screen (number output). 
My code is below:
Calculator.pde
Copy code
  2. import java.text.NumberFormat;
  3. import java.text.DecimalFormat;
  4. PFont f;
  5. final int topRowYPos = 80;
  6. final int secondRowYPos = 110;
  7. final int thirdRowYPos = 140;
  8. final int fourthRowYPos = 170;
  9. final int fithRowYPos = 200;
  10. final int sixthRowYPos = 230;
  11. final int buttonWidth = 35;
  12. final int buttonHeight = 25;
  13. final int firstRowXPos = 45;
  14. final int secondRowXPos = 85;
  15. final int thirdRowXPos = 125;
  16. final int fourthRowXPos = 165;
  17. final int fithRowXPos = 205;
  18. int digitCount;
  19. boolean isDecimal;
  20. double numOneBuffer;
  21. double numTwoBuffer;
  22. double screenBuffer;
  23. double numMemorY;
  24. Button mc = new Button(firstRowXPos, secondRowYPos);
  25. Button percent = new Button(firstRowXPos, thirdRowYPos);
  26. NumberFormat formatter = new DecimalFormat("0.0000000");
  27. void setup() {
  28.   size(250, 280);
  29. }
  30. void draw() {
  31.   if (mc.overButton()) {
  32.     println("Over MC");
  33.   }
  34.   screenBuffer = 00.111;
  35.   rectMode(CENTER);
  36.   fill(255);
  37.   rect(125, 140, 240, 270);
  38.   rect(125, 40, 220, 30);
  39.   rect(firstRowXPos, secondRowYPos, buttonWidth, buttonHeight, 7);
  40.   rect(firstRowXPos, thirdRowYPos, buttonWidth, buttonHeight, 7);
  41.   rect(firstRowXPos, fourthRowYPos, buttonWidth, buttonHeight, 7);
  42.   rect(firstRowXPos, fithRowYPos, buttonWidth, buttonHeight, 7);
  43.   rect(firstRowXPos, sixthRowYPos, buttonWidth, buttonHeight, 7);
  44.   rect(secondRowXPos, secondRowYPos, buttonWidth, buttonHeight, 7);
  45.   rect(secondRowXPos, thirdRowYPos, buttonWidth, buttonHeight, 7);
  46.   rect(secondRowXPos, fourthRowYPos, buttonWidth, buttonHeight, 7);
  47.   rect(secondRowXPos, fithRowYPos, buttonWidth, buttonHeight, 7);
  48.   rect(secondRowXPos, sixthRowYPos, buttonWidth, buttonHeight, 7);
  49.   rect(thirdRowXPos, secondRowYPos, buttonWidth, buttonHeight, 7);
  50.   rect(thirdRowXPos, thirdRowYPos, buttonWidth, buttonHeight, 7);
  51.   rect(thirdRowXPos, fourthRowYPos, buttonWidth, buttonHeight, 7);
  52.   rect(thirdRowXPos, fithRowYPos, buttonWidth, buttonHeight, 7);
  53.   rect(thirdRowXPos, sixthRowYPos, buttonWidth, buttonHeight, 7);
  54.   rect(fourthRowXPos, topRowYPos, buttonWidth, buttonHeight, 7);
  55.   rect(fourthRowXPos, secondRowYPos, buttonWidth, buttonHeight, 7);
  56.   rect(fourthRowXPos, thirdRowYPos, buttonWidth, buttonHeight, 7);
  57.   rect(fourthRowXPos, fourthRowYPos, buttonWidth, buttonHeight, 7);
  58.   rect(fourthRowXPos, fithRowYPos, buttonWidth, buttonHeight, 7);
  59.   rect(fourthRowXPos, sixthRowYPos, buttonWidth, buttonHeight, 7);
  60.   rect(fithRowXPos, topRowYPos, buttonWidth, buttonHeight, 7);
  61.   rect(fithRowXPos, secondRowYPos, buttonWidth, buttonHeight, 7);
  62.   rect(fithRowXPos, thirdRowYPos, buttonWidth, buttonHeight, 7);
  63.   rect(fithRowXPos, fourthRowYPos, buttonWidth, buttonHeight, 7);
  64.   rect(fithRowXPos, 215, buttonWidth, 55, 7);
  65.   f = createFont("Georgia", 16);
  66.   textFont(f);
  67.   textAlign(CENTER, CENTER);
  68.   fill(0);
  69.   text("MC", firstRowXPos, secondRowYPos);
  70.   text("%", firstRowXPos, thirdRowYPos);
  71.   text("+/-", firstRowXPos, fourthRowYPos);
  72.   text("C", firstRowXPos, fithRowYPos);
  73.   text("AC", firstRowXPos, sixthRowYPos);
  74.   text("MR", secondRowXPos, secondRowYPos);
  75.   text("7", secondRowXPos, thirdRowYPos);
  76.   text("4", secondRowXPos, fourthRowYPos);
  77.   text("1", secondRowXPos, fithRowYPos);
  78.   text("0", secondRowXPos, sixthRowYPos);
  79.   text("M-", thirdRowXPos, secondRowYPos);
  80.   text("8", thirdRowXPos, thirdRowYPos);
  81.   text("5", thirdRowXPos, fourthRowYPos);
  82.   text("2", thirdRowXPos, fithRowYPos);
  83.   text("=", thirdRowXPos, sixthRowYPos);
  84.   text("√", fourthRowXPos, topRowYPos);
  85.   text("M+", fourthRowXPos, secondRowYPos);
  86.   text("9", fourthRowXPos, thirdRowYPos);
  87.   text("6", fourthRowXPos, fourthRowYPos);
  88.   text("3", fourthRowXPos, fithRowYPos);
  89.   text(".", fourthRowXPos, sixthRowYPos);
  90.   text("off", fithRowXPos, topRowYPos);
  91.   text("÷", fithRowXPos, secondRowYPos);
  92.   text("*", fithRowXPos, thirdRowYPos);
  93.   text("-", fithRowXPos, fourthRowYPos);
  94.   text("+", fithRowXPos, 215);
  95.   text(formatter.format(screenBuffer), 125, 40);
  96. }
  97. void mousePressed() {
  98.   String debugString = mouseX +","+ mouseY;
  99.   println(debugString);
  100. }

Button.pde:
Copy code
  2. class Button {
  3.   final int buttonWidth = 35;
  4.   final int buttonHeight = 25;
  5.   int xPos, yPos;
  6.   Button (int y, int x) {
  7.     xPos = x;
  8.     yPos = y;
  9.   }
  10.   boolean overButton() {
  11.     if (mouseX >= xPos && mouseX <= xPos+buttonWidth && 
  12.       mouseY >= yPos && mouseY <= yPos+buttonHeight) {
  13.       return true;
  14.     }
  15.     else {
  16.       return false;
  17.     }
  18.   }
  19. }
Thanks in advance.

Replies(4)

GoToLoop
GoToLoop

Re: Button detection not working for some reason

2 months ago
I've spotted the bug -> You use a diff. rectMode()! Rather than default's CORNER, you've chosen CENTER.
The catch is, a rect() is now drawn from its center coordinates rather than its left-top 1s.
So, your isOverButton() method fails miserably! 

Below lies your Button class fixed:
Copy code
    class Button {
  static final int BTN_DX = 35, BTN_DY = 25; // diameters x, y
  static final int BTN_RX = BTN_DX >> 1;     // radius x
  static final int BTN_RY = BTN_DY >> 1;     // radius y

  final int left, right, top, bottom; // button edges

  Button (int cx, int cy) {  // centralized positions x, y
    left = cx - BTN_RX;
    right = cx + BTN_RX;
    top = cy - BTN_RY;
    bottom = cy + BTN_RY;
  }

  boolean isOverButton() {
    return mouseX > left & mouseX < right 
      & mouseY > top & mouseY < bottom;
  }
}
Chrisir
Chrisir

Re: Button detection not working for some reason

2 months ago

err....
If I am not mistaken,
  • you swapped x and y in line 6 of Button.pde
  • also try to save your sketch as calc.pde because processing doesn't like it when sketch has the same name as a class in it (Button)
also, I noticed you use
Copy code
  1.   rectMode(CENTER);
Therefore x,y is the center whereas in line 11/12 you assume it is the upper left corner.
Therefore you need to hover little below and right from the text "MC" to get a result.
So either delete rectMode(CENTER); or change line 11/12


Arrays and class
your code is very long winded
I understand you are turning it into a class. Good.
The goal is:
  • have rect painted in the class
  • have text in the class
You don't have that yet.

But if you do:
You would end up with code like
Copy code
  1. mc.display();
  2. percent.display();
  3. ....
etc. for all (!) buttons.
 
To avoid long code like this I recommend using an array of that class.

Thus you could for loop over the class
Copy code
  1. for i......... {
  2. buttons[i].display();
  3. }


Defining the array
So instead of
Copy code
  1. Button mc = new Button(firstRowXPos, secondRowYPos);

you'd say

Copy code
  1. int max = 36; // max buttons
  2. Button[] buttons = new Button [max];  // array
  3. buttons[0] = new Button (firstRowXPos, secondRowYPos, "MC");  // 1st object in the class
(That part would still be long since you need to define the buttons, but then the rest will be short)

Greetings, Chrisir

If you need an answer, please send me a personal message since this forum doesn't notify.
Chrisir
Chrisir

Re: Button detection not working for some reason

2 months ago


GoToLoop, I was working on it and hadn't seen yours
well spotted

Greetings, Chrisir

GoToLoop
GoToLoop

Re: Button detection not working for some reason

2 months ago
Top Reply

Statistics

  • 4 Replies
  • 284  Views
  • 1 Followers

Actions

Related Posts