Timeline for "Have a safe trip"… or not, depending on four conditions
Current License: CC BY-SA 3.0
16 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Feb 10, 2016 at 2:29 | answer | added | k_rollo | timeline score: 1 | |
| Feb 9, 2016 at 22:47 | review | Close votes | |||
| Feb 10, 2016 at 1:49 | |||||
| Feb 9, 2016 at 21:54 | answer | added | corsiKa | timeline score: 4 | |
| Feb 9, 2016 at 17:39 | history | protected | Mathieu Guindon | ||
| Feb 9, 2016 at 15:16 | answer | added | Jordan | timeline score: 8 | |
| Feb 9, 2016 at 13:32 | history | tweeted | twitter.com/StackCodeReview/status/697050375443124224 | ||
| Feb 9, 2016 at 13:28 | answer | added | Adriano Repetti | timeline score: 15 | |
| Feb 9, 2016 at 10:33 | comment | added | k_rollo | Hello, yes I'm doing validation. I managed to trim it down to 3 booleans by combining the destination checks (if the destination is blank, it's also invalid anyway). It's much more manageable now. :) | |
| Feb 9, 2016 at 10:30 | comment | added | Blacklight Shining |
It looks like you're doing validation, and I feel like at this point you have enough to justify factoring that out into its own method on your Trip class or whatever.
|
|
| Feb 9, 2016 at 8:58 | history | edited | 200_success | CC BY-SA 3.0 |
edited title
|
| Feb 9, 2016 at 8:02 | vote | accept | k_rollo | ||
| Feb 9, 2016 at 7:53 | answer | added | Zesar | timeline score: 61 | |
| Feb 9, 2016 at 7:40 | comment | added | 200_success |
This is a rather poor Code Review question. If you gave us more contextual information (e.g. How did these variables get set? Is the message going to be used in a System.out.println() or embedded in a JSON HTTP response?) we could actually provide more useful advice than just shuffling your tokens around.
|
|
| Feb 9, 2016 at 6:53 | answer | added | Legato | timeline score: 26 | |
| Feb 9, 2016 at 6:14 | history | edited | k_rollo | CC BY-SA 3.0 |
added 1 character in body
|
| Feb 9, 2016 at 6:05 | history | asked | k_rollo | CC BY-SA 3.0 |