Два метода или один с условием?

Question

[Посторонним В.]

Постоянно встречаюсь с задачей вытягивать из БД данные того или иного объекта. Причем как правило вытягивать их надо или по ID, или по имени (оба ключа уникальны).
Вопрос - создавать в классе менеджера материалов два метода getById и getByName или создать один get($key), в котором проверять, $key - это int или string, и соответственно менять where clause на id=$key или name=$key?
Так как кроме этого поля оба метода вышли бы идентичными.

Answer 1

[Владимир Болиев]

Я бы не ориентировался на тип, если в будущем понадобится вытаскивать данные по другому полю с таким же типом - ваша логика рухнет, и будет у вас метод get(), который работает только с id и name, но не работает с другими полями.

Как вариант, можно сделать метод get($field_name, $value)

Comments

[Посторонним В.]

Да, вы правы, уже сталкивался с таким. Пожалуй, действительно, лучше всего включить в аргументы имя поля.

[Посторонним В.]

А будет ли правильней для безопасности прописать в MenuManager константы:

const ID = 'id';
const NAME = 'name';

И потом вызывать например так:
MenuManager::getMenu(MenuManager::ID, 1)

[Посторонним В.]

впрочем, да, всё равно можно передать что угодно. Но может всё-таки ввести константы, чтобы ограничить список запрашиваемых полей?

[Владимир Болиев]

Посторонним В.: Можно и так. Я для таких случаев обычно использую массив возможных значений, просто проверяю if(!in_array($field, $this->available_fields)) {throw new Exception...}

[Владимир Болиев]

Посторонним В.: Но это если подставлять поле сразу в sql.

Answer 2

[Mikhail Osher]

getBy($field, $value);

Comments

[Виталий Хоменко]

Скорее наоборот: getBy( $value, $field = 'id' );

Answer 3

[65536]

2