Igor Deyashkin

• Глобальные переменные — зло
  
• На лицо некий код, который и с сессией работает и с БД — явное нарушение общепринятных принципов, например SRP
  
• $dao как инициализирована? Явно не через конструктор
  
• Использование цифры, вот сиди гадай по ней — это плохо, используйте именованную константу. Например 3 — это некий статус для desktop, вот и назовите ее STATUS_DESKTOP, хотя далее...
  
• вы обращаетесь к одному методу, который шибко умный — работает с флагами, тем самым нарушая SRP и сам содержит if/else, сделайте просто 2 раздельных метода:
  

  getMobileOpinionList()
  getDefaultOpinionList()

  
  
• Предикат лучше вынести в отдельный метод isMobileOnly(), тк условия могут поменяться в нем, не будете же везде в if менять по всему проекту
  

Если быть более, конкретным, зачем нужна эта опция, при генерации модели через Gii?

class TrainingQuery extends \yii\db\ActiveQuery
{
    public function actual($date)
    {
        return
            $this->alias('t')
            ->joinWith(['lesson l' => function($q){
                $q->joinWith('studio s');
            }])
            ->andWhere(['l.active' => 1])
            ->andWhere(['s.active' => 1])
            ->andWhere(['t.date' => $date])
            ->all();
    }
}

class Training extends \yii\db\ActiveRecord
{
...
public static function find()
    {
        return new \mynamespace\entities\query\TrainingQuery(get_called_class());
    }

User::find()->active()->haveSale()->bornToday()->all();

[mysqld]
init_connect='SET collation_connection = utf8mb4_unicode_ci'
character-set-server = utf8mb4
collation-server = utf8mb4_unicode_ci

[client]
default-character-set = utf8mb4

+--------------------------+--------------------+
| Variable_name                  | Value                       |
+------------------------------+------------------------+
| character_set_client          | utf8mb4                  |
| character_set_connection | utf8mb4                  |
| character_set_database    | utf8mb4                  |
| character_set_filesystem  | binary                      |
| character_set_results       | utf8mb4                   |
| character_set_server        | utf8mb4                   |
| character_set_system       | utf8                          |
| collation_connection        | utf8mb4_general_ci |
| collation_database          | utf8mb4_unicode_ci |
| collation_server               | utf8mb4_unicode_ci |
+-----------------------------+------------------------+
10 rows in set (0.01 sec)

$posnamevalend = mb_strpos($findstr, "\"", $posnamevalstart + 1);

$namevalue = mb_substr($findstr, ($posnamevalstart + 1), ($posnamevalend - $posnamevalstart - 1));

• Что лучше: запущенный проект с несжатыми стилями или незапущенный со сжатыми?
  
• Что лучше: не запущенный проект с десятью страницами или запущенный с тремя?
  
• Что лучше: запущенный проект c jQuery или не запущенный без jQuery?
  

function random(digits) {
	if(digits > 10) return false;
	var dg = "";
  var k = Math.floor(Math.random()*9)+1; // 0 вначале не модет идти
  dg += k;
  for(var i=0;i-1) {
  		var k = Math.floor(Math.random()*10);
    }
    dg += k;
  }
  return parseInt(dg);
}

• Поставить в переменную цифру от 1 до 9 (0 не может быть первым)
  
• ПАРАМЕТР-1 раз повторить:
  
  • Сгенерировать случайную цифру от 0 до 9
    
  • Если цифра уже есть, вернуться к предыдущему шагу
    
  • Дозаписать цифру в переменную