Ik heb geen idee hoe je het nu doet zodat je errors krijgt. Maar een check lijkt mij wel minimaal.
Link gekopieerd
Bij alle chechboxen die niet ingevuld zijn krijg ik deze error:
Link gekopieerd
Je zal elke checkbox moeten checken met isset().
Link gekopieerd
Hmm, het werkt nog steeds niet zie hier mijn code:
Link gekopieerd
Post alléén relevante code en geen compleet script.
Link gekopieerd
Ik zou het anders doen. Ik zou van 'huis', 'uitzicht', 'toiletten', e.d. arrays maken, waar de gekozen opties in zitten. Dat gaat denk ik veel makkelijker. Dan denk ik ook dat je datamodel niet meer klopt, je zult alles nog eens over moeten denken, en het beter in moeten delen. Soort bij soort, categorie bij categorie. Ga [google]normaliseren[/google] ; ), zo blijft het enorm veel werk, te veel werk.
In $_POST maak je zo arrays:
<input type="checkbox" name="huis[]" value="Vrijstaand" />
<input type="checkbox" name="huis[]" value="Twee-onder-een-kap" />
<input type="checkbox" name="uitzicht[]" value="Zeezicht" />
<input type="checkbox" name="uitzicht[]" value="Duinzicht" />
enzovoorts...
Link gekopieerd
Waarom heb je zo een super lange code? Je kan dit toch veel makkelijker via een array en for each doen?
edit: van 340 naar 26 regels. Je moet wel zelf de array nog helemaal invullen:
<?php
include("config.php");
include("checksession.php");
$huisnaam = $_SESSION['huisnaam'];
$waardering_accommodatie = $_POST['waardering'];
$waardering_situering = $_POST['waardering_situering'];
$checkbox_namen = array('kamer."','kamer2."','kamer_pensioen');
foreach($checkbox_namen AS $checkbox)
{
if(isset($_POST[$checkbox])) {
${$checkbox} = $_POST[$checkbox];
}
else {
$kamer = "";
}
}
$query = mysql_query("UPDATE accommodaties SET waardering_accommodatie = '".$waardering_accommodatie."', waardering_situering = '".$waardering_situering."', 'kamer = '".$kamer."', kamer2 = '".$kamer2."', kamer_pension = '".$kamer_pension."', kamer_pension_2 = '".$kamer_pension_2."', kamer_in_bb = '".$kamer_in_bb."', kamer_in_bb2 = '".$kamer_in_bb2."', kamer_in_particulier = '".$kamer_in_particulier."', kamer_in_particulier_2 = '".$kamer_in_particulier_2."', kamer_in_hotel = '".$kamer_in_hotel."', kamer_in_hotel2 = '".$kamer_in_hotel2."', rijtjes_huis = '".$rijtjes_huis."', hoekwoning = '".$hoekwoning."', vrijstaand_huis = '".$vrijstaand_huis."', twee_onder_een_kap = '".$twee_onder_een_kap."', zomerhuis = '".$zomerhuis."', studio_in = '".$studio_in."', studio_in2 = '".$studio_in2."', studio_in3 = '".$studio_in3."', appartement_in = '".$appartement_in."', appartement_in2 = '".$appartement_in2."', appartement_in3 = '".$appartement_in3."', anders = '".$anders."', anders2 = '".$anders2."', accommodaties_in_pand = '".$accommodaties_in_pand."', entree = '".$entree."', aantal_personen = '".$aantal_personen."', roken = '".$roken."', honden = '".$honden."', aantal_woonlagen = '".$aantal_woonlagen."', aantal_woonlagen2 = '".$aantal_woonlagen2."', woonoppervlakte = '".$woonoppervlakte."', woonoppervlakt2 = '".$woonoppervlakt2."', grondoppervlakte = '".$grondoppervlakte."', grondoppervlakte2 = '".$grondoppervlakte2."', balkon_oppervlakte = '".$balkon_oppervlakte."', balkon_oppervlakte2 = '".$balkon_oppervlakte2."', aantal_kamers = '".$aantal_kamers."', aantal_kamer2 = '".$aantal_kamer2."', aantal_toiletten = '".$aantal_toiletten."', aantal_toiletten2 = '".$aantal_toiletten2."', drukte = '".$drukte."', uitzicht_woningen = '".$uitzicht_woningen."', uitzicht_duinen = '".$uitzicht_duinen."', uitzicht_zee = '".$uitzicht_zee."', uitzicht_plein = '".$uitzicht_plein."', uitzicht_terras = '".$uitzicht_terras."', uitzicht_tuin = '".$uitzicht_tuin."', uitzicht_bomen = '".$uitzicht_bomen."', uitzicht_straat = '".$uitzicht_straat."', uitzicht_zwembad = '".$uitzicht_zwembad."', uitzicht_jachthaven = '".$uitzicht_jachthaven."', uitzicht_dorp = '".$uitzicht_dorp."', uitzicht_anders = '".$uitzicht_anders."', uitzicht_anders2 = '".$uitzicht_anders2' WHERE huisnaam = '".$huisnaam'")
or die(mysql_error());
echo "ok";
?>
Link gekopieerd
En waar is dit voor:
$waardering_accommodatie = $_POST['waardering'];
$waardering_situering = $_POST['waardering_situering'];
Als $_POST['waardering'] niet geset is, krijg je een notice.
Link gekopieerd
Waarom wordt er niet gewoon een HTML index gecreeerd en die gewoon verwerkt? Naampjes van de index hetzelfde noemen als de velden in de DB.
Komt er dus op neer dat je dan alleen de relevante data verwerkt....
<?php
if (count($_POST) > 0)
{
$updatesql = array();
foreach ($_POST['checkbox'] AS $field => $value) $updatesql[] = "`" . mysql_escape_string($field)."`='" . mysql_escape_string($value) . "'";
$sql = "UPDATE `tablename` SET " . implode(",",$updatesql) . " WHERE `huisnaam`='" . mysql_escape_string($_SESSION['huisnaam']) . "'";
// Verwerk sql
}
?>
<form method="post" action="">
....
<input type="checkbox" name="checkbox[uitzicht_snackbar]" value="1" />
<input type="checkbox" name="checkbox[uitzicht_mcdonalds]" value="1" />
<input type="checkbox" name="checkbox[uitzicht_hoeren]" value="1" />
....
</form>
Iets in deze geest
Nog iets waar ik me mateloos aan irriteer:
EN GEBRUIK ALLEMAAL EENS EEN ESCAPE FUNCTIE Excuses voor de caps, maar de meeste fouten ontstaan hierdoor!!!! Niemand op dit forum gebruikt het!! Leer het jezelf aan ALTIJD een escape functie te gebruiken ookal komt de parameter van jezelf!!!!
Link gekopieerd
Leon schreef op 11.09.2009 16:13
EN GEBRUIK ALLEMAAL EENS EEN ESCAPE FUNCTIE Excuses voor de caps, maar de meeste fouten ontstaan hierdoor!!!! Niemand op dit forum gebruikt het!! Leer het jezelf aan ALTIJD een escape functie te gebruiken ookal komt de parameter van jezelf!!!!
Iets "irriteert" je, of je "ergert je aan". "Ik irriteer me aan" is fout Nederlands.
Verder zou ik zeggen: geef het goede voorbeeld in je eigen scripts. Je kunt niet na pas dertien posts stellen dat "niemand" op dit forum het gebruikt. En al die uitroeptekens staan bijzonder schreeuwerig.
Link gekopieerd
