Ein Newslöschscript

Nachdem ich nun ein halben Jahr PHP lerne, von ein paar Seiten und aus einem Buch wollte ich jetzt mahl alles ohne Buch oder sonstiges Nachschlage Werk machen und was ist draus geworden mein Script funktioniert nicht.
Wenn ich nun einen der Links anklicke um die News zu löschen, läd die Seite erst so wie sie soll, aber es kommt dann "Du hast hier keinen Zugriff", das versteh ich einfach nicht, warum ist die Variable die in der Session ist auf einmahl nicht mehr vorhanden?, beim News-schreibe-Script funktioniert alles.
Hier nun das Script:

Menü:
session_start();
include("menu.inc.php");
?>

if ($name==abcde) {
?>
Adminbereich
Nachrichten löschen:
session_start();
echo 'Ausloggen
';
if(isset($_GET["asd4sf5sdfe4wadfg"])) {
include("zugriff.inc.php");
$sql1="DELETE FROM `news` WHERE `id`='".$_GET['asd4sf5sdfe4wadfg']."' LIMIT 1;";
mysql_query($sql1);
}
include("zugriff.inc.php");
$sql1 = "SELECT * FROM `news`";
$result2 = mysql_query($sql1);
while ($row = mysql_fetch_assoc($result2)) {
echo "[ Eintrag löschen ] ";
foreach ($row as $name => $key) {
echo $key." ";
}
echo "
";
}
}
else {
echo "Du hast hier keinen Zugriff.";
}
?>

nimm mal alle session_start() aus dem script und setz ein session_start() wieder in die datei und zwar als allererste zeile - zum weiter durchgucken bin ich noch net gekommen aber das viel mir innerhalb von 2 sec auf

Sorry ich habe versucht da etwas Ordnung in das Script zu bringen und wenigstens erstmal die Dinger einzurücken, doch Sorry aber das Script ist ein einziger Fehler.. Falsch Klammern, fehlende Anführungszeichen, falsch gesetzter HTML-Code.. Tut mir leid aber die Bücher die du da hast.. wenn die dir sowas beibringen dann sollte ich mal eins schreiben und damit Geld verdienen.

Ich hab auch versucht das Script neu zu schreiben und da etwas Ordnung reinzubringen aber so ganz steige ich durch dein System da nicht durch..

am geilsten find ich

$_GET['asd4sf5sdfe4wadfg']

OK habs jetzt so halbwegs debuggt.
ein

taq existierit nicht. man schreibt ins div align=center rein.
session_start(); kommt nur einmal pro Seite und zwar ganz oben da es mit Cookies Probleme gibt wenn vorher schon Ausgabe kam. Man nennt Sachen eindeutig wie id und nicht einmal auf die Tastatur hauen.
In mysql_querys setzt man als 2tes Argument einen Link Identifier sonst kanns Probleme bei mehreren DBs geben. Includes setzt man genauso obenhin und net irgendwo außer die machen Ausgaben.
In deinem foreach bezeichnest du das value als key!?
$name wär der key. Bei Querys prüft man auch normalerweise nochmal ob sie geglückt sind.
Wo ist bei deinen Tabellen bitte das taq?
Und nie nie niemals irgendwelche Dateien mit Endung inc machen wie dus anscheinend schon versucht hast. Denn normalerweise wird die Endung nicht interpretiert und man sieht schön den Code. Bei deinem DB connect wär damit deine Seite schon in Hand böser Menschen

Ähm.. gibt es: http://de.selfhtml.org/html/text/bereiche.htm#block_zentriert

Ist das konform? Wenn ja sry.
Bin nur davon ausgegangen da hier mehrere Editoren dieses Taq nicht kennen.

Original von Gambler
Wo ist bei deinen Tabellen bitte das taq?

Das war nur der mittlere Teil der Tabelle, da fehlt noch einiges.

Vielen dank für die Hilfe, jetzt schreib ich meine Script schöner, bei mir im Editor war es schon eingerückt, ich hatte auch einen überblick, man merkt halt das ich noch ein Anfänger bin. Das mit dem .inc.php Dateien habe ich mir halt so angewöhnt, werd es ab jetzt nicht mehr machen.


Noch was

Original von Mudder

Sorry aber das Script ist ein einziger Fehler.. Falsch Klammern, fehlende Anführungszeichen, falsch gesetzter HTML-Code..
Die klammern waren richtig, da war keine zuviel oder zuwenig und der HTML-Code war doch auch richtig oder wo war er falsch? Man hätte es geschickter schreiben können, da will ich ja nichts gegen sagen, aber ein einziger Fehler wars doch wohl nicht, vielleicht in deinen Augen.

Also falsche Klammern oder Anführungszeichen hab ich jetzt nicht gesehn. Das war eher vom Grundprinzip falsch. Und überall nen Mix aus PHP und HTML sollte man auch nicht schreiben. Aber als Anfänger ist das schon ok
Du kannst die Dateien gerne noch .inc.php nennen nur war das ein Anzeichen für dass du vl auch welche nur .inc nennst. Das wäre unklug.

Also ich benutze immernoch ganz gerne... So aus gewohnheit!
Finde ich auch netter, wenn ich sowohl Bilder als auch Text in die Mitte stellen will... dafür brauche ich dann nur ein machen!
MfG