Problemen met Notice: Undefined index: A1 in ... on line 22

Door Kristof Bruyneel op 07-10-2019 18:23 gewijzigd op 07-10-2019 18:25

4.749 views

Ik ben net begonnen voor mijn hobby om PHP te leren en zou graag een enquête maken welke gelinkt is aan een database.
Wanneer ik echter een vraag oningevuld laat krijg ik steeds de melding 'Notice: Undefined index...'.

Ik begrijp dat dit betekent dat de vraag niet ingevuld is, maar dacht dit te laten melden door volgende code:

<?php
if (empty($Email) || empty($Voornaam) || empty($Achternaam) || empty($Geslacht) || empty($Postcode) || empty($A1) || empty($A2) || empty($A3) || empty($A4) || empty($A5) || empty($A6) || empty($A7) || empty($A8) || empty($A9) || empty($B1) || empty($B2) || empty($B3) || empty($B4) || empty($B5) || empty($B6) || empty($B7) || empty($B8) || empty($B9)) {
echo "Gelieve alle vragen in te vullen.";
?>

Het gevolg is nu dat de Notice meldingen nog steeds op de webpagina verschijnen mét daaronder mijn melding "Gelieve alle vragen in te vullen.'

Ik heb hieronder de code van mijn .php bestand geplaatst. Kan iemand me uitleggen wat ik verkeerd doe en hoe ik het beter kan doen? Alvast bedankt!!!

<?php

/* Onderstaande code maakt connectie met de database */

function Connect_DB () {
global $connection;
$connection = mysqli_connect ('localhost', 'root', '', 'DatabaseHobby'); /* 'Servernaam', 'gebruikersnaam', 'wachtwoord', 'databasenaam' */
if (!$connection) {
die ('connection failed' . mysqli_error());
}
}

/* Haalt alle gegevens uit het formulier en stockeert ze in database */

if (isset($_POST ['Submit'])){

$Email = $_POST['Email'];
$Voornaam = $_POST['Voornaam'];
$Achternaam = $_POST['Achternaam'];
$Geslacht = $_POST['Geslacht'];
$Postcode = $_POST['Postcode'];
$A1 = $_POST['A1'];
$A2 = $_POST['A2'];
$A3 = $_POST['A3'];
$A4 = $_POST['A4'];
$A5 = $_POST['A5'];
$A6 = $_POST['A6'];
$A7 = $_POST['A7'];
$A8 = $_POST['A8'];
$A9 = $_POST['A9'];
$B1 = $_POST['B1'];
$B2 = $_POST['B2'];
$B3 = $_POST['B3'];
$B4 = $_POST['B4'];
$B5 = $_POST['B5'];
$B6 = $_POST['B6'];
$B7 = $_POST['B7'];
$B8 = $_POST['B8'];
$B9 = $_POST['B9'];

if (empty($Email) || empty($Voornaam) || empty($Achternaam) || empty($Geslacht) || empty($Postcode) || empty($A1) || empty($A2) || empty($A3) || empty($A4) || empty($A5) || empty($A6) || empty($A7) || empty($A8) || empty($A9) || empty($B1) || empty($B2) || empty($B3) || empty($B4) || empty($B5) || empty($B6) || empty($B7) || empty($B8) || empty($B9)) {
echo "Gelieve alle vragen in te vullen.";
}

Connect_DB ();

$query = "INSERT INTO DatabaseTabel (Email, Voornaam, Achternaam, Geslacht, Postcode, A1, A2, A3, A4, A5, A6, A7, A8, A9, B1, B2, B3, B4, B5, B6, B7, B8, B9) "; /* Naam tabel (kolommen) */
$query .= "VALUES ('$Email','$Voornaam','$Achternaam','$Geslacht','$Postcode','$A1','$A2','$A3','$A4','$A5','$A6','$A7','$A8','$A9','$B1','$B2','$B3','$B4','$B5','$B6','$B7','$B8','$B9')"; /*ingevulde waarde */

$result = mysqli_query ($connection, $query);
if (!$result) {
die ('query failed' . mysqli_error());
}

}

?>

- SanThe -

08-10-2019 17:25

Als een checkbox niet is aangevinkt zit die ook niet in $_POST.

Michael -

08-10-2019 17:32 gewijzigd op 08-10-2019 20:35

Zoiets zou het moeten doen. Al verdient het nog geen schoonheidsprijs :-) Inderdaad controle of een veld gevuld is zoals Arien zegt. En de checkboxen op deze manier werkt niet. Waarschijnlijk bedoel je een "radio".

