Scherzfrage des Tages

Cujo

Habe ein paar Minuten zeit gehabt:
Die Tabelle:

Code

CREATE TABLE `tabelle` (
 `id` int(11) NOT NULL auto_increment,
 `frage` text NOT NULL,
 `antwort` text NOT NULL,
 `datum` date NOT NULL default '0000-00-00',
 PRIMARY KEY  (`id`)
)

Da ich nicht wusste, wo ich diese Zeilen eingeben muss, habe ich die Tabelle von Hand angelegt. Hat auch einwandfrei funktioniert. Der nachher angezeigte Code sah so aus wie deiner.

Einlesen der CSV

PHP

while (($data = fgetcsv ($handle, 1000, ",")) !== FALSE) 
    { 
        $query = "Insert 
                    into tabelle
                  set
                     frage   = '" . mysql_real_escape_string($data[0]) . "',
                     antwort = '" . mysql_real_escape_string($data[1]) . "'";

        mysql_query($query) or die ("MySQL-Fehler: " . mysql_error());
    }

    fclose ($fp);
  }
  else
    die ("Fehler beim Oeffnen der CSV-Datei");

Alles anzeigen

Mit der while-Schleife stimmt was nicht. Sie wird nicht verlassen. Es erscheinen tausende von Zeilen mit dieser Warnung:

Warning: fgetcsv() expects parameter 1 to be resource, null given in /home/www/tabelle.php on line 13

Ich habe dann nach einer Minute abgebrochen. Da waren dann 200.000 Einträge in der Tabelle

Zuerst dachte ich, es läge daran, dass in der csv-Datei als Trennzeichen ";" und nicht "," eingestellt ist. Aber die entsprechende Änderung im Code hat nicht geholfen.

Kann sich bitte nochmal jemand den Code angucken?

Bandit

$handle ist falsch, versuche es mal mit $fp

Cujo

Zitat von bandit600

$handle ist falsch, versuche es mal mit $fp

Jetzt passiert gar nix. Es gibt keine Ausgabe und die Tabelle bleibt leer.

Bandit

Schreibe an den Anfang:

error_reporting(E_ALL);
ini_set("display_errors", true);

Wenn es dann immer noch keine Fehlermeldung gibt, poste dein aktuelles Script.

Cujo

Nee, gab immer noch keine Fehlermeldung. Hier ist das Script:

PHP

<?php
error_reporting(E_ALL);
ini_set("display_errors", true);
  $fp = @fopen ("scherzfragen.csv","r");

  if ($fp)
  {
    @mysql_connect("host", "name", "passwort") or
      die ("Fehler beim Connect");

    @mysql_select_db("datenbank") or
      die ("Datenbank nicht vorhanden");

    while (($data = fgetcsv ($fp, 992, ";")) !== FALSE) 
    { 
        $query = "Insert 
                    into scherzfrage
                  set
                     frage   = '" . mysql_real_escape_string($data[0]) . "',
                     antwort = '" . mysql_real_escape_string($data[1]) . "'";

        mysql_query($query) or die ("MySQL-Fehler: " . mysql_error());
    }

    fclose ($fp);
  }
  else
    die ("Fehler beim Oeffnen der CSV-Datei");
?>

Alles anzeigen

Die Felder "host", "name", "passwort" und "datenbank" sind natürlich sinnvoll gefüllt. Die csv-Datei heißt "scherzfragen.csv", die Tabelle "scherzfrage". Die Anzahl habe ich auf 992 geändert, da es nur 496 sind

Bandit

So auf den ersten Blick erkenne ich da keinen Fehler. Da hilft nur:

Richtig debuggen
1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(E_ALL); und ini_set("display_errors", true);
3. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
4. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde (oder auch nicht).
5. Schritt 3 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
6. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
7. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
8. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

Cujo

Ich weiß nicht genau wieso und warum, aber auf jeden Fall sind jetzt die 992 Datensätze in der Tabelle drin

Aber beim Frage-ermitteln-Script gibt es leider auch wieder eine Fehlermeldung:

Parse error: syntax error, unexpected '[' in /home/www/frage.php on line 34

Hier ist nochmal der Code:

PHP

