Меню навигации PHP/MySQL

Я думаю, что ваша проблема может иметь какое-то отношение к тому факту, что ваша функция является рекурсивной, но строка, которую вы создаете, каждый раз сбрасывается вверху вашей функции, вместо того, чтобы снова передаваться в функцию. Кроме того, я нигде не вижу, чтобы сабвуфер был обнулен для вашей последней итерации. Кроме того, кажется, что вам не нужно запрашивать каждую отдельную строку. Было бы более эффективно сделать запрос один раз и построить все меню. Я думаю, что рекурсия может быть угроблена. Кроме того, я бы рекомендовал хранить в ваших данных флаг «под» вместо использования какой-то сложной для понимания логики php относительно того, является ли данная строка подменю. Я внес изменения на основе этих концепций, понятия не имею, работает это или нет, поскольку у меня нет/не хочу создавать данные для его проверки:

function build_navbar()
{
    global $db;
    //first things first, i'd recommend putting a "sub" flag in your database. This example will use it.

    //start off by getting all of the rows. No need for recursion.
    $query = $db->simple_select("navbar", "*", "1", array("order_by" => "disporder"));
    $menu_build = "<ul id=\"navigation\">\n";

    //keep track of what level we're at
    $level = 1;

    while($menu = $db->fetch_array($query))
    {
      //get sub from data
      $sub = $menu['sub']

      //we need to go back to root level
      if($sub == 0 && $level == 2){
        $level--;
        $menu_build .= "</ul></li>\n";
      }
      else $menu_build .= "</li>\n";

      //we need to go up one level
      if($sub == 1 && $level == 1)
      {
        $level++;
        $menu_build .= "<ul><li>\n";
      }
      else $menu_build .= "<li>";
      //always print out a link
      $menu_build .= "<a href=\"#\">".$menu['title']."</a>\n";
    }
    $menu_build .= "</ul>\n";

    return $menu_build;
}

ОБНОВИТЬ:

Попробуй это:

function build_navbar($pid, $sub=0)
{
    global $db;

    $class = $sub ? "sub" : "navigation";

    $menu_build = "<ul class=\"$class\">\n";

    $query = $db->simple_select("navbar", "*", "pid='".$pid."'");
    while($menu = $db->fetch_array($query))
    {
        $menu_build .= "<li><a href=\"#\">".$menu['title']."</a>\n";

        // build child links
        $menu_build .= build_navbar($menu['id'],1);
    }

    $menu_build .= "</ul>";

    return $menu_build;
}

Что мы здесь делаем, так это разрешаем каждой функции создавать группу <ul><li>, переменная $sub будет определять, каким будет идентификатор <ul>, что позволяет вам по-разному оформлять каждую ul.

РЕДАКТИРОВАТЬ:

Я нашел это!

эта линия

build_navbar($menu['id'],1);

нужно изменить на это:

$menu_build = build_navbar($menu['id'],1);

Похоже, это должно сработать для меня.

Что бы я сделал, так это добавил несколько операторов echo, отображающих SQL-запрос, который выполняется каждый раз, а затем вы можете скопировать его в phpmyadmin (или любой другой браузер базы данных, который вы используете). Посмотрите, возвращает ли он также нулевой результат. Если это так, возможно, что-то не так с вашими данными.

так например:

echo "SELECT FROM navbar * WHERE pid='$pid' ORDER_BY disporder;<br>";