[MySQL][PHP]Problem z odswieżaniem zapytania Opcje

[andrzej7322]

Witam, mam problem napisalem klasę galerii i mam dwie osobne funkcje jedna odpowiedzialna jest za wyswietlanie tytułów rekordów z bazy danych a druga odpowiedzialna jest za wyświetlanie operacji.
Jedną z operacji jest zmiana pozycji wyświetlania i gdy robie UPDATE do bazy danych pozycja rekordu sie zmienia lecz zmiana w tytulach jest widoczna dopiero gdy ponownie odswieze stronę.
oto fragment klasy

[PHP] pobierz, plaintext 
class album {private function options($id){
$wyn=mysql_fetch_array(mysql_query("SELECT * FROM ".tabela." WHERE id='$id'"));
if($_POST['check']==$id){
if(isset($_POST['top'])){
$poz_minus=$wyn['poz']-1;
mysql_query("UPDATE ".tabela." SET poz=poz+1 WHERE poz='$poz_minus'");
mysql_query("UPDATE ".tabela." SET poz='$poz_minus' WHERE id='$id'");
alert('Operacja przebiegła pomyślnie','1');
}
if(isset($_POST['down'])){
$poz_plus=$wyn['poz']+1;
mysql_query("UPDATE ".tabela." SET poz=poz-1 WHERE poz='$poz_plus'");
mysql_query("UPDATE ".tabela." SET poz='$poz_plus' WHERE id='$id'");
alert('Operacja przebiegła pomyślnie','1');
}
$wyn=mysql_fetch_array(mysql_query("SELECT * FROM ".tabela." WHERE id='$id'"));
}
echo'<form method="POST"action="?gallery"class="form">';
echo'<input type="hidden" name="id" value="'.$id.'" /><input type="hidden" name="check" value="'.$id.'" />
<b class="icon">
<div class="inputhover"><input type="submit" class="m_edit" value=" " name="edit"/></div><div class="inputhover"><input type="submit" class="m_del" value=" " name="del"/></div></b>
<b class="icon">';
if($wyn['poz']!=1){
echo'<div class="inputhover"><input type="submit" class="m_top" value=" " name="top"/></div>';
}
else echo'<input type="submit" class="m_ntop" value=" "disabled/>';
if($wyn['poz']!=$this->end()){
echo'<div class="inputhover"><input type="submit" class="m_down" value=" " name="down"/></div>';
}
else echo'<input type="submit" class="m_ndown" value=" "disabled/>';
echo'</b>';
echo'</form>';
}
 
private function first(){
$this->ends();
echo'<div class="titlebox">
<input type="checkbox" style="float:left;"/>
<b class="bbox">Nazwa albumu</b>
<b class="icon"><img src="images/mm/_del.gif"alt="delete"></b><b class="icon"><img src="images/mm/_edit.gif"alt="edit"></b>
</div><div class="contentbox">';
$c=0;
$zap=mysql_query("SELECT * FROM ".tabela." order by poz");
if(mysql_num_rows($zap)>0){
while($wyn=mysql_fetch_array($zap)){
$class=($c++%2==1) ? 'conbox' : 'conboxodd';
echo'<div class="'.$class.'">';
$this->pokaz($wyn['id']);
echo'<form method="POST"action=""class="formleft">';
echo'<input type="hidden" name="id" value="'.$wyn['id'].'" />';
echo'<input type="submit" class="bboxsubmit"name="edit" style="width:310px;"value="'.$wyn['poz'].'. '.$wyn['nazwa'].'"/>';
echo'</form>';
$this->options($wyn['id']);
echo'</div>';
}
if(isset($_POST['end_edit']) OR ($_POST['del']) OR ($_POST['id'])){
echo'<a href="?gallery" style="clear:both;"class="submita">Zakończ edycję</a>';
}
}
else echo'<div class="warr"><b>Brak albumów</b></div>';
echo'</div>';
}
}
[PHP] pobierz, plaintext 

Z góry dziękuję za pomoc.

Ten post edytował andrzej7322 25.09.2010, 21:08:15

[bastard13]

Zmienna $wyn w jednej funkcji to nie jest ta sama co w drugiej. Poza tym porozdzielaj funkcje w taki sposób, że jedna wyświetla, a kolejna robi update, bo w tej chwili wywołujesz mnóstwo zapytań, których można uniknąć.

[PHP] pobierz, plaintext 
class album{
update()
{
  if($_POST['check'])
  { /*sprawdź i uaktualnij*/ }
}
 
pobierz()
{ /*zwracanie rekordów*/ }
 
pokaz($id)
{ /*czy pokazac*/ }
 
wyswietl($id)
{ /*wyświetl rekord*/ }
 
first()
{
  $this->update();
  $elements=$this->pobierz();
  while($row=mysql_fetch_array($elements))
  {if($this->pokaz($row['id']))
    $this->wyswietl($row['id']);    
  }
}
 
}
[PHP] pobierz, plaintext 

Ogólnie chodzi o to, że wszystkie operacje powinny być od siebie oddzielone, bo później, jeżeli będziesz chciał zmienić sposób np. wyświetlania, to zmieniasz jedną funkcję, a nie przelatujesz przez klasę i szukasz, gdzie masz wprowadzić zmiany.