Как лучше поступить с конструктором?

Question

[ZelibobA1706]

Как правильно присвоить новый адрес через функцию add переменной second и последующим left, right и т.д.?

public class First {
    Second second;
    public First() {
        second = null;
        add(second);
    }
    public void add(Second sec) {
        sec = new Second(11);
        sec.left = new Second(15);
    }
}
public class Second {
    public Second left;
    public Second right;
    public Integer value;
    public Second(Integer value) {
        left = null;
        right = null;
        this.value = value;
    }
}

Answer 1

[Shockoway]

public class First {
    Second second;
    public First() {
        second = add(second);
    }
    public Second add(Second sec) {
        sec = new Second(11);
        sec.left = new Second(15);
        return sec;
    }
}
class Second {
    public Second left;
    public Second right;
    public Integer value;
    public Second(Integer value) {
        this.value = value;
    }
}

А вообще не совсем ясно к чему так усложнять.

public class First {
    Second second;
    
    public First(int value, int left, int right) {
    	second = new Second(value, left, right);
    }
    public First(int value) {
    	second = new Second(value);
    }
}

class Second {
    public Second left;
    public Second right;
    public int value;
    
    public Second(int value, int left, int right) {
        this.value = value;
        this.left = new Second(left);
        this.right = new Second(right);
    }
    
    public Second(int value) {
    	this.value = value;
    }
    
}

Comments

[ZelibobA1706]

А если я хочу только left задать или left у left?

[Shockoway]

ZelibobA1706: конструктор можно перегрузить для каждого из вариантов. Да и по хорошему переменные должны быть приватными, а доступ к ним через геттеры и сеттеры. Ну а если последнее всё таки не важно, то самый просто вариант такой:

public class First {
Second second;
public First(int value) {
second = new Second(value);
}
}
class Second {
Second left;
Second right;
int value;
Second(int value) {
this.value = value;
}
}
....
First first = new First(11);
first.second.left = new Second(2);

[Shockoway]

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

[Shockoway]

public class First {
private Second second;
public First(int value) {
second = new Second(value);
}
Second getChild() {
return second;
}
}
class Second {
private Second left;
private Second right;
private int value;
Second(int value) {
this.value = value;
}
Second(int value, int left, int right) {
this.value = value;
this.left = new Second(left);
this.right = new Second(right);
}
void setValue() {
this.value = value;
}
void addLeftChild(int value) {
left = new Second(value);
}
void addRightChild(int value) {
right = new Second(value);
}
int getValue() {
return value;
}
Second getLeftChild() {
return left;
}
Second getRightChild() {
return right;
}
}

Answer 2

[aol-nnov]

public class Second {
    public Second left;
    public Second right;
    public Integer value;
    public Second(Second left, Second right, Integer value) {
        this.left = left;
        this.right = right;
        this.value = value;
    }
    public Second(Integer value) {
        this(null, null, value);
    }

Comments

[ZelibobA1706]

Ничего же не изменилось. Проблема в том, что переменной second в классу First не присваивается новый адрес через функцию add.

[aol-nnov]

ZelibobA1706: "переменной second в классу First не присваивается новый адрес" - там вообще ересь написана.
функция add должна возвращать этот секонд тогда и снаружи ты присваиваешь этот результат полю класса.
но то, что у тебя сейчас в примере написано в First, не поддается осмыслению.

[ZelibobA1706]

aol-nnov: так я поэтому и вопрос задал. Я понимаю, что у меня second null и не имеет адреса, а я его в функцию передаю, а как правильно сделать не знаю. И у меня должна быть возможность рекурсивно идти по дереву ( add(second.left, value) ).

[aol-nnov]

ZelibobA1706: First тебе вообще зачем? задачу, может, озвучишь? :)

[ZelibobA1706]

aol-nnov: ну вообще там бинарное дерево, и First - само дерево, a Second - узел