Witam

Mam taką funkcję i chciałbym zapisać ją bardziej profesjonalnie. Do tego chciałem zmienić połącznie z bazą na mysqli, ale pojawia się problem z mysql_fetch_array.
Jak zastąpić echo jakimś sensownym responsem ?

[PHP] pobierz, plaintext 
function display($section) {
        global $section;
 
        $query = "SELECT * FROM `$section` ORDER BY date DESC";
        $result = mysql_query($query);
 
        while ($row = mysql_fetch_array($result)) {
 
        $title = $row['title'];
        $date = $row['date'];
        $content = $row['content'];
        $autor = $row['autor'];
 
        echo('<h1>'.$title.'</h1><p class="right">'.$date.'</p>');
        echo('<p class="justify">'.$content.'</p>');
        echo('<p>Add by: <b>'.$autor.'</b></p>');
        echo('<hr />');
        };
[PHP] pobierz, plaintext 

Będę wdzięczny za rady i uwagi.

A. Co to znaczy bardziej 'profesjonalnie'?
B. Wyniki można zapisać np. w tablicy lub dopisać do zmiennej ($zmienna .= $date)
C. Jeżeli chcesz przejść na mysqli to mam nadzieje pamietasz o tym, że nie korzystasz z mysql_fetch_array tylko mysqli_fetch_array?

ps. korzystaj z mysql_fetch_assoc (przyjemniejsze )

onkel nie masz na myśli or die (mysql_error()); na końcu SELECTa ?

a. Zależy mi na tym, żeby funkcja była obiektowa.
b. Rozumiem, że można wrzucić do zmiennej, ale nie bardzo wiem w jaki sposób dołączyć do wyniku tagi HTML.
c. O tym, że powinno być mysqli_fetch_array to wiem, ale to nie wszystko

znalazłem coś takiego

[PHP] pobierz, plaintext 
        $query = "SELECT title, date, content, autor FROM music ORDER by date DESC";
        $result = $mysqli->query($query);
        while ($row = $result->fetch_array(MYSQLI_ASSOC)) {
        printf ("%s (%s)\n", $row["title"], $row["date"], $row["content"], $row["autor"]); };
[PHP] pobierz, plaintext 

Niestety zwraca mi tylko title i date.

Jakieś inne korzyści mogą wynikać z używania mysql_fetch_assoc ?

1. Zrób najpierw klasę, a później przepisz w/w f-cję jako metodę stworzonej klasy

2. Jaki sens ma to:

function display($section) {
global $section;



Albo przekazujesz parametr do funkcji, albo czynisz zmienną $section globalną w ciele funkcji, ale nie jedno i drugie, bo to nie ma sensu. W tym momencie w funkcji nadpisujesz przekazany parametr wartością zmiennej $section z zewnątrz funkcji (sic!), a pewnie nie o to chodziło.

// edit

Jednak lepiej będzie zwracać z funkcji określone wartości niż bezpośrednio wypluwać za pomocą echo czy print

darko Masz racje global $section; nie powinno się tam pojawić .


zmieniłem funkcję i teraz wygląda tak.

[PHP] pobierz, plaintext 
 
function display($section) {
        $mysqli = new mysqli("localhost", "root", "root", "baza");
 
        $query = "SELECT title, date, content, autor FROM music ORDER by date DESC";
        $result = $mysqli->query($query);
 
        while ($row = $result->fetch_array(MYSQLI_ASSOC)) {
 
        $a=    $row["title"];
        $b=    $row["date"];
        $c=    $row["content"];
        $d=    $row["autor"];
 
        echo ($a);
        echo ($b);
        echo ($c);
        echo ($d); };
 
        }
[PHP] pobierz, plaintext 

1. Jak mogę dodać komunikat błędu w przypadku gdy w bazie nie ma żadnego rekordu?

2. Jak dodać do przykładowo $a <h1>, aby funkcja miała nadal charakter obiektowy. Czytałem, że nie poleca się umieszczanie tagów HTML wewnątrz funkcji.

1. np.
$q = "SELECT count(autor) FROM music";
$count = $mysqli->query($q);
// fetch row dla mysqli oraz ustaw fetch type na fetch_num żeby wyciągać po indeksie (?) żeby poniższe zadziałało (nie wiem jak to dokładnie jest w mysqli, sprawdź, korzystam z PDO)
if($count[0] == 0) {
die "w bazie nie ma rekordów";
}

2. ee tam
$a= "<h1>".$row["title"]."</h1>";