Тернарный оператор с несколькими условиями, используемыми в массиве в PHP - PullRequest
Новая
       40

Тернарный оператор с несколькими условиями, используемыми в массиве в PHP

0 голосов
/ 26 сентября 2019

Если API дает следующий ответ, учитывая, что массив [0] и массив [1] иногда могут быть значением MX, а иногда могут быть значением AR: 

["published_sites"]=>
array(3) {
  [0]=>
  object(stdClass)#1565 (5) {
    ["site"]=>
    string(2) "MX"
    ["sale_price"]=>
    int(956)
    ["status"]=>
    string(6) "active"
    ["item_id"]=>
    string(12) "MLM724808"
    ["url"]=>
    string(108) "http://articulo.mercadolibre.com.mx/MLM-722808-apple-apple-watch-series-4-gps-cellular-44mm-space-bla-_JM"
  }
  [1]=>
  object(stdClass)#1569 (3) {
    ["site"]=>
    string(2) "AR"
        ["sale_price"]=>
        int(956)
        ["status"]=>
        string(6) "active"
        ["item_id"]=>
        string(12) "MLM724808"
        ["url"]=>
        string(108) "http://articulo.mercadolibre.com.ar/MLM-722808-apple-apple-watch-series-4-gps-cellular-44mm-space-bla-_JM"
}

и для сохранения в базе данных высоздайте массив со следующими полями для назначения значений для AR и для MX: 

    Channels::updateOrCreate(['mpid' => $response->mpid], [
    'mpid' => $response->mpid,
    'published_site_ar_item_id' => $assign item_id for AR, eg: $item_ar_assign
    'published_site_mx_item_id' => $assign item_id for MX,

Учитывая, что массив может использовать только троичный оператор для условий, которые я пробовал со следующей структурой, но использование Or для нескольких условий неработай.-> 

item_ar_assign = ($response->published_sites['0']->site == 'AR' ?: $response->published_sites['0']->item_id)
                ||($response->published_sites['1']->site == 'AR' ?: $response->published_sites['1']->item_id)
            ||($response->published_sites['2']->site == 'AR' ? : $response->published_sites['2']->item_id)

возможен ли этот подход троичного оператора + множественные условия с массивами?

(примечание: я прочитал SO для дубликатов, таких как Тернарный оператор с несколькими условиями , но я не думаю, что это тот же подход.)

1 Ответ

3 голосов
/ 26 сентября 2019

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

$site0 = $response->published_sites['0']->site;
$id0 = $response->published_sites['0']->item_id;

$site1 = $response->published_sites['1']->site;
$id1 = $response->published_sites['1']->item_id;

$site2 = $response->published_sites['2']->site;
$id2 = $response->published_sites['2']->item_id;

$item_ar_assign = 
    ($site0 == 'AR' ?: $id0)
    || ($site1 == 'AR' ?: $id1)
    || ($site2 == 'AR' ? : $id2);

Причина, по которой это не сработает, не имеет ничего общего с массивами, а потому что вы использовали неправильные операторы:

  • Оператор PHP || всегда возвращает логическое значение - true или false, независимо от того, на какие значения он смотрит.
  • Форма $a ?: $b означает "$a, если онаили $b если нет ".В этом случае $site0 == 'AR' ?: $id0 будет означать «true, если $site0 равно 'AR', или значение $id0, если нет»

То, что я пытаюсь сделать, этоВложите полную форму троичного оператора: $test ? $valueIfTrue : $valueIfFalse.

Обратите внимание, что ассоциативность немного странная, поэтому вам нужно аккуратно использовать скобки.Я думаю, что это должно сработать: 

$item_ar_assign =
    ($site0 == 'AR') ? $id0
    : (
        $site1 == 'AR' ? $id1
        : (
            $site2 == 'AR' ? : $id2
        )
    );

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

if ($response->published_sites[0]->site == 'AR' ) {
    $item_ar_assign = $response->published_sites[0]->item_id;
}
elseif ($response->published_sites[1]->site == 'AR' ) {
    $item_ar_assign = $response->published_sites[1]->item_id;
}
elseif ($response->published_sites[2]->site == 'AR' ) {
    $item_ar_assign = $response->published_sites[2]->item_id;
}

Однако вместоскопировав и вставив условие, вы можете поместить его в цикл: 

foreach ( [0, 1, 2] as $site_number ) {
    if ($response->published_sites[$site_number]->site == 'AR' ) {
        $item_ar_assign = $response->published_sites[$site_number]->item_id;
        break; // stop at first match
    }
}

или даже проще: 

foreach ( $response->published_sites as $published_site ) {
    if ($published_site->site == 'AR' ) {
        $item_ar_assign = $published_site->item_id;
        break; // stop at first match
    }
}
...