Author: tchemit Date: 2013-09-26 10:03:59 +0200 (Thu, 26 Sep 2013) New Revision: 2940 Url: http://chorem.org/projects/jtimer/repository/revisions/2940 Log: fix javadoc typo + improve code Modified: trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java Modified: trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java 2013-09-24 07:37:24 UTC (rev 2939) +++ trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java 2013-09-26 08:03:59 UTC (rev 2940) @@ -40,7 +40,6 @@ import org.chorem.jtimer.entities.TimerAlert; import org.chorem.jtimer.entities.TimerProject; import org.chorem.jtimer.entities.TimerTask; -import org.nuiton.util.ArrayUtil; /** * Gere les donnees. Des objets peuvent s'enregistrer pour etre notifies des @@ -201,7 +200,7 @@ vetoableDataEventListener.checkSetTaskTime(task, date, value); } - task.setTime(date, Long.valueOf(value)); + task.setTime(date, value); // send notification for (DataEventListener dataEventListener : dataEventListeners) { @@ -249,9 +248,8 @@ task.getParent().getSubTasks().remove(task); // send notification - Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator(); - while (itDataEventListener.hasNext()) { - itDataEventListener.next().deleteTask(task); + for (DataEventListener dataEventListener : dataEventListeners) { + dataEventListener.deleteTask(task); } } @@ -270,9 +268,8 @@ projectList.remove(project); // send notification - Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator(); - while (itDataEventListener.hasNext()) { - itDataEventListener.next().deleteProject(project); + for (DataEventListener dataEventListener : dataEventListeners) { + dataEventListener.deleteProject(project); } } @@ -324,9 +321,8 @@ task.setClosed(!task.isClosed()); // send notification - Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator(); - while (itDataEventListener.hasNext()) { - itDataEventListener.next().changeClosedState(task); + for (DataEventListener dataEventListener : dataEventListeners) { + dataEventListener.changeClosedState(task); } } @@ -565,17 +561,17 @@ } /** - * Find corresponding task for path. + * Return the corresponding task for path. * - * @param taskPath - * @return + * @param taskPath task path to find + * @return corresponding task for path */ public TimerTask getTaskForPath(String taskPath) { String[] paths = taskPath.split("/"); // make sure can't select only project - if (paths == null || paths.length <= 1) { + if (paths.length <= 1) { throw new IllegalArgumentException("Can't parse argument '" + taskPath + "' as task"); }