Archief - Website eindwerk, help.

Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.
K

Kwintus5

Legacy Member
Hoihoi,

Ik moet een website maken als eindwerk voor mijn dagopleiding. Ik heb deze opleiding vooral gedaan uit interesse en merk dat ik het dus haat. :P

Nu is mijn vraag of jullie een handje kunnen helpen? Ik heb namelijk verschillende vragen.

Eerste vraag: ik heb een registratiepagina gemaakt en deze geeft een melding weer dat ie normaal niet mag weergeven. Indien ik deze in commentaar zet geeft ie gewoon de volgende melding weer. Ergens verkeerd afgesloten zeker? Maar ik vind het niet. :(

Hier mijn code:
PHP: 
 codekwintus5registratie - Pastebin.com[/url]

Ow ja, ik wil ook dat ze een foto uploaden als ze op registratie klikken, hoe doe ik dat?

Danku :)
g

gille

Legacy Member
Lekker specifiek? Een melding en ander een andere melding... Iets meer details graag, 't is uw huiswerk he.

Soit, ik gok dat hij neut over het wachtwoord? Ge gebruikt eerst $password en dan gaat ge plots $paswoord gebruiken, dus het ligt waarschijnlijk daar aan.
m

meuh

Legacy Member
Idd, komt waarschijnlijk door $pasword <> $password

Als ik u was, zou 'k mijn code wat beter structureren. Is nogal een chaos :/
s

shadowstep0705

Legacy Member
Gebruik mysql_real_escape_string() om je te beveiligen tegen sql injection
Code: 
$password = mysql_real_escape_string($password);
wordt dat dan, voor meer informatie: SQL-injectie - Wikipedia
[

[JK]

Legacy Member
Code: 
if ($password == $password2)
                {
                }
                        else
                        {
                        $meldingpaswoord = "De paswoorden zijn niet gelijk!";
                        }
Beter:
if ($password !== $password2)
                        {
                       $meldingpaswoord = "De paswoorden zijn niet gelijk!";
of
                        echo "De paswoorden zijn niet gelijk!";
                        }

ook dit:

if ($actie === "Aanmelden" or $actie === "Afmelden") {

moet zijn:

if ($actie == "Aanmelden" || $actie == "Afmelden") {


nog iets:

  $sql = "SELECT * FROM gebruikers WHERE email = '$emailadres';";

moet zijn:

  $sql = "SELECT * FROM gebruikers WHERE email = '$emailadres'";


Foto:

Code: 
<form enctype="multipart/form-data" id="form" name="form" method="post" action="<?php $_SERVER['PHP_SELF']; ?>">
 <input type="file" name="foto" id="foto">
<input name="toevoegen" type="submit" id="toevoegen" value="Toevoegen">
</form>

if(isset($_POST["toevoegen"]))
{
$huidig = getcwd();
$test = $huidig . "/foto";

if (file_exists($huidig . $_FILES["foto"]["name"]))
      {
      echo $_FILES["foto"]["name"] . " Bestaat al. ";
      }
    else
      {
		$name = $_FILES["foto"]["name"];  
      move_uploaded_file($_FILES["foto"]["tmp_name"], "$test/$name");
      echo "Geplaatst in: " . $test;

	 //Als je het in je db wilt
        $foto = "foto/".$name;
      }
$query="INSERT INTO tabel (foto) VALUES ('$foto')";
$result=mysql_query($query) or die ("fout:".mysql_error());
}

een voorbeeld van hoe het zou kunnen.
d

design1

Legacy Member
Hmm als je het haat, waarom doe je dan verder? Zoek mss iets wat wel binnen je interesse ligt?

Nuja snel de code bekeken, en is eerlijk gezegd nogal onoverzichtelijk.
Wat ik er kon uithalen:


1. Onderstaande schrijf je best anders, aangezien je niets doet als het ene wachtwoord gelijk is aan het andere ...
if ($password == $password2)
{
}
else
{
$meldingpaswoord = "De paswoorden zijn niet gelijk!";
}


2. variabele $paswoord bestaat niet, vermoedelijk een typfout?

if (strlen($paswoord) < 4)
{
$melding = "Uw paswoord moet uit minimaal 4 karakters!!!";
}


3. Onnodige commentaren best verwijderen, dan wordt het wat overzichtelijker.


4. Nog een extra hint: MD5 gebruiken om de wachtwoorden te coderen.

Veel succes verder.
K

Kwintus5

Legacy Member
Dankjewel alvast, ik ga het straks allemaal even uitproberen en laat het weten hoe het gaat. En ja, ik heb deze opleiding gekozen omdat ie interessant was, ik had nooit gedacht dat het zoveel programmeren was en dat het zo saai is. Ik wil dan ook gewoon mijn diploma halen. :p
a

adrianhates

Legacy Member
if($_SERVER['REQUEST_METHOD'] == "POST")

beats

if(isset($_POST['submit&#8217;])
v

voltje

Legacy Member
adrianhates zei:
if($_SERVER['REQUEST_METHOD'] == "POST")

beats

if(isset($_POST['submit’])
Klik om te vergroten...

Dus dat eerste gebruik je bij het clicken op een knop?
Nog nooit gebruikt, kan je eens verduidelijken waarom het beter is ? Dan weten we dat ook :D

Thanks !
K

Kwintus5

Legacy Member
Ik krijg "Er is een onbekende fout opgetreden. Controleer het e-mailadres en klik op registreer. "

Dat is de melding op lijn 77 in de code. Als ik die in de les eens in commentaar had gezet kreeg ik gewoon de volgende melding. :(
Z

Zir0h

Legacy Member
Als ik deze tread lees krijg ik het gevoel dat je deze code nieteens zelf hebt geschreven?

Anyway, controleer $_POST en vergelijk met:
PHP: 
$actie = $_POST['actie'];          // Uit te voeren actie bepalen
if ($actie === "Aanmelden" or $actie === "Afmelden") { ...

Eerlijk gezegd ziet het er niet goed uit voor uw eindwerk.
K

Kwintus5

Legacy Member
Zir0h zei:
Als ik deze tread lees krijg ik het gevoel dat je deze code nieteens zelf hebt geschreven?

Anyway, controleer $_POST en vergelijk met:
PHP: 
$actie = $_POST['actie'];          // Uit te voeren actie bepalen
if ($actie === "Aanmelden" or $actie === "Afmelden") { ...

Eerlijk gezegd ziet het er niet goed uit voor uw eindwerk.
Klik om te vergroten...

Dankje voor de moed, en nee idd, groot stuk van die code is overgenomen van de docent en dan aangepast. Blech.
Z

Zero Grav

Legacy Member
Het is niet de bedoeling dat andere mensen hier uw eindwerk voor u gaan maken, ook al is het niet uw ding. Mensen kunnen tips en hints geven waaruit gij kunt leren om op die manier te bereiken wat ge wilt op zelfstandige wijze. Zomaar wachten tot iedereen u de oplossing geeft is echter niet de bedoeling.
K

Kwintus5

Legacy Member
Ik dacht dat er wel nerds zouden bestaan die het toch leuk vinden om dit in hun vrije tijd wat te doen. Ach, we zien wel dan. Toch bedankt.
Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.
Terug
Bovenaan