Как объединить в один код 2 строчки $array[$key]?

Question

[TatianaKrd]

На разных страницах сайта был одинаковый код, отличие только эти 2 строчки. Необходимо объединить в один код на одной странице.
Пожалуйста, подскажите, как объединить эти две строчки в одну?

$array[$key]['ColorGroup'] = join(",", $row['ColorGroup']);

$array[$key] = array_merge(array("Nom1" => 0, "Abs" => 0, "Nom3" => 0, "MarkFirst" => 0, "MarkSecond" => 0, "MarkThird" => 0, "RingWcf" => 0, "RingWcf2" => 0, "RingMono" => 0), $row);

Comments

[alexalexes]

А зачем? У них разное назначение.
Разберитесь по смыслу кода, какой результат хотите получить, те строчки и используйте.
Если надо одну, то одну, если другую, то другую. Если обе операции нужны, то разберитесь с последовательностью выполнения.

[Adamos]

Превратить эти строчки в колбэк и передавать его в функцию, если это действительно оправдано.

[TatianaKrd]

У меня сейчас 2 страницы с формами, на которых я выбираю галками участие в рингах.
Не удобно постоянно переключаться между страницами при большом потоке заявок, есть необходимость, чтобы на одной странице выбрать каждому участнику все ринги...
Я не сильна в PHP, поэтому решила попросить здесь помощи, если это не сложно...
Если это проблематично, тогда ладно, буду продолжать использовать по старому...

[TatianaKrd]

Может быть я не правильна сформулировала, если так, извините...
Есть одна таблица (CMS NetCat), каждая строка - один участник и у него в "карточке" куча галок: в первом коде множественный выбор из одного списка ColorGroup (1 поле в таблице), во втором коде отдельные галки (Nom1, Nom3 и т.д. - разные поля в таблице).
Сейчас у меня 2 страницы:
1. Перечень всех участников и в строке все галки одного типа:

if($_POST['query_form'] == 1 AND $_POST['var'] AND is_object($perm) AND $perm->isSupervisor()){
  foreach($_POST['var'] as $key => $row){
    $array[$key] = array_merge(array("Nom1" => 0, "Abs" => 0, "Nom3" => 0, "MarkFirst" => 0, "MarkSecond" => 0, "MarkThird" => 0, "RingWcf" => 0, "RingWcf2" => 0, "RingMono" => 0), $row);
    foreach($array[$key] as $k => $v){
      $where[$key][] = "`".$k."` = '".$v."'";
    }
    $nc_core->db->query("UPDATE `Message".$classID."` SET ".join(", ", $where[$key])." WHERE `Message_ID` = ".$key."");
  }
}

2. Перечень всех участников и в строке множественный выбор из одного списка ColorGroup

if($_POST['query_form'] == 1 AND $_POST['var'] AND is_object($perm) AND $perm->isSupervisor()){
  foreach($_POST['var'] as $key => $row){
    $array[$key]['ColorGroup'] = join(",", $row['ColorGroup']);
    foreach($array[$key] as $k => $v){
      $where[$key][] = "`".$k."` = '".$v."'";
    }
    $nc_core->db->query("UPDATE `Message".$classID."` SET ".join(", ", $where[$key])." WHERE `Message_ID` = ".$key."");
  }
}

Просто хотелось бы на одной странице работать сразу со всеми галками, а не переключаться...
Наверное не все так просто, как хотелось бы...

[Adamos]

Если вы хотите затянуть обе эти формы на одну страницу, вам нужно обе и обрабатывать в коде.
От повторения десятка строк еще никто не умирал.
Скорее, сунувшись в код с такими знаниями, вы наломаете дров по дороге.

Answer 1

[Ипатьев]

Вам надо не "объединить эти строчки в одну," а добавить условие

if (первая форма) {
    $array[$key]['ColorGroup'] = join(",", $row['ColorGroup']);
} else {
    $array[$key] = array_merge(array("Nom1" => 0, "Abs" => 0, "Nom3" => 0, "MarkFirst" => 0, "MarkSecond" => 0, "MarkThird" => 0, "RingWcf" => 0, "RingWcf2" => 0, "RingMono" => 0), $row);
}

Какое условие сделать - мы не знаем, это вам решать. Например, в форме есть какое-то поле, которого нет в другом.

И это будет работать только если отличие действительно только в этой строчке. В чём я сильно сомневаюсь

Answer 2

[Agrarian]

$array[$key] = array_merge(
    array(
        "Nom1" => 0, "Abs" => 0, "Nom3" => 0,
        "MarkFirst" => 0, "MarkSecond" => 0, "MarkThird" => 0,
        "RingWcf" => 0, "RingWcf2" => 0, "RingMono" => 0
    ),
    $row,
    array(
        "ColorGroup" => is_array($row['ColorGroup']) ? join(",", $row['ColorGroup']) : $row['ColorGroup']
    )
);

Если у вас современная версия php то

$array[$key] = array_merge(
    [
        "Nom1" => 0,
        "Abs" => 0,
        "Nom3" => 0,
        "MarkFirst" => 0,
        "MarkSecond" => 0,
        "MarkThird" => 0,
        "RingWcf" => 0,
        "RingWcf2" => 0,
        "RingMono" => 0,
    ],
    $row,
    [
        "ColorGroup" => is_array($row['ColorGroup'] ?? null)
            ? implode(",", $row['ColorGroup'])
            : ($row['ColorGroup'] ?? "")
    ]
);