[Java] 능력 단위 평가

능력 단위 평가 

국비수업 10일 차에는 능력 단위 평가라는 시험을 보았습니다. 

 

시험이라고 하지만 사실 팀 프로젝트의 코드를 올리는 게 시험문제였습니다. 

 

---

코드리뷰

오전에는 코드와 노션의 아쉬운 부분을 수정하였고 오후가 되기 전 12시부터 강사님께 팀 프로젝트를 했던 코드를 가지고 가서 팀원분들과 함께 코드 리뷰느낌으로 간단한 평가를 진행하였습니다.

 

각자 맡은 부분을 설명하며 강사님께 설명을 드리면 강사님이 질문을 하는 형식으로 진행되었습니다. 

 

엄청 긴장되었지만 긴장되는 마음을 참으며 제가 맡은 부분을 설명하였습니다. 

 

무사히 코드리뷰가 마무리 되었고 간단히 피드백받은 부분을 정리해 보았습니다. 

 

---

팀 프로젝트 코드 

package test03;

import java.util.Random;
import java.util.Scanner;

class Pokemon {
	static final Random rand = new Random(); // 가시성을 높이기 위한 순서 변경
	static final int MAX_LEVEL = 50; // 최대 레벨 50

	int num; // 고유값 PK
	int level; // 1~5 랜덤값
	int exp; // 현재(기존)경험치 0

	String type; // 반드시 예)물, 에스퍼, ...
	String name; // 반드시

	Pokemon(int num, String type, String name) {
		this.num = num; //PK값
		this.type = type;
		this.name = name;
		this.level = rand.nextInt(5) + 1; // 초기화
		// nextInt로 레벨 설정
		this.exp = 0;
	}

	void attack() {
		System.out.println("");

		int cntAttack = 0; // attack()에서만 사용되는 변수
		// 3번의 공격시도
		for (int i = 0; i < 3; i++) {
			int newExp; // 획득경험치 50~500 램덤숫자
			int randAttack; // 공격 난수 0~1
			randAttack = rand.nextInt(2); // 성공50%과 실패50%의 확률을 내는 난수
			if (randAttack == 1) {
				newExp = rand.nextInt(451) + 50; // 경험치 50~500을 내는 난수
				System.out.println("===============");
				System.out.println("효과가 굉장했다. 상대 트레이너의 포켓몬이 쓰러졌습니다.");
				System.out.println(newExp + "의 경험치를 얻었습니다.");
				System.out.println("포켓몬 트레이너 레드와의 승부에서 이겼다!");
				this.levelUp(newExp);
				break;
			} 
			else {
				System.out.println("===============");
				System.out.println((i + 1) + "번째 공격에 실패하였습니다.");
				System.out.println((2 - i) + "번의 공격시도가 남았습니다.");
				System.out.println("다시 공격을 시도합니다.");
				cntAttack++; // 공격횟수 + 1
				if (cntAttack == 2) {
					System.out.println("===============");
					System.out.println("마지막 공격마저 실패하였습니다.");
					System.out.println("포켓몬 트레이너 레드와의 승부에서 패배하였습니다...");
					System.out.println("획득할 경험치가 없습니다.");
					break;
				}
			}
		}
	}

	void hello() {
		// 피카츄는 삐까삐까, 꼬부기는 꼬북꼬북 등의 울음소리를 출력
		System.out.println("Pokemon안에서 hello메서드 사용");
	}

	void levelUp(int newExp) {
		
		if (this.level >=  MAX_LEVEL) { // 최고레벨 50일 때,경험치 0으로 고정
			System.out.println("최고 레벨 50");
			this.level = MAX_LEVEL;
			this.exp = 0;
		}
		
		// 경험치가 100을 채울때마다 레벨은 +1
		this.level = newExp / 100 + this.level;
		this.exp = newExp % 100 + this.exp;
		if (this.exp >= 100) { // 경험치가 100 넘으면 +1 레벨업
			this.exp = this.exp - 100;
			this.level++;
		}
		
	}

