Насколько критично дублирование кода в несколько строк?

Question

[thorii]

Допустим у меня есть кусок кода (смысловое дублирование, то есть по смыслу реализуют одно и то же)

$query = (new QueryChain())->select('*')
    ->from($this->store())
    ->where([$pk => ":$pk"])
    ->limit(1)
    ->build(); //Естественно этот кусок индивидуален для каждого запроса
$statement->bindValue($pk, $key); //И это тоже индивидуально!
$statement = $this->connection->prepare($query);

Или стоит завести отдельный метод вида prepareQuery($query, $params) //@return PDO Statement

UPD. Дублируются строки максимум в 2-3 методах.
Я понимаю что можно потом запутаться, но я уже выработал привычку генерить phpDOC для каждого метода.

Answer 1

[Ilya]

Честно? Я бы не стал объединять это в отдельный метод. Экономия на спичках получается.

Comments

[thorii]

Я тоже так думаю. Но я не опытен и часто ошибаюсь. Спасибо за ответ

[Ilya]

thorii: Рад помочь. Подобные сомнения возникают у всех, это нормально. Просто всегда всё зависит от задачи и реалий. Если со временем ваш код разрастется и вы увидите. что все эти куски кода можно реально объединить в один метод/функцию без каких-то лютых костылей и быстро, то вот тогда и займетесь рефакторингом.

[thorii]

Ilya: Ну, думаю право на ошибку у меня есть, я учусь на локальном проекте которые не уйдет в продакшен, и со временем хочу сделать его как площадку тренировок (То есть накидать кода и рефакторить его). Спасибо

Answer 2

[Михаил]

Как сказал один человек: если код дублируется 2 раза, то это не критично, а если 3 и больше - нужно рефакторить.