problem med if sætning

Der er vel ikke noget galt ?


Hvad skulle være galt med den ?

Der er intet galt.

Jeg har disse sætinger, men de to første if sætninger virker ikke..
Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in .........

if ($_SESSION['soeger']=='tidligere fer') {
                  $query = mysql_query("SELECT medlemmer.*, fer.* FROM medlemmer, fer WHERE medlemmer.brugernavn = fer.brugernavn && medlemmer.koen LIKE '%$_SESSION[koen]%' && fer.maal LIKE '%$_SESSION[maal]%' && fer.soegerfer LIKE '%$_SESSION[soegerfer]%' && medlemmer.brugernavn LIKE '%".preg_replace("/[_]/","\\\\_",$_SESSION['brugernavn'])."%' "); }
                  if ($_SESSION['soeger']=='kommende fer') {
                  $query = mysql_query("SELECT medlemmer.*, fer.* FROM medlemmer, fer WHERE medlemmer.brugernavn = fer.brugernavn && medlemmer.koen LIKE '%$_SESSION[koen]%' && fer.maal LIKE '%$_SESSION[maal]%' && fer.soegerfer LIKE '%$_SESSION[soegerfer]%' && medlemmer.brugernavn LIKE '%".preg_replace("/[_]/","\\\\_",$_SESSION['brugernavn'])."%' "); }

                  if ($_SESSION['soeger']=='') {
                  $query = mysql_query("SELECT medlemmer.*, fer.* FROM medlemmer, fer WHERE medlemmer.brugernavn = fer.brugernavn && medlemmer.koen LIKE '%$_SESSION[koen]%' && fer.maal LIKE '%$_SESSION[maal]%' && fer.soegerfer LIKE '%$_SESSION[soegerfer]%' && medlemmer.brugernavn LIKE '%".preg_replace("/[_]/","\\\\_",$_SESSION['brugernavn'])."%'"); }

       
                   
                 
                  $taeller=0;
                  while ($row = mysql_fetch_array($query)) { $taeller++; }

Hvad får du, hvis du bruger:
echo $_SESSION['soeger'];

?

<ole>

Er MySQL-fejlen mon det største problem? Jeg har i hvertfald umiddelbart ikke fantasi til at forestille mig et mere ineffektivt DB-kald  =)

/mvh
</bole>

mysql_fetch_array(): supplied argument is not a valid MySQL result resource in...

Fortæller at det er ugyldigt argurment.. og som jeg læser det er -- at det mangler en Array til sidste.. du kaldet en Tæller frem som du har sat til at være 0 OG SKAL tælle op med tæller ++ -- den argument er vist ugyldigt.. for det skrives :

while($row = mysql_fetch_array($query)
{
echo $row["navn"];
}
} else {
echo "Søgningen fandt ingenting!";
}

Jeg kom på en anden ide..

Hvis du sætter den stump kode i linjen i if sætningen kan man få vist hvor det opstå fejlen.. Altså du sætter den lille stump kode - efter den første if sætningen -- for at se hvis det er der hvor der gå galt.. If -- Or die og viser dig så den fejl..

or die(mysql_error());

DEN BEDSKED kommer også når det er ikke forbindelse med databasen.
Så skal db_connection se sådan her ud ikke??

<?
mysql_connect("Host", "XXXXXXX", "12345 - kode");
mysql_select_db("dit databasenavn");
?>

Jeg løb nu tør for ideer. :-)

Tilføj dette:
echo mysql_error();

Før dette:
$taeller=0;
while ($row = mysql_fetch_array($query)) { $taeller++; }

Så kan du se din SQL fejl (som skaber problemet)

For en let genkendelig fejlmeddelelse er det somregel en fordel at fejlsøge lige efter stedet, hvor det går galt. F.eks:

