Блог

Я работаю над проектом, в котором есть категории, подкатегории, вложенные подкатегории и т.д. Пользователь может их создавать. Я хочу отобразить их на столе. Я использую эту функцию, но возвращаемый результат дает мне только родительские категории (parent_id = 0). В нем отсутствуют подкатегории и вспомогательный подраздел … Если я помещаю var_dump , это показывает мне, что конкатенация работает, но когда функция находит другого родителя категории, другие данные теряются. Что я делаю не так, пожалуйста, помогите.

Я хочу вернуть переменную, содержащую все категории и подкатегории.

 function categoryTree($catsSubcats = "", $parent_id = 0, $sub_mark = ""){ var_dump($catsSubcats);

        $stmt = Conexion::conectar()->prepare("SELECT * FROM categorias_compra WHERE parent_id = $parent_id");
        $stmt -> execute();
        $respuesta = $stmt -> fetchAll(); 

        if($respuesta){
            foreach($respuesta as $key => $value){
                $catsSubcats = $catsSubcats . $sub_mark.$value['nombre'].'<br>'; 
                categoryTree($catsSubcats, $value['id'], $sub_mark.str_repeat('amp;nbsp;', 5));
            }
            return $catsSubcats;
        }
    }
  

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

Я думаю, вам не нужно передавать основную строку рекурсивному вызову. Это то, что приведет к повторным результатам. Вам нужно добавить только результат рекурсивного вызова, а не этот результат плюс все, что у вас уже было на тот момент.

Кроме того, если вы передаете свой подготовленный оператор рекурсивно, вы должны иметь возможность повторно использовать его вместо повторной подготовки одного и того же.

Я не могу ее протестировать, но я думаю, что это будет работать лучше.

 function categoryTree($parent_id = 0, $sub_mark = "", $stmt = null) {
    if (!$stmt) {
        // only prepare the first time, then pass the prepared statement to subsequent calls
        $stmt = Conexion::conectar()->prepare("SELECT * FROM categorias_compra WHERE parent_id = ?");
    }

    // bind $parent_id to the ? placeholder in the prepared statement
    $stmt->execute([$parent_id]);

    $respuesta = $stmt->fetchAll();

    // initialize the result of each call to an empty string
    $catsSubcats = "";

    if ($respuesta) {

        // you can also move this before the loop so it only has to execute once
        $sub_mark .= str_repeat('amp;nbsp;', 5);

        foreach ($respuesta as $key => $value) {
            $catsSubcats .= $sub_mark.$value['nombre'].'<br>';
            $catsSubcats .= categoryTree($value['id'], $sub_mark, $stmt);
        }
    }

    // always return a string
    return $catsSubcats;
}
  

Вы не используете возвращаемое значение при рекурсии. Вероятно, вы хотите что-то вроде этого:

 function categoryTree($parent_id = 0, $sub_mark = ""){ 
        $catsSubcats='';
        $stmt = Conexion::conectar()->prepare("SELECT * FROM categorias_compra WHERE parent_id = $parent_id");
        $stmt -> execute();
        $respuesta = $stmt -> fetchAll(); 

        if($respuesta){
            foreach($respuesta as $key => $value){
                $catsSubcats .= $sub_mark.$value['nombre'].'<br>'; 
                $catsSubcats .= categoryTree($value['id'], $sub_mark.str_repeat('amp;nbsp;', 5));
            }
        }
        return $catsSubcats;
    }