<?php


  function resetDate()
  {
     $query = "Update
                scherzfrage
               set
                 datum='0000-00-00'";

     mysql_query($query) or 
       die ("MySQL-Fehler: " . mysql_error());
  }

  function getQuestion(&$id, &$question)
  {
      $result = false;

      // Frage des Tages schon mal ermittelt?
      $query = "Select
                  id,
                  question
                from
                  scherzfrage
                where
                  datum=curdate()";

       $result = mysql_query($query) or 
         die ("MySQL-Fehler: " . mysql_error());


       if (mysql_num_rows($result))
       {
           $row = mysql_fetch_array($result);
           $id  = row['id'];
           $question  = row['frage'];
           $result = true;
       }
       else
       {
          // Es wurde noch keine Frage des Tages ermittelt
          $query = "Select
                  id,
                  question
                from
                  scherzfrage
                where
                  datum='0000-00-00'
                order by rand()";

          $result = mysql_query($query) or 
             die ("MySQL-Fehler: " . mysql_error());

          if (mysql_num_rows($result))
          {
             $row = mysql_fetch_array($result);
             $id  = row['id'];
             $question  = row['frage'];

             // Die Frage haben wir, jetzt Datum aktualisieren  
             $query = "Update
                         scherzfrage
                       set
                         datum=curdate()
                       where
                         id = $id";

             mysql_query($query) or 
                die ("MySQL-Fehler: " . mysql_error());

             $result = true;
          }
          else
          {
             // Alle Fragen durch, Datum resetten
             resetDate();

             $result = getQuestion($id, $question);
          }


       }

       return $result;
  }

  $result = getQuestion($id, $question);

  if ($result === true)
    echo "Id = $id, Frage = $question";
  else
    echo "Mist, da ist was schiefgelaufen.";
?>

Alles anzeigen

Kannst du bitte nochmal drübergucken?Den Feldnamen in der Tabelle habe ich von "frage" auf "question" geändert.

Edit: So, ich habe den Fehler gefunden. Bei "row" hat an manchen Stellen das "$" davor gefehlt. Ich versuche, das jetzt mit der Antwort hinzukriegen.

Vielen Dank, Bandit. Du hast mir sehr geholfen

Nochmal Edit: So, ich bin noch einen Schritt weiter. Die passende Antwort habe ich jetzt auch. Jetzt hätte ich gerne noch ein bisschen Javascript eingebunden:

Code

echo '<script type="text/javascript"><img style="width: 78px; height: 33px;" onclick="alert('$antwort')" alt="Lösung" src="buttons/raetsel-loesung.png" /></script>';

Die Antwort soll durch ein Klick auf einen Button per Alert angezeigt werden. Wenn ich das einbinde, gibt es aber folgende Fehlermeldung:

Parse error: syntax error, unexpected T_VARIABLE, expecting ',' or ';' in /home/www/frage.php on line 97

Wahrscheinlich habe ich die Einbindung völlig falsch gemacht. Kann sich das bitte nochmal jemand angucken?

Bandit

Die Kombination von " und ' ist nicht richtig, nochmal genau hinsehen. Außerdem ist der Script-Teil falsch.

gelöschter User

csv_to_database.php

PHP

<?php
$file = "scherz.csv";
$dbHost = "localhost";
$dbUser = "root";
$dbPass = "";
$dbName = "test";
$dbTableName = "tabelle";


$fp = @fopen ($file,"r");


echo 'CSV-Datei auslesen('.$file.')...';  
if ($fp)
{
    @mysql_connect($dbHost, $dbUser, $dbPass) or die ("Error: Keine Verbindung zum Datenbankserver");

    @mysql_select_db($dbName) or die ("Error: Keine Verbindung zur Datenbank");

    while (($data = fgetcsv ($fp, 1000, ",")) !== FALSE) 
    { 
        $query = "INSERT INTO `$dbTableName` (frage,antwort) VALUES ('".mysql_real_escape_string($data[0])."','" . mysql_real_escape_string($data[1]) . "');";
        echo 'Datensatz hinzuf&uuml;gen("'.$data[0].'","'.$data[1].'")...<br/>';            
        mysql_query($query) or die ("MySQL-Error: ".mysql_error());
    }
    fclose ($fp);
    echo 'CSV-Datei schlie&szlig;en('.$file.')...<br/>';
}
else
    die ("Error: Fehler beim &Ouml;ffnen der CSV-Datei");
echo 'CSV-Datei wurde erfolgreich in die Datenbank &uuml;bernommen!';


?>

Alles anzeigen

get_question.php

PHP

<?php
$dbHost = "localhost";
$dbUser = "root";
$dbPass = "";
$dbName = "test";
$dbTableName = "tabelle";
@mysql_connect($dbHost, $dbUser, $dbPass) or die ("Error: Keine Verbindung zum Datenbankserver");

@mysql_select_db($dbName) or die ("Error: Keine Verbindung zur Datenbank");




