Admin Interface Code

tuFx

Well-known member
ID: 292318
L
3 April 2007
241
11
PHP:
<?php
$blog_title="Admin Bereich":
include("https://")
include("#_funcitons.inc");
$titel = PostVar("titel");
$beitrag = PostVar("beitrag");
$rubrik = PostVar("rubrik");
$link = PostVar("link");
$url= PostVar("url");
$status= $_POST['status'];
$datum = datum("now")
if(!empty($url)) && empty($link) $link ="Quelle";
if (!empty($status)) {
if (empty($titel)) $fehler.= "Kein Titel vorhanden.</br>";
if (empty($beitrag)) $fehler.="Kein Beitrag vorhanden.</br>";
if (empty($rubrik)) $fehler.="Es wurde keine Kategorie angegeben.<br>";
if (!empty($link) && empty($url)) $fehler.= "Link ohne URL.<br>";
                     }
?>
<html>
<head>
<title><?=$blog_title?></title>
<link rel="stylesheet" type="text/css" href="style.css" />
</head>
<body>
<div align="center">
<table width="500" id="container">
<tr>
<td id="blog" colspan="2">
<?
if ($fehler) echo "<p>$fehler</p>";
elseif ($status =="Vorschau") {
echo "<p>";
 }
?>
<small><?= $datum ?></small><br>
<big><?=format($title)?></big><br>
<?= format($beitrag)?><br>
<small>Rubrik: <?= format($rubrik)?></small><br>
<?
if (!empty($url)) {
?>
<small><a href="<?=$url?>"><?=format($link)?></a></small>
<?
}
echo "</p>";
}
elseif ($status =="Eintragen") {
$query= "INSERT INTO blog SET
datum=now(),
titel='"addslashes($titel)."'.
beitrag='".addshlashes($beitrag)."',
rubrik='".addshlashes($rubrik)."',
link='".addshlashes($link)."',
url='".addshlashes($url)."'
";
$sql = mysql_query($query)
 or die (mysql_error());
unset($status,$titel,$beitrag,$rubrik,$link,$url);
}
?>
</td>
</tr>
<tr>
<td>
<form method="post" action="admin.php">
<table>
<tr>
<td><small><b>Titel</b></small></td>
<td><input type="text" name="titel" value="<?htmlentities($titel)?> 
style="width:380px;></td>
</tr>
<tr>
<td><small><b>Post</b></small></td>
<td><textarea name="beitrag" style="width:380px;height:200px;> 
<?htmlentities($beitrag)?> </textarea>
</td>
</tr>
<tr>
<td><small><b>Kategorie</b></small></td>
<td><input type="text" name="rubrik" value="<?htmlentities($rubrik)?> 
style="width:380px;></td>
</tr>
<tr>
<td><small><b>Link-Titel</b></small></td>
<td><input type="text" name="link" value="<?htmlentities($link)?> 
style="width:380px;></td>
</tr>
<tr>
<td><small><b>Link-Adresse</b></small></td>
<td><input type="text" name="url" value="<?htmlentities($url)?> 
style="width:380px;></td>
</tr>
<tr>
<td></td>
<td>
<input type="submit" name="status" value="Vorschau">
<input type="submit" name="status" value="Absenden">
</td>
</tr>
</table>
</form>
</td>
</tr>
</table>
</div>
</body>
</html>

Ich habe das Admin Interface gecodet, wo sind Fehler, was kann man verbessern?

Würde mich freuen, wenn jemand den Code ein bisschen "verbessern" würde.
 
In der zweiten Zeile steht ein Doppelpunkt statt einem Semikolon, in der dritten Zeile fehlt das Semikolon ganz.
Eine Frage noch: Warum inkludierst du "https://"?
 
Du hast nicht mal die Fehler ausgebessert, die ich dir schon genannt hab :roll:
theHacker schrieb:
  • Zeile 10: (PHP) Inkonsistenz des Codes; einmal PostVar(), einmal $_POST[]
  • Zeilen 12 und 17: (PHP) Copy&Paste ? Guck dir mal an, wie die Klammern gesetzt sind ? So korrekt ?
  • Zeile 14: (PHP) $fehler ist nicht gesetzt, ".=" führt zu E_NOTICE
  • Zeile 20: (HTML) DOCTYPE fehlt
  • Zeile 33, 46: (HTML) Bedingungen zum gleichzeitigen Öffnen und Schließen des Paragraph-Tags sind nicht gleich
  • Zeile 49-56: (MySQL) INSERT INTO...SET hab ich mal gehört, benutzt man afair nicht gerne
  • Zeile 69, 74, ...: (HTML) Für Kopfzeilen in der Tabelle muss man TableHead(<th>)- nicht TableData(<td>)-Tags benutzen
  • Zeile 75: (CSS) Syntaxfehler nach "height" fehlt der Doppelpunkt
  • Zeilen 70, 76, ...: (HTML) kritischer Syntaxfehler: value- und style-Attribute nicht geschlossen
  • Zeilen 70, 76, ...: (PHP) Syntaxfehler ";" fehlt. Wenn du die Kurzschreibweise für Ausgabe benutzen willst, fehlt das =-Zeichen