[pago]

hi ich habe folgendes script:

PHP-Code:

1 <?php
2 $empfaenger=$_SESSION["user_nickname"];
3
4 $connect = mysql_connect ("localhost", "user", "pw");
5 mysql_select_db ("user", $connect)
6    
7 $sql"SELECT".
8 "sender, betreff, text, datum FROM datenbank WHERE (empfaenger like ' ".$_SESSION["user_nickname"]." ') AND ".
10 "(lesen = '-1')";
11
12 $ergebnis=mysql_query($sql);
13 $row = mysql_fetch_array($ergebnis);
14 ?>

ich finde an dem script kein fehler aber trotzdem gibt er diesen fehler aus:

Parse error: parse error, unexpected T_VARIABLE in .../newtelegramm.php on line 7

sieht einer von euch vielleicht den fehler?

[Orolhawion]

klar,
1. in Zeile 5 fehlt das ;
2. nach dem $sql würd ich jetzt mal ein Gleichheitszeichen setzen...

noch ein paar tips:

vielleicht in diesem fall nicht mit dem usernamen arbeiten, sondern mit der userid und dann auch nicht mit LIKE sondern mit =
außerdem ist es evtl. sinnvoller mit 0 und 1 gelesene mails zu unterscheiden, als mit -1 und was auch immer du noch verwendest..

[Jann Hendrik]

du solltest dir weiterhin die Funktionen die() und mysql_error() anschauen

Somit kannst du noch mehr Sicherheit hineinbringen..

[pago]

das ; war der fehler, das = was hier ncoh gefehlt hat war nur ein tip fehler.
also das klappt also soweit jedoch kommt jetzt ncoh eine andere fehlermeldung

