Ошибка syntax 'foreach'?

Question

[SHNM]

Здравствуйте! Подскажите пожалуйста, как использовать цикл в switch? Показывает ошибку синтаксис foreach

public static function act_page($data = []) {
  if (isset($_GET['act'])) {
    switch ($_GET['act']) {
      foreach($data as $key => $value) {
        case $key:
          self::$path .= '/act/'.$key;
          self::display_page($value['title']);
          exit;
          break;
      }
      default:
      self::display_error(404);
      break;
    }
  }
  return null;
}

Comments

[Adamos]

Либо внутри case, либо снаружи switch, но уж никак не посредине.

[SHNM]

Adamos, мне нужно создавать case и брать из массива ключи, как это сделать?

[FanatPHP]

а зачем здесь свитч, я стесняюсь спросить?

[SHNM]

FanatPHP, Ты предлагаешь сделать это через if?

[FanatPHP]

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

Answer 1

[FanatPHP]

функция целиком - ад. я бы выкинул всё это оопэ из картона, все эти статики, публики, селфи - и шарашил обычным спагетти. Не будет вызывать настолько сильный когнитивный диссонанс.

Но если уж пытаться спасти этот ужас,

public static function act_page($data, $key) {
      if ($key && isset($data[$key])) {
          self::$path .= '/act/'.$key;
          self::display_page($data[$key]['title']);
      } else {
          self::display_error(404);
    }
}

Comments

[SHNM]

Спасибо тебе большое))

Answer 2

[Радость моя]

У вас структура switch нарушена. case внутри foreach не может быть в данном случае. Вы как будто в текстурах застряли :)

Comments

[SHNM]

Аххахаах, внатуре. Все, я исправил, сделал через if :)