EDIT: Heb je de opmerking van SanThe eerder ook gelezen? geslacht vs Geslacht. Het is hoofdlettergevoelig. Dus als je een vrouw kies, krijg je ook een foutmelding.

<?php
if ($_SERVER['REQUEST_METHOD'] == "POST") {

//echo'<pre>';print_r($_POST);echo'</pre>';

if (!isset($_POST['Email'], $_POST['Voornaam'], $_POST['Achternaam'], $_POST['Geslacht'], $_POST['Postcode'], $_POST['A1'], $_POST['A2'], $_POST['A3'], $_POST['A4'], $_POST['A5'], $_POST['A6'], $_POST['A7'], $_POST['A8'], $_POST['A9'])) {
echo "Gelieve alle vragen te beantwoorden.";
} else {

$connection = mysqli_connect ('localhost', 'root', '', 'wolf'); /* 'Servernaam', 'gebruikersnaam', 'wachtwoord', 'databasenaam' */
if (!$connection) {
die ('connection failed');}

$query = "INSERT
INTO
klantentevredenheid_2019
(
Email,
Voornaam,
Achternaam,
Geslacht,
Postcode,
A1,
A2,
A3,
A4,
A5,
A6,
A7,
A8,
A9
)
VALUES
(
'".mysqli_real_escape_string($connection, $_POST['Email'])."',
'".mysqli_real_escape_string($connection, $_POST['Voornaam'])."',
'".mysqli_real_escape_string($connection, $_POST['Achternaam'])."',
'".mysqli_real_escape_string($connection, $_POST['Geslacht'])."',
'".mysqli_real_escape_string($connection, $_POST['Postcode'])."',
'".mysqli_real_escape_string($connection, $_POST['A1'])."',
'".mysqli_real_escape_string($connection, $_POST['A2'])."',
'".mysqli_real_escape_string($connection, $_POST['A3'])."',
'".mysqli_real_escape_string($connection, $_POST['A4'])."',
'".mysqli_real_escape_string($connection, $_POST['A5'])."',
'".mysqli_real_escape_string($connection, $_POST['A6'])."',
'".mysqli_real_escape_string($connection, $_POST['A7'])."',
'".mysqli_real_escape_string($connection, $_POST['A8'])."',
'".mysqli_real_escape_string($connection, $_POST['A9'])."'
)"; /*ingevulde waarde */

$result = mysqli_query ($connection, $query);
if (!$result) {
die ('query failed' . mysqli_error());
}

echo $query;

}
}

?>

Kristof Bruyneel

08-10-2019 17:35

- Ariën - op 08/10/2019 17:21:40

Laat je relevante code nu eens zien tussen code-tags? En heb je mijn validatie al toegepast?

let ook op SQL-injection!!!!

Laat je relevante code nu eens zien tussen code-tags? Wat bedoel je met relevante code?
Heb je mijn validatie al toegepast? Ik heb je check met <?php
if(trim($_POST['Email'])=="") {
echo "Is niet ingevuld";
}
?> toegepast en als ik niets invul krijg ik je boodschap "is niet ingevuld" en als ik wel iets invul krijg ik geen enkele boodschap.

SQL injectie: Waar in mijn query zet ik mysqli_real_escape_string?

Ik begrijp niet goed hoe ik die if-elseif-elseif-else constructie moet inbouwen?

[size=xsmall]Toevoeging op 08/10/2019 17:38:31:[/size]

Oei, ik zie dat er al antwoord geschreven is op mijn vragen toen ik mijn vragen naar Ariën nog aan het schrijven was. :)

Michael -

08-10-2019 17:40

Kristof Bruyneel op 08/10/2019 17:35:54

SQL injectie: Waar in mijn query zet ik mysqli_real_escape_string?

Zie mijn vorige post.

Kristof Bruyneel op 08/10/2019 17:35:54

Ik begrijp niet goed hoe ik die if-elseif-elseif-else constructie moet inbouwen?

