Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP & SQL

register sluiten

Appie_Dijk - 14/01/2006 13:17

Lid

Hallo iedereen,

Wat is er fout aan de volgende code:

php code - Bekijk de code zonder highlighting - Klap code in

<?php
include('beveiliging.php');
include ("config.php");
?>
<?php
$id = $_GET['id'];
$select = "SELECT * FROM register WHERE id = '".$_GET['id']."'";

$query = mysql_query($select)or die(mysql_error());
$lijst = mysql_fetch_object($query);
$bestaat = mysql_num_rows($query);

if ($bestaat == 0) {
        echo "Dit register bestaat niet (meer)!<br>";
}

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

mysql_query("UPDATE register SET open='' WHERE id='".$id."'");  

echo "Het register is gesloten.<br>";
echo "klik <a href=\"register.php?id='.$id.'\">hier</a> om terug te gaan.<br>";
} else {
?>
<form method='POST' action=''>
	<input type='submit' value='JA, register sluiten'>
</form>
<?
}
?>

<?php
include('beveiliging.php');
include ("config.php");
?>
<?php
$id = $_GET['id'];
$select = "SELECT * FROM register WHERE id = '".$_GET['id']."'";
 
$query = mysql_query($select)or die(mysql_error());
$lijst = mysql_fetch_object($query);
$bestaat = mysql_num_rows($query);
 
if ($bestaat == 0) {
        echo "Dit register bestaat niet (meer)!<br>";
}
 
if(isset($_POST['submit'])) {
 
mysql_query("UPDATE register SET open='' WHERE id='".$id."'");  
 
echo "Het register is gesloten.<br>";
echo "klik <a href=\"register.php?id='.$id.'\">hier</a> om terug te gaan.<br>";
} else {
?>
<form method='POST' action=''>
	<input type='submit' value='JA, register sluiten'>
</form>
<?
}
?>

Ik wil dat als ik op "Ja, register sluiten" klik dat in de database in het veld open geen tekst meer heb staan.

8 antwoorden

Gesponsorde links

Tuinstoel - 14/01/2006 13:21
PHP expert		Je moet sowieso een action meegeven, en gebruik SVP double quotes. Verder moet je even goed kijken, want je kijkt of $_POST['submit'] bestaat, maar die staat niet eens in je formulier.

BigBug - 14/01/2006 13:22

PHP expert

value='JA' zou al beter zijn.. ik zou zelf een GET waarde meegeven in de url bijv: paag.php?id=12&verwijder=ja

en dan bovenin de pagina

php code - Bekijk de code zonder highlighting - Klap code in

<?php
if(isset($_GET['verwijder']) && isset($_GET['id']))
{

// numeric controles ect

     if($_GET['verwijder'] == 'ja')
     {
            // delete
      }
     else
        {
             // return button
         }
}

<?php
if(isset($_GET['verwijder']) && isset($_GET['id']))
{
 
// numeric controles ect
 
     if($_GET['verwijder'] == 'ja')
     {
            // delete
      }
     else
        {
             // return button
         }
}

Tuinstoel - 14/01/2006 13:26
PHP expert		Dat maakt weinig uit BigBug, er is niks beter aan. Je zou dan namelijk nog beter true/false kunnen gebruiken, die "Ja" slaat namelijk nergens op.

Appie_Dijk - 14/01/2006 13:29 (laatste wijziging 14/01/2006 13:29)
Lid		@Tuinstoel och thnx :S

ThAlmighty - 14/01/2006 14:05

HTML beginner

moet het niet zijn

php code - Bekijk de code zonder highlighting - Klap code in

<form method='POST' action='<?php echo $PHP_SELF ?>'>
  <input type='submit' value='JA, register sluiten'>
</form>

<form method='POST' action='<?php echo $PHP_SELF ?>'>
  <input type='submit' value='JA, register sluiten'>
</form>

ik weet niet hoor, misschien werkt het zo ook wel

ThAlmighty

riekele - 14/01/2006 14:11
PHP beginner		je moet nog een name atribuut geven aan die input..

Appie_Dijk - 14/01/2006 16:06
Lid		ja ik heb t al hoor... thnx to tuinstoel... ik was idd de name vergeten!

Ibrahim - 14/01/2006 16:09 (laatste wijziging 14/01/2006 16:10)

PHP expert

edit: bekijk mijn aangepaste versie van je script en leer

en heb ff je code bekeken en beetje gewijzigd:

php code - Bekijk de code zonder highlighting - Klap code in

<?php
include('beveiliging.php');
include ("config.php");
error_reporting(E_ALL); ## belangrijk! vertelt je waarom je script het niet doet !

## Waarom hier php afsluiten en meteen weer openen :s

## $id is onnodig .. 

$select = mysql_query("SELECT * FROM register 
					  WHERE id = ".htmlentities(addslashes( $_GET['id'] ))."") ## volgens mij horen geen ' ' om integers...?
					  or die("Select fout: <br />".mysql_error());

## mysql query kun je ook meteen doen ;-)

## waarom hier mysql fetchen als je het toch niet gebruikt?

$bestaat = mysql_num_rows($select);

if ($bestaat == 0) 
{
        echo "Dit register bestaat niet (meer)!<br>";
}

if($_SERVER['REQUEST_METHOD'] == "POST") 
{
	$Update	=	mysql_query("UPDATE register 
							SET open='' 
							WHERE id=".htmlentities(addslashes($_GET['id']))."") ## volgens mij horen geen ' ' om integers...?
							or die("UpdateFout: <br />".mysql_error());

	echo "Het register is gesloten.<br>";	
	echo "klik <a href=\"register.php?id=".$_GET['id']."\">hier</a> om terug te gaan.<br>"; ## vars buiten quotes ;-)
} 
else 
{
?>
	<form name="form1" method="post" action="">
	  <input type="submit" name="Submit" value="JA, register sluiten">
	</form>
<?
}

## volgende keer netter scripten en ook een tutorial beveiliging lezen !
?>

<?php
include('beveiliging.php');
include ("config.php");
error_reporting(E_ALL); ## belangrijk! vertelt je waarom je script het niet doet !
 
## Waarom hier php afsluiten en meteen weer openen :s
 
## $id is onnodig .. 
 
$select = mysql_query("SELECT * FROM register 
					  WHERE id = ".htmlentities(addslashes( $_GET['id'] ))."") ## volgens mij horen geen ' ' om integers...?
					  or die("Select fout: <br />".mysql_error());
 
## mysql query kun je ook meteen doen ;-)
 
## waarom hier mysql fetchen als je het toch niet gebruikt?
 
$bestaat = mysql_num_rows($select);
 
if ($bestaat == 0) 
{
        echo "Dit register bestaat niet (meer)!<br>";
}
 
if($_SERVER['REQUEST_METHOD'] == "POST") 
{
	$Update	=	mysql_query("UPDATE register 
							SET open='' 
							WHERE id=".htmlentities(addslashes($_GET['id']))."") ## volgens mij horen geen ' ' om integers...?
							or die("UpdateFout: <br />".mysql_error());
 
	echo "Het register is gesloten.<br>";	
	echo "klik <a href=\"register.php?id=".$_GET['id']."\">hier</a> om terug te gaan.<br>"; ## vars buiten quotes ;-)
} 
else 
{
?>
	<form name="form1" method="post" action="">
	  <input type="submit" name="Submit" value="JA, register sluiten">
	</form>
<?
}
 
## volgende keer netter scripten en ook een tutorial beveiliging lezen !
?>

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten