Author: fdesbois Date: 2010-01-12 17:38:10 +0100 (Tue, 12 Jan 2010) New Revision: 1736 Modified: trunk/src/main/java/org/nuiton/util/PeriodDates.java Log: Resolve NPE Modified: trunk/src/main/java/org/nuiton/util/PeriodDates.java =================================================================== --- trunk/src/main/java/org/nuiton/util/PeriodDates.java 2010-01-12 15:42:11 UTC (rev 1735) +++ trunk/src/main/java/org/nuiton/util/PeriodDates.java 2010-01-12 16:38:10 UTC (rev 1736) @@ -133,7 +133,7 @@ } public Date getFromDate() { - return this.fromCalendar.getTime(); + return this.fromCalendar != null ? this.fromCalendar.getTime() : null; } public int getFromMonth() { @@ -153,7 +153,7 @@ } public Date getThruDate() { - return this.thruCalendar.getTime(); + return this.thruCalendar != null ? this.thruCalendar.getTime() : null; } public List<Date> getMonths() {
Le Tue, 12 Jan 2010 17:38:10 +0100 (CET), fdesbois@users.nuiton.org a écrit :
public Date getFromDate() { - return this.fromCalendar.getTime(); + return this.fromCalendar != null ? this.fromCalendar.getTime() : null; }
On peut éviter de mettre les this. sauf dans le cas où une variable locale porte le même nom. C'est vrai qu'on a le droit de faire mais cela allourdit inutilement le code :) -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 email: chemit@codelutin.com http://www.codelutin.com
participants (2)
-
fdesbois＠users.nuiton.org -
Tony Chemit