Je kunt dit op meerdere manieren doen. Zoals je nu hebt geef je een foutmelding "Niet alles ingevuld"
Je kunt ook zeggen "Voornaam niet ingevuld" "Achternaam niet ingevuld" etc.

<?php
if(trim($_POST['Email'])=="") {
echo "E-mailadres is niet ingevuld";
}
if(trim($_POST['Voornaam'])=="") {
echo "Voornaam is niet ingevuld";
}
?>
etc.
Je kunt dit ook met else-if maar dan krijg je de foutmeldingen een voor een.

- Ariën -

08-10-2019 17:45 gewijzigd op 08-10-2019 17:47

<?php
if (isset($_POST['Email'], $_POST['Voornaam'], $_POST['Achternaam'], $_POST['Geslacht'], $_POST['Postcode'], $_POST['A1'], $_POST['A2'], $_POST['A3'], $_POST['A4'], $_POST['A5'], $_POST['A6'], $_POST['A7'], $_POST['A8'], $_POST['A9'])) {
// Alle velden bestaan, start de validatie

// Simpele validatie
if(trim($_POST['Voornaam'])=="") {
echo "Voornaam is niet ingevuld<br>";
} elseif(trim($_POST['Email'])=="") {
echo "Email is niet ingevuld<br>";
} else {
// Er wordt aan alle validatie-controles voldaan. Voer hier je query uit, en toon een melding dat het gelukt is.
}

}
?>

Of nog beter: Sla ze op in een array, en lees de array uit en kijk of de error-array niet gevuld is (count($array)==0) en voer alles uit. Dan kan je in één klap alle foutmeldingen op een rij tonen, en eventueel in je formulier verwerken, door velden te laten oplichten.

Thomas van den Heuvel

08-10-2019 17:47 gewijzigd op 08-10-2019 17:50

Twee dingen.

#1
Als die vragen A1 t/m B9 allemaal soortgelijke HTML produceren, wordt het dan niet eens tijd voor een arraytje in PHP die alle HTML genereert in een enkele of dubbele loop?

#2
Voordat je alles in de database probeert te frommelen en je applicatie vervolgens ontploft loont het de moeite om $_POST en de opgebouwde query eerst eens te inspecteren. Dan kun je namelijk precies zien waar er dingen foutgaan. Dump deze data eens naar je scherm. Dit heet ook wel debuggen. Nu ben je tegen een trits foutmeldingen aan het staren waarvan je niet weet waar deze precies vandaan komen. Het wordt dan (hoog) tijd dat je al je data gaat inspecteren voordat je er iets mee probeert te doen.

En ja, je formulierdata eerst valideren voordat je die ergens in probeert te duwen loont wellicht ook de moeite. Anders past iets mogelijk op voorhand al niet...

Kristof Bruyneel

08-10-2019 17:52

Michael - op 08/10/2019 17:32:24

Zoiets zou het moeten doen. Al verdient het nog geen schoonheidsprijs :-) Inderdaad controle of een veld gevuld is zoals Arien zegt. En de checkboxen op deze manier werkt niet. Waarschijnlijk bedoel je een "radio".

EDIT: Heb je de opmerking van SanThe eerder ook gelezen? geslacht vs Geslacht. Het is hoofdlettergevoelig. Dus als je een vrouw kies, krijg je ook een foutmelding.

