отключение элементов (php)

root@localhost:~# · 27 Июн 2022

недавно начал учить php
есть функция которая парсит из базы итемы, я сделал условия которые будут отключать некоторые символы в случае если допустим итем 1 - у него отключается вывод количества, или если итем не точенный у него не выводится заточка
пример

так вот, хотелось бы узнать насколько это костыльно у меня это реализовано, можно ли так пускать на лайв, и как лучше в целом это сделать?
кусок кода:

PHP:

    while ($row = $result->fetch(PDO::FETCH_OBJ)){
        $before = '(';
        $after = ')';
        $plus = ' +';
        if($row->enchant == 0){
            unset ($row->enchant);
            unset ($plus);
        }
        if($row->amount == 1){
            unset ($row->amount);
            unset ($before);
            unset ($after);
        }

            echo '<a><span data-title="' . $row->item_name . ' ' . $before . $row->amount . $after .  $plus . $row->enchant . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></span></a>';
    }

Mex-Vision · 27 Июн 2022

root@localhost:~# написал(а):
недавно начал учить php
есть функция которая парсит из базы итемы, я сделал условия которые будут отключать некоторые символы в случае если допустим итем 1 - у него отключается вывод количества, или если итем не точенный у него не выводится заточка
пример
Посмотреть вложение 44199
Посмотреть вложение 44200
так вот, хотелось бы узнать насколько это костыльно у меня это реализовано, можно ли так пускать на лайв, и как лучше в целом это сделать?
кусок кода:

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){ $before = '('; $after = ')'; $plus = ' +'; if($row->enchant == 0){ unset ($row->enchant); unset ($plus); } if($row->amount == 1){ unset ($row->amount); unset ($before); unset ($after); } echo '<a>item_name . ' ' . $before . $row->amount . $after . $plus . $row->enchant . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></a>'; }

Я бы конечно сделал вот так.
Кода меньше и код гораздо читаемый, как мне кажется.

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){
    $enchantStr = '';
    $amountStr = '';
    
    if($row->enchant > 0)
        $enchantStr = ' +' . $row->enchant;
    
    if($row->amount > 0)
        $amountStr = ' (' . $row->amount . ')';

    echo '<a><span data-title="' . $row->item_name . $amountStr . $enchantStr . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></span></a>';
}

Да и как мне кажется, код более оптимизирован. Меньше временных переменных, меньше канкатинаций.

root@localhost:~# · 27 Июн 2022

Mex-Vision написал(а):
Я бы конечно сделал вот так.
Кода меньше и код гораздо читаемый, как мне кажется.

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){ $enchantStr = ''; $amountStr = ''; if($row->enchant > 0) $enchantStr = ' +' . $row->enchant; if($row->amount > 0) $amountStr = ' (' . $row->amount . ')'; echo '<a>item_name . $amountStr . $enchantStr . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></a>'; }

Да и как мне кажется, код более оптимизирован. Меньше временных переменных, меньше канкатинаций.

хм, тоже неплохо, не додумался почему то

Mex-Vision · 27 Июн 2022

root@localhost:~# написал(а):
хм, тоже неплохо, не додумался почему то

Я просто уже не помню до конца, но если включить варнинги в php, скорее всего будет ругаться на обращение к перменной которая не установлена. А вы как раз обращаетесь к переменным которые удалили функцией unset

Попробуйте включить ошибки:
ini_set('display_errors', 1);
error_reporting(E_ALL);

и отработать свой код, будут ли варнинги.

òbi · 27 Июн 2022

или так

PHP:

$items = $result->fetchAll( PDO::FETCH_OBJ );

foreach ( $items as $item )
{
    $title = $item->item_name;
    
    if ( $item->enchant > 0 )
        $title .= ' +' . $item->enchant;
    
    if ( $item->amount > 0 )
        $title .= ' (' . $item->amount . ')';
    
    echo '<a><span data-title="' . $title . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></span></a>';
}

root@localhost:~# · 27 Июн 2022

òbi написал(а):

