[미션 피드백] 메서드&변수 네이밍, 자바 날짜/시간 클래스 LocalDateTime, import에 대해

리뷰어 Dion께서 자바 시간 표시 및 객체생성, import에 대한 좋은 피드백을 주셔서 기록으로 남긴다! 감사합니다🙇‍♀️

메서드 네이밍

메서드에는 메서드 내부 동작을 드러내기 보다는 메서드가 수행할 행동을 적어주세요.
간단하게 home으로 갈것이니 home()정도여도 충분합니다.

아 그렇군요! 메서드명은 동사로 하는 게 좋다는 말을 들어서 이렇게 했는데, 동사에 집착하기보다는 메서드의 행동을 나타내는 것에 더 중점을 두면 되나요?😮

네 동사 보다는 메서드가 어떤 일을 해줄 것인가가 더 중요하답니다!

+줄여쓰기 지양하기 (inputPw)

변수 네이밍

임시변수라도 변수명에 관사는 붙이지 않는 게 일반적이다. aUser(❌)

자바 날짜/시간 클래스 LocalDateTime 및 import에 대해

public String makeNewQuestion(Question question) {
    SimpleDateFormat format = new SimpleDateFormat("yyyy-MM-dd HH:mm");
    String formatTime = format.format(System.currentTimeMillis());
    question.setDateTime(formatTime);
    // ...
}

객체 생성 시점에 시간이 생성되도록 하는 방법이 있지 않을까요~

Date객체를 사용하도록 수정하겠습니다!
질문이 있는데, Date를 쓰면 따로 import를 해야 하지만 System.currentTimeMillies()는 import할 필요가 없어서 저는 이걸 선택했었는데요 (날짜출력 메서드가 이미 있는데 굳이 import java.util.Date를 하는 게 더 안좋을거라는 짐작 때문이었습니다)
시간을 메서드로 받아오는 것보다 객체 생성 시점에 시간을 생성하는 게 더 좋은 건가요?🤔

답변 :

일단은 Java8 이전의 날짜/시간 API인 Date, Calendar는 사용하지 마시구요!
Java8에서 추가된 날짜/시간 API를 사용해주시면 되겠습니다. 일단 LocalDateTime의 사용을 추천드릴게요.

질문에 대한 답변입니다.

import를 한다는 것은 그 코드가 필요해서 가져다 쓴다는 의미입니다. import를 한다고, 그것이 잘못된 행동이 아닙니다. 간단히 정리해드리자면, 더 전문적인 친구에게 일을 맡기는 것입니다.

우리가 날짜를 하루 뒤로 조작한다고 가정해볼게요. import를 해준 전문적인 친구는 날짜를 조작하는 메서드를 제공하기 때문에 편하게 이용할 수 있습니다.
하지만 우리가 import를 하지않고 직접 사용했다면, 두 가지 문제가 발생합니다. 코드가 더러워지고, 오류가 발생할 수 있습니다.

제가 생각하기에 뒤의 질문은 어떻게 하는가를 정하는건 개발자의 몫인데요. 저의 경우에는 객체 생성시점을 넣어주는 것을 선호하는 편입니다. 일단 set을 사용하지 않아도 되거든요!

+

왜 자바 날짜,시간 표현에 LocalDateTime을 사용해야 하는지는 아래 링크에 잘 정리되어 있어서, 이걸 보고 공부해야겠다.

[LocalDate,LocalTime,LocalDateTime 총 정리](