function resetDate($dbTableName)
{
    $query = "UPDATE $dbTableName SET datum = '0000-00-00'";
    mysql_query($query) or die("MySQL-Fehler: ".mysql_error());
}


function getQuestion(&$id, &$question,$dbTableName)
{
    $result = false;
    // Frage des Tages schon mal ermittelt?
    $query = "SELECT id, frage FROM $dbTableName WHERE datum = CURDATE()";
    $result = mysql_query($query) or die ("MySQL-Fehler: <br/>".$query."<br/>".mysql_error());


    if (mysql_num_rows($result))
    {
       $row = mysql_fetch_array($result);
       $id  = $row['id'];
       $question  = $row['frage'];
       $result = true;
    }
    else
    {
        // Es wurde noch keine Frage des Tages ermittelt
        $query = "SELECT id,frage FROM $dbTableName WHERE datum = '0000-00-00' ORDER BY RAND()";

        $result = mysql_query($query) or die ("MySQL-Fehler: " . mysql_error());

        if (mysql_num_rows($result))
        {
            $row = mysql_fetch_array($result);
            $id  = $row['id'];
            $question  = $row['frage'];

            // Die Frage haben wir, jetzt Datum aktualisieren  
            $query = "UPDATE $dbTableName SET datum = CURDATE() WHERE id = '$id'";

            mysql_query($query) or die ("MySQL-Fehler: ".mysql_error());

            $result = true;
        }
        else
        {
            // Alle Fragen durch, Datum resetten
            resetDate($dbTableName);

            $result = getQuestion($id, $question,$dbTableName);
        }
   } 
   return $result;
}
$result = getQuestion($id, $question,$dbTableName);


if ($result === true)
    echo "Id = $id, Frage = $question";
else
    echo "Mist, da ist was schiefgelaufen.";
?>

Alles anzeigen

So müsste bandits code funktionieren

Cujo

So, ich habe es geschafft, das Script fertigzustellen. Mit Antwort und Javascript. Ich kopiere es hier nochmal rein. Für zukünftige Generationen

PHP

<?php
@mysql_connect("Mein Host", " Mein Benutzername", "Mein Passwort") or
      die ("Fehler beim Connect");

    @mysql_select_db("Meine Datenbank") or
      die ("Datenbank nicht vorhanden");
  function resetDate()
  {
     $query = "Update
                scherzfrage
               set
                 datum='0000-00-00'";

     mysql_query($query) or 
       die ("MySQL-Fehler: " . mysql_error());
  }

  function getQuestion(&$id, &$question, &$antwort)
  {
      $result = false;

      // Frage des Tages schon mal ermittelt?
      $query = "Select
                  id,
                  question,
                  antwort
                from
                  scherzfrage
                where
                  datum=curdate()";

       $result = mysql_query($query) or 
         die ("MySQL-Fehler: " . mysql_error());


       if (mysql_num_rows($result))
       {
           $row = mysql_fetch_array($result);
           $id  = $row['id'];
           $question  = $row['question'];
           $antwort = $row['antwort'];
           $result = true;
       }
       else
       {
          // Es wurde noch keine Frage des Tages ermittelt
          $query = "Select
                  id,
                  question,
                  antwort
                from
                  scherzfrage
                where
                  datum='0000-00-00'
                order by rand()";

          $result = mysql_query($query) or 
             die ("MySQL-Fehler: " . mysql_error());

          if (mysql_num_rows($result))
          {
             $row = mysql_fetch_array($result);
             $id  = $row['id'];
             $question  = $row['question'];
             $antwort = $row['antwort'];

             // Die Frage haben wir, jetzt Datum aktualisieren  
             $query = "Update
                         scherzfrage
                       set
                         datum=curdate()
                       where
                         id = $id";

             mysql_query($query) or 
                die ("MySQL-Fehler: " . mysql_error());

             $result = true;
          }
          else
          {
             // Alle Fragen durch, Datum resetten
             resetDate();

             $result = getQuestion($id, $question, $antwort);
          }


       }

       return $result;
  }

  $result = getQuestion($id, $question, $antwort);

  if ($result === true) {
    echo "$question ";
    echo "<img style=\"width: 78px; height: 33px;\" onclick=\"alert('$antwort')\" alt=\"Lösung\" src=\"buttons/raetsel-loesung.png\">";
    }
  else
    echo "Mist, da ist was schiefgelaufen.";
?>

Alles anzeigen

Edit: Aaargggh, es gibt doch noch ein Problem Wenn ich das Skript einfach so als Datei aufrufe, funktioniert es einwandfrei. Wenn ich es aber mit

PHP

<? include ("http://www.raetselstunde.de/frage.php"); ?>

