PHP — prosta zagnieżdżona nieuporządkowana tablica (UL)

To powinno wystarczyć:

$result = mysql_query("SELECT * FROM News");
$topicname = '';

// open list of topics
echo '<ul>';

// loop through topics
while($row = mysql_fetch_array($result)) {
    if (!$row['TopicID']) {
        // fake topic name for unsorted stuff
        $row['TopicName'] = 'Sort Me';
    }
    if ($topicname != $row['TopicName']) {
        if($topicname != ''){
            // had a topic name, means we opened a list
            // that hasn't been closed, close it.
            echo '</ul>';
        }
        // print this topic and open the list of articles
        echo '<li>' . $row['TopicName'] . '</li><ul>';
        // update the current topic to be this TopicName
        $topicname = $row['TopicName']; 
    }
    // the news item
    echo '<li>' . $row['NewsID'] . '"</li>';
}
if($topicname != ''){
    // we saw at least one TopicName, we need to close
    // the last open list.
    echo '</ul>';
}
// end topic list
echo '</ul>';

Myślę, że twoim prawdziwym problemem jest to, że za każdym razem otwierałeś dwie listy, ale zamykałeś tylko jedną (nawet przesuwając ostatni blok wewnątrz listy).

W drugiej części Twojego (nowego) pytania:

Zastrzeżę, że w przypadku większych list (powiedzmy, więcej niż 300 pozycji) kompromis, który robię w odniesieniu do przechowywania listy w pamięci i dwukrotnego iterowania, zamiast po prostu pytać o potrzebne zliczenia, zmieniłby się w drugą stronę. Oznacza to, że poniższe rozwiązanie umieszcza wszystko w pamięci, a następnie wykonuje iterację po raz drugi, aby to wydrukować; alternatywą byłoby uruchomienie dwóch zapytań, jednego w celu znalezienia liczby unikalnych nazw tematów, a drugiego w celu znalezienia łącznej liczby elementów na liście.

Ponadto, jeśli chodzi o wyświetlanie, naprawdę chcesz rozwiązać pewną optymalizację układu, zrobię to naiwnie i po prostu utworzę (z grubsza) równą liczbę tematów w kolumnie, a gdy podział nie wyjdzie, będzie to obciążać w lewo. Zobaczysz, gdzie możesz poprawić lub zastąpić kod, aby uzyskać inne (i lepsze?) wyniki.

$columns = // user specified;

$result = mysql_query("SELECT * FROM News");
$num_articles = 0;

// $dataset will contain array( 'Topic1' => array('News 1', 'News2'), ... )
$dataset = array();
while($row = mysql_fetch_array($result)) {
    if (!$row['TopicID']) {
        $row['TopicName'] = 'Sort Me';
    }
    $dataset[$row['TopicName']][] = $row['NewsID'];
    $num_articles++;
}

$num_topics = count($dataset);

// naive topics to column allocation
$topics_per_column = ceil($num_topics / $columns);

$i = 0; // keeps track of number of topics printed
$c = 1; // keeps track of columns printed
foreach($dataset as $topic => $items){
    if($i % $topics_per_columnn == 0){
        if($i > 0){
            echo '</ul></div>';
        }
        echo '<div class="Columns' . $columns . 'Group' . $c . '"><ul>';
        $c++;
    }
    echo '<li>' . $topic . '</li>';
    // this lists the articles under this topic
    echo '<ul>';
    foreach($items as $article){
        echo '<li>' . $article . '</li>';
    }
    echo '</ul>';
    $i++;
}
if($i > 0){
    // saw at least one topic, need to close the list.
    echo '</ul></div>';
}