Warning: mysql_fetch_array( supplied argument is not a valid MySQL result resource in /home/www/web1/html/16669/site/newtelegramm.php on line 12

[Orolhawion]

dann mal mit mysql_error() arbeiten, wie von Jann Hendrik schon erwähnt, es liegt offensichtlich ein fehler im SQL-Statement vor. den findest du aber selber...

[pago]

okay ich glaube es lag daran das er keine ergebnisse aus der datenbank geholt hat und deshlab zickt er rum.
das mit dem die befehl ist keine gute idee weil wenn ich das mache werden ja alle sachen die darunter stehen nciht mehr ausgeführt ink. der grafik...

[Jann Hendrik]

das mit der die()-Funktion ist schon sinnvoll. Nämlich dann, wenn du keine Verbindung zum Server aufgebaut hast.

du kannst die() ja auch erweitern.

die('leider keine Verbindung zum Server aufgebaut');

oder so. Dann weiß man schonmal wo der Fehler liegt!

[pago]

ich hab das script jetzt total umgeräumt

PHP-Code:

<?php
// Datenbankverbindung aufbauen
$connectionid = mysql_connect ("localhost", "user", "pw");
if (!mysql_select_db ("user", $connectionid))
{
  die ("Keine Verbindung zur Datenbank");
}

$sql = "SELECT ".
    "sender, betreff, text, datum".
  "FROM ".
    "datenbank".
  "WHERE ".
    "(empfaenger like '".$_SESSION["user_nickname"]."') AND ".
    "(lesen = '0')";
$result = mysql_query ($sql);

if (mysql_num_rows ($result) > 0)
{
  // Benutzerdaten in ein Array auslesen.
  $ergebnis = mysql_fetch_array ($result);

  // Sessionvariablen erstellen und registrieren
  $zeigen["sender"] = $ergebnis["sender"];
  $zeigen["betreff"] = $ergebnis["betreff"];
  $zeigen["text"] = $ergebnis["text"];
  $zeigen["datum"] = $ergebnis["datum"];
}
echo $zeigen["sender"];
?>

aber immer wieder kommen sollche fehler meldungen!

Warning: mysql_num_rows( supplied argument is not a valid MySQL result resource in /home/www/web1/html/16669/site/oldtelegramm.php on line 18

ich hab davon jetzt teile aus meinen anderen funktionieren scripts geholt und trotzdem kommt sowas

[Jann Hendrik]

dann poste doch bitte nochmal den Code - und zeig, welche die Zeile 18 ist.

denn oben gibt es eine solche nicht...

[Ben]

Schreib doch einfach mal

PHP-Code:

<?php

    # [..]
    $result = mysql_query($sql) or die(mysql_error());
    # [..]

?>

Dann wirst Du eine hilfreichere Fehlermeldung bekommen.

Außerdem kannst Du Dir mal vor der oben genannten Zeile die Variable $sql ausgeben lassen.
Mach das mal so:

PHP-Code:

<?php 
 
   echo printf( '<pre>%s</pre>', $sql );

 ?>

Das sollte weiterhelfen. Poste uns mal das Ergebnis.

Grüße Ben.

[Orolhawion]

bzgl. Ben's Beitrag:
zusammengefaßt kann das dann auch so aussehen:

PHP-Code:

<?php

    # [..]
    $result = mysql_query($sql) or die('Fehler im Statement: '.$sql.'<br />MySQL meldet: '.mysql_error());
    # [..]

?>

[list][/list:u]

[pago]

also ich es jetzt geschafft:

PHP-Code:

<?php
// Datenbankverbindung aufbauen
$link = mysql_connect ("localhost", "user", "pwh");
if (!mysql_select_db ("user", $link))
{
  die ("Keine Verbindung zur Datenbank");
}

$sql = "SELECT `sender` , `betreff` , `text` , `datum` FROM ".
    "datenbank".
  "WHERE ".
    "(`empfaenger` like '".$_SESSION["user_nickname"]."') AND ".
    "(`lesen` = '0')";
// Benutzerdaten in ein Array auslesen.
$result = mysql_query($sql, $link);
if(mysql_num_rows($result)>0)
{
    for($i=0;$i<mysql_num_rows($result);$i++)
    {
        $ergebnis[$i]=mysql_fetch_array($result);
    }
    echo "<table width=100%>";
    for($i=0;$i<count($ergebnis);$i++)
    {
    echo "<tr><td><a href='nachricht.php?".$ergebnis[$i]."' onMouseOver=".$ergebnis[$i]['sender'].">";
    echo $ergebnis[$i]['betreff'];
    echo "</a></td></tr>";
    }
echo "<tr><td></td></tr></table>";
}
else
{echo "<b>Keine neuen Nachrichten!</b>";}
?>

[Jann Hendrik]

du kannst da ruhig noch mehr Sicherheit mit reinbringen...

was zB. wenn mysql_connect() nicht erfolgreich war?
Dann gibt das wieder nur häßliche Fehler...

btw: dein Code mal ein wenig gekürzt - ich habe inhaltlich nichts geändert!

PHP-Code:

<?php
  // Datenbankverbindung aufbauen
  $link = mysql_connect ("localhost", "user", "pwh");
  if (!mysql_select_db ("user", $link))
    die ("Keine Verbindung zur Datenbank");
  $sql = "SELECT `sender` , `betreff` , `text` , `datum` FROM ".
         "datenbank".
         "WHERE ".
         "(`empfaenger` like '".$_SESSION["user_nickname"]."') AND ".
         "(`lesen` = '0')";
  // Benutzerdaten in ein Array auslesen.
  $result = mysql_query($sql, $link);
  if(mysql_num_rows($result)>0)
  {
    for($i=0;$i<mysql_num_rows($result);$i++)
      $ergebnis[$i]=mysql_fetch_array($result);
   echo "<table width=100%>";
   for($i=0;$i<count($ergebnis);$i++)
     echo "<tr><td><a href='nachricht.php?".$ergebnis[$i]."' onMouseOver=".$ergebnis[$i]['sender'].">".$ergebnis[$i]['betreff']."</a></td></tr>";

   echo "<tr><td></td></tr></table>";
  }
  else
    echo "<b>Keine neuen Nachrichten!</b>";
?>

auch solltest du unbedingt ganz oben folgende Zeile stehen haben:

PHP-Code:

<?php
  error_reporting(E_ALL);
?>