Fixed a nasty bug in DataHandler.populateList related to comparison of strings. Also optimized function by using for-each loop.

This commit is contained in:
Robert Kolner committed 2011-09-28 19:54:36 +02:00
1 parent 7f2bc2fd61
commit 1ab75cee3c
1 file changed
+7 -9
@@ -25,16 +25,14 @@ public class DataHandler {
public Event[] populateList(String category) {
LinkedList<Event> sorted = new LinkedList<Event>();
if (category == "all") {
if (category.equals("all")) {
return events.toArray(new Event[events.size()]);
} else {
for (int i = 0; i < events.size(); i++) {
Event tmp = events.get(i);
if (category.equals(tmp.category))
sorted.add(tmp);
}
return sorted.toArray(new Event[sorted.size()]);
}
for (Event event : events) {
if (category.equals(event.category))
sorted.add(event);
}
return sorted.toArray(new Event[sorted.size()]);
}
}