Usuwanie rekordów Opcje

[renault12]

Witam,

Napotkałem problemik dotyczacy usuwania rekordow z bazy danych.
Otóż mam taki kod:

[PHP] pobierz, plaintext 
  <?
$start = $_GET['start'];
$na_stronie = 8;
$wykonaj=mysql_query("SELECT * FROM images");
$znaleziono=mysql_num_rows($wykonaj);
if($znaleziono>$na_stronie) {
    print '<center>Strona ';
for($i=0; $i<ceil($znaleziono/$na_stronie); $i++)
  print '<a href="galeria.php?start='.($i*$na_stronie).'">'.($i+1).'</a> | ';
}  
print '</center>';
 
$query = "SELECT * FROM images LIMIT ".($start).",".$na_stronie."";
$result = mysql_query ($query);
        while ($row = mysql_fetch_assoc($result))
        {
			$obraz = $row['tn_filename'];
			$obrazduzy = $row['filename'];
			?>
  <div id="mejn"> <?
  $uss = $_SESSION['user_name'];
  $edytt = mysql_query("SELECT user_name FROM users WHERE ranga='1'");
		while ($e = mysql_fetch_assoc($edytt)) {
		$tp = $e['user_name'];
		}
		if ($tp == $uss) {
 		 if ($e['ranga'] = 1) {
if(isset($_GET['usun'])) {
     mysql_query("DELETE FROM `images` WHERE `idx` = '".$_GET['usun']."'");
	 header ("usuwanie.php");
}
    $rezultat = mysql_query("SELECT * FROM `images` WHERE idx = ".$row['idx']." limit 1") or die("Błąd 2");
    $i = 0;
	while ($record = mysql_fetch_array($rezultat, MYSQL_ASSOC)) {
        ?> <center><font size="2"> <? echo "<a href=\"galeria.php?usun=".$record['idx']."\"> - Usuń - </a>"; ?> </font></center> <?
        $i++;
	}
		 }
		}
echo "<a href='galleria/images/".$obrazduzy."'><img src='galleria/images/".$obraz."' border='0' />"; ?> </div> 
  <?
} 
?>
[PHP] pobierz, plaintext 

W tym kodzie mam zawarte stronnicowanie, ktore dobrze chodzi, lecz jest problem w kodzie gdzie zaczyna sie usuwanie rekordów. Nad obrazkiem jest link usun i przypisany do niego id ktore wyglada mniej wiecej tak po najechaniu na niego myszka "http://blablabla.blabla.pl/blabla/galeria.php?usun=7". W momencie kiedy go naciskam, wraca do strony galeria.php i nic sie nie dzieje, nie usuwa sie zaden wpis do bazy danych.

Prosze o pomoc

Ten post edytował renault12 27.01.2010, 20:02:13

[Fifi209]

Pierwszy rzut oka:

[PHP] pobierz, plaintext 
mysql_query("DELETE FROM `images` WHERE `idx` = '".$_GET['usun']."'");
header ("usuwanie.php");
}
[PHP] pobierz, plaintext 

header('Location: usuwanie.php');

i czemu jak wstawiasz liczby to dajesz w apostrofy ? lubisz generować dodatkowe zadania bazie w postaci konwertowania stringa na int?

[renault12]

1. Zmieniłem kod na taki jak kazałeś lecz dalej nie usuwa z bazy rekordow
2. Po nacisnieciu nie przenosi na podana strone : /
3.A dodawanie z apostrofami koliduje z czyms ? tzn . moze zaszkodzic rekordom etc.?

[Fifi209]

1. Zmieniłem kod na taki jak kazałeś lecz dalej nie usuwa z bazy rekordow

Nic nie kazałem, tylko wskazałem jeden z błędów który akurat rzucił mi się w oczy.

2. Po nacisnieciu nie przenosi na podana strone : /

Czyżby: headers already sent?

3.A dodawanie z apostrofami koliduje z czyms ? tzn . moze zaszkodzic rekordom etc.?

Koliduje z wydajnością

Zrób sobie w php warunek:

