Java timer issue not running correctly

[alexandra12]

Hello,


Just looking for some help here.

I have created a separate timer class that runs once every 60 minutes. This class works fine.

public Initialize(int seconds){
	
		Timer timer = new Timer();
		timer.schedule(new timerTask(), 0, 1*600000);
	
	}
	
	
	class timerTask extends TimerTask{		
		public void run(){
			 Date date = new Date();
			 try{
			
			System.out.println(date);
	
			 }catch (Exception e) {
				 System.out.println(e);
				 running = false;
				 //stopping the execution of the thread if their is an error
				 System.exit(0);
				 while (running = false){
					 System.out.println("in the while command");
					//after killing the thread I now need to restart the timer process thread automatically.
					 //setting it to restart after 10 minutes
					 Timer timer = new Timer();
					
					 timer.schedule(new timerTask(),600000);
				 }
				 
			} 
			 running=true;
		}
		
	}


	public static void main(String[] args) {
		new Initialize(0);

If I update the above code and add the code to implement a new class as per below...it seems to keep looping through the constructor, wont go into the main and wont come out of the constructor.
It also seems to loose the option to do it only once every 10 minutes.

public void run(){
			 Date date = new Date();
			 try{
				 running=true;
			
			System.out.println(date);
			north = new north();

I need to get it to run through the start element() end element to parse the data.

Any advice

[quad64bit]

while (running = false){

Should be

while (running == false){

[DarrylBurke]

Should be

while (running == false){

No, it should be

while (!running) {

db

[quad64bit]

No, it should be
Java Code:
1
while (!running) {

Oops, I posted faster than I read :P

[kammce]

No, it should be

while (!running) {

db

Does that even matter? They are identical statements.

[Tolls]

You get problems like the above, where the assignment is valid in the statement as it produces a boolean result...so the simple rule is don't do it that way and you'll not end up with silly mistakes like that.

[DarrylBurke]

Technically OT, but it's certainly bad form to name a variable running and then continue around a while loop while the value of that variable is false.

db

[alexandra12]

Thanks guys,

I will try that out !