[Fabio]

Ich habe da ein kleines Problem mit einem (selbstprogrammierten) PHP-Script.

Das Script ist eigentlich ganz simpel (und doch habe ich mehr als zweieinhalb Stunden gebraucht). Drei for schleifen, die Daten in einem Array speichern und eine, die das ganze dann ausgibt (Vielleicht hat hier jemand eine Idee, wie man das vereinfachen kann?). Das funktioniert allerdings nicht korrekt. Scheinbar wird nach dem 3. Durchlauf durch irgendwas gestoppt.

Ausserdem habe ich nervige Notices weil ich angeblich einen 11. Offset anspreche. Wo liegt das? Klar, ich könnte einfach die error_reporting Stufe runterstellen, aber das ist ja nicht der Zweck meiner "kleinen" Übung.

Hier das Script, die Datenbank ist im Anhang und mit SQLite 2 erzeugt worden. Die echos in den for-Schleifen sind zu Debugzwecken: Intressanterweise klappt es hier auch.

PHP-Code:

<?php
 error_reporting(E_ALL);

 if($database = sqlite_open("part.db", 0444, $error))
 {
  $query = sqlite_array_query($database, "SELECT * from sda"); // structure sda: Partition Number | OS | use

  if($number = array_chunk($query, count($query), true)) { // structure sda: 0 | table | element
   $tmparray = array(array());

   for($runid = 0; $stepid = $number[0][$runid][0]; $runid++) {
    $tmparray[0][] = $stepid;
    echo $stepid."<br>";
   }
   for($runos = 0; $stepos = $number[0][$runos][1]; $runos++) {
    $tmparray[1][] = $stepos;
    echo $stepos."<br>";
   }
   for($runuse = 0; $stepuse = $number[0][$runuse][2]; $runuse++) {
    $tmparray[2][] = $stepuse;
    echo $stepuse."<br>";
   }

   echo "<table border='1'>";
   echo "<tr><th>ID</th><th>OS</th><th>Verwendung</th></tr>";
   for($rundisplay = 0; $rundisplay < count($tmparray); $rundisplay++) {
    echo "<tr><td>".$tmparray[0][$rundisplay]."</td><td>".$tmparray[1][$rundisplay]."</td><td>".$tmparray[2][$rundisplay]."</td></tr>";
   }
  }
 }
 else
 {
  echo "ERROR";
  die($error);
 }
?>

Vielen Dank
Fabio

[WarrenFaith]

Du hast in allen drei for-Schleifen einen kapitalen Fehler, denn du weist einen Wert zu, anstatt dass du ihn vergleichst. Du nutzt also = statt ==

Dann noch ne kleine Anmerkung für bessere Lesbarkeit im Quellcode:
Sogenanntes CamelCase nutzen. Sprich aus $rundisplay wird $runDisplay. Das liest sich gerade bei längeren Variablennamen deutlich leichter.

[Ben]

Zitat von WarrenFaith

Du hast in allen drei for-Schleifen einen kapitalen Fehler, denn du weist einen Wert zu, anstatt dass du ihn vergleichst. Du nutzt also = statt ==

Bisschen durcheinander?

Er will doch gar nichts vergleichen, er will doch zuweisen!
Wobei das $number = array_chunk(..) nicht ganz sauber ist.

Hab mir den Code jetzt aber nicht genauer angeschaut, gerade nicht den Kopf dafür

[Fabio]

Meine Überlegung war, dass die Schleife ausgeführt wird, solange das noch funktioniert und das auch gleich noch ausführen. Aber irgendwie muss man das verhindern können.

In welcher Weise "nicht ganz sauber"? Mir ist schlichtweg keine andere Methode eingefallen. Was gäbe es dann da?

[WarrenFaith]

Zitat von Ben

Bisschen durcheinander?

Dito?!
Ich meine nicht innerhalb der for-Schleife sondern im for-Schleifenkopf.

for(Wertzuweisung für Start; Vergleich bis wann die Schleife gilt; Was machen nach Schleifendurchlauf)
Er macht an 2. Stelle aber eine Wertzuweisung.

*update*
deutlicher und richtig:

for($runos = 0; $stepos == $number[0][$runos][1]; $runos++) {
$tmparray[1][] = $stepos;
echo $stepos."<br>";
}

(ohne php-tags, da sonst das fette hervorheben nicht geht)

[Ben]

Genau. Ich habs mal wieder verbockt ..

[Fabio]

Wie schon gesagt, meine Überlegung war, dass die Schleife ausgeführt wird, solange das noch funktioniert und dass das auch gleich ausgeführt wird.

Gibt es so etwas wie "As_long_this_works_errorless()" oder so? Ansonsten müsste ich mein Programm nochmal komplett neu schreiben

Edit: Ich werde mal deine Variante ausprobieren und neu verwerten.

Edit 2: Scheinbar wird mit deiner Variante gar nichts zurückgegeben. Absoulut keine Ausgabe mehr.

