nicht angemeldet
Hallo Henning,
$db=mysql_connect("localhost","xxxxx-x","password");
Fehlerbehandlung fehlt.
mysql_select_db("xxxxx-x");
dito.
if(mysql_query($anfrage)){
echo"Tabelle wurde angelegt";
}
else
{
echo"Tabelle wurde erstellt";
wo ist der Unterschied zwischen angelegt und erstellt? und wo ist eine sinnvolle Fehlerbehandlung (mysql_error())?
<html>
Doctype fehlt.
<head>
<link href="style.css" rel="stylesheet" type="text/css">
</head>
title-Element fehlt
<form action="<?php PHP_SELF; ?>" method="post">
was steht in der Konstanten PHP_SELF?
if($sent or !$sent){
wo kommmt $sent her? - ich nehme mal an, du meinst $_POST['sent'], oder? (mal abgesehen davon, das es vollkommen egal ist, ob ein $sent true ist, oder nicht - die Bedingung ist immer erfüllt)
<table border="0" bgcolor="aaaaaa">
die Attribute lassen sich durch css ersetzen - aber warum überhaupt eine Tabelle?
if($auswahl=="Tabelle erstellen"){
du möchtest auch hier das Array $_POST verwenden.
if($auswahl=="Daten eintragen"){
if($auswahl=="Daten auslesen"){
if($auswahl=="Tabelle loeschen"){
arum kein switch?
Löschen PHP:
$db=mysql_connect("localhost","xxxxx-x","password");
mysql_select_db("xxxxx-x");
mysql_query($anfrage);
hier fehlt wieder die Fehlerbehandlung.
Eintragen PHP:
$db=mysql_connect("localhost","xxxxx-x","password");
mysql_select_db("xxxxx-x");
du weißt ja: Fehlerbehandlung.
$name=strip_tags($name);
$text=strip_tags($text);
ich würde nicht strip_tags verwenden, sondern htmlspecialchars (strip_tags entfernt evtl. zu viel - htmlspecialchars maskiert die zeichen nur - wobei ich die Daten zum Eintragen in die DB unverändert lassen würde (bis auf das behandeln mit mysql_escape_string()))
$datum=date("y,m,d");
$anfrage="INSERT INTO Tabellenname VALUES('','$name','$text','$datum')";
warum lässt du mysql das Datum nicht selbst eintragen (->NOW()) - ich hoffe doch, die 4. Spalte hat einen Datumstype? btw: du solltest immer alle Spaltennamen angeben (->http://dclp-faq.de/q/q-sql-select.html).
if(mysql_query($anfrage)){
echo"<a href='start.php'>Zurück</a><br><br>";
echo"Daten wurden eingetragen";
mail("me@world.net","Ein Eintrag in ihrem Gästebuch","$name hat am $datum $text geschrieben!");
Verwende *nie* Daten die von außen kommen ungeprüft in Funktionsaufrufen.
Lesen PHP:
$anfrage="SELECT * FROM Tabellenname";
"SELECT *" solltest du nicht verwenden (siehe Link oben)
echo"<textarea rows='6' colls='20'>$inhalt[2]</textarea>";
'cols' schreibt sich mit einem 'L' (in der Seite zum Eintragen auch) - außerdem ist:
echo "text $var text";
wesentlich langsamer als:
echo 'text '.$var.' text';
CSS Datei:
A:active {
color: #black;
hier ist ein '#' zu viel.
body {
font-family: Verdana;
hier fehlt eine generische Schriftfamilie.
font-size: 10px;
das ist keine geeignete Schriftgröße - verwende em oder %
tr {
font-family: Arial;
font-size: 9px;
nachdem tr keine Text enthalten darf, ist eine Schriftart- bzw. Schriftgrößenangabe imho überflüssig (bzw. es wäre sinnvoller td o.ä. als Selector zu verwenden.
textarea {
font-size:10;
10 was?
color: #000000;
border:1px #000000;
du solltest auch noch einen Wert für border-style angeben - der Standardwert ist nämlich 'none'.
Will später dann auch ein Tutorial auf meine Homepage legen, deshalb bitte diesen Thread nicht löschen. Danke!! :)
nein, Threads werden hier idR nicht gelöscht (sie wandern nur ins Archiv).
Grüße aus Nürnberg
Tobias
Selfcode: sh:( fo:) ch:? rl:( br:< n4:& ie:% mo:| va:) de:] zu:) fl:( ss:| ls:[ js:|