Bedingung in IF Funktion einbauen?

[Pseiko]

Ich habe folgenden Code:

$sql = mysql_query("SELECT * FROM {$prefix}pages WHERE url = '$url'");
    if (mysql_num_rows($sql) > 0){
     
        $error_duplication = true;
    }else{
        if (strlen($title) < 5 || strlen($title) > 100) { $error_variables = true; $errors["title"] = true; };
        if (strlen($description) < 40 || strlen($description) > 500) { $error_variables = true; $errors["description"] = true; };
        if (strlen($keywords) < 20 || strlen($keywords) > 200) { $error_variables = true; $errors["keywords"] = true; };
        if (strlen($email) < 1 || strlen($email) > 200) { $error_variables = true; $errors["email"] = true; };
        if (strlen($adresse) < 0 || strlen($adresse) > 200) { $error_variables = true; $errors["adresse"] = true; };
        if (strlen($telefon) < 0 || strlen($telefon) > 30) { $error_variables = true; $errors["telefon"] = true; };
        if (strlen($telefax) < 0 || strlen($telefax) > 30) { $error_variables = true; $errors["telefax"] = true; };
        if (strlen($bild) < 0 || strlen($bild) > 200) { $error_variables = true; $errors["bild"] = true; };
        if (strlen($url) < 0) { $error_variables = true; $errors["url"] = true; };
        if (!$error_variables){
            mysql_query("INSERT INTO {$prefix}pages (url,title,description,keywords,email, adresse, telefon, telefax, bild, accepted,ip,category) VALUES ('$url','$title','$description','$keywords','$email','$adresse','$telefon','$telefax','$bild','n','".getenv("REMOTE_ADDR")."',$c)");
        }; 

ich möchte nun hier irgendwie hinzufügen, dass wenn der Tabellenwert bei 'url' "http://" ist kein Duplication-Error entsteht:

    $sql = mysql_query("SELECT * FROM {$prefix}pages WHERE url = '$url'");
    if (mysql_num_rows($sql) > 0){
     
        $error_duplication = true; 

[garyx7de]

http://www.scripts-online.net/?scriptid=222

Beschreibung:
Mit strpos() kann man in einem String einen String suchen.
Hier suchen wir "bs" in "tri tra tralala".
bs nicht vorhanden ist , ist $pos = false
if ($pos === false) <-- hier vergleicht man mit ===
Quelltext:

<?
$mystring = "tri tra tralala";

$pos = strpos ($mystring, "bs");
if ($pos === false) { echo "nicht gefunden";}
else { echo "gefunden"; }


?>

[Pseiko]

geht sowas auch:

if not ($pos === false) { echo "nicht gefunden";}

[garyx7de]

ne wenn dann
if ($pos !== false) { echo "nicht gefunden";}

[Pseiko]

Ich habe es so geändert aber es geht nicht.

    $sql = mysql_query("SELECT * FROM {$prefix}pages WHERE url = '$url'");
    if (mysql_num_rows($sql) > 0){
     
        $error_duplication = true;
    }else{
        if (strlen($title) < 5 || strlen($title) > 100) { $error_variables = true; $errors["title"] = true; };
        if (strlen($description) < 40 || strlen($description) > 500) { $error_variables = true; $errors["description"] = true; };
        if (strlen($keywords) < 20 || strlen($keywords) > 200) { $error_variables = true; $errors["keywords"] = true; };
        if (strlen($email) < 1 || strlen($email) > 200) { $error_variables = true; $errors["email"] = true; };
        if (strlen($adresse) < 0 || strlen($adresse) > 200) { $error_variables = true; $errors["adresse"] = true; };
        if (strlen($telefon) < 0 || strlen($telefon) > 30) { $error_variables = true; $errors["telefon"] = true; };
        if (strlen($telefax) < 0 || strlen($telefax) > 30) { $error_variables = true; $errors["telefax"] = true; };
        if (strlen($bild) < 0 || strlen($bild) > 200) { $error_variables = true; $errors["bild"] = true; };
        if (strlen($url) < 0) { $error_variables = true; $errors["url"] = true; };
        if (!$error_variables){
            mysql_query("INSERT INTO {$prefix}pages (url,title,description,keywords,email, adresse, telefon, telefax, bild, accepted,ip,category) VALUES ('$url','$title','$description','$keywords','$email','$adresse','$telefon','$telefax','$bild','n','".getenv("REMOTE_ADDR")."',$c)");
        };

Das habe ich eingefügt:

    $mystring = "http://";
    $pos = strpos ($mystring, "$url");
    if ($pos !== false) { 
        $error_duplication = false;}
    }; 

[3DMax]

strpos ist ungeeignet - findet ja alles, wo http:// vorkommt.
besser:

$error_duplication=(bool)preg_match('|^[url]http://[/url]$|i', $url);

und brich deinen code um!

[jhaase]

erstens heisst es IF Bedingung und nicht IF Funktion.
Zweitens solltest Du die sql-Abfrage überhaupt nicht machen, wenn url=http:// ist
Die Logik bis zum else-Zweig wird dadurch etwas komplizierter aber das bringst Du sicher hin.
Hier ist ein Versuch.

if($url=='http://') $error_duplication=false;
else {
  $sql = mysql_query("SELECT * FROM {$prefix}pages WHERE url = '$url'");
    if (mysql_num_rows($sql) > 0){    
        $error_duplication = true;
    }else{
        $error_duplication = false;
   }
}

if($error_duplication ==false)  [edit: anstatt !=true ]
{
   ...all deine weiteren tests
   ...dein INSERT
} 

[garyx7de]

strpos -- Find position of first occurrence of a string

[3DMax]

Original geschrieben von garyx7de
strpos -- Find position of first occurrence of a string

und genau darum ist diese funktion ungeeignet, wenn man herausfinden möchte, ob der string nur (ausschließlich) 'http://' ist.

@jhaase - die programmlogik habe ich mal so stehen gelassen, hätte zu lange gedauert, alles absurde aufzuzählen.
lustig fand ich aber auch: strlen($string) < 0 - frage mich, ob diese bedingung jemals greift

[garyx7de]

Original geschrieben von 3DMax
und genau darum ist diese funktion ungeeignet, wenn man herausfinden möchte, ob der string nur (ausschließlich) 'http://' ist.

@jhaase - die programmlogik habe ich mal so stehen gelassen, hätte zu lange gedauert, alles absurde aufzuzählen.
lustig fand ich aber auch: strlen($string) < 0 - frage mich, ob diese bedingung jemals greift

ich versteht deine aussage so das man es auch so machen könnten

if($url == 'http://'){
//
}

[jhaase]

Original geschrieben von 3DMax
@jhaase - die programmlogik habe ich mal so stehen gelassen, hätte
zu lange gedauert, alles absurde aufzuzählen.

WAS hast du stehen gelassen - wessen Programmlogik??

Problem war ja bloss, dass seine if-Schleife einen sql-duplication error gibt, wenn
es keine url hat. Der Rest ist im, für den deutschsprachigen Raum, üblichen Niveau.

Vermutlich sind alle bisherigen Lösungen, inklusive meine, falsch. Er müsste, falls es
keine url hat, eine sql-NULL in die Datenbank eintragen. Weiss jemand, ob sich das
mit UNIQUE verträgt?

[garyx7de]

Original geschrieben von jhaase
WAS hast du stehen gelassen - wessen Programmlogik??

Problem war ja bloss, dass seine if-Schleife einen sql-duplication error gibt, wenn
es keine url hat. Der Rest ist im, für den deutschsprachigen Raum, üblichen Niveau.

Vermutlich sind alle bisherigen Lösungen, inklusive meine, falsch. Er müsste, falls es
keine url hat, eine sql-NULL in die Datenbank eintragen. Weiss jemand, ob sich das
mit UNIQUE verträgt?

Teste es doch

[uwe59]

if (strlen($adresse) < 0 

Kann mir mal jemand einen String zeigen, der dabei einen negativen Wert liefert?

Gruß
Uwe

[jhaase]

Original geschrieben von uwe59

if (strlen($adresse) < 0 

Kann mir mal jemand einen String zeigen, der dabei einen negativen Wert liefert?

Man muss das im Zusammenhang sehen, es hat auch Abfragen der Art:
if (strlen($keywords) < 20 || strlen($keywords) > 200)

Insofern ist es entschuldbar oder sogar als Platzhalter erklärbar.

[jhaase]

Original geschrieben von garyx7de
Teste es doch

Die Frage, ob es schon jemand weiss, braucht weniger Buchstaben und ist deswegen naheliegender. Im Durchschnitt über alle Fragen ist es effizienter, zuerst zu fragen und dann mit einer gewissen Wahrscheinlichkeit zu warten und mit einer
noch kleineren Wahrscheinlichkeit nochmals fragen oder auszuprobieren oder nie die Antwort zu bekommen.
(Markoff Wahrscheinlichkeiten und Erwartungswert etwa so:
Summe p^k * frage + (1-p)^n * probieren)