OOP - 다른 함수 내에서 함수를 호출하거나 큰 컨트롤러 하나를 사용하는 것이 더 좋습니까?

Question

몇 주 전에 작성한 상당히 복잡한 스크립트를 다시 작성합니다. 몇몇 사람들에게 보여준 후에, 그들은 모두 특정한 목적을 가진 더 작은 조각들 (클래스들)로 나누어야한다는 것에 동의했고, SOLID object-oriented programming principles

독학 프로그래머로서,이 개념들은 나에게 꽤 새롭다. 하나의 함수/클래스에서 다른 클래스로 데이터를 가장 잘 전달하는 방법에 대해서는 약간 혼란 스럽습니다. 예를 들어, 여기에 내가 짓고 있어요 무엇의 단순화 된 구조는 다음과 같습니다

내 질문은 :

나는 이러한 기능 /의 모든 클래스를 호출하는 "컨트롤러"스크립트가해야 필요한 곳에 데이터를 전달합니까? 위의 다이어그램에서 for loop을 넣을 곳이 Function 2이 될 수있는 Function 1의 일부분 일 뿐이겠습니까?

Answer 1

컨트롤러/서비스 스크립트가 없으면 실제로는 단일 책임 원칙을 충족시킬 수 없습니다. 내가 당신이라면 구현할 수있는 고급 설계가 있습니다.

FileRepository
파일을 가져 오는 방법을 폭로해야합니다. 이름은 GetFileById입니다.

MongoDBRepositoryCRUD 연산 방법. 이러한 SelectById, SelectByQuery, 업데이트, 로 는, 이 삭제 만듭니다.

객체 생성 로직
당신의 createNewObj 로직 코드가 그렇게 보이도록이 개체의 생성자로 이동하기에 충분 것이다 간단한 경우 : var newObj = new MyObj(json[i]); 를하지만 상대적으로 복잡 경우 일부를 사용 어쩌면 때문에 검증을 위해 제 3 자 프레임 프레임을 사용하거나 공장을 만드는 것이 좋습니다.
마지막으로 당신이 컨트롤러/서비스 개체를 구현해야합니다 var newObj = MyObjFactory.Create(json[i]);

서비스/컨트롤러 : 그럼 당신은 이런 식으로 보일 것입니다 수 있습니다. WorkService의 이름은 DoWork이며 위와 같이 기록 된 리포지토리와 다른 개체 간의 모든 상호 작용을 처리해야합니다. DoWork의 대략적인 모습은 다음과 같습니다.

function DoWork(fileId){ 
    var json = fileRepository.SelectById(fileId); 
    for(var i=0;i<json.length;i++){ 
    var newObj = new MyObj(json[i]); // or calling factory method 
    var dbRecord = MongoDBRepository.SelectByQuery(newObj.name, newObj.date); 
    if(dbRecord){ 
     MongoDBRepository.Update(newObj); 
    } 
    else{ 
     MongoDBRepository.Create(newObj); 
    } 
    } 
} 

단지 자바 스크립트 구문 의사 코드입니다. 실제 코드는 절대적으로 다르게 보일 수 있지만 어떤 디자인을 가져야하는지에 대한 느낌을줍니다. 또한 나는 내부적으로 DoWork 메쏘드 안에 저장소 객체를 생성하지 않았기 때문에 그들을 삽입하여 의존성 반전 원리를 충족시키는 방법을 생각해 보았다.

희망이 있습니다.