[Vorstellung]Homepage

Mikasuki · 29. März 2012, 22:01

Abend Leute.
Ich habe mich mal intensiv an meine Homepage rangesetzt und habe viel Zeit, Nerven und
ganz viel Liebe darein investiert.

Funktionalitäten:
Registrierung
Login
Logout
Passwort Ändern
Passwort vergessen
Umfrage
Grafische Auswertung in Form von Balkendiagramm der Umfrage
Produktsuche
Zufällige Wiedergabe der Banner
Zufällige Wiedergabe eines Produktes auf der Startseite
Zufällige Wiedergabe der Produkte unten in den Div-Boxen
Im News Link werden die Neuesten Spiele wiedergegeben
kleine Javascript funktionen
Homepage funktioniert einwandfrei
Evtl. könnten ein paar Sicherheitslücken vorhanden sein da ich mich mit der Sicherheit
nicht auseinander gesetzt habe
...
..
.

Bilder teil aus dem Internet und teils bearbeitet.
Quellcodes sind 100% von mir geschrieben.(Kein Copy and Paste oder sonst so ein Quatch)
Design auch von mir.
Eigentlich alles von mir außer Bilder und Beschreibungstexte für die Spiele.

Nebenbei muss ich sagen, das dies meine Anfangsphase ist. Das heißt ich habe erst vor ein paar Monaten
damit angefangen mich mit PHP, MYSQL auseinanderzusetzen.

Bewertet doch bitte meine Homepage
Würde mich über ein Feedback freuen.

Hier ein paar Screenshots:

mmo-core.de/attachment/2040/?s…3777ed14e5419607b4af8e815

mmo-core.de/attachment/2041/?s…3777ed14e5419607b4af8e815

mmo-core.de/attachment/2042/?s…3777ed14e5419607b4af8e815

mmo-core.de/attachment/2043/?s…3777ed14e5419607b4af8e815

mmo-core.de/attachment/2044/?s…3777ed14e5419607b4af8e815

mmo-core.de/attachment/2045/?s…3777ed14e5419607b4af8e815

mmo-core.de/attachment/2048/?s…3777ed14e5419607b4af8e815

mmo-core.de/attachment/2049/?s…3777ed14e5419607b4af8e815

MFG
Mikasuki :)

denninho · 29. März 2012, 22:15

Hei, ersteinmal Respekt für die Funktionalität, vor allem wenn du noch nicht so lange dabei bist!
Ich finde das Design, sagen wir mal durchwachsen.
1. Der Banner.
Gib dir mit dem Banner noch etwas Mühe. Die ersten 10 Sekunden entscheiden, ob man auf einer Website bleibt oder sie verlässt. Habe ich letztens wo gelesen, und ca. 4 Sekunden schaut man auf den Banner. Also setz dich da nimma ran, denn er ist dein Aushängeschild wenns um das Design geht.
2. Hintergrund
Ich finde die Farbe, muss ich leider sagen fast grausam. Ist natürlich jetzt mein Geschmack, aber ich finde die Farbe ist sehr unvorteilhaft.
3. Die Grafiken
Bau Mario und die Tussi mehr in den Hintergrund mit ein, das sieht so aufgesetzt aus, verstehste was ich meine? Am besten dezenter Hintergrund und die Grafiken irgendwie mit eingearbeitet.
4.Der Aufbau
Den Aufbau an sich finde ich ganz gut.
Alles klar strukturiert, gefällt mir eigentlich ganz gut.

Nich ein wenig Arbeit hier und da investieren und dan. Wird die Seite sicherlich sehr gut! ;)
gruß

Mikasuki · 29. März 2012, 22:23

denninho;288202 schrieb:

