Почему канал inc / de c ничего не делает?

Question

В следующем коде мои тесты показывают, что методы tvObject.incChannel(...) и tvObject.decChannel(...) ничего не меняют в моем коде. Почему?

package lab2;

public class TV {

    private int channelNumber;
    private int volumeLevel;
    private String status = "ON";

    public TV(int channel, int vol, String stat) {

        status = stat;
        setChannelNumber(channel);
        setVolumeLevel(vol);
    }
    public void setChannelNumber(int channel) {
        if (status.equalsIgnoreCase("ON")) {
            channelNumber = channel;
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }   
    }
    public int getChannelNumber() {
        return channelNumber;
    }
    public void setVolumeLevel(int vol) {
        if (status.equalsIgnoreCase("ON")) {
            volumeLevel = ((vol >= 0 && vol <= 100)?vol:0);
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }       
        volumeLevel = ((vol >= 0 && vol <= 100)?vol:0);
    }
    public int getVolumeLevel() {
            return volumeLevel; 
    }
    public void incVol(int vol) {
        if (status.equalsIgnoreCase("ON")) {
            volumeLevel = ((vol >= 0 && vol <= 98)?vol += 2:volumeLevel);
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }   
    }
    public void decVol(int vol) {
        if (status.equalsIgnoreCase("ON")) {
            volumeLevel = ((vol >= 2 && vol <= 100)?vol -= 2:volumeLevel);
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }           
    }

    // does not do anything
    public void incChannel(int channel) {
        if (status.equalsIgnoreCase("ON")) {
            channelNumber = channel++;
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }       
    }

    // does not do anything
    public void decChannel(int channel) {
        if (status.equalsIgnoreCase("ON")) {
            channelNumber = channel--;
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }       
    }
    public void Toggle() {
        //String status = "ON";  //default on

        if (status.equalsIgnoreCase("ON")) {
            status = "OFF";
        } else if (status.equalsIgnoreCase("OFF")) {
            status = "ON";
        }       
    }
}


// code to test

package lab2;

public class Test {

    public static void main(String[] args) {

            TV tvObject = new TV(53, 35, "ON");
            //TV tvObject2 = new TV(3, -10, "on");
            //TV tvObject3 = new TV(3, -10, "OFF");

            tvObject.Toggle();

            tvObject.setChannelNumber(53);
            tvObject.incChannel(53);

            tvObject.setVolumeLevel(35);
            tvObject.decVol(35);

            System.out.println(tvObject.getChannelNumber());
            System.out.println(tvObject.getVolumeLevel());
    }
}

Answer 1

Должно быть так:

public void incChannel(int channel) {
        if (status.equalsIgnoreCase("ON")) {
            channelNumber = ++channel; // use either this
            // channelNumber = channel+1; // or this
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }       
    }

public void decChannel(int channel) {
        if (status.equalsIgnoreCase("ON")) {
            channelNumber = --channel; // use either this
            // channelNumber = channel-1; // or this
        } else if (status.equalsIgnoreCase("OFF")) {
            System.out.println("Error TV is off");
        }       
    }

channelNumber = channel++ означает сначала назначить канал для channelNumber, а затем увеличить его значение.

channelNumber = ++channel означает сначала увеличьте значение канала, затем присвойте его cannelNumber.

Answer 2

Инкрементальный постфиксный оператор (таким образом, "++" после имени переменной) сначала вычисляет выражение и увеличивает его только после этого.

Таким образом, channelNumber = channel++ является эквивалентом следующего

channelNumber = channel;
channel = channel + 1;

Becuse channel является только параметром в вашем методе, его увеличение ничего не изменит (оно не будет использовано позже в вашем методе), уменьшая ваше утверждение до channelNumber = channel; выражение.

Самое простое исправление вашего кода:

// so it already works
public void incChannel(int channel) {
    if (status.equalsIgnoreCase("ON")) {
        channelNumber = ++channel;
    } else if (status.equalsIgnoreCase("OFF")) {
        System.out.println("Error TV is off");
    }       

... потому что оператор приращения префикса (= "++" перед имя переменной) сначала увеличивается, а оценивает только потом. Таким образом, оператор проблематики c работает как

channel = channel + 1;
channelNumber = channel;

, что, вероятно, то, что вы хотите.