Как работает наследование классов в java?

Question

[daMage]

Помогите понять почему код возвращает 0 вместо 12:

class App {
	public static void main(String[] args) throws Exception {
		Params params = new Params();
		params.MACDFast = 12;
		params.MACDSlow = 16;
		params.MACDMA = 9;

		DSO dso = new DSO(params);
	}
}

class TSParams {
	public int MACDFast;
}

class Params extends TSParams {
	public int MACDFast, MACDSlow, MACDMA;
}

class TS {
	public TS(TSParams params) {
		System.out.println(params.MACDFast);
	}
}

class DSO extends TS {
	public DSO(Params params) {
		super(params);
	}
}

Такое ощущение, что конструктор класса TS работает не с обьектом, а со статической переменной класса TSParams. Собственно, как вывести истинное значение MACDFast из класса TS?

Answer 1

[bimeg]

В вашем случае поле Params.MACDFast спрятало (hided) поле TSParams.MACDFast (их стало два).
Более того, поля разруливаются используя compile-time type.
Поэтому в main вы задали поле Params.MACDFast, а поле TSParams.MACDFast осталось с дефолтным значением - 0.
Но во конструкторе TS у вас compile-time type - TSParams, поэтому вы прочитали поле TSParams.MACDFast (0), а не Params.MACDFast (12).

Чтобы это исправить в вашем примере достаточно удалить MACDFast из класса Params.

Comments

[daMage]

А если в TS нужно использовать те свойства, что известны TSParams, а в DSO - всё, что известно Params, такой код подойдет или его можно оптимизировать? Что из себя представляют переменные TS.params и DSO.params. Это одна и та же переменная?

class App {
	public static void main(String[] args) throws Exception {
		Params params = new Params();
		params.MACDFast = 12;
		params.MACDSlow = 16;
		params.MACDMA = 9;

		DSO dso = new DSO(params);
	}
}

class TSParams {
	public int MACDFast;
}

class Params extends TSParams {
	public int MACDSlow, MACDMA;
}

class TS {
	public TSParams params;

	public TS(TSParams params) {
		this.params = params;
	}

	public void out() {
		System.out.println(this.params.MACDFast);
	}
}

class DSO extends TS {
	public Params params;

	public DSO(Params params) {
		super(params);
		this.params = params;
		this.out();
	}
}

[bimeg]

Нет. Это разные переменные. Опять же вторая хайдит первую. Это решается например так:

class TS {
	private TSParams params;

	public TSParams getParams()
	{
		return params;
	}

	public TS(TSParams params) {
		this.params = params;
	}

	public void out() {
		System.out.println(getParams().MACDFast);
	}
}

class DSO extends TS {
	public DSO(Params params) {
		super(params);
		this.out();
	}

	public void out()
	{
		super.out();
		System.out.println(getParams().MACDSlow);
	}

	public Params getParams()
	{
		return (Params) super.getParams();
	}
}

[daMage]

@bimeg спасибо огромное!

Answer 2

[anyd3v]

У вас в классах
class TSParams {
public int MACDFast;
}

class Params extends TSParams {
public int MACDFast, MACDSlow, MACDMA;
}

одинаковое поле MACDFast, уберите его из Params и будет работать как ожидается.

When you declare a field with the same name as an existing field in a superclass, the new field hides the existing field. The existing field from the superclass is still present in the subclass, and can even be used ... subject to the normal Java access rules.

References:

docs.oracle.com/javase/tutorial/java/IandI/hidevar...
docs.oracle.com/javase/specs/jls/se7/html/jls-8.ht...

Comments

[daMage]

@anyd3v не могли бы вы ответить на комментарий к @bimeg?