Eingabedaten überprüfen

[Afrika]

Ich mach mal einen neuen Threat auf, da es ein anderes Problem ist.

PHP-Code:

 // inserting a new link
if(isset($_POST['newlink'])){
        if(eregi("http://", $_POST['url'])){
                $_POST['url']=eregi_replace("http://", "", $_POST['url']);
                }
        $newlinkquery="INSERT INTO ".DB_PREFIX."links (link, url, description, submitter, submitter_email, category, uptime) VALUES ('".$_POST['link']."', '".$_POST['url']."', '".$_POST['description']."', '".$_POST['submitter']."', '".$_POST['submitter_email']."', '".$_POST['category']."', $time)"; 
        $newlinkresult=mysql_query($newlinkquery);
        if($newlinkresult==1){
                $ws->assign("_ROOT.TITLE", "Link vorschlagen");
                $ws->assign("_ROOT.MESSAGE", "Ihre Eingabe war erfolgreich. Ein Editor wird es überprüfen.");
                $ws->newBlock("redirect");
                $ws->assign("LOCATION", "index.php?cat=".$_POST['cat']);
        }else{
                $ws->assign("_ROOT.TITLE", "Link vorschlagen");
                $ws->assign("_ROOT.MESSAGE", "Fehler:".mysql_error());
        }
        }elseif(isset($_POST['newcat'])){ 


Nun habe ich das Problem, dass sich doch viele doppelt eintragen.
Mit einer if Abfrage sollte es doch möglich sein das zu verhindern.
So in der Art:

PHP-Code:

 // inserting a new link

if($_POST['url']) == ??????)

    {

if(isset($_POST['newlink'])){
        if(eregi("http://", $_POST['url'])){
                $_POST['url']=eregi_replace("http://", "", $_POST['url']);
                }
        $newlinkquery="INSERT INTO ".DB_PREFIX."links (link, url, description, submitter, submitter_email, category, uptime) VALUES ('".$_POST['link']."', '".$_POST['url']."', '".$_POST['description']."', '".$_POST['submitter']."', '".$_POST['submitter_email']."', '".$_POST['category']."', $time)";
        $newlinkresult=mysql_query($newlinkquery);
        if($newlinkresult==1){
                $ws->assign("_ROOT.TITLE", "Link vorschlagen");
                $ws->assign("_ROOT.MESSAGE", "Ihre Eingabe war erfolgreich. Ein Editor wird es überprüfen.");
                $ws->newBlock("redirect");
                $ws->assign("LOCATION", "index.php?cat=".$_POST['cat']);
        }else{
                $ws->assign("_ROOT.TITLE", "Link vorschlagen");
                $ws->assign("_ROOT.MESSAGE", "Fehler:".mysql_error());
        }
        }elseif(isset($_POST['newcat'])){
}
else
    {
    echo "Fehler die Url ist schon eingetragen";
    } 


[xabbuh]

Auf was willst du denn überprüfen?

[|Coding]

Zitat von Afrika

PHP-Code:

if($_POST['url']) == ??????) 


was willst du denn damit prüfen?
ausserdem, wenn du doch weist was du machen willst, dann mach es doch auch so. bzw. probier es aus.

[Afrika]

Wenn ich wüsste was ist anstelle der ????? einsetzen sollte hätte ich das schon gemacht glaube mir ;)
Es soll geprüft werden ob die URl schon vorhanden ist.

[|Coding]

Zitat von Afrika

Wenn ich wüsste was ist anstelle der ????? einsetzen sollte hätte ich das schon gemacht glaube mir ;)
Es soll geprüft werden ob die URl schon vorhanden ist.

vorhanden? hä? wie meinst du denn das?

[Afrika]

na ob die url schon mal eingetragen worden ist.
Ich versuche es gerade mit einem SQL Befehl, aber das klappt alles nicht.
Es sind gerade jetzt 3500 Datensätze eingetragen.
Davon ist aber die Hälfte doppelt.
Nun will ich zum einen die doppelten Datensätze anhand der doppelt eingetragenen Url löschen und zusätzlich die Eingabe von doppelten in zukunft verhindern.
select distinct sollte vielleicht die Lösung sein. Allerdings weiß ich nun nicht wie ich das auch einbauen kann in den Code oben.

Zum Löschen der bestehenden Daten habe ich probiert:

PHP-Code:

 SELECT one.`url` 
FROM  `links`  AS one
JOIN  `links`  AS two ON one.url = two.url 


Klappt aber nicht.
Fehlermeldung:

You have an error in your SQL syntax near 'ON one.url = two.url LIMIT 0, 30' at line 3

[|Coding]

du solltest dir

mysql.com
php.net
tut.php-q.net

auf jedenfall mal ansehen.
du kannst ja wirklich nicht das geringste selbst, was ja nicht schlimm ist, so hat jeder mal angefangen.

es bringt aber nichts, wenn wir dir hier helfen und helfen und müssen dir dann trotzdem immer wieder alles vorkauen, weil du es nicht nachvollziehen kannst.

wie gesagt, ist nichts gegen dich, doch ohne das kleinste bissel grundwissen nutzt alles nichts.

[Afrika]

Kann sein, das du Recht hast, aber ich lerne nun mal bei doing.
Deshalb wähle ich gerne Foren, weil dort solche Sachen eben beim "doing"
gelöst werden.
Wenn ich hier nerve, dann tut es mir leid.
ps
So ganz ohne Grundwissen bin ich auch wieder nicht, ich tue mich eben etwas schwer mit neuen Funktionen.

[|Coding]

[deleted]

[Afrika]

Wie kommst Du denn auf de Code?