Стоит ли выносить значения в переменные или подставлять их на пряму?

Question

[andr2017]

2 варианта, в одном значения вынесены в переменные, в другом подставляются на прямую, как правильнее делать, в каких случаях вы подставляете на прямую, в каких выносить отдельно?

//данные вынесенны в переменные
    public function actionGet_places($term) {
        if(!Yii::app()->request->isAjaxRequest)
            throw new CException('Попытка прямого доступа');

        $term = CHtml::encode($term);

        $sellersId = Yii::app()->user->getSellerId( ! Yii::app()->user->isSeller( Yii::app()->user->id) );

        $places = Yii::app()->db->createCommand()
                ->selectDistinct('place')
                ->from('trips')
                ->where('place LIKE :place AND sellers_id = :sellers_id',
                        [':place' => '%'.$term.'%', ':sellers_id' => $sellersId ])
                ->queryColumn();

        echo CJSON::encode($places);
    }

 	//данные подставляются на прямую
    public function actionGet_places($term) {
        if(!Yii::app()->request->isAjaxRequest)
            throw new CException('Попытка прямого доступа');

        $places = Yii::app()->db->createCommand()
                ->selectDistinct('place')
                ->from('trips')
                ->where('place LIKE :place AND sellers_id = :sellers_id',
                        [':place' => '%'.CHtml::encode( $term ).'%', ':sellers_id' => Yii::app()->user->getSellerId( ! Yii::app()->user->isSeller( Yii::app()->user->id) )])
                ->queryColumn();

        echo CJSON::encode($places);
    }

Comments

[Сергей delphinpro]

Пишите так, чтобы читать было удобно.

[SharuPoNemnogu]

если в присвоении тернарник, или вычисления какие то, или переменная будет использоваться дальше по коду несколько раз, то стоит заводить. Иначе, это захломление памяти лишним мусором

Answer 1

[Yustas Alexu]

Лично я создаю переменную, когда начинаю писать одно и то же по несколько раз. но вообще о таких вещах я уже давно не задумываюсь, все делается на автомате.