Php eller MySQL buffrar...

drew skrev:

I varje loop i while-satsen ändras datan i $sql och ska skrivas in i en tabell. Jag har med echo kollat att innehållet i $sql verkligen ändras. Ändå skrivs samma sak ner varje gång. ...

Någon som har något förslag?

Hm... Testa att byta till nya variabelnamn inne i while, jämfört med första sql-satsen. Alltså:

<?php
$sql = 'SELECT * FROM users_table';  //$sql
$result = mysql_query($sql);  //$result
while($row = mysql_fetch_array($result, MYSQL_ASSOC)) //$row
{
/* Klipp */

	$to_id = $row['user_id'];
	//Här använder vi $insSQL istället
	$insSQL = 'INSERT INTO ' . PRIVMSGS_TABLE . " (author_id, author_ip, message_time, message_subject, message_text, bbcode_bitfield, bbcode_uid, to_address, bcc_address)
		VALUES ($from_id, '$avs_ip', $nu, '$pm_sub', '$pm_text', '$pm_bbkod_bits', '$pm_bbkod_id', 'u_{$to_id}', '')";
	mysql_query($insSQL);
}

?>

Jag fick ett liknande fel tidigare idag när jag råkade återanvända samma namn utanför och innanför while...

Det behövs ingen php-loop, du kan göra det direkt i SQL med

insert into t(<columns>) select ...

Jag testade att byta variabelnamnet. Ingen skillnad. Jag har också tidigare provat unset($sql); utan att det hjälpte.

php-loopen behövs för att bl a $pm_sub och $pm_text innehåller olika text beroende på inställningarna som hämtas med den första queryn.

Jag skulle råda dig till att skapa en enkel db.php som du inkluderar där du behöver databasen. Innehållande två funktioner, en för Query och en för Insert. Som båda öppnar en koppling till databasen och sen stänger den när de är klara. Då borde du slippa problem med buffrar som hänger kvar. (Och det är smidigare att programmera fler grejor sen.)

<?php
function Insert($sql)
{
	$db = mysql_connect("adress.till.databasen.com", "username", "pass") or die("Databasfel.");
	mysql_select_db("databasens_namn", $db);
	mysql_query($sql);
	mysql_close($db);
}
function Query($sql)
{
	$db = mysql_connect("adress.till.databasen.com", "username", "pass") or die("Databasfel.");
	mysql_select_db("databasens_namn", $db);
	$result = mysql_query($sql) or die("SQL-satsen gav inget resultat.");
	mysql_close($db);
	return $result;
}?>

Jag har aldrig varit med om att samma sida skulle behöva två eller flera connections. Det blir ju rejält overhead att öppna och stänga dem hela tiden. Betänk att hans insert ligger i en whileslinga! Nog måste det gå att hitta en vettigare lösning... ? :q

. :r :r
Jag har hittat felet. Det låg inte i skripet jag höll på att göra utan ett annat som förutsatte att samma användare inte skrev mer än ett pm per sekund.

aasah skrev:

Det blir ju rejält overhead att öppna och stänga dem hela tiden. Betänk att hans insert ligger i en whileslinga!

Naturligtvis helt sant som du säger. Så går det när man snabbläser kod =)

php-loopen behövs för att bl a $pm_sub och $pm_text innehåller olika text beroende på inställningarna som hämtas med den första queryn.

Så du har alltså php-kod som ser ut som

if  ($row['someColumn'] == 'someValue')
{
    $pm_sub = 'tjo';
}
else
{
    $pm_sub = 'flöjt';
}

?

Du kan lägga in sådana översättningar direkt i select med hjälp av case.