Timer and TimerTask

[Kishan Kumar]

Hi,
Why dosent the following code work as intended.
public class testTimer {
public static void main(String args[]) {
Timer t = new Timer(false);
t.scheduleAtFixedRate((new Ta()),System.currentTimeMillis(),1000);
}
}

public class Ta extends TimerTask {
public void run()
{try {
FileOutputStream fw = new FileOutputStream("D:/timer.txt",true);
PrintWriter pw = new PrintWriter(fw);
pw.println("Timer is working");
pw.close();
}catch(Exception e) {}
System.out.println("Timer is working");
}
}
when testTimer is run the timertask Ta is not scheduled...the run method is not called. Is there any mistake Iam doing here.
Thanks

[Jim Yingst]

Check the API for scheduleAtFixedRate(TimerTask, long, long). The middle argument doesn't mean what you think it means.

[Michael Morris]

Hi Kishan,
The problem is this line of code:

t.scheduleAtFixedRate((new Ta()),System.currentTimeMillis(),1000);

The second parameter is the delay before the timer is called. What you are passing is the current number of millsecs since the epoch (Jan. 1, 1970), I was a junior in high school! So for your program to work as expected, you'll have to wait over 30 years before that timer starts firing. So if you don't want to waste 30 years of your life, change the second parameter to something like 0-1000.
Hope this helps,
Michael Morris
SCJP2

[Kishan Kumar]

Hi,
Thanks all, I have learnt one though .. never start anything new in the middle of nights.
Thanks again.