no, not Timer, it's horrible, incredibly easy to generate memory leaks with it.
Use this instead http://docs.oracle.com/javase/6/docs/api/java/util/concurrent/Executors.html#newScheduledThreadPool(int)
but I think current Thread solution is fine