Помогите найти ошибку shop

Помогите найти ошибку. Оно пишет “магазин открыть можно только в выживание” в двух режимах в креативе и в виживание помогите исправить ошибку.

В 80 строке убери ту хрень и напиши:

if($p->isCreative()){
return $p->sendMessage("Нельзя открыть в креативе");
}

зачем там ретурн)

Чтобы код не выполнялся дальше

а как же else?)

Сори, я ненавижу if else

Да и какой смысл использовать тут иф елсе?

тогда $p->isCreative() ? $p->sendMessage(‘Нельзя открыть’) : $this->openChest($p, $nick);

Уверен, что он не знает, что такое тернартный оператор. Пусть хотя бы так учится

да и насколько я понял по функций звука это код для генки лайта

Это хорошо конечно…но громоздские конструкции делать, это не есть хорошо.
Проще вернуть

Все конечно хорошо, но тернарный я бы не советовал использовать.
Да, он делает свою работу. Но для дальнешего поддержания кода очень плох, да и читабельность падает. Если везде его использовать.

Тернарный меня также раздражает, как и лямбда в Пайтоне.
Хорошо конечно что она есть, но вот эти любители все в одну строку забывают, что их код может быть поддержан другим человеком. Или просто даже измениться, и потом надо переписывать все, что связано с этим. (Логику работы)

хз хз мне проще разглядеть где нужная скобка у меня код лесенкой