Почему не срабатывает тернарный оператор?

Question

[denism300]

Есть конструкция

foreach ($seasonsData as $s) : 
    echo isset($s['priceweek']) ? $s['priceweek'] : '';
endforeach;

которая выдает предупреждение Notice: Undefined index: priceweek in ...
Однако, если ее записать так:

foreach ($seasonsData as $s) : 
    if (isset($s['priceweek'])): 
        echo $s['priceweek'];
    endif;
endforeach;

то все ок.
Почему тернарный оператор не работает?

Comments

[Илья]

Могу предположить, что ваша ошибка выдаётся где-то в другом месте, потому что первый код не должен выдавать ошибки

[denism300]

Илья, в коде только в двух местах вызывается $s['priceweek'], где используется тернарный оператор.
Если я комментирую эти строки или меняю на конструкцию с if, то все ок.

[Ипатьев]

При чем здесь "в двух местах", если в сообщении об ошибке указано конкретное место?

[Ипатьев]

И при чем здесь вообще тернарный оператор? В сообщении об ошибке написано, что отсутствует индекс при обращении к массиву, а не отсутствует индекс при вызове тернарного оператора

[denism300]

Ипатьев, вроде, понятно написал.
еще раз:
1. в коде $s['priceweek'] упоминается дважды, допустим, это строки 5 и 6, оба раза в конструкциях

echo isset($s['priceweek']) ? $s['priceweek'] : '';

2.

При чем здесь "в двух местах", если в сообщении об ошибке указано конкретное место?

выводится два предупреждения для строк 5 и 6
3.

И при чем здесь вообще тернарный оператор? В сообщении об ошибке написано, что отсутствует индекс при обращении к массиву, а не отсутствует индекс при вызове тернарного оператора

наверное, потому, что в тернарном операторе я проверяю наличие индекса, и если он существует, вывожу его значение.
если isset($s['priceweek']) = false, то echo $s['priceweek'] не должно выполнятся, а должно выполнится echo '', соответственно обращения к несуществующему индексу не должно быть, соответственно, не должно быть предупреждения

[Илья]

denism300, Вы приведите полный код, который вызывает ошибку, тогда всё сразу станет ясно. Вот прям отдельный php файл сделайте у себя, напишите вручную ваш массив $seasonsData, запустите. Если будет ошибка, скидывайте сюда ПОЛНЫЙ код этого файла. А по вашим кускам нифига не понятно

[Ипатьев]

это не могут быть строки 5 и 6.
Если в 6 строке ошибка, то в пятой будет foreach.

Приведите уже здесь нормальный код и полное сообщение об ошибке. Точно указав в коде строку, упомянутую в ошибке.
А то вы со всеми этими "допустим" и сами уже запутались, и нам не даете возможности указать вам на очевидную опечатку.

В том же seasonData не может лежать

Array
(
    [start] => 15.06.2023
    [end] => 30.06.2023
    [price] => 7654
    [priceweek] => 
    [price3] => 
    [price7] => 456
    [price14] => 
    [price30] => 345
)

потому что в нем по определению нет вложенного массива с индексом priceweek

[denism300]

Ипатьев, весь код данного блока.
переменная $currency - символ валюты
переменная $editing - id редактируемой записи