или так

PHP:

$items = $result->fetchAll( PDO::FETCH_OBJ );

foreach ( $items as $item )
{
    $title = $item->item_name;
   
    if ( $item->enchant > 0 )
        $title .= ' +' . $item->enchant;
   
    if ( $item->amount > 0 )
        $title .= ' (' . $item->amount . ')';
   
    echo '<a><span data-title="' . $title . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></span></a>';
}

а почему не while? просто из интереса спрашиваю

òbi · 27 Июн 2022

root@localhost:~# написал(а):
а почему не while? просто из интереса спрашиваю

в данном случае не будет иметь значения, как больше нравится

Desquire · 27 Июн 2022

Сильно костьільно из-за чистого пхп).

У тебя есть операторьі ?? ?:
зачем тебе делать ансет перед вьіводом?

Можно сделать несколько вариантов. ( один менее костьільний, второй)
Єто переделаньій вариант Mex-Vision, тоже самое, только для пхп 8 и в 1 строку)

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){
    echo '<a><span data-title="' . $row->item_name . ($row->enchant > 0 ? ' +' . $row->enchant : '') . ($row->amount > 0 ? ' (' . $row->amount . ')' : '') . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></span></a>';
}

Может нужно будет добавлять какие-то еще лебльі или т.п.
То можно сделать вот так

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){
    $data_titles = [];

    $data_titles[] = $row->item_name;
    if($row->enchant > 0)
        $data_titles[] = '+' . $row->enchant;

    if($row->amount > 0)
        $data_titles[] = '(' . $row->amount . ')';

    // Можно добавить еще что-то в $data_titles.
    echo '<a><span data-title="' . implode(' ', $data_titles) .'"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></span></a>';
}

root@localhost:~# · 27 Июн 2022

Desquire написал(а):
Сильно костбьіли.

У тебя есть операторьі ?? ?:
зачем тебе делать ансет перед вьіводом?

Можно сделать несколько вариантов. ( один менее костьільний, второй)
Єто переделаньій вариант Mex-Vision, тоже самое, только для пхп 8 и в 1 строку)

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){ echo '<a>item_name . ($row->enchant > 0 ? ' +' . $row->enchant : '') . ($row->amount > 0 ? ' (' . $row->amount . ')' : '') . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></a>'; }

Может нужно будет добавлять какие-то еще лебльі или т.п.
То можно сделать вот так

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){ $data_titles = []; $data_titles[] = $row->item_name; if($row->enchant > 0) $data_titles[] = '+' . $row->enchant; if($row->amount > 0) $data_titles[] = '(' . $row->amount . ')'; // Можно добавить еще что-то в $data_titles. echo '<a><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></a>'; }

я еще не пришел к тому, что бы делать по уму, я еще даже не весь базовый синтаксис выучил) поэтому и спросил как лучше

Desquire · 27 Июн 2022

root@localhost:~# написал(а):
я еще не пришел к тому, что бы делать по уму, я еще даже не весь базовый синтаксис выучил) поэтому и спросил как лучше

У тебя сразу идет вьівод.
Собери все в 1 и вьіведи в нужном месте еще)

GhgostCobalt · 27 Июн 2022

Desquire написал(а):
Сильно костьільно из-за чистого пхп).

У тебя есть операторьі ?? ?:
зачем тебе делать ансет перед вьіводом?

Можно сделать несколько вариантов. ( один менее костьільний, второй)
Єто переделаньій вариант Mex-Vision, тоже самое, только для пхп 8 и в 1 строку)

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){ echo '<a>item_name . ($row->enchant > 0 ? ' +' . $row->enchant : '') . ($row->amount > 0 ? ' (' . $row->amount . ')' : '') . '"><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></a>'; }

Может нужно будет добавлять какие-то еще лебльі или т.п.
То можно сделать вот так

PHP:

while ($row = $result->fetch(PDO::FETCH_OBJ)){ $data_titles = []; $data_titles[] = $row->item_name; if($row->enchant > 0) $data_titles[] = '+' . $row->enchant; if($row->amount > 0) $data_titles[] = '(' . $row->amount . ')'; // Можно добавить еще что-то в $data_titles. echo '<a><img src="/img/items/' . $row->item_type . '.jpg" width="32" height ="32"></a>'; }

Вы принципиально отключили русский язык на телефоне?)

Desquire · 27 Июн 2022

GhgostCobalt написал(а):
Вы принципиально отключили русский язык на телефоне?)

