Timeline for SonarQube is complaining about: "Use isEmpty() to check whether the collection is empty or not."
Current License: CC BY-SA 3.0
7 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Oct 2, 2016 at 17:09 | answer | added | Karl Bielefeldt | timeline score: 4 | |
| Oct 2, 2016 at 16:37 | review | Close votes | |||
| Oct 17, 2016 at 3:02 | |||||
| Oct 2, 2016 at 15:50 | comment | added | Eric King | Well, no... I actually think that isEmpty() is more readable, too. But that's subjective. I feel that if the readability isn't compromised too much, picking the more performant option is better. | |
| Oct 2, 2016 at 15:47 | comment | added | Arturas M | @EricKing so you kind of agree that we should always do it for performance instead of readability, correct? | |
| Oct 2, 2016 at 15:42 | answer | added | Eric King | timeline score: 8 | |
| Oct 2, 2016 at 15:38 | comment | added | Eric King | See this answer to a similar question on StackOverflow: stackoverflow.com/a/11152624/18938 | |
| Oct 2, 2016 at 15:35 | history | asked | Arturas M | CC BY-SA 3.0 |