{"id":8361,"date":"2022-09-13T04:58:01","date_gmt":"2022-09-12T23:28:01","guid":{"rendered":"https:\/\/jassweb.com\/solved\/solved-why-is-the-comma-in-two-text-fields-put-at-the-same-time-without-focusing\/"},"modified":"2022-09-13T04:58:01","modified_gmt":"2022-09-12T23:28:01","slug":"solved-why-is-the-comma-in-two-text-fields-put-at-the-same-time-without-focusing","status":"publish","type":"post","link":"https:\/\/jassweb.com\/solved\/solved-why-is-the-comma-in-two-text-fields-put-at-the-same-time-without-focusing\/","title":{"rendered":"[Solved] Why is the comma in two text fields put at the same time without focusing?"},"content":{"rendered":"

[ad_1]
\n<\/p>\n

\n
\n
<\/div>\n
\n
\n
\n

Why is the comma placed without focusing text fields?<\/p>\n<\/blockquote>\n

Because you are using the global keyPressed()<\/code> event.
\nThis condition: if (e.getKey() == ',')<\/code> checks that the ,<\/code> key is pressed and it’s redundant to check twice in your case. It’s equivalent to this simpler\/cleaner snippet:<\/p>\n

public void keyPressed(KeyEvent e) {\n  if (key == ','){\n    X9.setText(X9.getText() + ',');\n    X10.setText(X10.getText() + ',');\n  }\n}\n<\/code><\/pre>\n
There isn’t any check if a field is focused or not.<\/p>\n
You probably meant something like this ?:<\/p>\n
public void keyPressed(KeyEvent e) {\n  if (key == ',') {\n    if (X9.isFocus()) {\n      X9.setText(X9.getText() + ',');\n    }\n    if (X10.isFocus()) {\n      X10.setText(X10.getText() + ',');\n    }\n  }\n}\n<\/code><\/pre>\n
Overall it’s unclear what the main goal is. Perhaps there’s a simpler way to achieve it ?<\/p>\n
Remeber that you can listen for individual controlP5 component events via void controlEvent(ControlEvent event)<\/code>. See Processing > Examples > Contributed Libraries > ControlP5 > controllers > ControlP5Textfield<\/strong> for a nice demo.<\/p>\n
Additionally I recommend formatting\/keeping code tidy. It may be a quick throw-away sketch, but buiilding a good habbit will pay off as you’ll spend way more time reading code than writing code. As your programs become larger and larger you’ll want to make life easier for your future self \ud83d\ude42<\/p>\n
Update<\/strong> Based on your comments here is a minimal sketch that should allow you control a floating point number between 7.4 and 16.8 and an integer number between 1800-1900:<\/p>\n
import controlP5.*;\n\nControlP5 cp5;\n\nint serialInt = 1800;\n\nfloat serialFloat = 7.4;\n\nvoid setup() {\n  size(300,300);\n  noStroke();\n  cp5 = new ControlP5(this);\n  \n  cp5.addNumberbox(\"serialInt\")           \/\/ notice the component name matches the variable name: controlP5 links the two for you\n     .setBroadcast(false)                 \/\/ disable events while we update value specific properties \n     .setPosition(100,100)                \/\/ screen location\n     .setSize(100,20)                     \/\/ screen dimensions\n     .setMultiplier(10)                   \/\/ set the sensitifity of the numberbox: each step is 10\n     .setDirection(Controller.HORIZONTAL) \/\/ change the control direction to left\/right\n     .setRange(1800, 9000)                \/\/ set minimum, maximum value\n     .setBroadcast(true)                  \/\/ enable events (after setting range)\n     ;\n  \n\n  cp5.addNumberbox(\"serialFloat\")\n     .setBroadcast(false)                 \/\/ disable events while we update value specific properties\n     .setPosition(100,140)                \/\/ screen location\n     .setSize(100,20)                     \/\/ screen dimensions\n     .setMultiplier(0.01)                 \/\/ set the sensitifity of the numberbox: each step is 0.01\n     .setDirection(Controller.HORIZONTAL) \/\/ change the control direction to left\/right\n     .setRange(7.4,16.8)                  \/\/ set minimum, maximum value\n     .setBroadcast(true)                  \/\/ enable events (after setting range)\n     ;\n  \n}\nvoid draw() {\n  background(0);\n}\n\/\/ gets called whenever a component updates value\nvoid controlEvent(ControlEvent event){\n  println(event.getController().getName(),\"changed value to\",event.getValue(),\"serialInt = \",serialInt,\"serialFloat = \",serialFloat);\n}\n<\/code><\/pre>\n
Click and drag horizontally to change values.
\nNotice a few helpful things that ControlP5 provides:<\/p>\n