Created by 한재중, last modified on 11월 12, 2020

시험 문제


답안 및 평가

김정래

항목수행 내용평가 점수평가 내용
답안​https://github.com/wdfo7765/CodingTestForGT2/tree/JeongRaeKIM_Branch1B

​총평

  • 기본 기능은 거의 만족했으나 코드 품질이 아쉬움
작업 순서잘 지켰음AN/A
FTP 정보 입출력

Console 을 통해 입력 받도록 했음

  • Address / IP / PW

XML 을 Save 만 하고 Load 는 하지 않음

B

입력 내용을 XML 로 저장할 수 있음

그런데 활용하는 코드가 없음

의미없는 코드를 왜 만들었을까?

XmlSerializer 를 사용하지 않고 XmlDocument/XmlNode 를 사용함

node 이름이 하드코딩 됨

Local Path 경로Console 을 통해 입력 받도록 했음AN/A
파일/폴더 구분반환값의 주소 형식을 기반으로 구분B+더 직관적인 나이스한 것이 없었을까?
FTP Client 구현 방식FtpWebRequest 사용AN/A
다운로드 결과 출력구현하지 않음C

관련 코드를 확인할 수 없었음

언제 어떤 FTP 서버에서 어떤 파일들을 어디에 복사했는지, 또는 그 요약에 대한 코드가 없음

리팩토링 (Option)수행했음B

오히려 의미없는 함수가 보임. 대표적으로 isExist() 임. 함수의 이름 상, 존재여부를 return 해야 하는데 내부적으로 있으면 삭제하고 있음 

모두 public 임. private 이어야 함

테스트 코드 (Option)수행했음B-

테스트 코드의 목적성이 보이지 않음

코드의 품질보다는 했다는 시늉 정도의 느낌임

요구사항 외 잘한 점

주석을 열심히 달았음

commit 의 단위가 작았음. 하지만 의미없는(질문) commit 도 있었음 (습관적인 commit?)

N/AN/A
요구사항 외 아쉬운 점

예외처리가 아쉬움

  • 생성 가능한 폴더 경로가 입력으로 들어왔는지 체크하지 않음
  • FTP 연결 실패에 대한 처리가 없음
  • 다운로드 할 것이 하나도 없더라도 성공 메시지가 찍힘

접근 한정자의 잘못된 사용

  • 모두 public 으로 선언되었음
  • 외부에 노출하지 않을 항목들을 private 로 해야 함
N/AN/A


금기룡

항목수행 내용평가 점수평가 내용
답안

https://github.com/infachala/CodingTestForGT2  

B+

​총평

  • 기본 기능은 거의 만족했으나 코드 품질이 아쉬움
    • XmlSerializer 등을 사용하지 못한 부분이 아쉬움
  • 리팩토링과 테스트 코드에 대한 개념은 모르고 있는 것인지 확인 필요함
작업 순서잘 지켰음A
FTP 정보 입출력XML 파일 사용A

입출력 모두 사용함

XmlSerializer 를 사용하지 않고 XmlDocument/XmlNode 를 사용함

node 이름이 하드코딩 됨

Local Path 경로XML 파일 사용AN/A
파일/폴더 구분구분함Afile/folder type 정보를 이용하여 구분함
FTP Client 구현 방식FtpWebRequest 사용AN/A
다운로드 결과 출력구현하지 않음C

다 받아졌는지 아닌지만 출력하고 있음

디테일한 정보...하다 못해 개수라도...표기했다면 좋아을걸...

리팩토링 (Option)수행하지 않음N/AN/A
테스트 코드 (Option)수행하지 않음N/AN/A
요구사항 외 잘한 점

N/A

N/AN/A
요구사항 외 아쉬운 점

Branch 당 commit 한번씩 밖에 없음

comment 없음

추가 점수에 대한 욕심이 없음

N/AN/A


최홍준

항목수행 내용평가 점수평가 내용
답안



​총평

작업 순서


FTP 정보 입출력


Local Path 경로

N/A
파일/폴더 구분


FTP Client 구현 방식


다운로드 결과 출력


리팩토링 (Option)

N/A
테스트 코드 (Option)

N/A
요구사항 외 잘한 점


N/AN/A
요구사항 외 아쉬운 점


N/AN/A