if ($_SESSION['soeger']=='tidligere fer') {
    $sql = "SELECT medlemmer.*, fer.* FROM medlemmer, fer WHERE medlemmer.brugernavn = fer.brugernavn && medlemmer.koen LIKE '%$_SESSION[koen]%' && fer.maal LIKE '%$_SESSION[maal]%' && fer.soegerfer LIKE '%$_SESSION[soegerfer]%' && medlemmer.brugernavn LIKE '%".preg_replace("/[_]/","\\\\_",$_SESSION['brugernavn'])."%' ";
    $query = mysql_query($sql) or die ($sql."<br>\r\n".mysql_error());
}

- men som sagt vil jeg mene, der er langt større problemer med koden. Alene slutningen er rent overkill. I stedet for at bladre resultatet igennem og inkremere en tæller, kan du få samme resultat betydeligt mere effektivt med:
    $tæller = mysql_num_rows($query);

- men selve query'en virker også ekstremt ineffektiv. Er det f.eks. nødvendigt at hente samtlige felter fra begge tabeller? Og er det virkelig nødvendigt med 4 (FIRE...!) dobbelte wildcard-søgninger - hvoraf den ene ovenikøbet bygger på en RegExp?

Som sagt, har jeg svært ved at forestille mig et væsentligt mere ineffektivt kald til en MySQL-DB - og koden får mig til at frygte, du heller ikke har indekseret tabellerne ... men det er selvfølgelig kun et gæt  =)

ole
nej det er ikke nødvendigt at hente samtlige felter fra begge tabeller....

Hvad mener du med 4 dobbelte wildcard løsninger ???

Hvorfor gætter du på at jeg ikke har indekseret tabellerne ??

Jeg må lige kigge på jeres forslag til fejlsøgning.

1) så bør du ikke kalde med "medlemmer.*, fer.*", men specifisere de ønskede felter i stedet for at bruge asteriks (*)

2) du bruger fire gange LIKE med et wildcard (%) i hver ende af det søgte ord

3) da koden savner effektivitet på alle hylder, er det vel en nærliggende tanke  =)

I din sidste linje skal du ændre (Tæller) og angive Arrays

den linje er dødelig!

$taeller=0;
                  while ($row = mysql_fetch_array($query)) { $taeller++; }

I stedet for skal du ændre den til
while($row = mysql_fetch_array($query)
{
echo $row[" "];
}
} else {
echo "Søgning bla bla!";

det er nok lettere at skrive koden om igen !! :-)

victor22 >> kan du elaborere lidt på, hvad du finder 'dødeligt' i de to linjer?

Jeg ville også gerne høre, hvad du mener, han skulle bruge din ændring til? Hvad tror du selv, den kan/gør?  =)

I de to linjer sætter han en tæller til 0 - og løber derefter recordset'et igennem, mens han for række tæller tælleren én op. Det er der absolut intet 'dødeligt' i!
Som sagt kan det dog gøres nemmere med:
    $taeller = mysql_num_rows($sql);

Dit eget ændringsforslag er vist helt ude i hampen! Jeg har i hvertfald aldrig hørt om en 'while/else ting' i PHP ... har du?  ;o)

ole
ved godt koden nok ikke er optimeret helt...
Men det skulle jo virke uanset ,
og kan ikke se hvorfor ,mit ikke skulle virke.. det virker som sagt når : $_SESSION['soeger']==''

men ikke ved de 2 andre tilfælde...

Ok Olebole :-) Har du ikke hørt While / else?? :-) Koder hører man jo ikke om -- man læser dem!

hoejgaard - jeg håber du få det afklaret. Jeg kom blot med de betragtninger jeg mener kan undersøges -- og som Olebole mener er ude i rampen.. som eksempel nævner jeg i starten efter hans kritik uden mening - at du kunne bruge en check :or die(mysql_error()); så du kan få en præcisering af fejlen i din IF sætningen.
Så vil jeg kigge nærmere på Array sagde jeg i min første kommentar og connections.
OLEBOLe er god til at kritisere alle forslag men han fremkommer ikke selv med noget særlige konstruktiv andet end at vi andre skal føles som idioter der ikke kan tænke. :-) Olebole har vist nogen private Frustrationer der skal udløses.. jeg er den der er gået!
Ha -- Pøj pøj!

