цикломатическая сложность метода пружинного контроллера: ** передовая практика ** - PullRequest
0 голосов
/ 28 мая 2019

У меня есть контроллер spring-boot, который получает данные со service и возвращает страницу thymeleaf html.Все работает нормально.

Проблема здесь в том, что мне нужно создать список и карту, которые будут использоваться thymeleaf, а метод контроллера становится больше.

   @Controller
    public class MyController {

        @Autowired
        MyService myService; 

        @RequestMapping("/my-page")
        public String getMypage(Model model) {

            //get data from service

            //i have alot of code to extract data and prepare different list to be used by thymeleaf

        }
}

Теперь мой вопросо передовой опыт .

Хорошая идея создать метод внутри контроллера, чтобы код был более читабельным, как показано ниже

@Controller
public class MyController {

    @Autowired
    MyService myService; 

    @RequestMapping("/my-page")
    public String getMypage(Model model) {

        //get data from service

        myMethod1();
        //some code here
        //myMethod2();
        // code here

    }

    private void myMethod1(){

        //myMethod 1 code here
    }

    private void myMethod2(){

        //method 2 code here
    }
}

1 Ответ

0 голосов
/ 28 мая 2019

Да, это, безусловно, хорошая идея.

Знаете ли вы книгу «Чистый код» Роберта К. Мартина:

https://www.amazon.de/Clean-Code-Handbook-Software-Craftsmanship/dp/0132350882

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...