[Vorstellung]Homepage

Mikasuki

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:

Der Inhalt kann nicht angezeigt werden, da er nicht mehr verfügbar ist.

MFG
Mikasuki :)

denninho

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

Zitat von denninho;288202

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ß

Alles anzeigen

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

Links unten am Post. ;)

~Kevin~

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

Zitat von ~Kevin~;288207

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 :)

PHP

<?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&uuml;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&uuml;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&uuml;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&uuml;r ihre Stra&szlig;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&szlig;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&uuml;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&uuml;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&uuml;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&uuml;r ihr Schl&uuml;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&uuml;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&auml;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&auml;tigen:</td><td><input type="password" id="passwort" maxlength="32" name="Passwortwiederholung"></td></tr>
        <tr><td>Schl&uuml;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

PHP

$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

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.

Zitat

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.

Zitat

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

Zitat von felix;288250

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

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

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

Zitat

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.

Zitat

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

Zitat von Lalox3;288309

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

Alles anzeigen

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

Zitat von felix;288353

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

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

Zitat von ulle;288370

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

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

Mikasuki

Zitat von ulle;288374

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

PHP

//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

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.');
}

Alles anzeigen

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

Zitat von Mikasuki;288413

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

Zitat von Well;288433

PHP

//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

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.');
}

Alles anzeigen

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.

Zitat von ulle;288455

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

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

//Mikasuki

ulle

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".

[Vorstellung]Homepage

Jetzt mitmachen!

SCHNELLZUGRIFF

HILFE & COMMUNITY

RECHTLICHES