	@Override
	public String toString() {
		return "[" + this.name + " " + this.type + " lv." + this.level + " exp." + this.exp + "]";
	}
}

class Lucky extends Pokemon { // 럭키
	Lucky(int num, String name) {
		super(num, "노말", name);
	}	

	Lucky(int num) {
		this(num, "럭키");
	}

	@Override
	void hello() {
		System.out.println("[ " + this.name + " : 럭키! ]");
	}

	@Override
	void attack() {
		System.out.println("몸통박치기!");
		super.attack();
	}
}

class Pangdori extends Pokemon { // 팽도리
	Pangdori(int num, String name) {
		super(num, "물", name);
	}

	Pangdori(int num) {
		this(num, "팽도리");
	}

	@Override
	void hello() {
		System.out.println("[ " + this.name + " : 팽팽! ]");
	}

	@Override
	void attack() {
		System.out.println("물대포!");
		super.attack();
	}
}

class Natu extends Pokemon { // 네이티오
	Natu(int num) {
		this(num, "네이티오");
	}

	Natu(int num, String name) {
		super(num, "에스퍼", name);
	}

	@Override
	void hello() {
		System.out.println("[ " + this.name + " : 네오~티오! ]");
	}

	@Override
	void attack() {
		System.out.println("염동력!");
		super.attack();
	}
}

class Rizarmong extends Pokemon { // 리자몽
	Rizarmong(int num, String name) {
		super(num, "불", name);
	}

	Rizarmong(int num) {
		this(num, "리자몽");
	}

	@Override
	void hello() {
		System.out.println("[ " + this.name + " : 리자아! ]");
	}

	@Override
	void attack() {
		System.out.println("화염방사!");
		super.attack();
	}
}

class Mew extends Pokemon { // 뮤
	Mew(int num) {
		this(num, "뮤");
	}

	Mew(int num, String name) {
		super(num, "에스퍼", name);
	}

	@Override
	void hello() {
		System.out.println("[ " + this.name + " : 뮤우뮤우! ]");
	}

	@Override
	void attack() {
		System.out.println("염동력!");
		super.attack();
	}
}

class Houdin extends Pokemon { // 후딘
	Houdin(int num) {
		this(num, "후딘");
	}

	Houdin(int num, String name) {
		super(num, "에스퍼", name);
	}

	@Override
	void hello() {
		System.out.println("[ " + this.name + " : 후디니! ]");

	}

	@Override
	void attack() {
		System.out.println("염동력!");
		super.attack();
	}
}

public class Test03 {

	public static boolean isEmpty(int cnt) {
		if(cnt <= 0) {
			System.out.println("사용할 수 있는 포켓몬이 없습니다");
			return true;
		}
		return false;
	}

	public static boolean isFull(int cnt) { //포켓몬이 없을 때
		if(cnt >= 6) {
			System.out.println("포켓몬이 가득 찼습니다");
			return true;
		}
		return false;
	}
	//포켓몬이 가득 찼을 때

	public static void printList(Pokemon[] datas, int cnt) {
		System.out.println("\n=== 나의 포켓몬 ===");
		for (int i = 0; i < cnt; i++) {
			System.out.println((i+1) + "번 포켓몬 " + datas[i]);
		}
		System.out.println("===============");
	}

	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		int PK = 1;
		String[] pokemon = {"럭키", "리자몽", "후딘", "팽도리", "네이티오", "뮤"}; //포켓몬 종의 배열

		Pokemon[] caughtPokemon = new Pokemon[6]; //잡은 포켓몬의 배열
		caughtPokemon[0] = new Lucky(PK++);
		caughtPokemon[0].level = 5; // 소유하고 있는 럭키의 레벨을 5로 고정!

		int cnt = 1;

		// 상수정의:메뉴 번호
		final int EXIT = 0, PLAY = 1, INFO = 2, HOWLING = 3, CATCH = 4; //배열 활용 상수화해서 유지보수 용이하게 만듬
		//메뉴 항목 배열
		String[] menus = new String[] {"[게임종료]", "[게임하기]", "[전체상태출력]", "[울음소리듣기]", "[포켓몬잡기]"};
		//메뉴 텍스트 생성
		String menuText = "메뉴출력 \n"; 

