Nem en!

\"select * from favorites where user_id = \'1\' and msg_text = \'$tousername\'\"

if(!row = mysql_query(\"select * from favorites where user_id = \'1\' and msg_text = \'$tousername\'\") echo \"ingen bruger\";
else
echo \"Der er en bruger\";

where user_id = \'1\' a fejl >>

where user_id = \'to_userid\' a

nu kludre jeg rundt i dit script


if(!row = mysql_query(\"select * from favorites where user_id = \'$userdata[user_id]\' and msg_text = \'$tousername\'\") echo \"ingen bruger\";
else
echo \"Der er en bruger\";

nu tror jeg den er der

Det duer squ ikke!
For det første har du glemt en afsluttende parantes, for det andet giver det bare en parse eror pga. din if(row....

Hvis morten har forstået det korrekt, men han bare ikke kan finde ud af syntaxen :O), så vil dette nok virke::

$q = mysql_query(\"select * from favorites where user_id = \'$userdata[user_id]\' and msg_text = \'$tousername\'\");
if(!mysql_num_rows())
  echo \"ingen bruger\";
else
  echo \"Der er en bruger\";

if(!$row = mysql_query(\"select * from favorites where user_id = \'$userdata[user_id]\' and msg_text = \'$tousername\'\")) echo \"ingen bruger\";
else
echo \"Der er en bruger\";

der mangelde også et $ - den burde da virke

nej den duer ikke

$row = mysql_query(\"select * from favorites where user_id = \'$userdata[user_id]\' and msg_text = \'$tousername\'\");
if(!$row) echo \"ingen bruger\";
else
echo \"Der er en bruger\";

sådan så

hvad med den her

if(!mysql_query(\"select * from favorites where user_id = \'$userdata[user_id]\' and msg_text = \'$tousername\'\")) echo \"ingen bruger\";
else
echo \"Der er en bruger\";

...måske vil de sidste 2 virke...

Intet af det virker ordenligt.. Jeg kan ikke få lov at indsætte med følgende kode, uanset hvordan jeg vender og drejer det... Det skal lige nævnes at user_id skal være to_userid istedet.. og $action bliver self. sat til add!

#//// Add favorite function
if ($action == \'add\')
{

if(!$row = mysql_query(\"select * from favorites where to_userid = \'$userdata[user_id]\' and msg_text = \'$tousername\'\"))

{
$time = date(\"d M, Y H:i\");

$sql = \"INSERT INTO favorites ( to_userid, msg_time, msg_text) VALUES (\'$userdata[user_id]\', \'$time\', \'$tousername\')\";

if(!mysql_query($sql, $db)) {
echo $sql . \" : \" . mysql_error() . \"<br>\";
die(\"<p align=\\\"center\\\">Fejl - Kunne ikke opdatere databasen. Klik venligst tilbage og prøv igen\");
}
}
else
{
echo \"Der er en bruger i forvejen\";
}
#////

Det lader til at problemet ligger her:

#////Denne kode virker ikke...
if(!$row = mysql_query(\"select to_userid from favorites where to_userid = \'$userdata[user_id]\' and msg_text = \'$tousername\'\"))
#////Slut på uduelig kode...

--> jeg har da også rettet den !!

if(!mysql_query(\"select * from favorites where to_userid = \'$userdata[user_id]\' and msg_text = \'$tousername\'\")) echo \"ingen bruger\";

virker lige så lidt som:

if(!$row = mysql_query(\"select to_userid from favorites where to_userid = \'$userdata[user_id]\' and msg_text = \'$tousername\'\"))

og bedre er denne heller ikke:

  $row = mysql_query(\"select * from favorites where to_userid = \'$userdata[user_id]\' and msg_text = \'$tousername\'\");
if(!$row) 

I det sidste eksempel returnere $row jo en Resource id lige meget hvad der sker! Derfor har $row jo fået en værdi og kan derfor ikke bruges som !$row.....

Ha! Fandt løsningen!! Wuhuu!

#//// Add favorite function
if ($action == \'add\')
{

$row = mysql_query(\"select * from favorites where to_userid = \'$userdata[user_id]\' and msg_text = \'$tousername\'\");

$res = mysql_fetch_array($row); // get\'s 0 since there\'s no return

echo $res;

if(!$res)

{
    $time = date(\"d M, Y H:i\");
    $sql = \"INSERT INTO favorites ( to_userid, msg_time, msg_text) VALUES (\'$userdata[user_id]\', \'$time\', \'$tousername\')\";

    if(!mysql_query($sql, $db)) {
        echo $sql . \" : \" . mysql_error() . \"<br>\";
        die(\"<p align=\\\"center\\\">Fejl - Kunne ikke opdatere databasen. Klik venligst tilbage og prøv igen\");
    }

}
else

echo \"Der er en bruger i forvejen\"; 


}
#////

Godt nytår! :-)

Tak for hjælpen!

det er vist det her

$row = mysql_query(\"select * from favorites where user_id = \'$userdata[user_id]\' and msg_text = \'$tousername\'\");
if(!$row) ......

undskyld jeg rodede så meget , men jeg sad selv med noget der drillede

GODT NYTÅR

/morten

Jeg synes I roder rundt i det, så man kan have svært ved at se hvad der er
korrekt og hvad der ikke er. Det der kommer tilbage fra en mysql_query kan testes
for om det var korrekt SQL-syntaks eller ej - IKKE om der var tupler i eller ej. Man
kan altså godt:

    $result = mysql_query(\"SELECT ....\");
    if (!$result) {
      echo \"Fejl: \".mysql_error().\"<br>\\n\";
      exit();
    }

Men spørgsmålet er netop om der er, eller ikke er, et tupel i svaret. Linierne ovenover
kan derfor med fordel efterfølges af

    if (mysql_num_rows($result)==0) {
      echo \"Den var da da godt nok ikke<br>\\n\";
    } else {
      echo \"Der var gevinst<br>\\n\";
    }

Strengt taget er der ingen grund til at hente række, med mysql_fetch_row og
lignende, blot for at checke eksistensen - men det gør nok ingen forskel
i praksis.

Så er den vist på plads.

Jeg kunne ikke bruge mortenfn\'s svar til noget, ej heller jd\'s, da jeg ikke kunne gennemskue den korrekte syntaks i problemet.

Kunne ikke forstå at mysql_query gav en resource id når array\'et burde indeholde ingenting??

BTW Happy new year!!

Den angivne løsning (min ;-)) virker, men hvis din er bedre (mindre res. krævende) vil jeg da bruge den!

Det gør nok ingen forskel, el_barto, med mindre du skal køre et par
tusind i sekundet... ;-)  Og så må jeg da sige, at jeg ikke ved hvilken
der er bedst, men så kan man jo sætte sig ned og måle.

Jeg kunne bare ikke holde ud at se alle de forkerte svar, og havde
såmænd overset din løsning ;-O

Jeg viste at jeg havde ret!! :)

Jeg sagde også if(!mysql_num_rows($result)) echo ingen bruger\"; osv...

jd--> ja da, men så må denne også virke

Kommentar fra mortenfn 
31/12 2000 16:25:55 

morten-> Det tror jeg ikke, da den jo altid vil returnere true, med mindre der opstår en fejl?

Jeg kiggede i manualen, og jeg har ret!! :O)

snip::

mysql_query() returns TRUE (non-zero) or FALSE to indicate whether or not the query succeeded. A return value of TRUE means that the query was legal and could be executed by the server. It does not indicate anything about the number of rows affected or returned. It is perfectly possible for a query to succeed but affect no rows or return no rows.

jd - ok du har ret - jeg har også testet og nu læst. Det min if gør er at krontrolere om mysql_query() lykkes, ikke om resultatet er tomt.

Mit script kan bruges, hvis man vil krontrollere om f.eks. \"select\" udføres  - ikke om select giver et resultat.