Hei, ersteinmal Respekt für die Funktionalität, vor allem wenn du noch nicht so lange dabei bist!
Ich finde das Design, sagen wir mal durchwachsen.
1. Der Banner.
Gib dir mit dem Banner noch etwas Mühe. Die ersten 10 Sekunden entscheiden, ob man auf einer Website bleibt oder sie verlässt. Habe ich letztens wo gelesen, und ca. 4 Sekunden schaut man auf den Banner. Also setz dich da nimma ran, denn er ist dein Aushängeschild wenns um das Design geht.
2. Hintergrund
Ich finde die Farbe, muss ich leider sagen fast grausam. Ist natürlich jetzt mein Geschmack, aber ich finde die Farbe ist sehr unvorteilhaft.
3. Die Grafiken
Bau Mario und die Tussi mehr in den Hintergrund mit ein, das sieht so aufgesetzt aus, verstehste was ich meine? Am besten dezenter Hintergrund und die Grafiken irgendwie mit eingearbeitet.
4.Der Aufbau
Den Aufbau an sich finde ich ganz gut.
Alles klar strukturiert, gefällt mir eigentlich ganz gut.

Nich ein wenig Arbeit hier und da investieren und dan. Wird die Seite sicherlich sehr gut! ;)
gruß

Danke für dein schnelles und ehrliches Feedback
Ich finde auch das die Grafiken schöner aussehen könnten. Nur bin ich leider nicht so gut im GFX Bereich. Aber ich hoffe das, dass noch wird. Noch ist nämlich kein Meister vom Himmel gefallen und nur Übung macht den Meister.

//EDIT
Wie kann man einen Beitrag liken? Bzw. Danken? :-)

//Mikasuki

denninho · 29. März 2012, 22:27

Links unten am Post. ;)

~Kevin~ · 29. März 2012, 22:53

Du hast noch nicht genug Beiträge um sich bei jemandem zu bedanken.

Aber mal davon abgesehen: Die oben aufgelisteten Funktionen sind, für deine Anfangsphase (wie du es nennst), ein großer Fortschritt und bestimmt auch nicht wenig Arbeit für dich gewesen. Respekt, dass du das hinbekommen hast!
Du sagst selber, dass du kein GFX - Künstler bist, also nehm ich dir das nicht soo schöne Design nicht übel. Könnte man vielleicht wen engagieren der damit arbeitet oder jemanden suchen, der dir ein kostenloses Design erstellt!
Wie genau der Code aussieht weiss man nicht, wenn er ordentlich ist hast du gute Arbeit geleistet. :)

Mikasuki · 29. März 2012, 23:42

~Kevin~;288207 schrieb:

Du hast noch nicht genug Beiträge um sich bei jemandem zu bedanken.

Aber mal davon abgesehen: Die oben aufgelisteten Funktionen sind, für deine Anfangsphase (wie du es nennst), ein großer Fortschritt und bestimmt auch nicht wenig Arbeit für dich gewesen. Respekt, dass du das hinbekommen hast!
Du sagst selber, dass du kein GFX - Künstler bist, also nehm ich dir das nicht soo schöne Design nicht übel. Könnte man vielleicht wen engagieren der damit arbeitet oder jemanden suchen, der dir ein kostenloses Design erstellt!
Wie genau der Code aussieht weiss man nicht, wenn er ordentlich ist hast du gute Arbeit geleistet. :)

Ich danke dir :-)

Ich poste mal meine registrierung.inc.php :)

Quellcode

