Как передать название метода в переменную в Laravel?

Question

[jazzus]

Мне нужно название метода со скобками передать в переменную. И как это прописать я не знаю. Пробовал ‘metod()’ и другие подборы паролей - не выходит.

Статусы в модели

//Опубликован
  public function active()
  {
      return 1;
  }

  //Не опубликован
  public function inactive()
  {
      return 2;
  }

  //Проверка 
  public function isActive()
  {
      if ($this->status_id==$this->active()) return true;
      return false;
  }
  //Обновление статуса
  public function statusUpdate($status)
  {
      $this->update(['status_id'=>$this->$status]);
  }

Мне нужно из контроллеров в statusUpdate($status) в переменную $status передать название метода. Как это сделать? Пробовал по разному в контроллере писать, например

$product->statusUpdate('active()');
не работает

Answer 1

[Алексей Уколов]

Что за ужас вы делаете?

Хорошо, в лоб будет так:

//Обновление статуса
public function statusUpdate($status)
{
    $this->update(['status_id'=>$this->{$status}()]);
}

$product->statusUpdate('active');

Нормально, вот так:

class Foo
{
    public const ACTIVE  = 1;
    public const INACTIVE  = 2;
    
    public function isActive()
    {
        return $this->status_id === static::ACTIVE;
    }

    public function statusUpdate($status)
    {
        $this->update(['status_id' => $status]);
    }
}

(new Foo)->statusUpdate(Foo::ACTIVE);

Comments

[jazzus]

Спасибо) сейчас буду разбирать, но я понял

[jazzus]

В итоге сделал так:

В модели Product

//Статусы
  public function statuses()
  {
     $data=[
       'active' => 1,
       'inactive' => 2
     ];
     return $data;
  }

  //Проверка
  public function isActive()
  {
      $id = $this->statuses()['active'];
      if ($this->status_id==$id) return true;
      return false;
  }

  //Смена статуса
  public function statusUpdate($status)
  {
      $id = $this->statuses()[$status];
      $this->update(['status_id'=>$id]);
  }

Алексей, а в чем преимущество работы с константами? Имхо итоговое
$product->statusUpdate('active');
более читабельно чем
(new Foo)->statusUpdate(Foo::ACTIVE);

[jazzus]

Проверки активного в контроллерах так:

if($product->isActive()){
...
}

[Алексей Уколов]

Может, и читабельнее, зато потом рефакторить проще.

[developer007]

jazzus, не читабельнее.

[vism]

jazzus, Вот сказали как сделать. Сделай по нормальному.
Щас вот такое вот говно разгребаю и на константы переделываю.. блин, вот учат и не учитесь.

[jazzus]

vism, В смысле делать не спрашивая? А если интересно?))

[vism]

Всмысле делать как посоветовали, а не говнокодить по своему, аргументируя, что вам так больше нравится. jazzus,

[jazzus]

vism, Не хотите потом в моем говнокоде возиться?)) На самом деле я очень внимательно отношусь ко всем советам здесь, реально знающие люди учат, как правильно и помогают разобраться