Да, єтот портал единстенное место, где я общаюсь на єтом язьіке.
Можно каждый раз копировать букву от куда-то, но мне лень.

m1ckey · 27 Июн 2022

Посмотрите в сторону MVC , на таком подходе далеко не уедешь

root@localhost:~# · 27 Июн 2022

m1ckey написал(а):
Посмотрите в сторону MVC , на таком подходе далеко не уедешь

пожалуй когда хотя бы месяца 3 опыта будет

Mex-Vision · 27 Июн 2022

root@localhost:~# написал(а):
пожалуй когда хотя бы месяца 3 опыта будет

Если синтаксис знаешь, то лучше не терять время, и сразу почитать про паттерны проектирования. MVC это некая база для небольшой веб разработки на PHP, которая не должна вызвать сложностей, так как очень проста в понимании.

root@localhost:~# · 27 Июн 2022

Mex-Vision написал(а):
Если синтаксис знаешь, то лучше не терять время, и сразу почитать про паттерны проектирования. MVC это некая база для небольшой веб разработки на PHP, которая не должна вызвать сложностей, так как очень проста в понимании.

с синтаксисом пока проблемы, я начал пилить некое подобие статистики сервера для себя, пилил на mysqli, потом решил перекинуться на pdo, потом понял что лучше делать функциями
по сути я еще даже циклы толком не освоил, из того что я уже понял это работа с массивами, условия, работа с БД, сессии, функции и прочая мелочь

m1ckey · 27 Июн 2022

Вы не можете просматривать ссылку пожалуйста воспользуйтесь следующими ссылками Вход или Регистрация

Mex-Vision · 27 Июн 2022

root@localhost:~# написал(а):
с синтаксисом пока проблемы, я начал пилить некое подобие статистики сервера для себя, пилил на mysqli, потом решил перекинуться на pdo, потом понял что лучше делать функциями
по сути я еще даже циклы толком не освоил, из того что я уже понял это работа с массивами, условия, работа с БД, сессии, функции и прочая мелочь

Тогда лучше всё такие основы выучить, и долго не задерживаться на функциональном программировании, изучайте ООП. Наступит переломный момент, когда вам покажется что перед вами открылся целый мир=) Потом конечно вы сломаете ноги и руки изучая паттерны, и запутаетесь еще сильнее чем вначале пути, но это уже совсем другая история.

root@localhost:~# · 27 Июн 2022

Mex-Vision написал(а):
Тогда лучше всё такие основы выучить, и долго не задерживаться на функциональном программировании, изучайте ООП. Наступит переломный момент, когда вам покажется что перед вами открылся целый мир=) Потом конечно вы сломаете ноги и руки изучая паттерны, и запутаетесь еще сильнее чем вначале пути, но это уже совсем другая история.

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

отключение элементов (php)

Дополнительные настройки

root@localhost:~#

Легендарный

Mex-Vision

Бессмертный

root@localhost:~#

Легендарный

Mex-Vision

Бессмертный

òbi

-Wan Kenobi

root@localhost:~#

Легендарный

òbi

-Wan Kenobi

Desquire

Гений мысли

root@localhost:~#

Легендарный

Desquire

Гений мысли

GhgostCobalt

Пляшущий с бубном

Desquire

Гений мысли

m1ckey

Прославленный

root@localhost:~#

Легендарный

Mex-Vision

Бессмертный

root@localhost:~#

Легендарный

m1ckey

Прославленный

Mex-Vision

Бессмертный

root@localhost:~#

Легендарный