<?php
if ($_SERVER['REQUEST_METHOD'] == "POST") {

//echo'<pre>';print_r($_POST);echo'</pre>';

if (!isset($_POST['Email'], $_POST['Voornaam'], $_POST['Achternaam'], $_POST['Geslacht'], $_POST['Postcode'], $_POST['A1'], $_POST['A2'], $_POST['A3'], $_POST['A4'], $_POST['A5'], $_POST['A6'], $_POST['A7'], $_POST['A8'], $_POST['A9'])) {
echo "Gelieve alle vragen te beantwoorden.";
} else {

$connection = mysqli_connect ('localhost', 'root', '', 'wolf'); /* 'Servernaam', 'gebruikersnaam', 'wachtwoord', 'databasenaam' */
if (!$connection) {
die ('connection failed');}

$query = "INSERT
INTO
klantentevredenheid_2019
(
Email,
Voornaam,
Achternaam,
Geslacht,
Postcode,
A1,
A2,
A3,
A4,
A5,
A6,
A7,
A8,
A9
) /* Naam tabel (kolommen) */
VALUES
(
'".mysqli_real_escape_string($connection, $_POST['Email'])."',
'".mysqli_real_escape_string($connection, $_POST['Voornaam'])."',
'".mysqli_real_escape_string($connection, $_POST['Achternaam'])."',
'".mysqli_real_escape_string($connection, $_POST['Geslacht'])."',
'".mysqli_real_escape_string($connection, $_POST['Postcode'])."',
'".mysqli_real_escape_string($connection, $_POST['A1'])."',
'".mysqli_real_escape_string($connection, $_POST['A2'])."',
'".mysqli_real_escape_string($connection, $_POST['A3'])."',
'".mysqli_real_escape_string($connection, $_POST['A4'])."',
'".mysqli_real_escape_string($connection, $_POST['A5'])."',
'".mysqli_real_escape_string($connection, $_POST['A6'])."',
'".mysqli_real_escape_string($connection, $_POST['A7'])."',
'".mysqli_real_escape_string($connection, $_POST['A8'])."',
'".mysqli_real_escape_string($connection, $_POST['A9'])."'
)"; /*ingevulde waarde */

$result = mysqli_query ($connection, $query);
if (!$result) {
die ('query failed' . mysqli_error());
}

echo $query;

}
}

?>

Ik heb je code hier bovenstaand geprobeerd maar ik krijg dan geen reactie, niet als ik iets niet invul maar ook niet als ik alles invul.

Wbt de opmerking over de checkbox en radio: ik heb dit reeds aangepast in mijn html/css file :)

Wbt volgende codes
<?php
if(trim($_POST['Email'])=="") {
echo "E-mailadres is niet ingevuld";
}
if(trim($_POST['Voornaam'])=="") {
echo "Voornaam is niet ingevuld";
}
?>

zet ik deze boven de if (!isset($_POST['Email'], $_POST['Voornaam'], $_POST['Achternaam'], $_POST['Geslacht'], $_POST['Postcode'], $_POST['A1'], $_POST['A2'], $_POST['A3'], $_POST['A4'], $_POST['A5'], $_POST['A6'], $_POST['A7'], $_POST['A8'], $_POST['A9'])) {
echo "Gelieve alle vragen te beantwoorden.";
}?

- Ariën -

08-10-2019 17:55

je wilt liever controleren of de variabelen bestaan, en dit niet met een ! voor je isset() ontkennen.
Daarna kan je doe en laten wat je wilt.

Trouwens, je weet wat // doet? ;-)

Michael -

08-10-2019 17:55

Ik denk dat je dit /* Naam tabel (kolommen) */ even moet weghalen op regel 32. foutje.

"zet ik deze boven de ..?"
Is al beantwoord door Arien.

Kristof Bruyneel

08-10-2019 18:07

Michael - op 08/10/2019 17:55:46

Ik denk dat je dit /* Naam tabel (kolommen) */ even moet weghalen op regel 32. foutje.

"zet ik deze boven de ..?"
Is al beantwoord door Arien.

Ik heb je code gecorrigeerd volgens je instructies en krijg nu idd al mijn data in mijn databank, maar krijg bovenaan de melding:
INSERT INTO klantentevredenheid_2019 ( Email, Voornaam, Achternaam, Geslacht, Postcode, A1, A2, A3, A4, A5, A6, A7, A8, A9 ) VALUES ( '[email protected]', 'Kristof', 'Bruyneel', 'Man', '9032', 'Niet tevreden', 'Slecht', 'Niet', 'Reclame op de camionetten', 'Niet', 'Niet', 'Niet tevreden', 'Niet tevreden', 'fdsgw' )

Als ik één vraag niet invul krijg ik de gewenste boodschap 'Gelieve alle vragen in te vullen.' :)

[size=xsmall]Toevoeging op 08/10/2019 18:08:42:[/size]

- Ariën - op 08/10/2019 17:55:25

je wilt liever controleren of de variabelen bestaan, en dit niet met een ! voor je isset() ontkennen.
Daarna kan je doe en laten wat je wilt.

Trouwens, je weet wat // doet? ;-)

// comments in PHP voor één enkele lijn
/* comment voor meerdere lijnen */

:)

Reageren

Inloggen om te reageren