Timeline for Beginner GUI Calculator
Current License: CC BY-SA 3.0
10 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Jul 20, 2015 at 3:22 | history | edited | Legato | CC BY-SA 3.0 |
added 2 characters in body
|
| Jul 20, 2015 at 0:55 | comment | added | Maxim | Truly amazing feedback! Using Java 8 features really makes the code easier to understand. Thank you all very much for your time, guys! | |
| Jul 20, 2015 at 0:54 | vote | accept | Maxim | ||
| Jul 19, 2015 at 23:33 | comment | added | AndrewC |
@Maxim Legato makes a good point - your setTextTo should really be called appendText or something similarly descriptive.
|
|
| Jul 19, 2015 at 23:19 | comment | added | Legato | Hm. Thanks for that, OP, you should make note of this. setText usually means erase contents and replace, but yours doesn't work like that. This is why it's important to follow naming conventions. | |
| Jul 19, 2015 at 23:17 | comment | added | AndrewC |
Great answer, and your revised loop with addActionListener means I've deleted the comment I was writing to address the clunky actionPerformed. One edit I'd suggest to that is that the OP probably prefers their setTextTo(int) instead of textField.setText(val), since setTextTo actually appends rather than clobbers the current input.
|
|
| Jul 19, 2015 at 23:01 | history | edited | Legato | CC BY-SA 3.0 |
added 399 characters in body
|
| Jul 19, 2015 at 22:53 | history | edited | Legato | CC BY-SA 3.0 |
added 399 characters in body
|
| Jul 19, 2015 at 22:37 | history | edited | Legato | CC BY-SA 3.0 |
added 706 characters in body
|
| Jul 19, 2015 at 22:29 | history | answered | Legato | CC BY-SA 3.0 |