Какой лучший путь для получение список заказов?

Question

[yativ_sobb]

На данный момент я пишу функцию для получение заказов на обе стороны. Стал вопрос использовать один метод или для каждого роли отдельный.

+ Использовать один метод, сократит код, и повторений кода
- Хотя сейчас это функции сильно похожи, то в будущем может сильно изменится

Или сейчас использовать один метод, но когда сильно пути разветвляется сделать два методы на основе первичного метода.

Comments

[yativ_sobb]

Решил использовать две функция GetCustomerOrder and GetExecutorOrder, а то что похоже на 2 функция раскидывал и положил их GetCustomerOrder and GetExecutorOrder, прадо мне до сих пор не нравится

Answer 1

[Yuriy Vorobyov]

Напишите общую функцию и две поменьше, в которых есть различия. Затем из общей вызывайте одну из тех двух.

Comments

[yativ_sobb]

Это интересный вариант, я думал сделать наоборот из меньшой функций вызывать общую

[Yuriy Vorobyov]

yativ_sobb, если бы вы писали на классах, то был бы класс, у которого была единая точка входа, а дальше вызов функций бы отличался для конкретного случая. Классов у нас нет, поэтому можно применить вот такой подход)

[yativ_sobb]

Это тоже интересный путь я пишу на структарах если бы точно

Answer 2

[Виктор П.]

Добрый день,
Процесс, который вы описываете, называется рефакторинг. Советую по данной теме лёгкое чтиво refactoring.guru
Так же, существуют общие советы, так называемые принципы, вот первая ссылка в гугле, поверхностно, но должно быть понятно.
Отвечая на ваш вопрос, дублирования кода не должно быть. Делаете один метод. Потом, если, вдруг, появится разветвление, то в этой фукнции остаётся общий код и появляются две другие, в которых будет разный код

Comments

[yativ_sobb]

Все это читал) и писал на практики на работу. Это типо мой проект, хочется все идеально сделать, для себя пишу)

[Виктор П.]

Тогда странный вопрос. В приведённых ссылках есть все интересующие ответы

[yativ_sobb]

Ага спасибо, особенно refactoring.guru, когда заблудишь основные правило можно открыты и посмотреть или когда перегоришь