victor tak for dine forslag, men kan ikke lige gennemskue dem.
Denne virker jo:

  if ($_SESSION['soeger']=='') {
                  $query = mysql_query("SELECT medlemmer.*, fer.* FROM medlemmer, fer WHERE medlemmer.brugernavn = fer.brugernavn && medlemmer.koen LIKE '%$_SESSION[koen]%' && fer.maal LIKE '%$_SESSION[maal]%' && fer.soegerfer LIKE '%$_SESSION[soegerfer]%' && medlemmer.brugernavn LIKE '%".preg_replace("/[_]/","\\\\_",$_SESSION['brugernavn'])."%'"); }

Så der er jo ikke noget galt med tælleren..
Det er de 2 andre forespørgsler der er problemet...

if ($_SESSION['soeger']=='tidligere fer') {
                  $query = mysql_query("SELECT medlemmer.*, fer.* FROM medlemmer, fer WHERE medlemmer.brugernavn = fer.brugernavn && medlemmer.koen LIKE '%$_SESSION[koen]%' && fer.maal LIKE '%$_SESSION[maal]%' && fer.soegerfer LIKE '%$_SESSION[soegerfer]%' && medlemmer.brugernavn LIKE '%".preg_replace("/[_]/","\\\\_",$_SESSION['brugernavn'])."%' "); }
                  if ($_SESSION['soeger']=='kommende fer') {
                  $query = mysql_query("SELECT medlemmer.*, fer.* FROM medlemmer, fer WHERE medlemmer.brugernavn = fer.brugernavn && medlemmer.koen LIKE '%$_SESSION[koen]%' && fer.maal LIKE '%$_SESSION[maal]%' && fer.soegerfer LIKE '%$_SESSION[soegerfer]%' && medlemmer.brugernavn LIKE '%".preg_replace("/[_]/","\\\\_",$_SESSION['brugernavn'])."%' "); }

jeg kommer ikke ind i if sætningen så min
$_SESSION['soeger']=='tidligere fer'
er åbenbart ikke opfyldt....selvom jeg har valgt 'tidligere fer'

damn det var en skrivefejl..
'tidligere fer' skulle have været 'Tidligere fer'  og
'kommende fer'  skulle have været  'Kommende fer'

Nå læg svar dem af jer der vil have points for jeres hjælp og interesse !

Hi :-) Hoejgaard - Jeg synes at det er dig den der skal give sig selv Points. Tænkt at tidligere skulle skrives med stort T. Det er da herligt at du fik det til at virke!

victor22 >> "Koder hører man jo ikke om -- man læser dem!" >> Så undrer det mig, du ikke var i stand til at læse hoejgaards kode - og ligefrem kaldte den dødelig.

Derudover undrer det mig også, hvad det mere præcist er i mit forslag til kode, du ikke finder konstruktivt?

Til sidst må jeg korrigere dig lidt: Jeg kritiserer absolut ikke alle forslag ... kun de, der er grund til at kritisere  =)

ole
du skriver :
2) du bruger fire gange LIKE med et wildcard (%) i hver ende af det søgte ord

Hvordan skal jeg dog undgå det, når jeg vil søge i de 4 felter..??

Måske, du skal undlade at søge på de fire felter - det er ikke altid godt at gøre det, man gerne ville.

Jeg kender ikke din applikation i enkeltheder, så jeg kan ikke foreslå andre løsninger eller et andet design. Jeg kan kun konstatere, at søgningen er _ekstremt_ ineffektiv, og at de fleste kodere ville være villig til at begå mord for at undgå den søgning  =)

læg et svar, dem der vil ha points !

ingen der vil ha points ???

Vores opgave er at forsøge at løse dine kodemæssige problemer. Din opgave i projektet er derefter at vælge én eller flere, der bør have points. Hvem synes _du_, skal have points?  =)

olebole du får pointsne

olebole læg et svar :-)

okay lukker så..