in einer html-Datei aufrufe, gibt es folgende Fehlermeldungen:

Warning: include() [function.include]: URL file-access is disabled in the server configuration in /home/www/test.html on line 36

Warning: include(http://test.raetselstunde.de/scherzfrage-des-tages.php) [function.include]: failed to open stream: no suitable wrapper could be found in /home/www/test.html on line 36

Warning: include() [function.include]: Failed opening 'http://www.raetselstunde.de/frage.php' for inclusion (include_path='.:/usr/share/php') in /home/www/test.html on line 36

Woran liegt denn das jetzt schon wieder? Fehlen mir da irgendwelche Rechte?

Greg10

lies dir die fehlermeldung durch die sagt doch alles!
nimm nicht den absoluten pfad sondern einen relativen

sudeki05

Zitat von Cujo

Woran liegt denn das jetzt schon wieder? Fehlen mir da irgendwelche Rechte?

Ich mach es eben schnell udn schmerzlos:
Deine datei darf nicht .html heißen sonder "muss" .php heißen
Auch in .php Dateien kannst du ganz normalen HTML-Code problemlos ausgeben

Cujo

Zitat von Greg10

lies dir die fehlermeldung durch die sagt doch alles!
nimm nicht den absoluten pfad sondern einen relativen

Ja, daran lag's. Danke für den Tipp.

Allerdings ergibt sich dadurch dann ein neues Problem. Ich würde nämlich gerne anderen Webmastern die Möglichkeit bieten, dieses PHP-Script bei sich einzubinden (ähnlich wie bei "Das Zitat des Tages"). Das funktioniert allerdings nur, wenn ich es absolut einbinden kann. Oder würde ich sowas besser mit Javascript machen?

Zitat von sudeki05

Ich mach es eben schnell udn schmerzlos:
Deine datei darf nicht .html heißen sonder "muss" .php heißen

Nee, damit hatte es nichts zu tun. Ich habe "AddHandler application/x-httpd-php .html" in meiner .htaccess drin, also funktioniert es auch mit .html.

Greg10

ja sie könnens schon einbinden aber auf deinem server ist es nunmal nicht erlaubt urls einzubinden

Cujo

Wenn ich das richtig verstehe, müssten die Webmaster dann selbst eine Tabelle in einer MySQL-Datenbank anlegen, die Fragen hochladen und dann das Skript einbinden.

Mir wäre es aber lieber, wenn der Zugriff über meine Datenbank erfolgen würde und die Webmaster nur einen kleinen Code einbinden müssten. Das geht wohl nur mit Javascript, oder?

Greg10

natürlich du machstes einfach so wie jetzt nur benutzt du anstatt das absoluten pfad also http://www....... benutzt du einen realtiven wie z.b. ../irgendwas/dingens dann kannst dus includieren und die webmaster könnten dann die datei die DU inkludierst via frame einbinden, oder wenn es auf ihren server erlaubt ist, includieren

Cujo

Ich hab's hingekriegt

Die Datei test.html sieht wie folgt aus:

Code

<? include ("frage.php"); ?><br />

Und die Datei test2.html so:

Code

<iframe name="test" src="http://www.raetselstunde.de/test.html" frameborder="0" height="1000" width="100%" scrolling="auto" align="center"></iframe>

Vielen Dank nochmal für deine Hilfe

Cujo

PHP

<?php


  $fp = @fopen ("datei.csv","r");

  if ($fp)
  {
    @mysql_connect("host", "user", "passwort") or
      die ("Fehler beim Connect");

    @mysql_select_db("datenbank") or
      die ("Datenbank nicht vorhanden");

Alles anzeigen

Ich hätte noch eine Frage dazu. Ist das nicht ein Sicherheitsrisiko wenn ich in dem Skript meine ganze Daten wie Benutzername und Passwort eintrage? Oder hat man darauf von aussen keinen Zugriff?

Bandit

Solange der Server funktioniert, sprich, PHP-Files geparst werden, kann da nichts passieren. Ich habe bei mir immer ein Verzeichnis, auf dass man von außen nicht drauf zugreifen kann, sondern nur per Script. In diesem Verzeichnis befindet sich dann eine Konfigurationsdatei mit allen wichtigen Daten.

No0ob

Kannst du mal bitte erläutern, wie das geht? Ich suche schon lange nach einer Möglichkeit, bin aber mit .htaccess nicht so begabt. Als ich das mal probiert habe, konnte ich mit gar nichts drauf zugreifen, nicht einmal per Script. Es wäre toll, wenn du mal deine Lösung zeigen könntest

Tags