코드를 왜 이렇게 짠건지 궁금합니다.

Question

public function login(Request $request) { $this->validateLogin($request); // ... 생략 } protected function validateLogin(Request $request) { $request->validate([ $this->username() &#61;> 'required|string', 'password' &#61;> 'required|string', ]); }보시다시피 login() 함수에서 validateLogin() 함수를 호출합니다. validateLogin() 함수를 보면 그냥 validate() 함수를 이용한 부분 밖에 없는데.. 그냥 login() 함수에서 validate() 사용하는 부분을 쓰면 될텐데 왜 굳이 한 단계를 더 거치는 건가요?

멍충이이 · Answer

가독성, 재사용성 등등을 위해서요

코딩하는사람 · Answer

코드가 엄청 길어지면 한줄한줄 다 읽고 파악하는데 오래걸립니다. 그런데 저렇게 기능단위로 만들면 아, 여기서 validate 검증을 진행하였구나 라고 한줄만 읽고도 손쉽게 코드의 흐름을 파악 할 수가 있죠

카카오콘 · Answer

구문을 구분짓는 목적이 강합니다.1+11-11x11/1각각 서로에게 전혀 관련없는 코드들이 주욱 이어져있으면나중에 특정 부분의 코드를 보고 싶은데 이 긴 코드 중 어디지? 싶은게 첫번째 문제.그래도 찾으면 문제가 없는데, 어떤 사람은 1을 위에 선언해놓고 곱하기는 중간에서 하고 결과는 맨 아래서 출력하고이렇게 다른 로직과 전혀 관계 없는 하나의 프로세스를 위 중간 아래 흩뿌려놔서어디서부터 어디까지가 하나의 로직인거야? 싶은 때도 있습니다.의도치 않았는데 짜다보면 그렇게 되는 경우도 있구요.그래서 처음부터 저런 습관을 들이면 가독성이 매우 향상됩니다.

캐티 · Answer

public 으로 정의 하지 않았더나요. 이것은 다른 영역에서 사용하기 위함이어요. 굳이 한 단계를 더 거치는 이유는 접근 권한을 나누기 위함으로 볼 수 있겠어요.

nubie · Answer

하나의 함수는 하나의 기능만 한다 이렇게 해야 나중에 코드 분석하기 좋습이다

mirheeoj · Answer

원래는 뭔가 부가기능이 있었는데 버전업을 거치면서 저렇게 변화한 걸수도 있습니다. 콜 하는 곳들을 모두 수정하면 번거롭기도 하고 예기치 못한 문제가 생길수도 있으니까요. 해당 코드를 쓰는 줄도 몰랐던 곳에서 특정기능이 안된다고 연락이 온다든지 등등.

코드를 왜 이렇게 짠건지 궁금합니다.

6개의 답변

기타 목록