[PHP] pobierz, plaintext 
if ('1' === 1)
echo 'ok';
else
echo 'oj cos nie tak';
[PHP] pobierz, plaintext 

To nie jest to samo

[renault12]

No masz z tymi apostrofami racje.
Ale odswieze prosbe na temat usuwanie tych rekordow : P

mam tutaj kod który dziala do usuwania newsow :

[PHP] pobierz, plaintext 
$wyn = mysql_query("SELECT * FROM news ORDER BY id DESC LIMIT 10")
or die('Błąd zapytania');
if(mysql_num_rows($wyn) > 0) {
    while($s = mysql_fetch_assoc($wyn)) {
		?> <font color="blue"><b><font size="3"> <?
		echo "".$s['tytul']."";
		?> </font></b></font>
        <font size="1.5"> <?
		echo "<br/>";
		echo "Napisany w ".$s['czas'].", przez ".$s['userr']."";
		?> </font> <font size="2.5"><text align="left"> <?
		echo "<br/>";
        echo "<br/>".$s['newsy']."";
		echo "<br/>";
		echo "<br/>";
		echo "<br/>";
		$edytt = mysql_query("SELECT user_name FROM users WHERE ranga='1'");
		while ($e = mysql_fetch_assoc($edytt)) {
		$tp = $e['user_name'];
		}
 		 if ($tp = $uss) {
if(isset($_GET['usun'])) {
     mysql_query("DELETE FROM `news` WHERE `id` = '".$_GET['usun']."'");
	 header ("usuwanie.php");
}
    $rezultat = mysql_query("SELECT * FROM `news` WHERE id = ".$s['id']." limit 1") or die("Błąd 2");
    $i = 0;
	while ($record = mysql_fetch_array($rezultat, MYSQL_ASSOC)) {
        ?> <p align="right"> <? echo "<a href=\"index.php?usun=".$record['id']."\"> - Usuń - </a>";
        $i++;
	}
			 ?> </p><?
		 }
		echo "<br/>";
		echo "<br/></font>";
    }
    echo "</table>";
}
[PHP] pobierz, plaintext 

Prawie taki sam jest ten ktory podalem na gorze, lecz tamten nie dziala.

Ten post edytował renault12 27.01.2010, 21:12:36

[Fifi209]

No masz z tymi apostrofami racje.
Ale odswieze prosbe na temat usuwanie tych rekordow : P

