Нужно исправить мой код таймера Java

Эта секция:

time.schedule(bj, afterCurrent);//Schedule job "bj" at time = afterCurrent

//Now turn the bulb off at new time = newest afterTime
afterCurrent.setSeconds(currentSecs + 2 * offset);

назначает только одну задачу. Если вам нужно запланировать его дважды, сделайте это явно:

time.schedule(bj, afterCurrent);//Schedule job "bj" at time = afterCurrent

//Now turn the bulb off at new time = newest afterTime
afterCurrent.setSeconds(currentSecs + 2 * offset);
time.schedule(bj, afterCurrent);//Schedule job "bj" at time = afterCurrent

Также. эта строка:

bj.setBulbStateEqualTo(false);

выполняется в основном потоке, поэтому он будет выполняться перед обеими задачами. Вы должны запланировать запуск этого оператора между двумя задачами.

Код исправлен, но эта версия не может выйти из main в конце -

import java.util.*;

@SuppressWarnings( "deprecation" )
class BulbScheduler {

public static void main(String args[]) throws InterruptedException{

    System.out.println("This is the main program");

    Timer timeOn = new Timer();
    Timer timeOff = new Timer();
    Bulb b1 = new Bulb("B1");
    BulbJob bjOn = new BulbJob(b1);
    BulbJob bjOff = new BulbJob(b1);

    bjOn.setBulbStateEqualTo(true);//Task - Turn bulb on 
    bjOff.setBulbStateEqualTo(false);//Task - Then turn the bulb off later

    Date current = new Date();//Get current time and execute job ten seconds after this time
    Date afterCurrent = (Date) current.clone();

    System.out.println("Current time is - " + current);

    int currentSecs = current.getSeconds();
    int offset = 3;//number of seconds

    afterCurrent.setSeconds(currentSecs + offset);
    System.out.println("Future time is - " + afterCurrent);

    timeOn.schedule(bjOn, afterCurrent);//Schedule job "bj" at time = afterCurrent

    //Now turn the bulb off at new time = latest afterCurrent
    afterCurrent.setSeconds(currentSecs + 2 * offset);
    System.out.println("Future time is - " + afterCurrent);

    timeOff.schedule(bjOff, afterCurrent);

    System.out.println("Main program ends");

}

}

вы не правильно устанавливаете время. Необходимо использовать GreogarianCalendar.

java.util.Date используется, но не может использовать его setSeconds Прочтите Javadoc, это очень хорошо и очень поможет. public void setSeconds (целые секунды)

Устарело. Начиная с JDK версии 1.1, заменено на Calendar.set(Calendar.SECOND, int секунд). Устанавливает секунды этой даты в указанное значение. Этот объект Date изменен таким образом, что он представляет момент времени в пределах указанной секунды минуты, с тем же годом, месяцем, датой, часом и минутой, что и раньше, как интерпретируется в местном часовом поясе.

Вам нужно использовать java.util.GregorianCalendar # add(Calendar.SECOND, howManySeconds)

затем используйте getDate(), чтобы получить объект Date и отправить его на таймер.

вызов setSecond на дату не изменит другие поля. см. java-документ Calendar.add и сверните. http://docs.oracle.com/javase/1.5.0/docs/api/java/util/Calendar.html и ознакомьтесь с правилами во вступительной части класса.

Также можно использовать расписание объекта таймера (задача TimerTask, большая задержка). Планирует выполнение указанной задачи после указанной задержки (в миллисекундах). Модифицированный код -

import java.util.*;

class BulbScheduler {

    private static java.text.SimpleDateFormat sdf1 = new java.text.SimpleDateFormat ("yy MM dd HH mm ss");

//helper    
static String formatDate(Date d){
        return sdf1.format(d);
    }

    public static void main(String args[]) throws InterruptedException{
        System.out.println("This is the main method");
        java.util.GregorianCalendar cal = new java.util.GregorianCalendar();

        Bulb b1 = new Bulb("bulb 1", false);
        Bulb b2 = new Bulb("bulb 2", false);

        System.out.println("Time now " + formatDate(cal.getTime()));

        Timer timer = new Timer("bulbs");
        BulbJob b1On = new BulbJob(b1, true);
        BulbJob b1Off = new BulbJob(b1, false);
        BulbJob b2On = new BulbJob(b2, true);
        BulbJob b2Off = new BulbJob(b2, false);
        timer.schedule(b1On, 3 * 1000);//after 3 seconds
        timer.schedule(b2On, 7 * 1000);//after 4 seconds
        timer.schedule(b1Off, 6 * 1000);//after 6 seconds; before b2 on

        b1On = new BulbJob(b1, true);
        timer.schedule(b1On, 9 * 1000);


        //if you want main to wait need to add code here to make it wait,
        // but even if does the JVM wont exit. Its just a method. The JVM exits when all non daemon threads are done
        // or System.exit is called

        System.out.println("This is the main method ending; but other threads might be running ...");
        //main thread JVM waits for all other non dameons to end

    }

}

Изменено задание на лампочку

импортировать java.util.*;

класс BulbJob расширяет TimerTask{

private Bulb bulbToHandle;
private boolean bulbNewState;//dont start propert names with set

//why a seperate property when we need to set the new state everytime and cannot reuse jobs?
BulbJob(Bulb toHandle, boolean newState){
    this.bulbToHandle = toHandle;
    bulbNewState= newState;
}

public void run(){
    this.bulbToHandle.setState(bulbNewState);//Set on or off
}

}

класс Bulb ... public void setState (логическое состояние) { this.state = state; System.out.println("Bulb" + name + " is " + (state ? "on" : "off") + "at" + BulbScheduler.formatDate(new java.util.Date()));//if хорошо тоже

}