[PHP] Editor-Script zum selber basteln

**Lucleonhart** · 27.10.2006 08:29

Zitat von mitaki

PHP-Code:


file_put_contents('../$script_name/index_'.$script_name.'.txt', $_POST['text']);

Wie war das mit dem "schlechten Stil"? Ausserdem sind die Dateien im "scripts" Ordner. Also:

PHP-Code:


file_put_contents('scripts/'.$script_name.'/index_'.$script_name.'.txt', $_POST['text']);

Ausserdem funzt das ganze so überhaupt nicht.

Erstmal ist da eine else Verzweigung drin, die sich auf gar kein if bezieht, den block

PHP-Code:


}
else
{
echo "<p><strong>Der Scriptname ist falsch.</strong></p>";
}

Können wir also erstmal ganz rausschmeissen.
Ausserdem hat die Datei-Auswahl-Form ein seeehr merkwürdigen action tag. Warum willst du den da was mit Get dranhängen? Die schickst die variablen doch per submit button ab! Dazu muss aber auch das select Feld den richtigen Namen haben. Auch solltest du drauf achten, wann du welches Tag schließt. Wenn du erst das Formular öffnest, und dann eine Tabelle machst, solltest du auch erst die Tabelle beenden, und dann das Formular. Des weiteren beendet die else schleife die ausgabe bevor die
Tabelle geschlossen ist.. Alles etwas durcheinander. Insgesamt also:

PHP-Code:


<form action="editor.php" method="post">
<table>
<tr>
  <td style="padding:2px"><strong>Script-Name:</strong></td>
  <td style="padding:2px"><select id="auswahl" name="script_name"><option>news</option><option>guestbook</option></select></td>
</tr>
  <tr>
  <td align="center"><input name="Submit" type="submit" value="Enter" /></td>
</tr>
</table>
</form>

Befolge Mitaki's Rat und benutz Einrückungen, sonst blivkste da garnicht mehr durch!

Ein letztes ist, das wenn man den geänderten text speichern will, er wieder zur Auswahl springt, da die $script_name Variable nicht wieder mit übergeben wird. Das habe ich durch ein hidden Feld gelöst.

Nun der funzende Code: (mitaki hat sicher noch ein paar verbesserungsvorschläge! *XD*)

PHP-Code:


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html>
<head>
    <title>Mein Script-Editor</title>
</head>
<body>
<?php

error_reporting(E_ALL);

if (isset($_POST['script_name']))
{
$script_name = $_POST['script_name'];

    if (isset($_POST['text']))
        file_put_contents('scripts/'.$script_name.'/index_'.$script_name.'.txt', $_POST['text']);

?>
<form method="post" action="editor.php" name="source_edit" id="formular">
<input type="hidden" name="script_name" value="<?php echo ($script_name); ?>">
    <table>
        <tr>
            <td style="padding:2px"><strong><?php echo $script_name; ?>.txt Inhalt:</strong></td>
        </tr>
        <tr>
            <td style="padding:2px">
                <textarea cols="140" rows="15" name="text"><?php echo file_get_contents('scripts/'.$script_name.'/index_'.$script_name.'.txt'); ?></textarea>
            </td>
        </tr>
        <tr>
            <td align="center" style="padding:2px">
                <input name="submit" type="submit" id="speichern" value="Speichern" />
                <input name="loeschen" type="reset" id="loeschen" value="Zur&uuml;cksetzen" />
                <input name="sbm" type="submit" value="Close" onClick="self.close()" />
            </td>
       </tr>
    </table>
</form>
<?php  
}
else
{
?>
<form action="editor.php" method="post">
    <table>
        <tr>
            <td style="padding:2px"><strong>Script-Name:</strong></td>
            <td style="padding:2px"><select id="auswahl" name="script_name"><option>news</option><option>guestbook</option></select></td>
        </tr>
        <tr>
            <td align="center"><input name="Submit" type="submit" value="Enter" /></td>
        </tr>
    </table>
</form>
<?
}
?>
</body>
</html>

Thema: [PHP] Editor-Script zum selber basteln

Themen-Optionen

Anzeige

Baum-Darstellung

Berechtigungen