		for (int i = 0; i < menus.length; i++) {
			menuText += i + ". " + menus[i] + "\n";
		}

		while (true) {
			// System.out.println("메뉴출력 \n0. 게임종료\n1. 게임하기\n2. 전체상태출력\n3. 울음소리듣기\n4. 포켓몬잡기");
			System.out.println(menuText);
			System.out.print("번호를 선택하세요 : ");
			int menu = sc.nextInt();

			if (menu == EXIT) {
				System.out.println("게임 종료");
				break;
			} else if (menu == PLAY) { // 1. 게임하기 
				if(isEmpty(cnt)) {
					continue;
				}

				int selecNum = 0; // 메뉴를 번호로 선택 
				System.out.println("\n포켓몬 트레이너 레드가 승부를 걸어왔다!");
				System.out.print("몇 번째 포켓몬을 내보낼지 선택해주세요! (1 ~ "+ cnt+")" );
				while(true) {
					printList(caughtPokemon, cnt);
					selecNum = sc.nextInt(); //사용자가 입력한 포켓몬

					if(0 < selecNum && selecNum <= cnt) {
						System.out.println(caughtPokemon[selecNum-1].name + " 너로 정했다!");

						System.out.println(caughtPokemon[selecNum-1].name+ "은(는) 무엇을 할까?");

						caughtPokemon[selecNum-1].attack();
						printList(caughtPokemon, cnt);
						break;
					}
					System.out.println("※※※※※ 현재 가지고 있지 않은 포켓몬입니다.※※※※※\n다시 입력해주세요 ");
					// 없는 포켓몬을 선택할 경우 
				}  
			}else if (menu == INFO) { // 2. 전체상태출력
				if(isEmpty(cnt)) {
					continue;
				}
				printList(caughtPokemon, cnt);
			} else if (menu== HOWLING) { // 3. 울음소리듣기
				if(isEmpty(cnt)) {
					continue;
				}
				int selecNum; // 사용자가 선택할 숫자
				while(true) {
					printList(caughtPokemon, cnt);
					System.out.println("듣고 싶은 포켓몬번호 선택 >>");
					sc.nextLine(); // 버퍼닫기

					if(sc.hasNextInt()) { // 숫자인지 아닌지
						selecNum = sc.nextInt();
						if(0<selecNum && selecNum <= cnt) { // 유효성 검사
							caughtPokemon[selecNum-1].hello();
							break;
						}
						else {
							System.out.println("※※※※※ 유효한 숫자만 입력하세요. ※※※※※");
						}
					}
					else {
						System.out.println("※※※※※ 숫자만 입력해주세요. ※※※※※");
						continue;
					}
				}

			} else if (menu == CATCH) { // 4. 포켓몬잡기
				if(isFull(cnt)) {
					continue;
				}

				int selecMenu;//사용자가 입력한 메뉴 
				String pokemonName;
				while(true) {
					if(isFull(cnt)) {
						break;
					}
					System.out.println("포획할 포켓몬을 입력해주세요");
					//럭키, 리자몽, 후딘, 팽도리, 네이티오, 뮤 중에서 선택!
					System.out.println("[1. 럭키 2. 리자몽 3. 후딘 4. 팽도리 5. 네이티오 6. 뮤 ]");
					selecMenu = sc.nextInt();//사용자가 입력한 포켓몬

					//유효값 측정
					if(selecMenu<1 || selecMenu>pokemon.length) {
						System.out.println("※※※※※ 없는 번호입니다. 다시 입력해주세요 ※※※※※※");
						continue;
					}

					while(true) {
						System.out.println(pokemon[selecMenu-1]+"의 이름을 입력해주세요");
						pokemonName = sc.nextLine(); // 버퍼비우기
						pokemonName = sc.nextLine();
						//사용자가 작성한 이름을 한버더 확인
						System.out.print(pokemonName + "(으)로 하시겠습니까? (Y/N) : ");
						String answer = sc.next();//사용자가 입력한 대답
						
						if(answer.equalsIgnoreCase("Y") || answer.equalsIgnoreCase("YES")) {
							break;
						}
						
						System.out.print("다시 입력해주세요 : ");
					}

					// 이름을 입력하지 않은 경우 포켓몬의 이름은 종명
					if(pokemonName.isEmpty()) {
						pokemonName = pokemon[selecMenu-1]; 
					}
					if(selecMenu == 1) {
						caughtPokemon[cnt] = new Lucky(PK++, pokemonName);
					}
					else if(selecMenu == 2) {
						caughtPokemon[cnt] = new Rizarmong(PK++, pokemonName);
					}
					else if(selecMenu == 3) {
						caughtPokemon[cnt] = new Houdin(PK++, pokemonName);
					}
					else if(selecMenu == 4) {
						caughtPokemon[cnt] = new Pangdori(PK++, pokemonName);
					}
					else if(selecMenu == 5) {
						caughtPokemon[cnt] = new Natu(PK++, pokemonName);
					}
					else{
						caughtPokemon[cnt] = new Mew(PK++, pokemonName);
					} // 포켓몬의 이름을 지은 후, 포켓몬 종에 따른 객체 생성

					System.out.println(pokemonName + "을(를) 포획했습니다");
					printList(caughtPokemon, ++cnt);
					break;
				}
			} else {
				System.out.println("※※※※※※ 잘못된 입력입니다. ※※※※※※");
			}
		}
	}
}