<div class="publication-edit_fields-block season-prices_block">
    <div class="publication-edit_fields-head">
        <div class="publication-edit_fields-title">Сезонные цены</div>
    </div>
    <div class="publication-edit_fields-content">
        <div class="publication-edit_fields-table1">
            <div class="publication-edit_fields-table1_head">
                <div class="publication-edit_fields-table1_row">
                    <div class="publication-edit_fields-table1_item">Временной период</div>
                    <div class="publication-edit_fields-table1_item">Сутки</div>
                    <div class="publication-edit_fields-table1_item">Выходные</div>
                    <div class="publication-edit_fields-table1_item">3-6 дней</div>
                    <div class="publication-edit_fields-table1_item">7-14 дней</div>
                    <div class="publication-edit_fields-table1_item">15-25 дней</div>
                    <div class="publication-edit_fields-table1_item">от 26 дней</div>
                    <div class="publication-edit_fields-table1_item"></div>
                </div>
            </div>
            <div class="publication-edit_fields-table1_body">
                <?php $seasonsData = get_post_meta($editing, 'seasonsData', 1); ?>
                <?php if ($seasonsData) : ?>
                    <?php foreach ($seasonsData as $s) : ?>
                        <?php echo '<pre>' . print_r($s, 1) . '</pre>'; ?>
                        <div class="publication-edit_fields-table1_row">
                            <div class="publication-edit_fields-table1_item">
                                <span>с <?php echo isset($s['start']) ? $s['start'] : ''; ?> по <?php echo isset($s['end']) ? $s['end'] : ''; ?></span>
                                <input type="hidden" name="season-price-field_date-first" class="js-season-start season-price-field_date-first" value="<?php echo isset($s['start']) ? $s['start'] : ''; ?>">
                                <input type="hidden" name="season-price-field_date-last" class="js-season-end season-price-field_date-last" value="<?php echo isset($s['end']) ? $s['end'] : ''; ?>">
                            </div>
                            <div class="publication-edit_fields-table1_item">
                                <span class="season-price-field_currency js-price-currency"><?php echo $currency; ?></span>
                                <span class="season-price-field_value"><?php echo isset($s['price']) ? $s['price'] : ''; ?></span>
                                <input type="hidden" name="season-price-field_per-day" class="js-season-price season-price-field_per-day" value="<?php echo isset($s['price']) ? $s['price'] : ''; ?>">
                            </div>
                            <div class="publication-edit_fields-table1_item">
                                <span class="season-price-field_currency js-price-currency"><?php echo $currency; ?></span>
                                <span class="season-price-field_value"><?php echo isset($s['priceweek']) ? $s['priceweek'] : ''; ?></span>
                                <input type="hidden" name="season-price-field_per-weekend" class="js-season-price-weekend season-price-field_per-weekend" value="<?php echo (isset($s['priceweek']) ? $s['priceweek'] : ''); ?>">
                            </div>
                            <div class="publication-edit_fields-table1_item">
                                <span class="season-price-field_currency js-price-currency"><?php echo $currency; ?></span>
                                <span class="season-price-field_value"><?php echo isset($s['price3']) ? $s['price3'] : ''; ?></span>
                                <input type="hidden" name="season-price-field_per-3-6-day" class="js-season-price3 season-price-field_per-3-6-day" value="<?php echo isset($s['price3']) ? $s['price3'] : ''; ?>">
                            </div>
                            <div class="publication-edit_fields-table1_item">
                                <span class="season-price-field_currency js-price-currency"><?php echo $currency; ?></span>
                                <span class="season-price-field_value"><?php echo isset($s['price7']) ? $s['price7'] : ''; ?></span>
                                <input type="hidden" name="season-price-field_per-7-14-day" class="js-season-price7 season-price-field_per-7-14-day" value="<?php echo isset($s['price7']) ? $s['price7'] : ''; ?>">
                            </div>
                            <div class="publication-edit_fields-table1_item">
                                <span class="season-price-field_currency js-price-currency"><?php echo $currency; ?></span>
                                <span class="season-price-field_value"><?php echo isset($s['price14']) ? $s['price14'] : ''; ?></span>
                                <input type="hidden" name="season-price-field_per-15-25-day" class="js-season-price14 season-price-field_per-15-25-day" value="<?php echo isset($s['price14']) ? $s['price14'] : ''; ?>">
                            </div>
                            <div class="publication-edit_fields-table1_item">
                                <span class="season-price-field_currency js-price-currency"><?php echo $currency; ?></span>
                                <span class="season-price-field_value"><?php echo isset($s['price30']) ? $s['price30'] : ''; ?></span>
                                <input type="hidden" name="season-price-field_per-26-day" class="js-season-price30 season-price-field_per-26-day" value="<?php echo isset($s['price30']) ? $s['price30'] : ''; ?>">
                            </div>
                            <div class="publication-edit_fields-table1_item">
                                <button type="button" class="publication-edit_fields-table1_edit-btn" data-popup-src="season-price_popup-stage">
                                    <svg width="18" height="18" viewBox="0 0 18 18">
                                        <path d="M12.75 2.25023C12.947 2.05324 13.1808 1.89699 13.4382 1.79038C13.6956 1.68378 13.9714 1.62891 14.25 1.62891C14.5286 1.62891 14.8044 1.68378 15.0618 1.79038C15.3192 1.89699 15.553 2.05324 15.75 2.25023C15.947 2.44721 16.1032 2.68106 16.2098 2.93843C16.3165 3.1958 16.3713 3.47165 16.3713 3.75023C16.3713 4.0288 16.3165 4.30465 16.2098 4.56202C16.1032 4.81939 15.947 5.05324 15.75 5.25023L5.625 15.3752L1.5 16.5002L2.625 12.3752L12.75 2.25023Z" fill="#BDC8D3" />
                                    </svg>
                                </button>
                            </div>
                        </div>
                    <?php endforeach; ?>
                <?php endif; ?>
            </div>
        </div>
        <a href="#" class="publication-edit_season-prices_create-btn btn6" data-popup-src="season-price_popup-stage">
            <svg width="20" height="20" viewBox="0 0 20 20" fill="none">
                <path d="M10.0013 18.3346C14.6037 18.3346 18.3346 14.6037 18.3346 10.0013C18.3346 5.39893 14.6037 1.66797 10.0013 1.66797C5.39893 1.66797 1.66797 5.39893 1.66797 10.0013C1.66797 14.6037 5.39893 18.3346 10.0013 18.3346Z" stroke="#384654" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" />
                <path d="M10 6.66797V13.3346" stroke="#384654" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" />
                <path d="M6.66797 10H13.3346" stroke="#384654" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" />
            </svg>
            <span>Создать сезонную цену</span>
        </a>
    </div>
