Михаил Соловьев

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

  getMobileOpinionList()
  getDefaultOpinionList()

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

class Fireball: public IMagicSpell {
  public:
    Fireball() {
      SpellInfo.Name = "Огненный шар";
      SpellInfo.Description = "Мощный огненный шар, сметающий все на своем пути.";
      SpellInfo.Type = fire;
      SpellInfo.MinDamage = 30;
      SpellInfo.MaxDamage = 50;
      SpellInfo.Range = 40;
      SpellInfo.Distance = 120;
    }

    virtual ~Fireball() {}

    virtual void Use() {
      cout << "Boom!" << endl;
    }
};

1. private - приватный доступ, только для экземпляра текущего класса;
   
2. protected - ограниченный доступ, только для экземпляров текущего класса и всех публичных наследников; доступ ограничивается при ограниченном или приватном наследовании;
   
3. public - публичный доступ, для всех пользователей экземпляров класса.
   

class IMagicSpell {
  protected:
    MagicSpellInfo SpellInfo;

  public:
    virtual ~IMagicSpell() {}
    virtual void Use() {}

    MagicSpellInfo GetInfo() {
      return this->SpellInfo;
    }

    void ShowDebugInfo() {
      cout << "Название: " << SpellInfo.Name << endl;
      cout << "Описание: " << SpellInfo.Description << endl;
      cout << "Тип: " << SpellInfo.Type << endl;
      cout << "Мин. урон: " << SpellInfo.MinDamage << endl;
      cout << "Макс. урон: " << SpellInfo.MaxDamage << endl;
      cout << "Расстояние: " << SpellInfo.Distance << endl;
      cout << "Радиус: " << SpellInfo.Range << endl;
    }
};

$apiUrl = "https://api.shiptor.ru/public/v1";
$apiKey = "YOUR_KEY";
$client = new \JsonRPC\Client($apiUrl);

// Set headers
$client->getHttpClient()->withBeforeRequestCallback(function(HttpClient $client, $payload) {
    $client->withHeaders([
        "Content-Type: application/json",
        "x-authorization-token: {$apiKey}"
    ]);
});

// Fetch something
$method = "calculateShipping";
$params = ['paramName'=>"paramValue"];
$result = $client->execute($method, $params);