[PHP] pobierz, plaintext 
if ($e['ranga'] = 1) {
[PHP] pobierz, plaintext 

Teraz mi się rzuciło w oczy, ma być chyba podwójny znak równości.

[renault12]

No masz racje, juz poprawilem kod.

A co do tego usuwania to nie wiem co jest bo przy wcisnieciu "usun" przenosi i nie usuwa. Jak wpisuje naprzyklad

[PHP] pobierz, plaintext 
echo $record['idx']; 
[PHP] pobierz, plaintext 

wypisuje mi id w bazie naprzyklad "9". Adres odnosnika jest dobry rowniez. Nie wiem, widac to jakby nie wykonywalo akcji usuwania z bazy co do tego idzie nie wykrywa nacisniecia przycisku usun a powinno.

[Fifi209]

Pokaż mi przerobiony kod, według moich zaleceń i podpowiedzi.

[renault12]

[PHP] pobierz, plaintext 
  <div id="mainContent">
  <?
  if(isset($_GET['usun'])) {
     mysql_query("DELETE FROM `images` WHERE `idx` = '".$record['idx']."'");
}
$start = $_GET['start'];
$na_stronie = 8;
$wykonaj=mysql_query("SELECT * FROM images");
$znaleziono=mysql_num_rows($wykonaj);
if($znaleziono>$na_stronie) {
    print '<center>Strona ';
for($i=0; $i<ceil($znaleziono/$na_stronie); $i++)
  print '<a href="galeria.php?start='.($i*$na_stronie).'">'.($i+1).'</a> | ';
}  
print '</center>';
 
$query = "SELECT * FROM images LIMIT ".($start).",".$na_stronie."";
$result = mysql_query ($query);
        while ($row = mysql_fetch_assoc($result))
        {
			$obraz = $row['tn_filename'];
			$obrazduzy = $row['filename'];
			?>
  <div id="mejn"> <?
  $uss = $_SESSION['user_name'];
  $edytt = mysql_query("SELECT user_name FROM users WHERE ranga=1");
		while ($e = mysql_fetch_assoc($edytt)) {
		$tp = $e['user_name'];
		if ($tp == $uss) {
if(isset($_GET['usun'])) {
     mysql_query("DELETE FROM `images` WHERE `idx` = '".$_GET['usun']."'");
}
    $rezultat = mysql_query("SELECT * FROM `images` WHERE idx = ".$row['idx']." limit 1") or die("Błąd 2");
	while ($record = mysql_fetch_array($rezultat, MYSQL_ASSOC)) {
        ?> <center><font size="2"> 
		<?
		echo $record['idx']; //(wyświetla prawidłowo)
		echo "<a href=\"galeria.php?usun=".$record['idx']."\"> - Usuń - </a>"; ?> </font></center> <?
		}
		}
		}
echo "<a href='galleria/images/".$obrazduzy."'><img src='galleria/images/".$obraz."' border='0' />";
?> </div> 
  <?
} 
?>
</div>
<br class="clearfloat" />
   <div id="footer">
    <p><center><font size="-2">? Copyright 2010 by Johny</p></center></font>
  </div>
</div>
</body>
</html>
[PHP] pobierz, plaintext 

Ten post edytował renault12 28.01.2010, 20:01:28

[Fifi209]

A coś się tak uparł, że to usuwanie etc. musi być w jednym pliku? O wiele prościej jest to rozbić na kilka plików - łatwiej edytować i kod nie jest tak zagmatwany.

[szamburdsz]

Wystarczy, ze zrobisz tak:

[PHP] pobierz, plaintext 
mysql_query("DELETE FROM images WHERE idx = '$record[idx]'");
[PHP] pobierz, plaintext 

[renault12]

Zmieniłem i też zdjecie sie nie usuwa.
Co do rozbicia to nie wiem zabardzo jak to zrobic, poniewaz nie chce mi przypisywac zmiennej do usuwania przy odpowiednim obrazku po kolei.
Prosze o pomoc

Zauważyłem takze gdy usuwam warunek if i w ogole pozostawiam ta czesc kodu w takim stanie:

[PHP] pobierz, plaintext 
mysql_query("DELETE FROM `images` WHERE `idx` = '".$_GET['usun']."'");
[PHP] pobierz, plaintext 

Obrazek usuwa się po wejściu, lecz teraz co jest źle w tym ifie?

[PHP] pobierz, plaintext 
if(isset($_GET['usun'])) {
mysql_query("DELETE FROM `images` WHERE `idx` = '".$_GET['usun']."'");
}
[PHP] pobierz, plaintext 

Ten post edytował renault12 29.01.2010, 23:26:29

[Fifi209]

Zamiast sprawdzać isset czy zmienna istnieje, sprawdź czy nie jest pusta empty

[renault12]

Yhm, też nie działa ani !empty ani empty.

[szamburdsz]

Sprawdź na początku wartość $_GET['usun']

[PHP] pobierz, plaintext 
echo $_GET['usun'];
[PHP] pobierz, plaintext 

warunek masz prawidłowy

[PHP] pobierz, plaintext 
if(isset($_GET['usun']))
{
 
}
[PHP] pobierz, plaintext 

[renault12]

Gdziekolwiek wstawie to nie wyswietla sie wartosc usun.

[szamburdsz]

A czy wartość $_GET['usun'] pojawia się w adresie przeglądarki internetowej?

http://galeria.php?usun=wartosc

po kliknięciu w link Usuń

Ten post edytował szamburdsz 31.01.2010, 21:07:33

[renault12]

tak, zgodnie z id obrazka

[szamburdsz]

a masz register_globals na off w konfiguracji serwera?

[renault12]

Używam cba.pl jako hosting a oni chyba mają to właczone.