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 (IMG:style_emoticons/default/winksmiley.jpg)

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ą (IMG:style_emoticons/default/winksmiley.jpg)

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 (IMG:style_emoticons/default/winksmiley.jpg)

[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.

[szamburdsz]

Dla sprawdzenia czy dziala ci $_GET wrzuc taki pliczek test.php

[PHP] pobierz, plaintext 
<?php
   if(isset($_GET['test']))
   {
      echo 'OK';
   }
?>
<a href"test.php?test">TEST</a>
[PHP] pobierz, plaintext 

Ale nie wierzę, że nie działa bo na cba działa php bez problemów.

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

[renault12]

O dziwo nic sie nie dzieje, sprawdz sam http://szybkiezaproszenialockerz.cba.pl/test/test.php

[szamburdsz]

Masz jakies krzaki na poczatku pliku. Usun spacje i entery na samym poczatku

[renault12]

usunięte

[szamburdsz]

Mój bład. Popraw:

<a href="test.php?test">TEST</a>

[renault12]

tez nie zauwazylem. Teraz dziala.

[szamburdsz]

Czyli działa.
Daj teraz listing twojego pliku, zobacze co jest nie tak.

[renault12]

Jako listing chodzi ci o zawartosc pliku czy adres strony ?

Oto ta zawartosc:

[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(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 $_GET['usun'];
		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' />";
?>
[PHP] pobierz, plaintext 

[szamburdsz]

popraw

[PHP] pobierz, plaintext 
echo "<a href=\"galeria.php?usun=".$record['idx']."\"> - Usuń - </a>"; ?> </font></center> <?
na
echo '<a href="galeria.php?usun='.$record['idx'].'"> - Usuń - </a>'; ?> </font></center> <?
[PHP] pobierz, plaintext 

I dobrze jakbys dal linka jakiegos do strony abym mogl zobaczyc

I daj na sam poczatek

[PHP] pobierz, plaintext 
<?php
   if(isset($_GET['usun']))
   {
      echo 'OK';
   }
?>
[PHP] pobierz, plaintext 

Ten post edytował szamburdsz 31.01.2010, 21:49:53

[renault12]

http://szybkiezaproszenialockerz.cba.pl/te...ria.php?start=0

[szamburdsz]

Zobacz teraz

[PHP] pobierz, plaintext 
<?php
$start = $_GET['start'];
$na_stronie = 8;
$wykonaj = mysql_query("SELECT * FROM images");
$znaleziono = mysql_num_rows($wykonaj);
if($znaleziono>$na_stronie)
{
	echo '<center>Strona ';
	for($i=0; $i<ceil($znaleziono/$na_stronie); $i++)
  		echo '<a href="galeria.php?start='.($i*$na_stronie).'">'.($i+1).'</a> | ';
}  
echo '</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"> <?php
  	$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"> <?php
		echo $_GET['usun'];
		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' />";
?>
 
[PHP] pobierz, plaintext 

A bład masz w tej lini

[PHP] pobierz, plaintext 
echo '<a href="galeria.php?usun='.$record['idx'].'"> - Usuń - </a>'; ?> </font></center> <?
[PHP] pobierz, plaintext 

Musisz jeszcze dododac start

[PHP] pobierz, plaintext 
echo '<a href="galeria.php?start='.$start.'&usun='.$record['idx'].'"> - Usuń - </a>'; ?> </font></center> <?
[PHP] pobierz, plaintext 

[renault12]

Dzieki wielkie! Działa
piwo dla ciebie ;]

Mam jeszcze jeden problem dotyczacy usuwania pliku z serwera, kod opiera sie na zmiennej i unlinku wrzuconego w warunek ten sam gdzie jest usuwanie z bazy, lecz pliki sie nie usuwaja. Dzieki "echo" zwraca poprawna lokalizacje pliku ale ni usuwa sie, czemu? Oto kod

[PHP] pobierz, plaintext 
if(isset($_GET['usun'])) {
     mysql_query("DELETE FROM `images` WHERE `idx` = '".$_GET['usun']."'");
	 ?>
      <?
	  	$obraz2 = 'gallery/images/'.$obraz.'';
		echo $obraz2;
		unlink($obraz2);
}
[PHP] pobierz, plaintext 

Ten post edytował renault12 31.01.2010, 22:48:54

[szamburdsz]

Jeżeli zmienna $obraz przechowuje wyłącznie nazwę pliku bez rozszerzenia to rozwiązanie wygląda następująco:

[PHP] pobierz, plaintext 
$obraz2 = 'gallery/images/'.$obraz.'.jpg';
[PHP] pobierz, plaintext 

No i musisz mieć prawa modyfikacji CHMOD do katalogu image na serwerze. A jeżeli jest inaczej to muszę znać więcej szczegółów.

Ten post edytował szamburdsz 1.02.2010, 00:57:03