Funktionsaufruf bricht FOR Schleife ab

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Funktionsaufruf bricht FOR Schleife ab

    Also ich habe ein Script gebastelt das CSV Datenbankdateien verwaltet. Hierzu lese ich über FOR Schleifen jeweils zeilenweise aus der Datei in einen Array ein und wieder aus.
    Nun habe ich eine Schleife geschrieben die eine checkbox die hinter jeder zeile ist ausliest und wenn diese aktiviert ist die funktion delete_line ausführt. Leider funktioniert das aber nur mit der ersten markierten Zeile. Hat jemand eine idee woran das liegt?

    Quellcode

    1. for($i=1; $i<$zeile; $i++)
    2. {
    3. if(isset($_POST['del'][$i]) != "")
    4. {
    5. $his=$i;
    6. delete_line($datenbank,$i);
    7. echo "ok";
    8. $i=$his;
    9. }
    10. }
    12. echo "<br>";
    13. echo"<meta http-equiv=\"refresh\" content=\"0; URL=changes.php\">";
    14. echo"</table><a href=\"tedit.php\">Fortfahren</a>";
    15. unset($del);
    17. }
    19. function delete_line($file, $line, $change_in = "") {
    20. $entries = file($file);
    21. if(empty($change_in)) {
    22. unset($entries[$line-1]);
    23. } else {
    24. $entries[$line-1] = $change_in;
    25. }
    26. $newentries = implode("", $entries);
    27. $datei = fopen($file, "w");
    28. fputs($datei, $newentries);
    29. fclose($datei); }
    Alles anzeigen
    &#8222;Wenn man zwei Stunden lang mit einem Mädchen zusammensitzt, meint man, es wäre eine Minute. Sitzt man jedoch eine Minute auf einem heißen Ofen, meint man, es wären zwei Stunden. Das ist Relativität.&#8220; - Albert Einstein

  • hi,

    poste mal mehr code, damit wir wissen welchen wert $zeile hat
    außerdem ist die abfrage suboptimal - vielleicht funktioniert sie, aber sie ist nicht gut
    if(isset($_POST['del'][$i]) != "")

    isset liefert ein boolean - perfekt für eine schleifenbedinung
    => if(isset($_POST['del'][$i]))

    in die funktion delete_line() könntest du dann noch eine ausgabe machen
    z.b. echo 'versuche '.$line.' zu loeschen<br />';

    dann wissen wir ob die funktion aufgerufen wird

  • $Zeile

    $Zeile wird zu Beginn des Scripts auf die Zeilenanzahl der CSV Datei gesetzt.
    Das Problem ist aber das offenbar die FOR Schleife mit dem ersten Aufruf meiner Zeilenänder/lösch Funktion abgebrochn wird
    &#8222;Wenn man zwei Stunden lang mit einem Mädchen zusammensitzt, meint man, es wäre eine Minute. Sitzt man jedoch eine Minute auf einem heißen Ofen, meint man, es wären zwei Stunden. Das ist Relativität.&#8220; - Albert Einstein

  • Quellcode

    1. <html>
    2. <head>
    3. <meta http-equiv="expires" content="0">
    4. <meta http-EQUIV="pragma" content="no-cache">
    5. <title>Programm &Auml;ndern</title>
    6. <link href="../../style/css/text.css" rel="stylesheet" type="text/css">
    7. <link href="../../style/css/div.css" rel="stylesheet" type="text/css">
    8. </head>
    9. <div id="content">
    10. <form action="<? echo $PHP_SELF?>" method="POST" name=save>
    11. <table width="100%" border="1">
    12. <tr>
    13. <td>&nbsp;</td>
    14. <td><strong>Kursbezeichnung</strong></td>
    15. <td><strong>Wochentage</strong></td>
    16. <td><strong>Raum/Turnhalle</strong></td>
    17. <td><strong>Ort/Stadtteil</strong></td>
    18. <td>&nbsp;</td>
    19. </tr>
    20. <?php
    21. //CSV Datenbanktool von Johannes Gottschalk im Auftrag von Objekt+Grafik U.Lunenberg
    23. // Umgebungsvariablen
    24. $datenbank="../kurse.tab";
    26. // wichtige Aufrufe
    27. $zc=file($datenbank); //Lädt die Datenbank in ein Array
    28. $countedline=count($zc); //Zählt die Zeilen des Arrays
    29. unset($zeile); //löscht die Variable $Zeile
    30. $Dateizeiger = fopen($datenbank, "r"); //Öffnet die Datenbank zum lesen
    32. while(($Daten = fgetcsv($Dateizeiger, 1000, "|")) !== FALSE) //Liest die Datenbank Zeilenweise bis zum Ende aus
    33. {
    34. // $tdsum = count($Daten); // Zellen zählen oder fest einstellen
    35. $tdsum = "4"; //Anzahl der Zellen auf 4 festgesetzt
    36. $zeile++; //Zeilenzähler +1
    37. echo"<tr><td> ",$zeile,"</td>"; //Ausgabe HTML
    38. for($i = 0; $i < $tdsum; $i++)
    39. {
    40. echo"<td>",
    41. "<input type=\"text\" value=\"",
    42. $Daten[$i],"\"",
    43. "name=\"feld[",
    44. $zeile,
    45. $i,
    46. "]\">",
    47. "</td>";
    48. }
    50. if($countedline<2){
    51. echo"<td>Achtung - letzter Datensatz</td>"; }
    52. else{
    53. echo"<td><input type=\"checkbox\" name=\"del[",
    54. $zeile,
    55. "]\">l&ouml;schen</td></tr>"; }
    56. }
    58. // neuer Eintrag
    59. if(isset($_POST['new']) != "")
    60. {
    61. echo"<tr bgcolor=\"red\"><td>NEU</td>";
    62. for($i = 0; $i < $tdsum; $i++)
    63. {
    64. echo"<td>",
    65. "<input type=\"text\"",
    66. "name=\"newrow[",
    67. $i,
    68. "]\">",
    69. "</td>";
    70. }
    71. echo"<td><input type=\"submit\" name=\"addent\" value=\"Hinzuf&uuml;gen\"></td>";
    73. }
    74. else
    75. {
    76. echo"<input type=\"submit\" name=\"speichern\" value=\"speichern\">",
    77. "<input type=\"submit\" name=\"new\" value=\"neuer Eintrag\">";
    78. }
    79. fclose($Dateizeiger);
    81. if(isset($_POST['addent']) != "")
    82. {
    83. unset($ndat);
    84. for($i = 0; $i < $tdsum; $i++)
    85. {
    86. $ndat.=$_POST['newrow'][$i];
    87. $abfr=$tdsum;
    88. $abfr--;
    89. if($i<$abfr)
    90. {
    91. $ndat.="|";
    92. }
    93. else
    94. {
    95. $ndat.="\n";
    96. }
    97. $abfr++;
    98. }
    99. echo$ndat;
    100. $dateischreiber = fopen($datenbank, "a");
    101. fwrite($dateischreiber, $ndat);
    102. fclose($dateischreiber);
    103. echo"<meta http-equiv=\"refresh\" content=\"0; URL=changes.php\">";
    105. }
    107. ?>
    108. </table>
    109. </form>
    112. <?
    113. if(isset($_POST['speichern']) != "")
    114. {
    115. unset($ndat);
    116. $zeile++;
    117. for($irow=1; $irow<$zeile; $irow++)
    118. {
    119. for ($icol=0; $icol<$tdsum; $icol++)
    120. {
    121. unset($fln);
    122. $fln=$irow;
    123. $fln.=$icol;
    124. $ndat.=$_POST['feld'][$fln];
    125. $icol++;
    126. if($icol<$tdsum)
    127. {
    128. $ndat.="|";
    129. }
    130. else
    131. {
    132. $ndat.="\n";
    133. }
    134. $icol--;
    135. }
    136. }
    137. unlink($datenbank);
    138. $dateischreiber = fopen($datenbank, "a");
    139. fwrite($dateischreiber, $ndat);
    140. fclose($dateischreiber);
    143. for($i=1; $i<$zeile; $i++)
    144. {
    145. if(isset($_POST['del'][$i]) != "")
    146. {
    147. $his=$i;
    148. delete_line($datenbank,$i);
    149. echo "ok";
    150. $i=$his;
    151. }
    152. }
    154. echo "<br>";
    155. echo"<meta http-equiv=\"refresh\" content=\"0; URL=changes.php\">";
    156. echo"</table><a href=\"tedit.php\">Fortfahren</a>";
    157. unset($del);
    159. }
    161. // Zuerst: Dateiname, Zeilennummer (Anfang: 1!),
    162. // Text der ersetzt wird (optional, sonst "")
    163. function delete_line($file, $line, $change_in = "") {
    164. $entries = file($file);
    165. if(empty($change_in)) {
    166. unset($entries[$line-1]);
    167. } else {
    168. $entries[$line-1] = $change_in;
    169. }
    170. $newentries = implode("", $entries);
    171. $datei = fopen($file, "w");
    172. fputs($datei, $newentries);
    173. fclose($datei); }
    175. ?>
    176. </div>
    177. </body>
    178. </html>
    Alles anzeigen
    &#8222;Wenn man zwei Stunden lang mit einem Mädchen zusammensitzt, meint man, es wäre eine Minute. Sitzt man jedoch eine Minute auf einem heißen Ofen, meint man, es wären zwei Stunden. Das ist Relativität.&#8220; - Albert Einstein

  • deine fehlerbeschreibung stimmt nicht ganz
    es können doch auch mehrere einträge gleichzeitig gelöscht werden
    das problem ist, dass du zeilenbedingt löschst

    wenn du also zeilen 2 und 3 löschst, dann löschst du erst zeile2
    daraufhin werden alle zeilen nach 2 um 1 nach oben geschoben
    der 2te eintrag der gelöscht wird ist also der 4te eintrag

    lösen tust du das indem du einfach die komplette datei nochmal schreibst und eintrag 2 und 3 weglässt.
    das geht sowieso viel schneller als für jede zeile ein öffnen/schreiben operation durchzuführen

    du brauchst deine checkboxen übrigens nicht del[1-n] zu nennen... checkbox inhalte werden automatisch in ein array gepackt