</div>

В том же seasonData не может лежать потому что в нем по определению нет вложенного массива с индексом priceweek

согласен, тупанул. это лежит в $s,
seasonData - массив из таких $s

[Ипатьев]

И теперь сообщение об ошибке полностью, с номером строки

[Ипатьев]

И научитесь уже писать нормально,
<?= $s['start'] ?? : '' ?>
вместо

<?php echo isset($s['start']) ? $s['start'] : ''; ?>

[denism300]

Ипатьев,

Notice:  Undefined index: priceweek in /(тут путь к файлу)/edit_listing.php on line 37
Notice:  Undefined index: priceweek in /(тут путь к файлу)/edit_listing.php on line 38
Notice:  Undefined index: price3 in /(тут путь к файлу)/edit_listing.php on line 42
Notice:  Undefined index: price3 in /(тут путь к файлу)/edit_listing.php on line 43
Notice:  Undefined index: price14 in /(тут путь к файлу)/edit_listing.php on line 52
Notice:  Undefined index: price14 in /(тут путь к файлу)/edit_listing.php on line 53

т.е., предупреждения на строки, где значения массива $s пустые

[Ипатьев]

Пустое значение в принципе не вызывает эту ошибку. Она появляется, только если ключа вообще нет в массиве.

[denism300]

Ипатьев, я в курсе.
Содержимое массива $s я привел, так же, как писал в вопросе, если заменить тернарный оператор на обычный if else, то все ок.

[Ипатьев]

Значит вы пишете здесь не тот код, который выводит ошибку.
Вот я взял ВАШ же массив, и он не выводит никаких ошибок, ни с isset, ни без https://3v4l.org/jb158

[denism300]

Ипатьев, в общем, это была не ошибка.
На каком-то этапе, видимо, когда не было проверки isset, данное предупреждение было выведено в value hidden полей и сохранено в базу.
А дальше благополучно оттуда бралось и выводилось на экран.

Answer 1

[Ипатьев]

Есть конструкция которая выдает предупреждение Notice: Undefined index: priceweek in ...

Это неправда.

Comments

[Илья]

В теории, это может быть правдой, мы не знаем что там у автора лежит в $seasonsData.

Но это в порядке бреда)
https://3v4l.org/CDed5

[Илья]

но опять же, это не объясняет то, что во втором случае ошибки нет) понятно что, скорее всего автор чего-то не договаривает)

[denism300]

Илья, в seasonData лежит, к примеру, это:

Array
(
    [start] => 15.06.2023
    [end] => 30.06.2023
    [price] => 7654
    [priceweek] => 
    [price3] => 
    [price7] => 456
    [price14] => 
    [price30] => 345
)