메서드 코드 중복

Question

abstract class AirUnit extends Unit { private int power; public AirUnit(String name, int tribal, Point location, BaseStat baseStat, int power, double speed) { super(name, tribal, location, baseStat, speed); setPower(power); } // 공중공격 public void airAttack(AirUnit airTarget) { System.out.println(this.getName() + "의 공격!"); System.out.println(airTarget.getName() + "에게 " + power + "의 데미지를 입혔다!"); int remainDamage = power - airTarget.getBaseStat().getDef(); if(remainDamage > 0) { airTarget.getBaseStat().setDef(0); remainDamage = airTarget.getHp() - remainDamage; airTarget.setHp(remainDamage); if(remainDamage < 0) { airTarget.setHp(0); } } else { airTarget.getBaseStat().setDef(- remainDamage); } System.out.println(airTarget.getName() + "의 HP / def : " + airTarget.getHp() + " / " + airTarget.getBaseStat().getDef()); } //지상공격 public void groundAttack(GroundUnit groundTarget) { System.out.println(this.getName() + "의 공격!"); System.out.println(groundTarget.getName() + "에게 " + power + "의 데미지를 입혔다!"); if(groundTarget.getTribal() == PROTOSS) { // 타겟의 종족이 프로토스면 int remainDamage = power - ((ShieldStat)groundTarget.getBaseStat()).getShield(); if(remainDamage > 0) { // 내 공격력이 더 쎄면 ((ShieldStat)groundTarget.getBaseStat()).setShield(0); remainDamage = remainDamage - groundTarget.getBaseStat().getDef(); groundTarget.setHp(groundTarget.getHp() - (remainDamage >= 0 ? remainDamage : 0)); System.out.println(groundTarget.getName() + "의 HP: " + groundTarget.getHp()); } else { // 내 공격력이 더 낮으면 ((ShieldStat)groundTarget.getBaseStat()).setShield(- remainDamage); System.out.println(groundTarget.getName() + "의 HP / Shield : " + groundTarget.getHp() + " / " + ((ShieldStat)groundTarget.getBaseStat()).getShield()); } } else { // 프로토스말고 다른 종족 System.out.println(this.getName() + "의 공격!"); System.out.println(groundTarget.getName() + "에게 " + power + "의 데미지를 입혔다!"); int remainDamage = power - groundTarget.getBaseStat().getDef(); if(remainDamage > 0) { // 내 공격이 더 쎄면 groundTarget.getBaseStat().setDef(0); remainDamage = groundTarget.getHp() - remainDamage; groundTarget.setHp(remainDamage); if(remainDamage < 0) { groundTarget.setHp(0); } } else { // 내 공격력이 적의 방어력보다 약할 때, groundTarget.getBaseStat().setDef(- remainDamage); } System.out.println(groundTarget.getName() + "의 HP / def : " + groundTarget.getHp() + " / " + groundTarget.getBaseStat().getDef()); } }공중공격 메서드가 지상공격의 else부분(프로토스말고 다른 종족 else)이랑 코드가 동일한데 코드 중복으로 안좋게 생각할까요..?! 저런 건 어떻게 중복을 제거해야 하는지 궁금합니다 !

킁킁탐정 · Accepted Answer

누구에게 공격과 데미지를 입혔다 정도의 메시지 출력 관련만 별도 메소드로 빼서 공용으로 사용하고그 외에 조건문 안의 코드들은 굳이 하나로 합치기 위해 리팩토링하지 않아도 됩니다.중복 코드가 항상 나쁜게 아닙니다. 몇 번 사용하지 않는 중복 코드라면 괜찮습니다.지금 이 코드에서 중복을 제거하면 나중에 변화에 대응하기가 오히려 더 힘들 수 있습니다.참고하세요.

톰소여 · Answer

공중공격의 중복이라고 생각되는 해당 코드 블럭을 선택하시고 리펙토링(함수로 추출) 하세요.들어가는 인자와 리턴을 잘 생각해보시고 함수를 변경하세요.지상공격의 else부분에 해당 함수를 넣으시고 인자를 바꾸어서 호출하세요.대략 보면..AirUnit airTarget과 GroundUnit groundTarget 만 바뀌는 것 같은데 Unit을 상속하고 있으니 타입을 유닛으로 바꾸고 Unit targetUnit로 바꾸고 코드를 일부 수정하면 되겠네요.

자전거타고러닝하고 · Answer

저라면 프로토스를 체크 하는게 아니라 실드가 있는지만 체크해서 실드깨는 로직만 넣고 나머진 그대로 하면 될것 같아 보입니다...

메서드 코드 중복

3개의 답변

Q&A 목록