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

Delete fout maar vind ze niet

DenMette - 03/07/2006 11:05 (laatste wijziging 04/07/2006 20:33)

PHP gevorderde

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

<?php
if($status == 'beheerder' || $status == 'co-beheerder' || $status == 'admin')
{
	if($_GET['methode'] == 'reply')
	{
		$id   	= $_GET['id'];
		if(is_numeric($id)){
			$query 	= mysql_query("Select * FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
			$del   	= mysql_fetch_assoc($query);
			$geb  	= $del['poster'];
			$delete = deletebonuspunt($geb, 1);
			if($delete){
				echo 'Er is een fout opgetereden.';
			}else{
				mysql_query("DELETE FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
				header('Location: ?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
			}
		}else{
			echo 'FOUT';
			header('Refresh: 2; URL=?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
		}
	}
	elseif($_GET['methode'] == 'type' && ($status == 'beheerder' || $status == "co-berheerder"))
	{
		$id   	= $_GET['id'];
		if(is_numeric($id)){
			$query  = mysql_query("SELECT * FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
			$delT   = mysql_fetch_assoc($query);
			$gebT   = $delT['poster'];
			$delete = deletebonuspunt($gebT, 1);
			if($delete){
				echo 'Er is een fout opgetereden.';
			}else{
				mysql_query("DELETE FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
			}
			$sql = mysql_query("SELECT * FROM cm_replys WHERE top_id='".$id."'") or die(mysql_error());
			if(mysql_num_rows($sql) != 0)
			{
		  		while($asc = mysql_fetch_assoc($sql))
		  		{
					$user 	= $asc['poster'];
					$sql 	= mysql_query("SELECT * FROM cm_replys WHERE top_id=".$id." AND poster='".$user."'") or die(mysql_error());
					$aantal = mysql_num_rows($sql);
					deletebonuspunt($user, $aantal);
					mysql_query("DELETE FROM cm_replys WHERE top_id='".$id."' AND poster='".$user."'") or die(mysql_error());
				}
			}
			header('Location: ?pag=forum/topic_lijst&cat='.$_GET['cat']);
		}
	}else{
		echo 'FOUT';
		header('Refresh: 2; URL=?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
	}
}else{
	header('Location: ?pag=home');
}
?>

<?php
if($status == 'beheerder' || $status == 'co-beheerder' || $status == 'admin')
{
	if($_GET['methode'] == 'reply')
	{
		$id   	= $_GET['id'];
		if(is_numeric($id)){
			$query 	= mysql_query("Select * FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
			$del   	= mysql_fetch_assoc($query);
			$geb  	= $del['poster'];
			$delete = deletebonuspunt($geb, 1);
			if($delete){
				echo 'Er is een fout opgetereden.';
			}else{
				mysql_query("DELETE FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
				header('Location: ?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
			}
		}else{
			echo 'FOUT';
			header('Refresh: 2; URL=?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
		}
	}
	elseif($_GET['methode'] == 'type' && ($status == 'beheerder' || $status == "co-berheerder"))
	{
		$id   	= $_GET['id'];
		if(is_numeric($id)){
			$query  = mysql_query("SELECT * FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
			$delT   = mysql_fetch_assoc($query);
			$gebT   = $delT['poster'];
			$delete = deletebonuspunt($gebT, 1);
			if($delete){
				echo 'Er is een fout opgetereden.';
			}else{
				mysql_query("DELETE FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
			}
			$sql = mysql_query("SELECT * FROM cm_replys WHERE top_id='".$id."'") or die(mysql_error());
			if(mysql_num_rows($sql) != 0)
			{
		  		while($asc = mysql_fetch_assoc($sql))
		  		{
					$user 	= $asc['poster'];
					$sql 	= mysql_query("SELECT * FROM cm_replys WHERE top_id=".$id." AND poster='".$user."'") or die(mysql_error());
					$aantal = mysql_num_rows($sql);
					deletebonuspunt($user, $aantal);
					mysql_query("DELETE FROM cm_replys WHERE top_id='".$id."' AND poster='".$user."'") or die(mysql_error());
				}
			}
			header('Location: ?pag=forum/topic_lijst&cat='.$_GET['cat']);
		}
	}else{
		echo 'FOUT';
		header('Refresh: 2; URL=?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
	}
}else{
	header('Location: ?pag=home');
}
?>

Nu heb ik een probleem met het verwijderen van het topic de is_numeric doet moeilijk

12 antwoorden

Gesponsorde links

roelh - 03/07/2006 11:12
PHP gevorderde		kun je imsschien eventjes tussen <? en ?> zetten?

DenMette - 03/07/2006 11:17
PHP gevorderde		gebuert vergeten sorry

marten - 03/07/2006 11:22 (laatste wijziging 03/07/2006 11:24)
Beheerder		1. SQL commando's zijn altijd met hoofdletters dus Delete moet zijn DELETE 2. ik ga verder kijken mmm echt rare manier van werken heb jij :S Doe eens normaal ---> $var = mysql_query(); $var2 = mysql_fetch_assoc($var); enz enz

DenMette - 03/07/2006 11:25 (laatste wijziging 03/07/2006 11:27)
PHP gevorderde		Citaat: 1. SQL commando's zijn altijd met hoofdletters dus Delete moet zijn DELETE Toch niet ik gebruik soms hoofdletters en alles werkt altijd prima

marten - 03/07/2006 11:28 (laatste wijziging 03/07/2006 11:29)
Beheerder		wil je beetje proper scripten dan zijn het met hoofdletters maar dat moet jezelf weten. Net zoiets als bijvoorbeeld $sQuery meestal $query genoemd maar je kan in de variabele naam aflezen dat het een string is..... Dit werkt gewoon zoveel handiger..... maar dat moet je zelf maar weten..... deletebonuspunt($gebT, 1); Ik neem aan dat de functie deletebonuspunt wel gedefinieerd is? Je moet ook op je foutcontrole letten. Dus $sVar = deletebonuspunt($gebT, 1); if(!$sVar) { echo 'Fout in de functie'; }

DenMette - 03/07/2006 11:37 (laatste wijziging 03/07/2006 11:38)
PHP gevorderde		Zo wat betere code?? @marten Ik snap niet goed wat je bedoeld

marten - 03/07/2006 11:42 (laatste wijziging 03/07/2006 11:46)
Beheerder		je hebt in je code deletebonuspunt($gebT, 1); staan. Dit is een functie. Maar je controleert niet of die functie wel gelukt is daarna. Evenals je query's. Je doet nergens aan foutafhandeling. Dit is een zeer belangrijk punt in scripten / programmeren. En bij query's word trouwens or die(mysql_error()) sterk afgeraden!. Alleen om te debuggen mag je dit gebruiken. Als je website online draait (dus open is voor bezoeker en dergelijke) mag er nergens een or die komen! Verder variabelenamen. $var weet je niet wat dit nou is. Daarom zetten ze (bij grote projecten) de eerste letter van het type ervoor. Dus string ---> $var word $sVar double ---> $var word $dVar array --> $var word $aVar enzovoort.... Ditzelfde geld voor mysql commando's met hoofdletters. Het zal wel werken met kleine letters maar gebruikelijk is met hoofdletters. Dit is gewoon zoveel handiger voor een ander om te debuggen (of als je met een groep script). En zet eens voor de gein error_reporting(E_ALL) boven je scripts. en probeer dan de fouten eruit te halen Leer je echt veel van en als je dit vanaf het begin van een nieuw script doet is je script al van hoger niveau dan wanneer je het niet doet (Kan ook in je php.ini standaard aanstaan maar wanneer dit niet zo is even erboven zetten )

Lemon - 03/07/2006 11:47
PHP expert		@ marten: Ik heb nog geen enkel groot project gezien die de hungarian style gebruiken (1 letter code voor het type). Dat is zelfs afgeraden omdat het de code er enkel maar onduidelijker op maakt. En over die foutafhandeling in die functie. Die kan evengoed in de functie zelf zitten ;)

DenMette - 03/07/2006 13:45 (laatste wijziging 03/07/2006 13:45)

PHP gevorderde

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

<?
if($status == 'beheerder' || $status == 'co-beheerder' || $status == 'admin')
{
  if($_GET['methode'] == 'reply')
  {
    $id   	= $_GET['id'];
    $query 	= mysql_query("Select * FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
    $del   	= mysql_fetch_assoc($query);
    $geb  	= $del['poster'];
    $delete = deletebonuspunt($geb, 1);
	if(!$delete){
		echo 'Er is een fout opgetereden.';
	}else{
		mysql_query("DELETE FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
		header('Location: ?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
	}
  }
  elseif($_GET['methode'] == 'type')
  {
    $id   	= $_GET['id'];
    $query  = mysql_query("SELECT * FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
    $delT   = mysql_fetch_assoc($query);
    $gebT   = $delT['poster'];
    $delete = deletebonuspunt($gebT, 1);
	if(!$delete){
		echo 'Er is een fout opgetereden.';
	}else{
		mysql_query("DELETE FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
	}
    $sql = mysql_query("SELECT * FROM cm_replys WHERE top_id='".$id."'") or die(mysql_error());
    if(mysql_num_rows($sql) != 0)
    {
      while($asc = mysql_fetch_assoc($sql))
      {
        $user 	= $asc['poster'];
        $sql 	= mysql_query("SELECT * FROM cm_replys WHERE top_id=".$id." AND poster='".$user."'") or die(mysql_error());
        $aantal = mysql_num_rows($sql);
        deletebonuspunt($user, $aantal);
        mysql_query("DELETE FROM cm_replys WHERE top_id='".$id."' AND poster='".$user."'") or die(mysql_error());
      }
    }
    header('Location: ?pag=forum/topic_lijst&cat='.$_GET['cat']);
  }
}else{
  header('Location: ?pag=home');
} ?>

<?
if($status == 'beheerder' || $status == 'co-beheerder' || $status == 'admin')
{
  if($_GET['methode'] == 'reply')
  {
    $id   	= $_GET['id'];
    $query 	= mysql_query("Select * FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
    $del   	= mysql_fetch_assoc($query);
    $geb  	= $del['poster'];
    $delete = deletebonuspunt($geb, 1);
	if(!$delete){
		echo 'Er is een fout opgetereden.';
	}else{
		mysql_query("DELETE FROM cm_replys WHERE id='".$id."'") or die(mysql_error());
		header('Location: ?pag=forum/topic&id='.$_GET['tid'].'&cat='.$_GET['cat']);
	}
  }
  elseif($_GET['methode'] == 'type')
  {
    $id   	= $_GET['id'];
    $query  = mysql_query("SELECT * FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
    $delT   = mysql_fetch_assoc($query);
    $gebT   = $delT['poster'];
    $delete = deletebonuspunt($gebT, 1);
	if(!$delete){
		echo 'Er is een fout opgetereden.';
	}else{
		mysql_query("DELETE FROM cm_topics WHERE id='".$id."'") or die(mysql_error());
	}
    $sql = mysql_query("SELECT * FROM cm_replys WHERE top_id='".$id."'") or die(mysql_error());
    if(mysql_num_rows($sql) != 0)
    {
      while($asc = mysql_fetch_assoc($sql))
      {
        $user 	= $asc['poster'];
        $sql 	= mysql_query("SELECT * FROM cm_replys WHERE top_id=".$id." AND poster='".$user."'") or die(mysql_error());
        $aantal = mysql_num_rows($sql);
        deletebonuspunt($user, $aantal);
        mysql_query("DELETE FROM cm_replys WHERE top_id='".$id."' AND poster='".$user."'") or die(mysql_error());
      }
    }
    header('Location: ?pag=forum/topic_lijst&cat='.$_GET['cat']);
  }
}else{
  header('Location: ?pag=home');
} ?>

Dit is de code en nu doet hij het wel

marten - 03/07/2006 16:10
Beheerder		nice misschien nog je id controleren of het wel echt een id is

Thomas - 03/07/2006 16:34
Moderator		Ik hoop dat je (co)-beheerders en admins betrouwbare personen zijn, want op deze manier kun je volgens mij de hele tabel cm_topics of cm_replys leegmieteren .

DenMette - 03/07/2006 16:45 (laatste wijziging 04/07/2006 20:33)
PHP gevorderde		Ja natuurlijk ik ben Co-beheerder en de Beheerder doe moet mij maar vertrouwen maar ik ben er nog niet zeker van dat admins dit moeten kunnen!!

Gesponsorde links

Dit onderwerp is gesloten.

Actiefste leden van de maand

Actieve forumberichten