Почему возвращает default?

Question

[Александр Пичужкин]

import java.util.Scanner;
public class TestConditions {

	public static void main(String[] args) {
		Conditions cond = new Conditions();
		try (Scanner scanner = new Scanner(System.in)) {
			System.out.println("Введите первое число:");
			cond.setFirstChange(scanner.nextInt());
			System.out.println("Введите второе число:");
			cond.setSecondChange(scanner.nextInt());
			System.out.println("Введите номер операции: 1.Умножение 2.Деление 3.Сложение");
			cond.setThirdChange(scanner.nextInt());
			cond.operation(cond.getFirstChange(), cond.getSecondChange(), cond.getThirdChange());
		}
	}
}


public class Conditions {
	    private int firstChange;
		private int secondChange;
		private int thirdChange;
	
	
		public void setFirstChange(int a){
			a = firstChange;
		}
		public void setSecondChange(int b){
			b = secondChange;
		}	
		public void setThirdChange(int c){
			c = thirdChange;
		}
		
		public int getFirstChange(){
			return firstChange;
		}
		public int getSecondChange(){
			return secondChange;
		}	
		public int getThirdChange(){
			return thirdChange;
		}
			public void operation(int x, int y, int z) {
				switch(z) {
				case 1:
					int res = x*y;
					System.out.println(res);
					break;
				case 2:
					int res1 = x/y;
					System.out.println(res1);
					break;
				case 3:
					int res2 = x+y;
					System.out.println(res2);
					break;
				default:
					System.out.println("Операция не определена");
				}
				
			}
			
}

Возвращает "Операция не определена". Не могу понять, в чем дело?
P.S. Если вопрос сильно глупый, то заранее прошу прощения, я только начинаю осваивать джаву.

Comments

[Александр Пичужкин]

Введите первое число:
10
Введите второе число:
5
Введите номер операции: 1.Умножение 2.Деление 3.Сложение
2
Операция не определена

[Dmitry Roo]

Пожалуйста, оформите код тегом code

Answer 1

[BorLaze]

1. Текст надо форматировать.
2. В сеттерах класса Conditions надо присваивать значение параметра полям, а не наоборот.

Comments

[Александр Пичужкин]

2. Это эклипс так автоматически создал сеттеры.
1. Не совсем понятно.
Спасибо за обратную связь

[Александр Пичужкин]

Сделал пункт №2 и заработало, спасибо большое

[BorLaze]

Arome, не верю.

сеттеры (и форматирование) должны быть такими:

public void setFirstChange(int a){
    firstChange = a;
}

[Александр Пичужкин]

BorLaze, я ни один сеттер руками не написал, всё за меня предложил сделать эклипс, после того, как я поставил модификатор private. Сейчас переделал так, как вы сказали, и всё заработало. А в чем логика? Я так понимаю, правильнее в таких случаях читать код справа налево?

[BorLaze]

Arome, эммм... и правда "я только начинаю осваивать джаву" :-)

Я не припоминаю ни одного языка, где переменная, куда положить значение, указывалась бы справа. Даже термин такой есть - lvalue.

В общем, учитесь :-) учить еще много.

[Александр Пичужкин]

BorLaze, а, я понял) спасибо большое)