Freunde ( online < 15min )

[orcaexpress]

Hallo,

vorweg: ich bin neu hier wiel ich Hilfe suche da mein wissen leider nicht ausreicht....

Hat jemand Erfahrung mit Freundeslisten anzeigen?

Meine Freundesliste zeigt auf dem Profil alle Freunde an, "online" und "offline" die durch eine Grafik (src='img/menu_friend_offline.gif') > 15min getrennt werden.
Ich will nun aber nur die Freunde angezeigt haben die "online" sind! was wäre da zuändern?

// --- Freundesliste anzeigen:
function get_friends($get_id)
{
    global $dotted;
    // todo -abfrage- query kürzen und count
    $result = mysql_query("
        SELECT `users`.`id` AS uid,
        TIMESTAMPDIFF(MINUTE,`users`.`time`,NOW()) AS `min`
        FROM `users` INNER JOIN `friends`
        ON (`users`.`id` = `friends`.`right`) OR (`users`.`id` = `friends`.`left`)
        WHERE (`friends`.`right` = '" . $get_id . "' OR `friends`.`left` = '" . $get_id . "') AND `users`.`id` != '" . $get_id . "' AND `friends`.`status`='both'
        ORDER BY `min` ASC");

    echo mysql_error();
    $num_rows = mysql_num_rows($result);
    $i = 1;
    while ($friends = mysql_fetch_assoc($result)) {
        if ($friends['min'] > 15 && !isset($set)) {
            echo "<tr><td colspan='2' class='pl6 pt5 pb3'><img alt='' src='img/menu_friend_offline.gif'></td></tr>";
            $set = 1;
        }
        echo "<tr>
<td style='width:23px; padding-top: 1px; padding-left: 6px; padding-right: 1px;'>";
        echo pimg($friends[uid], 25, 30);   
        echo "</td>
<td class='s13' style='width:191px; height: 26px; padding-right: 4px;'>" . username($friends[uid]) . "";
        echo "</td></tr>"; 
        if ($i != $num_rows)  {
          echo "<tr><td></td><td " . $dotted . "></td></tr>"; 
        }
        $i++;
    }
} 

[Kropff]

Wo ist das Problem? Du hast doch schon eine entsprechende Bedingung, die du nur anpassen musst.

if ($friends['min'] > 15 && !isset($set)) { 

Peter

[orcaexpress]

Hallo Peter,

die Zeile

if ($friends['min'] > 15 && !isset($set)) { 

bezieht sich ja zur grafischen Trennung von online und offline.

 echo "<tr><td colspan='2' class='pl6 pt5 pb3'><img alt='' src='img/menu_friend_offline.gif'></td></tr>";
            $set = 1;
        } 

oder wie meinst Du das?

ich möchte ja das nur die Freunde, die gerade "online" sind angezeigt werden.

Gruß Ron

[h3ll]

1. global ist böse und sollte nicht verwendet werden.

2. Bei dir fehlt überall das SQL-Escaping. Davon abgesehen, dass das zu Fehlern führen kann, ist es auch eine riesige Sicherheitslücke.

3. Die mysql_* Funktionen sind seit vielen Jahren veraltet. mysqli_* ist der Nachfolger. Bei OOP sollte allerdings gleich PDO verwendet werden.

4. Du gibst eine Liste an Freunden aus, verwendest aber eine Tabelle. Wenn du Listen ausgeben möchtest, verwende auch Listen und missbrauche keine Tabellen. Das Aussehen bestimmt übrigens CSS und nicht die verwendeten HTML-Tags.

5. Eine Funktion sollte für eine Aufgabe zuständig sein. Entweder Daten abfragen, oder Daten ausgeben. Aber nicht beides gleichzeitig. Davon abgesehen hast du deine Funktion falsch benannt. Der Funktionsname sollte nämlich aussagen, was die Funktion macht. Und deine Funktion macht weit mehr als "get friends".

6. Warum fügst du nicht einfach `min` <= 15 bei der Abfragebedingung hinzu?

[orcaexpress]

Hallo h3ll,

`min` <= 15 hab ich zur Abfrage hinzugefügt, aber es ändert sich nur dass die "online" Freunde nun unten stehen und "offline" oben über dem offlinesymbol...

[h3ll]

H`min` <= 15 hab ich zur Abfrage hinzugefügt, aber es ändert sich nur dass die "online" Freunde nun unten stehen und "offline" oben über dem offlinesymbol...

Dann hast dus falsch hinzugefügt.

[orcaexpress]

Dann hast dus falsch hinzugefügt.

könntest Du es mir an einem Beispiel zeigen? oder genaue Position in Zeile von meinem Script?

Gruß Ron

[h3ll]

könntest Du es mir an einem Beispiel zeigen? oder genaue Position in Zeile von meinem Script?

WHERE `min` <= 15

[orcaexpress]

WHERE `min` <= 15

soll das hier mit rein? :

   // todo -abfrage- query kürzen und count
    $result = mysql_query("
        SELECT `users`.`id` AS uid,
        TIMESTAMPDIFF(MINUTE,`users`.`time`,NOW()) AS `min`
        FROM `users` INNER JOIN `friends`
        ON (`users`.`id` = `friends`.`right`) OR (`users`.`id` = `friends`.`left`)
        WHERE (`friends`.`right` = '" . $get_id . "' OR `friends`.`left` = '" . $get_id . "') AND `users`.`id` != '" . $get_id . "' AND `friends`.`status`='both'
        ORDER BY `min` ASC"); 

[h3ll]

Wär nicht schlecht. Aber wenn du davon überhaupt keine Ahnung hast, solltest du dich erstmal mit den SQL-Grundlagen beschäftigen.

[orcaexpress]

kannst Du mir bitte die genaue Position in der Abfrage nennen?

[h3ll]

kannst Du mir bitte die genaue Position in der Abfrage nennen?

Willst du selber programmieren oder suchst du jemanden, der für dich programmiert?

[orcaexpress]

hmmm egal wo ich "<= 15" einfüge habe ich immer Syntax error, wegen "<="

[TobiaZ]

Unwahrscheinlich. Eine Korrekte stelle dafür wird es sicher nicht geben. Wenn du sie nicht mit ausprobieren findest, versuch es mit verstehen. Ansonsten beschreibe wenigstens, was du konkret probiert hast.