<?php
include("connect.inc.php");
if(isset($_POST['suchen']))
{
include('suche.inc.php');
}
else
{
//Dient zur Überprüfung ob der Nickname oder die Email schon vorhanden sind
$nicknamen = array();
$emails = array();
$vorhanden = "SELECT Nickname, Email FROM user";
$ergebnis = mysql_query($vorhanden);
while($row = mysql_fetch_assoc($ergebnis))
{
$nicknamen[] = $row['Nickname'];
$emails[] = $row['Email'];
}
//Variablen übergeben falls die textfelder existieren
if(isset($_POST['Geschlecht']) && isset($_POST['Vorname']) && isset($_POST['Nachname']) && isset($_POST['Nickname'])
&& isset($_POST['Passwort']) && isset($_POST['Passwortwiederholung']) && isset($_POST['Strasse']) && isset($_POST['Hausnummer'])
&& isset($_POST['Plz']) && isset($_POST['Wohnort']) && isset($_POST['EMail']) && isset($_POST['Geburtstag']) && isset($_POST['Schluesselwort']))
{
$geschlecht = $_POST['Geschlecht'];
$vorname = $_POST['Vorname'];
$nachname = $_POST['Nachname'];
$nickname = $_POST['Nickname'];
$passwort = $_POST['Passwort'];
$passwort_wiederholung = $_POST['Passwortwiederholung'];
$strasse = $_POST['Strasse'];
$hausnummer = $_POST['Hausnummer'];
$plz = $_POST['Plz'];
$wohnort = $_POST['Wohnort'];
$email = $_POST['EMail'];
$geburtstag = $_POST['Geburtstag'];
$schluesselwort = $_POST['Schluesselwort'];
}
//Session Registrieren falls die variablen existieren
if(isset($_SESSION['Geschlecht']) && isset($_SESSION['Vorname']) && isset($_SESSION['Nachname']) && isset($_SESSION['Nickname']))
{
$_SESSION['Geschlecht'] = $geschlecht;
$_SESSION['Vorname'] = $vorname;
$_SESSION['Nachname'] = $nachname;
$_SESSION['Nickname'] = $nickname;
}
if(isset($_POST['registrieren']))
{
if(empty($geschlecht) || empty($vorname) || empty($nachname) ||
empty($nickname) || empty($passwort) || empty($passwort_wiederholung) ||
empty($strasse) || empty($hausnummer) || empty($plz) ||
empty($wohnort) || empty($geburtstag) || empty($email) ||
empty($schluesselwort))
{
echo "<p align='center'>Bitte füllen Sie alle Felder aus</p>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Vorname nur Buchstaben enthält
elseif(!preg_match('/^[a-zäöüA-ZÄÖÜ \/]*$/', trim($vorname)))
{
echo "<p align='center'>Benutzen Sie bitte nur Buchstaben für ihren Vornamen</p>";
echo "<style type=\"text/css\">#vorname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Vorname min. 3 Zeichen enthält
elseif(strlen(trim($vorname)) < 3)
{
echo "<p align='center'>Ihr Vorname muss min. 3 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#vorname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Nachname nur Buchstaben enthält
elseif(!preg_match('/^[a-zäöüA-ZÄÖÜ \/]*$/', trim($nachname)))
{
echo "<p align='center'>Benutzen Sie bitte nur Buchstaben für ihren Nachnamen</p>";
echo "<style type=\"text/css\">#nachname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Nachname min. 3 Zeichen enthält
elseif(strlen(trim($nachname)) < 3)
{
echo "<p align='center'>Ihr Nachname muss min. 3 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#nachname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Nickname min. 6 Zeichen enthält
elseif(strlen(trim($nickname)) < 6)
{
echo "<p align='center'>Ihr Nickname muss min. 6 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#nickname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Nickname nur Zahlen,Buchstaben oder Unterstriche enthält
elseif(!preg_match('/^\w+$/', trim($nickname)))
{
echo "<p align='center'>Benutzen Sie bitte nur alphanumerische Zeichen für ihren Nicknamen (Zahlen, Buchstaben und den Unterstrich)</p>";
echo "<style type=\"text/css\">#nickname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Nickname bereits vorhanden ist
elseif(in_array(trim($nickname), $nicknamen))
{
echo "<p align='center'>Dieser Nickname ist bereits vergeben</p>";
echo "<style type=\"text/css\">#nickname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob beide Passwörter übereinstimmen
elseif (trim($passwort) != trim($passwort_wiederholung))
{
echo "<p align='center'>Ihre Passwortwiederholung war nicht korrekt</p>";
echo "<style type=\"text/css\">#passwort { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob das Passwort min. 6 Zeichen enthält
elseif (strlen(trim($passwort)) < 6)
{
echo "<p align='center'>Ihr Passwort muss mindestens 6 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#passwort { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob die Strasse nur Buchstaben enthält
elseif(!preg_match('/^[a-zäöüA-ZÄÖÜ \/]*$/', trim($strasse)))
{
echo "<p align='center'>Benutzen Sie bitte nur Buchstaben für ihre Straße</p>";
echo "<style type=\"text/css\">#strasse { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Strasse min. 4 Zeichen enthält
elseif(strlen(trim($strasse)) < 4)
{
echo "<p align='center'>Ihre Straße muss min. 4 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#strasse { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob die Hausnummer nur Ziffern enthält
elseif(!is_numeric(substr($hausnummer, 0, 1)))
{
echo "<p align='center'>Benutzen Sie bitte nur Ziffern für ihre Hausnummer</p>";
echo "<style type=\"text/css\">#hausnummer { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob die PLZ nur Ziffern enthält
elseif(!is_numeric($plz))
{
echo "<p align='center'>Benutzen Sie bitte nur Ziffern für ihre PLZ</p>";
echo "<style type=\"text/css\">#plz { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob die PLZ min. 5 Zeichen enthält
elseif(strlen(trim($plz)) < 5)
{
echo "<p align='center'>Ihre PLZ muss 5 Zahlen lang sein!";
echo "<style type=\"text/css\">#plz { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Wohnort nur Buchstaben enthält
elseif(!preg_match('/^[a-zäöüA-ZÄÖÜ \/]*$/', trim($wohnort)))
{
echo "<p align='center'>Benutzen Sie bitte nur Buchstaben für ihren Wohnort</p>";
echo "<style type=\"text/css\">#wohnort { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Wohnort min. 3 Zeichen enthält
elseif(strlen(trim($wohnort)) < 3)
{
echo "<p align='center'>Ihr Wohnort muss min. 3 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#wohnort { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Nickname min. 3 Zeichen enthält
elseif(strlen(trim($nickname)) < 3)
{
echo "<p align='center'>Ihr Nickname muss min. 3 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#nickname { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob die Email Adressen Syntax richtig ist
elseif(!preg_match('§^[\w\.-]+@[\w\.-]+\.[\w]{2,4}$§', trim($email)))
{
echo "<p align='center'>Ihre Email Adresse hat eine falsche Syntax</p>";
echo "<style type=\"text/css\">#email { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob die Email Adresse bereits vorhanden ist
elseif(in_array(trim($email), $emails))
{
echo "<p align='center'>Diese Email-Adresse ist bereits vergeben</p>";
echo "<style type=\"text/css\">#email { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob das Geburtsdatum gültig ist
elseif(!check_date($geburtstag,"Ymd","."))
{
echo "<p align='center'>Ihr Geburtsdatum hat eine falsche Syntax</p>";
echo "<style type=\"text/css\">#geburtstag { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob der Nickname nur Zahlen,Buchstaben oder Unterstriche enthält
elseif(!preg_match('/^\w+$/', trim($schluesselwort)))
{
echo "<p align='center'>Benutzen Sie bitte nur alphanumerische Zeichen für ihr Schlüsselwort (Zahlen, Buchstaben und den Unterstrich)</p>";
echo "<style type=\"text/css\">#schluesselwort { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
//Überprüft ob das Schlüsselwort min. 4 Zeichen enthält
elseif(strlen(trim($schluesselwort)) < 4)
{
echo "<p align='center'>Ihre Schlüsselwort muss min. 4 Zeichen enthalten</p>";
echo "<style type=\"text/css\">#schluesselwort { border: 2px solid red; } </style>";
goto weiter;//Zur Sprungmarke
}
else
{
$sql = "INSERT INTO User (Geschlecht,
Vorname,
Nachname,
Nickname,
Passwort,
Strasse,
Hausnummer,
Plz,
Wohnort,
Geburtstag,
Email,
Schluesselwort,
Registrierungsdatum)
VALUES
('".mysql_real_escape_string(trim($geschlecht))."',
'".mysql_real_escape_string(trim($vorname))."',
'".mysql_real_escape_string(trim($nachname))."',
'".mysql_real_escape_string(trim($nickname))."',
'".md5(trim($passwort))."',
'".mysql_real_escape_string(trim($strasse))."',
'".mysql_real_escape_string(trim($hausnummer))."',
'".mysql_real_escape_string(trim($plz))."',
'".mysql_real_escape_string(trim($wohnort))."',
'".mysql_real_escape_string(trim($geburtstag))."',
'".mysql_real_escape_string(trim($email))."',
'".mysql_real_escape_string(trim($schluesselwort))."',
CURDATE())";
$result = mysql_query($sql);
if($result)
{
echo "<p align='center'>Sie haben sich erfolgreich Registriert!</p>";
echo "<script lang=\"javascript\">";
echo "document.Login.Nickname.focus();";
echo "</script>";
}
}//else insert to schließen
}//if isset schließen
else
{
weiter: //marke
if(empty($_SESSION['Nickname']))
{
?>
<form name="Registrieren" action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<br>
<table align="center">
<tr><td>Geschlecht:</td><td>
<select name ="Geschlecht">
<option value="M">Männlich</option>
<option value="W">Weiblich</option>
</select></td></tr>
<tr><td>Vorname:</td><td><input type="text" id="vorname" value="<?php if(isset($vorname)){ echo "".$vorname;} ?>" maxlength="30" name="Vorname"></td></tr>
<tr><td>Nachname:</td><td><input type="text" id="nachname" value="<?php if(isset($nachname)){ echo "".$nachname;} ?>" maxlength="30" name="Nachname"></td></tr>
<tr><td>Nickname:</td><td><input type="text" id="nickname" value="<?php if(isset($nickname)){ echo "".$nickname;} ?>" maxlength="30" name="Nickname"></td></tr>
<tr><td>Passwort:</td><td><input type="password" id="passwort" maxlength="32" name="Passwort"></td></tr>
<tr><td>Passwort bestätigen:</td><td><input type="password" id="passwort" maxlength="32" name="Passwortwiederholung"></td></tr>
<tr><td>Schlüsselwort*:</td><td><input type="text" id="schluesselwort" value="<?php if(isset($schluesselwort)){ echo "".$schluesselwort;} ?>" maxlength="30" name="Schluesselwort"></td></tr>
<tr><td>Strasse:</td><td><input type="text" id="strasse" value="<?php if(isset($strasse)){ echo "".$strasse;} ?>" maxlength="30" name="Strasse"></td></tr>
<tr><td>Hausnummer:</td><td><input type="text" id="hausnummer" value="<?php if(isset($hausnummer)){ echo "".$hausnummer;} ?>" maxlength="10" name="Hausnummer"></td></tr>
<tr><td>PLZ:</td><td><input type="text" id="plz" value="<?php if(isset($plz)){ echo "".$plz;} ?>" name="Plz" maxlength="5"></td></tr>
<tr><td>Wohnort:</td><td><input type="text" id="wohnort" value="<?php if(isset($wohnort)){ echo "".$wohnort;} ?>" maxlength="30" name="Wohnort"></td></tr>
<tr><td>Geburtstag:[JJJJ.MM.TT]</td><td><input id="geburtstag" type="text" value="<?php if(isset($geburtstag)){ echo "".$geburtstag;} ?>" name="Geburtstag"></td></tr>
<tr><td>Email:</td><td><input type="text" id="email" value="<?php if(isset($email)){ echo "".$email;} ?>" maxlength="70" name="EMail"></td></tr>
<tr><td><input type="submit" value="Daten Eintragen" name="registrieren"></td></tr>
<tr><td colspan = "2">*wird gebraucht falls man das Passwort vergisst</td></tr>
</table>
</form>
<?php
}//if empty session nickname
}//else von result schließen
}//Von dem If Suchen das Else schließen
?>

Alles anzeigen

Ich habe mir mühe gegeben immer schön einzurücken und alls schön mit Kommentaren zu versehen. :-)
Ich habe vor ca. 6 Monaten (Schule) mit PHP und MYSQL angefangen und es macht mir enorm viel Spaß in PHP zu coden. Ich habe ungefähr 20 - 30 Stunden hierfür investiert bis jetzt. Die hälfte der Zeit ging leider für die Fehlersuche flöten. Dennoch habe ich nicht aufgegeben und habe weitergemacht. Ich bin ziemlich stolz auf mich mit diesem "kleinen" Projekt. Ich weiß das es für die Profesionellen unter euch ein Klaks ist so eine Homepage zu erstellen. Aber für mich war es ein großes Werk :)

Zu dem bedanken -> Ich habe mich schon gewundert wieso der button fehlt. Lol

Auch dir vielen Dank für dein Feedback.
Ich würde eure Beiträge gerne "liken", kann es leider noch nicht.
MFG
Mikasuki

felix · 30. März 2012, 11:20

PHP-Quellcode

$vorhanden = "SELECT Nickname, Email FROM user";
$ergebnis = mysql_query($vorhanden);
while($row = mysql_fetch_assoc($ergebnis))
{
$nicknamen[] = $row['Nickname'];
$emails[] = $row['Email'];
}

dafuq?! Stell dir vor du hast 10.000 User, da kackt dein root doch ab :o

SQL-Abfrage

SELECT count(id) FROM user WHERE nickname = '". [align=left][COLOR=#0000BB][FONT=monospace]$_POST[/FONT][/COLOR][COLOR=#007700][FONT=monospace][[/FONT][/COLOR][COLOR=#DD0000][FONT=monospace]'Nickname'[/FONT][/COLOR][COLOR=#007700][FONT=monospace]][/FONT][/COLOR][/align]
."'

Oder mysql_num_rows nachlesen. Vorher möglichst escapen. Variablennamen etc. sollten
übrigens immer in Englisch sein.

Btw hast du nicht nur ein paar Sicherheitslücken, sondern deine Registration ist eigentlich
mehr Lücke als Script, da du allerdings wie du selbst sagst Anfänger bist ist das aber wohl
normal.

Dennoch habe ich nicht aufgegeben und habe weitergemacht.

Mein früherer Chef meinte mal zu mir Programmieren ist 20% Schreibarbeit und 80%
Fehlersuche. Das hört egal wie lange du dich mit einer Programmiersprache auseinandersetzt
wohl nie auf.

Ich weiß das es für die Profesionellen unter euch ein Klaks ist so eine Homepage zu erstellen.

Pass auf, sonst kommt der hier in dieser Community so hoch gefeierter Übercoder Sur*** und klaut
dir noch deinen Code. Würde mich wundern ob er sowas überhaupt selbst hinbekommt :o
Merk dir eins, in dieser Community gibt es keine Professionellen sondern nur hochgehypte
knowledgelose Kinder und ulle.

Mikasuki · 30. März 2012, 13:53

felix;288250 schrieb:

Btw hast du nicht nur ein paar Sicherheitslücken, sondern deine Registration ist eigentlich
mehr Lücke als Script, da du allerdings wie du selbst sagst Anfänger bist ist das aber wohl
normal.

Könntest du mir nennen wieso mein script aus mehr Lücken als aus das eigentliche Script besteht? :)

//Mikasuki

ulle · 30. März 2012, 16:27

@felix
soll ich das nun als Kompliment oder Beleidigung auffassen. Ich glaube meine arbeiten sprechen für sich und mich muss niemand hochloben. Ich weiß was ich kann und teile mein wissen gerne mit anderen. :) also bitte doch mal um Klärung deines Vorwurfes :p

Ansonsten sieht das System doch gut aus. Freut mich immervwieder wenn der Nachwuchs gefallen an der webentwicklung findet :)

Lalox3 · 30. März 2012, 17:21

Ganz abgesehen vom Code was du geschrieben hast.
Über Geschmack lässt sich ja bekanntlich streiten.
Das Design erinnert mich einwenig an Homepage
baukasten, was aber nicht sonderlich schlimm ist
wie ich gelesen habe bist du "Neuling". Was mich
noch Interessieren würde, wie lange du schon
am Lernen oder "können" bist?!

Aber die Funktionen und anderen möglichkeiten finde
ich schonmal sehr gut ;)

Greez

felix · 30. März 2012, 20:04

soll ich das nun als Kompliment oder Beleidigung auffassen. Ich glaube meine arbeiten sprechen für sich und mich muss niemand hochloben. Ich weiß was ich kann und teile mein wissen gerne mit anderen. :) also bitte doch mal um Klärung deines Vorwurfes :p

Ich meinte damit das du eine der wenigen Ausnahmen von der Regel bist.

Könntest du mir nennen wieso mein script aus mehr Lücken als aus das eigentliche Script besteht? :)

War natürlich übertreibung, ich meinte damit das man Post daten immer prüfen sollte, ich hab allerdings dein Script nur überflogen und garnicht registriert das du das bereits tust, von daher entschuldige ich mich für meine absolut falsche Aussage. Peinlich peinlich :)

Mikasuki · 30. März 2012, 21:30

Lalox3;288309 schrieb:

Ganz abgesehen vom Code was du geschrieben hast.
Über Geschmack lässt sich ja bekanntlich streiten.
Das Design erinnert mich einwenig an Homepage
baukasten, was aber nicht sonderlich schlimm ist
wie ich gelesen habe bist du "Neuling". Was mich
noch Interessieren würde, wie lange du schon
am Lernen oder "können" bist?!

Aber die Funktionen und anderen möglichkeiten finde
ich schonmal sehr gut ;)

Greez

dankeschön :) ne ist kein Homepage baukasten zeugs. Habe alles selber designed mit positionierten div boxen :)
Lerne PHP und MYSQL seit knapp einem halben Jahr in der Schule :-D

felix;288353 schrieb:

War natürlich übertreibung, ich meinte damit das man Post daten immer prüfen sollte, ich hab allerdings dein Script nur überflogen und garnicht registriert das du das bereits tust, von daher entschuldige ich mich für meine absolut falsche Aussage. Peinlich peinlich :)

:))

ulle · 30. März 2012, 21:33

Was meinst du mit positionierten Div Boxen?? Erkläre das mal bitte. Wenn es das ist was ich denke dann kann ich dir dabei helfen wie du es besser machen kannst.

Gruß
Ulle

Mikasuki · 30. März 2012, 21:38

ulle;288370 schrieb:

Was meinst du mit positionierten Div Boxen?? Erkläre das mal bitte. Wenn es das ist was ich denke dann kann ich dir dabei helfen wie du es besser machen kannst.

Gruß
Ulle

Ich habe in der CSS Datei z.B. #content { top:10px; left:30px; width:600px; .....} stehen und in in der Index.php <div id="content">Content goes here</div> Div Boxen habe ich quasi erstellt, die Größe festgelegt und positioniert. Verstanden? :-D

//Mikasuki

ulle · 30. März 2012, 21:45

als Tipp von mir, verzichte mal auf die Absoluten Angaben. Floate deine Elemente das geht bedeutend einfacher und ist auch X-Browser Kompatibel.

Mikasuki · 31. März 2012, 11:34

ulle;288374 schrieb:

als Tipp von mir, verzichte mal auf die Absoluten Angaben. Floate deine Elemente das geht bedeutend einfacher und ist auch X-Browser Kompatibel.

woher weißt du das ich absolute verwende??
Ich werde mich mal über's Floaten informieren. Vielen dank :-D

//Mikasuki

Well · 31. März 2012, 13:35

PHP-Quellcode

//Variablen übergeben falls die textfelder existieren
if(isset($_POST['Geschlecht']) && isset($_POST['Vorname']) && isset($_POST['Nachname']) && isset($_POST['Nickname'])
&& isset($_POST['Passwort']) && isset($_POST['Passwortwiederholung']) && isset($_POST['Strasse']) && isset($_POST['Hausnummer'])
&& isset($_POST['Plz']) && isset($_POST['Wohnort']) && isset($_POST['EMail']) && isset($_POST['Geburtstag']) && isset($_POST['Schluesselwort']))
{
$geschlecht = $_POST['Geschlecht'];
$vorname = $_POST['Vorname'];
$nachname = $_POST['Nachname'];
$nickname = $_POST['Nickname'];
$passwort = $_POST['Passwort'];
$passwort_wiederholung = $_POST['Passwortwiederholung'];
$strasse = $_POST['Strasse'];
$hausnummer = $_POST['Hausnummer'];
$plz = $_POST['Plz'];
$wohnort = $_POST['Wohnort'];
$email = $_POST['EMail'];
$geburtstag = $_POST['Geburtstag'];
$schluesselwort = $_POST['Schluesselwort'];
}

Alles anzeigen

Dafür würde ich lieber eine Funktion schreiben. Irgendwie so:

PHP-Quellcode

function getPostValue($name) {
return (isset($_POST[$name])) ? $_POST[$name] : null;
}
function getPostValue_2($name) {
if (isset($_POST[$name])) {
return $_POST[$name];
}
throw new RuntimeException('Post field '.$name.' does not exist.');
}

Denn das würde sich die ewig lange Bedingung vermeiden lassen.

Die Auswertung der Nutzerausgabe würde ich in sofern verändern, dass immer alle Fehler angezeigt werden (Folgefehler natürlich ausgenommen). So würde ich das als Nutzer ziemlich nervig finden.

Ferner sollte goto nicht verwendet werden und ich würde bei schließenden, geschweiften Klammern keinen Kommentar setzen. Wenn du ordentlich einrückst sieht man auch so ziemlich schnell, wo die Klammer aufgeht und aus welchem Grund.

ulle · 31. März 2012, 15:38

Mikasuki;288413 schrieb:

woher weißt du das ich absolute verwende??
Ich werde mich mal über's Floaten informieren. Vielen dank :-D

//Mikasuki

Wenn du Top und Left benutzt kann man davon ausgehen ;)

Mikasuki · 1. April 2012, 19:08

Well;288433 schrieb:

PHP-Quellcode

//Variablen übergeben falls die textfelder existieren

if(isset($_POST['Geschlecht']) && isset($_POST['Vorname']) && isset($_POST['Nachname']) && isset($_POST['Nickname'])

&& isset($_POST['Passwort']) && isset($_POST['Passwortwiederholung']) && isset($_POST['Strasse']) && isset($_POST['Hausnummer'])

&& isset($_POST['Plz']) && isset($_POST['Wohnort']) && isset($_POST['EMail']) && isset($_POST['Geburtstag']) && isset($_POST['Schluesselwort']))

{

$geschlecht = $_POST['Geschlecht'];

$vorname = $_POST['Vorname'];

$nachname = $_POST['Nachname'];

$nickname = $_POST['Nickname'];

$passwort = $_POST['Passwort'];

$passwort_wiederholung = $_POST['Passwortwiederholung'];

$strasse = $_POST['Strasse'];

$hausnummer = $_POST['Hausnummer'];

$plz = $_POST['Plz'];

$wohnort = $_POST['Wohnort'];

$email = $_POST['EMail'];

$geburtstag = $_POST['Geburtstag'];

$schluesselwort = $_POST['Schluesselwort'];

}

Alles anzeigen

Dafür würde ich lieber eine Funktion schreiben. Irgendwie so:

PHP-Quellcode

function getPostValue($name) {

return (isset($_POST[$name])) ? $_POST[$name] : null;

}

function getPostValue_2($name) {

if (isset($_POST[$name])) {

return $_POST[$name];

}

throw new RuntimeException('Post field '.$name.' does not exist.');

}

Denn das würde sich die ewig lange Bedingung vermeiden lassen.

Die Auswertung der Nutzerausgabe würde ich in sofern verändern, dass immer alle Fehler angezeigt werden (Folgefehler natürlich ausgenommen). So würde ich das als Nutzer ziemlich nervig finden.

Ferner sollte goto nicht verwendet werden und ich würde bei schließenden, geschweiften Klammern keinen Kommentar setzen. Wenn du ordentlich einrückst sieht man auch so ziemlich schnell, wo die Klammer aufgeht und aus welchem Grund.

Funktionen muss ich mir noch anschauen. :) Aber danke für den Tipp.

ulle;288455 schrieb:

Wenn du Top und Left benutzt kann man davon ausgehen ;)

Stimmt. Relative hätte ich aber auch verwenden können. :)

//Mikasuki

ulle · 1. April 2012, 20:26

Macht doch aber keinen Unterschied ob du absolute oder relative benutzt. Floate deine Elemente dann brauchst du das "nie" wieder machen für das "Layout".

Quellcode

PHP-Quellcode

SQL-Abfrage

PHP-Quellcode

PHP-Quellcode

PHP-Quellcode

PHP-Quellcode

Teilen