Be/main/yeoshin java lv2 #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

duckgii wants to merge 5 commits into main from be/main/yeoshin_Java_Lv2

duckgii commented Jan 9, 2025

Lv2. 구현 완료했습니다.
하드코딩된 리터럴이 없도록 주의해서 진행했습니다!

duckgii added 5 commits

January 9, 2025 16:06


          DOCS : 전체 구조 정리

bb8ddba


          알고리즘 인터페이스 상속 객체들 구현 완료

2d3c91a


          DOCS : 완료 버전으로 수정

fba412b


          FEAT : Wchae 객체 알고리즘 풀이 후 문자열 반환 기능

e5395df


          FEAT : Main 클래스 테스트와 출력 기능

755e01b

duckgii self-assigned this

chyo1 reviewed

View reviewed changes

chyo1 left a comment

리뷰에서 공통되는 부분 다 찾아서 바꿔보기~! (hint. 상수, 출력 문자열)
수고하셨습니다~

v1/backend/JAVA_LV_2_알고리즘의_굴레/src/main/java/com/example/algorithmbot/BinarySearch.java Outdated

Comment on lines +5 to +9

chyo1 Jan 12, 2025

Suggested change

      
            	private final String solution;
          
            	BinarySearch() {
          
            		solution = "반띵 ㄱㄱ";
          
            	}
          
            	private static final String solution = "반띵 ㄱㄱ";

변수로 둘 거라면 상수화를 추천합니다

v1/backend/JAVA_LV_2_알고리즘의_굴레/src/main/java/com/example/algorithmbot/Main.java Outdated

Comment on lines +6 to +22

chyo1 Jan 12, 2025

enum 내부 필드 활용 굿
벗 파일 분리 해.줘.

v1/backend/JAVA_LV_2_알고리즘의_굴레/src/main/java/com/example/algorithmbot/Main.java Outdated

chyo1 Jan 12, 2025

Enum 클래스 내부의 static 상수로 옮겨주세요

v1/backend/JAVA_LV_2_알고리즘의_굴레/src/main/java/com/example/algorithmbot/Main.java Outdated

Comment on lines +41 to +43

chyo1 Jan 12, 2025

int보다 랜덤한 AlgorithmType을 뽑은 후 해당 타입을 넘기는게 좋을 것 같아요

v1/backend/JAVA_LV_2_알고리즘의_굴레/src/main/java/com/example/algorithmbot/Main.java Outdated

chyo1 Jan 12, 2025

Suggested change

      
            		System.out.println(DEFAULT + algorithmTypes[firstAlgorithm].getType() + DELIMITER + algorithmTypes[secondAlgorithm].getType());
          
            		System.out.println("오늘의 알고리즘 : %s, %s", algorithm1.getType(), algorithm2.getType());

처럼 하나의 문자열로 만들어서 넘겨주세요

v1/backend/JAVA_LV_2_알고리즘의_굴레/src/main/java/com/example/algorithmbot/Main.java Outdated

Comment on lines +63 to +64

chyo1 Jan 12, 2025

상동

enaenen commented Jan 14, 2025 •

edited

Loading

다른분들이 다 잘 써주셨고, 또 잘 작성해주셔서 좋네요

but 군데군데 접근지정자가 비어보이네요 like my 알고리즘 기본기

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet