Стоит ли создавать константу для ключа массива?

Question

[Сергей]

Поставил на phpStrom SonarLint, почти везде одно и то-же предупреждение "String literals should not be duplicated"
Первый пример : Есть массив с полем например name, в разных функция я присваиваю ему какое-то значение и lint соответственно выдает предупреждение, что хорошо было бы заменить на константу.
Второй пример: метод контроллера возвращает ['success' => true]. Таких контроллеров десяток, стоит ли везде поменять на коснтанту?

Answer 1

[DevMan]

имхо, не стоит.

Answer 2

[synapse_people]

сделай лучше класс Result { bool success; } и возвращай его инстанс

Comments

[Сергей]

Подключать еще один класс во все контроллеры как-то не очень идея. Наверно проще в абстрактный класс добавить константу success и её брать

[synapse_people]

Сергей, всмысле подключать? Имеете ввиду, что лишняя дисковая операция?
Так это не проблема для opcache же..
А если пихать что-нить в абстрактный класс контроллера... Стоит вообще задуматся, нафига контроллерам абстрактный класс?