[PHP][MySQL]mysql_insert_id mały klopot Opcje

[acztery]

witam mam taki klopot

[PHP] pobierz, plaintext 
	public function sqlSave($array,$table,$id='') {
		$set = '';
 
		while( list($key, $val) = each($array['form'])) {
			$set .= '`'.$key.'`=\''. stripslashes($val) .'\', ';
		}
		$set = substr ($set,0, strlen($set) - 2 );
		if (empty($id)) {  //insert
			mysql_query("INSERT INTO `$table` SET $set");
			$id_ = mysql_insert_id();
		}
		else {  //update
			mysql_query("UPDATE `$table` SET $set WHERE id = '".$id."'");
			$id_ = $id;
 
		}
 
 
		$cache = new other_cache(); 
		$cache->clearch();
		return $id_;
	}
[PHP] pobierz, plaintext 

i i ta metoda ma zwracac id rekordu dodanego albo edytowanego. (edytowany dziala)
ale mysql_insert_id(); nie dziala

chodzi o zmienną $id_

moze ktos wie o co chodzi

[nospor]

A nowy rekord się dodaje?

[acztery]

dodaje i zwraca 0

jeszcze 1 kawalek kodu z modelu to cowczesniej to metoda obslugi bazy...

[PHP] pobierz, plaintext 
	function m_modules_desc_update($post,$img='')
	{
 
		if ($img != '')
		{
			$post['form']['img'] = $this->sqlFiltr($img);
		}
 
		if ($post['FOR'] != '')
		{	
			$this->sqlQuery("UPDATE modules_desc SET count_mod=count_mod+1 WHERE id='".$this->sqlFiltr($post['FOR'])."'");	
			$id=$this->sqlSave($post,'modules_desc',$this->sqlFiltr($post['FOR']));
		}else{
 
			$prefix = $this->MysqlGetOne("SELECT count(*) FROM modules_desc");
			$post['form']['id'] = DoRewrite($post['form']['name'])."_".$prefix;
			$post['form']['dateadd'] = date("Y-m-d"); 
			$id=$this->sqlSave($post,'modules_desc');
		}
		return $id;
	}
[PHP] pobierz, plaintext 

metoda sqlQuery odpowiada za dodania albo edycja jak przekazujemy jakies id ...

[nospor]

No jeśli nowy rekord się dodaje to ID powinno być zwrócone.
Czy na pewno ID jest autoincrement?

[acztery]

fuck!

zapomnaiłem.

Tylko, że nie mogę mieć autoincrement narzucone ze starej wersji oprogramowania i klient tak chce zachować.. to jest pole varchal.

tzn ze nie ma sposobu na to ze poznac wartosc ostatnio dodanego pola w kolunie id...

PS wiem ze moge pobrac ostatnio dodany rekord i tyle. ale czy nie bedzie z tym klopotow. jak np w ciagu jednego czasu 100 userow bedzie cos robilo ... wtedy moze się zrobić balagan

Ten post edytował acztery 6.05.2011, 07:11:13

[nospor]

Czegoś tu nie rozumiem.... to skąd ci się to ID bierze skoro to jest varchar?
Ty je gdzieś ustalasz czy jak? No bo jeśli to ty je gdzieś ustalasz to no przecież znasz to id

[acztery]

Naspor dzięki.

[PHP] pobierz, plaintext 
	function m_modules_desc_update($post,$img='')
	{
 
		if ($img != '')
		{
			$post['form']['img'] = $this->sqlFiltr($img);
		}
 
		if ($post['FOR'] != '')
		{	
			$this->sqlQuery("UPDATE modules_desc SET count_mod=count_mod+1 WHERE id='".$this->sqlFiltr($post['FOR'])."'");	
			$this->sqlSave($post,'modules_desc',$this->sqlFiltr($post['FOR']));
			$id=$this->sqlFiltr($post['FOR']);
		}else{
 
			$prefix = $this->MysqlGetOne("SELECT count(*) FROM modules_desc");
			$post['form']['id'] = DoRewrite($post['form']['name'])."_".$prefix;
			$post['form']['dateadd'] = date("Y-m-d"); 
			$this->sqlSave($post,'modules_desc');
			$id=$post['form']['id'];
 
		}
		return $id;
	}
[PHP] pobierz, plaintext 

teraz powinno działać


Jeszcze raz 100 dzięki, nie wiem jak mogłem to przeoczyć....
Masz u mnie piwo.

[thek]

Jjeśli już robisz strukturalnie, to wypadało by sprawdzić czy zapytanie z INSERT się wykonało i czy choć jeden rekord został dodany, zanim wywołasz mysql_insert_id Innymi słowy oprócz tego że to ma być int a nie varchar, myślę że bezpieczniejszym rozwiązaniem byłoby albo walnąć wszystko jako transakcję, albo jeśli nie jesteś aż tak obciążany:

[PHP] pobierz, plaintext 
$insert = mysql_query("INSERT INTO `$table` SET $set");
if( $insert AND mysql_affected_rows() > 0 ) {
    $id_ = mysql_insert_id();
}
[PHP] pobierz, plaintext 

Bez tego może dojść do sytuacji, że będziesz próbował ustawić $id_ zarówno dla insert jak i update choć żaden z nich nie zwróci informacji iż zmiany/dodanie zaszły, bo mogło do nich nie dojść, czego powody mogą być różne: wyczerpał się limit dla autoincrement, nie doszło do zmian w rekordzie (update z danymi mimo poprawnego zapytania zwróci dla mysql_query prawdę, ale już 0 dla affected_rows ) lub podobne atrakcje.