---

피드백 

1. 포켓몬을 레벨업 하는 과정에서 한번에 +5가 되는 것이 아니라 실제 게임처럼 +1, +1, +1, +1, +1 이 되는 형식으로 코드를 짜는 것이 좋았을 거 같다.

--> 타겟,목표,자료,시장,경쟁상대 조사가 부족했다

 

2. levelUp() 메서드에서는 this.level과 this.exp가 사용되고 있기 때문에 인자로 받을 필요가 없고 필요시 설명이 필요한 코드이다. 

--> 메서드 사용할때 인자여부/인자에 대한 설명 주석도 부족

 

3. 만들어둔 생성자가 1개 덜 사용되고 있었다. 

-->  럭키 객체를 생성하면 바로 부모생성자를 호출

 

4.  내가 작성한 코드에 주석이 없음 

--> 자신이 작성한 코드에 다른 사람이 봤을 때 이게 뭔지 이해할 수 있도록 주석을 작성 

 

---

좋았던 부분 

1. UI/UX 사용자 친화적이다. 

--> attack() 메서드에서와 main()의 게임하기 등에서 출력되는 멘트, 사용자의 포켓몬 정보 자료조사, 실제 포켓몬 게임 경험 등을 통한 멘트 설정이 좋았다. 

 

2. 코드를 개발자가 사용하기 편하게 작성하였다. 

// 상수정의:메뉴 번호
		final int EXIT = 0, PLAY = 1, INFO = 2, HOWLING = 3, CATCH = 4; //배열 활용 상수화해서 유지보수 용이하게 만듬
		//메뉴 항목 배열
		String[] menus = new String[] {"[게임종료]", "[게임하기]", "[전체상태출력]", "[울음소리듣기]", "[포켓몬잡기]"};
		//메뉴 텍스트 생성
		String menuText = "메뉴출력 \n"; 

		for (int i = 0; i < menus.length; i++) {
			menuText += i + ". " + menus[i] + "\n";
		}

			System.out.println(menuText);

 

--> 위의 코드처럼 개발자가 사용하기 편하도록 코드를 작성한 부분이 좋았습니다.

---

마무리 

이렇게 오늘 능력 단위 평가를 보았습니다.  피드백받은 부분을 기억하며 다음에는 저런 문제점들을 고칠 수 있도록 코드를 작성해 보는 게 중요 할거 같습니다. 

 

저는 다음 글로 찾아오겠습니다. 
봐주신 분들 모두 감사합니다.