[WarrenFaith]

Ich muss ganz ehrlich sagen, dass ich kaum nachvollziehen kann was dein Programm eigentlich machen soll?!
Versuch mir das bitte nochmal zu erklären. Also welche Daten sind in der Tabelle, was soll am ende rauskommen?

[Fabio]

Am Ende sollte eine HTML Tabelle ausgegeben werden, die so aussieht:

|Partitionsid|System, dass die Partition verwendet|Wie wird es verwendet|
|1|Ubuntu|/boot|

usw.

Das könnte man auch einfach mit einer Query lösen und auf dem Terminal ausgeben, aber dann ist es ja keine PHP-Übung mehr.

Struktur der Datenbank:
sda(nr integer, system varchar(20), use varchar(20)) (Eine Nummer als Integer, das System als Varchar und die Verwendung als varchar)

Inhalt der DB:
1|Ubuntu|/boot
2|All Linux|SWAP
3|All Linux|/home
5|Ubuntu|/tmp
6|Ubuntu|/
7|Gentoo|/
8|Fedora|/
9|All Linux|DiskIMG Parallels
10|All Linux|Games
11|CentOS|/
12|SuSE|/

[Ben]

Zitat von Fabio

Gibt es so etwas wie "As_long_this_works_errorless()" oder so?

Eine while()-Schleife?

[Fabio]

Bringt auch nichts. Es muss wohl am array_chunk() liegen, da alle while versuche auch gescheitert sind.

[WarrenFaith]

Ok verstehe.
Ich würde sagen, du gehst da definitiv zu kompliziert heran

PHP-Code:

<?php
  error_reporting(E_ALL);
  $database = sqlite_open("part.db", 0444, $error);
  if(!$database) {
    exit('Datenbankfehler!');
  }
  $query = sqlite_query("SELECT * from sda");
  while ($ergebnis = sqlite_fetch_array($query)) {
    echo $ergebnis[0] . "|" . $ergebnis[1] . "|" . $ergebnis[2] . "<br/>";
  }
?>

Das sollte dir die von dir genannte Liste ausgeben.

[Fabio]

Du hast dies wirklich ums vielfache einfacher gemacht.
Einfacher geht's nicht, dankeschön. Den Code hab ich verstanden und daraus gelernt, dass das KISS Prinzip in der Programmierung Fehler vermeidet.

Tatsächlich: Die Ausgabe stimmt auch. Noch ein wenig angepasst. Möchte jemand noch die Endgültige Version sehen?

[WarrenFaith]

Ja hau rein, kann für andere nicht schaden

Sagst mir noch schnell was KISS ist? Dann hab ich auch was gelernt

[Ben]

http://de.wikipedia.org/wiki/KISS-Prinzip

[Fabio]

Es waren vor allem Fehler in den Funktionen für SQLite. Steht alles in den Kommentaren.

PHP-Code:

<?php
  error_reporting(E_ALL);

  $database = sqlite_open("part.db", 0444, $error);
  if(!$database) {
    exit('Datenbankfehler!');
  }
  $query = sqlite_query($database, "SELECT * from sda"); // Fixed: $database als erster Parameter

  echo "<table border='1'>\n";
  echo "<tr><th>ID</th><th>System</th><th>Verwendung</th></tr>\n"; // Fixed: Tabelle eingefügt
  while ($ergebnis = sqlite_fetch_array($query, 3)) { // Fix
    echo "<tr><td>" . $ergebnis[0] . "</td><td>" . $ergebnis[1] . "</td><td>" . $ergebnis[2] . "</td></tr>\n"; // Fixed: Tabelle (Beendet)
  }
  echo "</table>";
?>

Und natürlich die HTML Tabelle eingefügt.

[WarrenFaith]

Den Code hab ich verstanden und daraus gelernt, dass das KISS Prinzip in der Programmierung Fehler vermeidet.

Nunja das stimmt voll und ganz, denn je weniger Codezeilen es gibt, desto weniger Fehler dürfte es haben.

Irgendwo hab ich mal gelesen, dass ein guter Programmierer auf 1000 Codezeilen 1 bis 3 Fehler macht...

[WarrenFaith]

Ergebnis_Typ spezifiziert, wie Sie das Ergebnis erhalten möchten. Der Standard-Wert ist SQLITE_BOTH. Damit werden die Spalten sowohl mit Ihrer Nummer als auch mit Ihrem Namen als Hashindex genutzt. SQLITE_ASSOC erzeugt ein Array, dass nur mit den Spaltennamen indiziert ist, SQLITE_NUM eine reine Indizierung mit den Spaltennummern.

Quelle: http://de3.php.net/sqlite_fetch_array
Demzufolge ist $sqlite_fetch_array($query, 3) falsch, denn damit gibst du nicht die Anzahl der Parameter an, sondern wie das Array gestaltet sein soll. Das solltest du beachten