때때로 컨트롤러 클래스의 액션 함수는 모델에서 뷰로의 데이터 흐름을 간단히 제어하기위한 많은 코드 줄로 인해 거대하고 불쾌해질 수 있습니다. 어떤 시점에서이 거대한 기능들은 좋은 코드의 기본 원칙을 완전히 잃어 버립니다.
이러한 거대한 액션 함수를 컨트롤러 클래스에서 더 작은 개인 함수로 나누는 것이 좋은 방법으로 간주됩니까? 아니면 그러한 최적화가 필요하다면 모델에 추가해야합니까?
나는 컨트롤러에서 더 작은 함수를 프라이빗으로 사용하여 동작과 관련이 있다고 투표하지만 모델이 거대하고 덩어리가 될 수있는 동안 컨트롤러가 간단해야한다는 주장을 들었습니다. 어느 것이 가장 선호되는 방법인지 궁금합니다.
답변:
가장 좋은 유추는 아니지만 스파이더 웹에 대해 생각하는 것과 같은 방식으로 컨트롤러에 대해 생각하십시오. 그것의 유일한 임무는 거미 (기본 레이어)가 소화하도록 파리 (요청)를 잡는 것입니다. 웹은 더 작거나 큰 파리 (모델)를 붙잡고 잡을 수 있습니다. 거미의 웹 역할은 먹이를 소화시키는 것이 아니라,이 목적으로 사용될 수 있습니다. 웹이 얇고 깨끗할수록 거미가 더 쉽게 살 수 있습니다.
MVC 응용 프로그램에 다소 동일한 논리를 적용 할 수 있습니다. 설명하는 거대하고 불쾌한 함수는 모델의 동작 일 가능성이 높으며 모델에 속해야합니다 (모델은 뷰에 표시되는 객체 만이 아니라는 점에 유의하십시오). 모델의 동작이 변경되면 모델을 처리하는 컨트롤러가 아니라 변경해야하는 모델입니다.
또한 컨트롤러에서 개인 메서드로 유지하면 컨트롤러가 복잡해지고 유지 관리가 어려워집니다. 또한 개발에 참여한 다른 사람들이 프로젝트에서 이전에 해왔 던 것을 보았 기 때문에 같은 일을하려는 유혹을 받기 때문에 나쁜 습관을 길러줍니다.
내가 줄 수있는 가장 좋은 대답은 로버트 마틴 (Robert Martin)의 위대한 책에서 "클린 코드 (Clean Code)"를 인용하는 것입니다.
함수의 첫 번째 규칙은 작아야한다는 것입니다. 두 번째 규칙은 그것들보다 작아야한다는 것입니다.
더 잘 말할 수 없습니다. 같은 책에서 또 다른 큰 인용문이 적용됩니다.
함수는 한 가지 일을해야합니다. 그들은 잘해야합니다. 그들은 그것을해야합니다.
코드를 더 많은 함수로 분할 할 때 코드의 가독성을 크게 향상시킬 수있는 의미있는 이름을 해당 함수에 제공해야합니다. 말할 것도없이, 클래스 외부에서 사용하도록 의도되지 않은 모든 함수는 전용이어야하므로 상속을 통해 코드를 쉽게 재사용 할 수 있습니다.
컨트롤러에 기능이 너무 많으면 아마도 너무 많은 기능을 수행한다는 표시입니다. 그런 다음 여러 독립적 인 조각으로 나누거나 다른 답변에서 언급 한 것처럼 일부 기능을 모델로 이동하려고 할 수 있습니다. 또한 뷰에 약간의 논리가 허용되는 비 클래식 MVC 풍미를 따르는 경우 적합 할 때마다 일부 함수를 배치 할 수 있습니다.
MVC에서는 컨트롤러가 가능한 "얇은"모델인지 확인하고 모델이 최대한 바보인지 확인합니다.
필요한 논리 및 도우미 함수는 별도의 독립형 도우미 클래스에 배치됩니다. 내 테스트도 훨씬 쉬워집니다 (테스트 중입니다. right ?? : D) 컨트롤러 테스트는 컨트롤러의 인스턴스를 만들려고 할 때 HTTP 컨텍스트와 가짜에 대해 생각해야 할 때마다 매우 어렵습니다. http 이것과 저것 그리고 그것의 고통, 그러나 그것의 고의적 인 고통. 컨트롤러는 HTTP와 웹에 밀접하게 연결되어 있기 때문에 모든 것이 필요합니다. 웹 앱의 진입 점입니다.
논리 및 도우미 기능은 웹과 관련이 없습니다. 그들은 전적으로 환경에 구애받지 않습니다 (또는 있어야합니다). 그것만으로는 그들이 같은 장소에 속해 있지 않다고 말할 수 있습니다. 또한 모든 응용 프로그램 논리를 웹이나 특정 웹 구현에 밀접하게 연결하면 절대로 사용할 수 없습니다.
우리는 모든 mvc 모델, 실제 db 엔터티가 아닌 모든 데이터베이스 엔터티, 저장소, 도우미 클래스 및 별도의 독립형 dll의 논리를 사용하여 MVC 사이트를 개발했습니다. 우리는 오직 하나의 웹 사이트 만 가지고 있지만 어쨌든 이렇게했습니다.
몇 달 전에 우리는 몇몇 프린지 시스템과 관련된 몇 가지 데스크탑 앱을 만들라는 요청을 받았습니다. 테스트 된 모든 코드를 쉽게 재사용 할 수 있으므로 쉽게 수행 할 수 있습니다. 코드를 웹 프로젝트에 넣거나 컨트롤러에 넣었다면 결코 그렇게 할 수 없었을 것입니다.
Dmitri Zaitsev와 우주인의 훌륭한 답변 외에도 다음이 PHP에도 유효한지 모르겠습니다. 자동 테스트 가능성이 없기 때문에 개인 메소드를 피해야합니다.
예, 메타 프로그래밍 또는 의존성 주입을 사용하여 개인 메소드를 테스트 할 수도 있지만 코드의 가독성에 큰 영향을 미치므로 그렇게하지 않아야합니다.
항상 KISS 원칙을 기억하십시오. 간단하고 어리석게 유지하십시오.