Java OOP : 객체 트리/객체 패밀리 구성

Question

Java를 사용하고 있었기 때문에이 설정이 괜찮은지 또는 올바른 방법인지 궁금해했습니다.

FYI의 userResults는 아래 코드에없는 JDBI 변수를 참조합니다.

더 나은 방법을 제안 해 주시기 바랍니다, 감사합니다.

public class Stat 
{ 
private int current; 
private int max; 

public int getCurrent() {return current;} 
public void setCurrent(int current) {this.current = current;} 

public int getMax() {return max;} 
public void setMax(int max) {this.max = max;} 
} 


public class Character 
{ 
Stat hp = new Stat(); 
Stat mp = new Stat(); 
} 

Character thisCharacter = new Character(); 

// Set the value of current & max HP according to db data. 
thisCharacter.hp.setCurrent((Integer) userResults.get("hpColumn1")); 
thisCharacter.hp.setMax((Integer) userResults.get("hpColumn2")); 

// Print test values 
System.out.println (thisCharacter.hp.Current); 
System.out.println (thisCharacter.hp.Max); 

Answer 1

맞습니까? 글쎄, 일 작품? 그런 다음 이 올 Y 르고입니다.

웨더이든 아니든간에 괜찮은 대답은 "어쩌면"입니다. 이 코드에서 어떤 맥락에서 얘기하기 어렵다하지만 마음에 유지할 수있는 몇 가지 생각이 있습니다.

• 하는 클래스 (또는 오히려 객체)에서 Stat 세트에? 이 작업을 수행하고 그것을 가져올 데이터베이스 값을 알고있는 클래스의 책임이라고 생각하십니까? 그렇지 않다면, 이것을하는 일종의 클래스를 만드는 것을 고려하십시오.
• thisCharacter.hp.setCurrent(...)과 같은 연결 전화는 principle of least knowledge의 위반입니다. 때때로 당신은 그것을 도울 수 없지만, 보통 그것은 끈적 거리는 코드로 연결됩니다. 통계를 둘러싼 모든 논리를 처리하는 것을 고려하십시오. 코드에서 loadStats(), saveStats() 및 takeDamage(int dmg) 및 revive(int health)과 같은 변형 연산자 동작과 같은 메서드가있는 HealthStatsHandler이 필요할 수 있습니다.
• 올바른 개체 디자인이있는 경우 문제가 발생하는 경우 SOLID principles에서 공부하십시오. 그들은 확장 가능하고 "깨끗한"코드를 원한다면 모든 개발자가 따라야하는 훌륭한 지침을 제공합니다.

Answer 2

이것은 실제로 나무가 아닙니다. 두 명이 둘 이상의 자식 계층을 가질 수는 없습니다.

보통 당신은 인터페이스가 모두 Stat 및 Character 그것을 구현하고 Character의 두 아이는 유형 Node을했을 경우의이 Node를 호출하도록 정의합니다.

Answer 3

나는 별도로 스탯 객체를 생성하고 문자로 전달하고, 문자를 다음과 같이 개인 속성을 만드는 고려할 것 :

public class Character 
{ 
    private Stat hp; 
    private Stat mp; 

    public Stat getHp()   {return hp;} 
    public void setHp(Stat h) {this.hp = h;} 

    public Stat getMp()   {return mp;} 
    public void setMp(Stat m) {this.mp = m;} 
} 

// Set the value of current & max HP according to db data. 
Stat hp = new Stat(); 
hp.setCurrent((Integer) userResults.get("hpColumn1")); 
hp.setMax((Integer) userResults.get("hpColumn2")); 

Character thisCharacter = new Character(); 
thisCharacter.setHp(hp); 
// do the same for mp 

하나 추가 간단한 단계는 걸릴 것 Character 생성자를 생성하는 것입니다 hp 및 mp