1

В MySQL есть таблица main:

id | order_id | sub1_arr | sub2_arr | sub3_arr | sub4_arr
 1 |        1 |  3, 6, 7 |          |       70 |         
 2 |        0 |          |          |          |         
 3 |        2 |       33 |25, 83, 9 |          |   39, 10  

Мне нужно вывести значения всех строк из столбцов sub1_arr, sub2_arr, sub3_arr, sub4_arr через запятую.

Пробую вот таким образом:

  $result=mysql_query("SELECT `sub1_arr`,`sub2_arr`,`sub3_arr`,`sub4_arr` FROM `main`");
  while ($arr=mysql_fetch_row($result, MYSQL_ASSOC)) {
    if ($arr['sub1_arr']) {echo $arr['sub1_arr'].','};
    if ($arr['sub2_arr']) {echo $arr['sub2_arr'].','};
    if ($arr['sub3_arr']) {echo $arr['sub3_arr'].','};
    if ($arr['sub4_arr']) {echo $arr['sub4_arr'].','};
  }

То более-менее всё работает как надо, но в конце всегда лишняя запятая. ¯_(ツ)_/¯

Но мне нужно просто вернуть (return) эти значения, а не отображать их (echo). И вот когда возвращаю (return) -- выводятся только записи из первой строки.

Вопрос: как оптимизировать этот запрос и выдачу результата, чтобы получить на выходе: 3, 6, 7, 70, 33, 25, 83, 9, 39, 10

4
  • Ну так последняя возвращается, все верно. Смотря что вам надо дальше делать.
    – user242433
    5 авг 2017 в 7:50
  • @Cactus нужно получить на выходе: 3, 6, 7, 70, 33, 25, 83, 9, 39, 10
    – Di-
    5 авг 2017 в 7:56
  • 2
    сделайте массив и по мере чтения таблицы добавляйте в него значения, потом этот массив возвращайте или собирайте в строку с помощью implode и возвращайте строку, если вам нужна действительно строка
    – Mike
    5 авг 2017 в 8:03
  • @Mike нужна действительно строка. :-)
    – Di-
    5 авг 2017 в 8:09

1 ответ 1

1

А вы не возвращайте сразу, а накапливайте в строке:

$str = "";
$result=mysql_query("SELECT `sub1_arr`,`sub2_arr`,`sub3_arr`,`sub4_arr` FROM `main`");
while ($arr=mysql_fetch_row($result, MYSQL_ASSOC)) {
    if ($arr['sub1_arr']) {
        $str .= $arr['sub1_arr'].',';
    }
    if ($arr['sub2_arr']) {
        $str .=  $arr['sub2_arr'].',';
    }
    if ($arr['sub3_arr']) {
        $str .= $arr['sub3_arr'].',';
    }
    if ($arr['sub4_arr']) {
        $str .= $arr['sub4_arr'].',';
    }
}
return $str;

Проблемы такого подхода вы заметили сами -- лишняя запятая в конце. Но есть типовой обходной путь для этого:

$str = "";
$is_first = true;
$result=mysql_query("SELECT `sub1_arr`,`sub2_arr`,`sub3_arr`,`sub4_arr` FROM `main`");
while ($arr=mysql_fetch_row($result, MYSQL_ASSOC)) {
    if($is_first)
    {
        $is_first = false;
    }
    else
    {
        $str .= ",";
    }
    if ($arr['sub1_arr']) {
        $str .= $arr['sub1_arr'];
    }
    if ($arr['sub2_arr']) {
        $str .=  $arr['sub2_arr'];
    }
    if ($arr['sub3_arr']) {
        $str .= $arr['sub3_arr'];
    }
    if ($arr['sub4_arr']) {
        $str .= $arr['sub4_arr'];
    }
}
return $str;

Я надеюсь, этот пример вам понятен, потому что правильнее делать накопление в массиве, а не в строке -- а итоговую строку уже собирать через implode (посмотрите комментарий @Mike).

$res = array();
$result=mysql_query("SELECT `sub1_arr`,`sub2_arr`,`sub3_arr`,`sub4_arr` FROM `main`");
while ($arr=mysql_fetch_row($result, MYSQL_ASSOC)) {
    if ($arr['sub1_arr']) {
        $res[] = $arr['sub1_arr'];
    }
    if ($arr['sub2_arr']) {
        $res[] = $arr['sub2_arr'];
    }
    if ($arr['sub3_arr']) {
        $res[] = $arr['sub3_arr'];
    }
    if ($arr['sub4_arr']) {
        $res[] = $arr['sub4_arr'];
    }
}
return implode(", ", $res);

Удобство implode заключается в том, что эта функция не поставит последнюю запятую: ваш вывод будет 3, 6, 7, 70, 33, 25, 83, 9, 39, 10, -- а implode вернёт 3, 6, 7, 70, 33, 25, 83, 9, 39, 10.

Это во-первых. А во-вторых, ещё в школе учат на уроках информатики, что алгоритм должен разделять шаги ввода информации, обработки и вывода. Так вот обработку лучше делать в массиве (данные структурированы), а не в строке, а вывод уже должен быть строкой. Возможно, на таком простом примере как ваш сложно почувствовать эту выгоду, но привыкнув мыслить чётко это потом будет выручать на сложных задачах.

Дополнительно. Склеить в одну строку можно ещё на SQL-сервере. Кроме того, непонятен смысл параметров arr1 - arr4 в таблице mysql, подозреваю, что можно было бы и нормализовать таблицу дальше.

2
  • Здорово. Спасибо за такой развёрнутый ответ. Теперь всё стало ясно и понятно. :-)
    – Di-
    5 авг 2017 в 10:10
  • 1
    Я проглядел одну ошибку, доберусь до компа - поправлю пока никто не заметил.
    – A K
    5 авг 2017 в 12:53

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.