Чистота функций важна по многим причинам, но в первую очередь, потому что она позволяет нам легче рассуждать о поведении функции. Если функция чистая и не имеет побочных эффектов, то мы можем с большей уверенностью рассуждать о производительности этой функции, поскольку нам нужно будет рассматривать меньше переменных.
Не то, что бы предложения.. А вот, например метод:Sam Dark писал(а):Есть конкрентные предложения?
Код: Выделить всё
$post=Post::model()->findByPk($postID,$condition,$params);
Ну дак а почему так задумано? Хорошо, что документация есть, а сходу не понять.Sam Dark писал(а):byPk != byPk only.
Нашел запись по ПК и проверил, есть ли на ней флажок или нет (уже вне БД).Sam Dark писал(а):Ради гибкости и лаконичности. Как бы вы искали, например, отмеченную флажком запись по PK? Да и так как вы понимаете сходу метод тоже работает.
Ну это да, не правильно. А если с условием, то:Sam Dark писал(а):То есть тащили бы данные вместо того, чтобы добавить условие в SQL?
Код: Выделить всё
$post=Post::model()->findByPkAndCondition($postID,$condition,$params);Что бы код был более понятным, будучи новичком в Yii / меньше читать API. =)Sam Dark писал(а):А если условия три? findByPkAndThreeConditions($postID, $condition1, $condition2, $condition3,$params)?
Ради чего именно переименовывать метод?
Код: Выделить всё
findByPkAndConditions($postID, array($condition1, $condition2, $condition3), $params) 