Optimering af javascript/ajax fil

Frac tillader åbenbart ikke direkte link til filer, så bare gå herind og hent den:

www.simplus.frac.dk

ingen?

hæ hæ Især hastigheden.. øhh det bliver jo liget svært.. køb dig en større Internetforbindelse.. og nu ved jeg ikke om Frac har limit på deres linje men så er der jo ikke meget at gøre..

hvormeget tekst/html sender search.php ?

Det virker skam lynhurtigt hos mig, men når databasen kommer til at indeholde 50.000 poster og der bliver søgt realtime kan det godt virke langsomt hos folk med dårlig internetforbindelse. Derfor vil jeg gerne optimere koden.

Jeg fik at vide af roenving herinde fra at min kode var outdated og ikke særlig klogt opbygget. Det er dog heller ikke mig selv som har lavet den, såeh :-)

search.php er en fil som søger i databasen efter et ord som man skriver, selve databasen kommer som sagt til at indeholde ca. 50.000 poster muligvis mere. Jeg har styr på selve php delen og den kan jeg sagtens selv optimere.

Sig til hvis du vil se php delen, det skal lige siges at jeg ikke har optimeret den endnu!

Håber du kan hjælpe ;-)

Jo men du kan jo ikke forvente at alle har en Internetforbindelse på 1024 eller mere.. men du behøver jo ikke sende alle 50.000 poster..

du kan vel sende 10 eller 20.. det er jo en søning.. ik..

Jo, men som sagt realtime. Dvs hver gang man skriver eller sletter et bogstav søger den automatisk. Der er ingen submit knap jo. Derfor skal den samtidigt søge alle poster igennem.

Men i bund og grund skal koden modificeres så den bliver mere up-to-date. Det er jo slamkode :-)

Jeg kan så kun se at det må være din SQL som skal optimeres.. du kan jo prøve den her : http://vbhansen.dk/javascript/suggest/optionbox/ den søg i 356933 DK-domæner

Hmm, den kan vidst ikke optimeres meget. Jeg ser de har brugt strlen i php og LEFT i mysql til at kombinere en løsning som kun søger efter ting der fylder det antal initialer som man har indtastet. Det kan jeg dog ikke i mit tilfælde.

Men måske har jeg udtrykt mig forkert. Min javascript/ajax kode skal renskrives så den bliver mere moderne. Det er ikke flot/god gode som er skrevet.

må man se din sql-sætning ?

Jeg har ikke gjort noget ved den endnu, så det er sådan set en hel almindelig sql:

mysql_query("SELECT * FROM artikler WHERE LOWER(overskrift) LIKE '%".$q."%' OR LOWER(indhold) LIKE '%".$q."%' ORDER BY id ASC")or die(mysql_error());

brug ikke * definer de felter som du skal bruge.. også selv om du skal bruge dem alle.

SELECT overskrift, indhold FROM artikler WHERE LOWER(overskrift) LIKE '%".$q."%' OR LOWER(indhold) LIKE '%".$q."%' ORDER BY id ASC

LOWER() ?? hvad skal du bruge den til?? køre du binær i dine felter ??

Hmm, har gjort det, men hvad skulle det hjælpe når jeg alligevel skal bruge alle? :-)

Har du andre tilføjelser, evt. til min javascript fil? Eller nogle små ændringer du ville foretage? Måske jeg evt. kunne få fjernet innerHTML og bruge noget andet...

med hensyn til hvorfor jeg bruger LOWER så gør jeg det jo fordi at alle bogstaver i mine felter skal laves om til små bogstaver så søgningen ikke bliver case-sensitive.

ja men så kan du vel bruge strtolower($results)

og så er det vel kun overskrift du skal ha ud..

SELECT overskrift FROM artikler WHERE overskrift LIKE '%".$q."%' OR indhold LIKE '%".$q."%' ORDER BY id ASC

Vi haved også et problem her med en langsom SQL-Server her : http://www.eksperten.dk/spm/556611 Så det kan væer noget sql opsætning hos Frac.

Jeg kan ikke huske om MATCH er hurtiger
men prøv fx.

SELECT overskrift, MATCH(overskrift, indhold) AGAINST ('".$q."' IN BOOLEAN MODE) AS Score
FROM artikler HAVING Score ORDER BY Score DESC LIMIT 0,10;

Den vil søge på hele ord, men du kan så bruge '".$q."*' så skulle den være der..

hvis du skal bruge dit id ;)

SELECT id, overskrift, MATCH(overskrift, indhold) AGAINST ('".$q."' IN BOOLEAN MODE) AS Score
FROM artikler HAVING Score ORDER BY Score DESC LIMIT 0,10;

Skulle måske lige sige at du lige skal se på:

[mysqld]
ft_min_word_len=3 <- minimum længde på ordet
ft_stopword_file  <- fil som indeholder ord som der ikke bliver søgt på

Tak, jeg vil regne genereringstider ud og se ad hvilke metoder der er bedst.

Men vil det sige at min ajax kode ikke har noget med hastigheden på søgningen at gøre?

Nej det er jo ikke Javascript (ALAX) som søger for Dig.. Det er jo din SQL Server.. og det er her det er en god ide at fortælle den hvad den skal gøre, så den ikke laver mere end Du har brug for.

På samme måde som hvis jeg siger du skal købe mælk og sukker.. så skal du jo ikke komme med mælk, sukker, kaffe, kage og meget mere. det vil jo tage længer tid når du skal købe alt muligt andet.. :D

Hehe, det er nu rigtig nok, men det er nu stadig AJAX delen som laver en forbindelse imellem HTML dokumentet og PHP dokumentet såeh den forbindelse har stadig er stor betydning for hastigheden som helhed hvis du forstår. :-)

Det jeg mener "SELECT *..." så skal SQL serveren først finde ud af hvilket felter der er i tablen (*), og så alt det andet den skal..

men nu er LIKE heller ikke serlig hurtiger i forvejn men det kan vi jo ikke lave om på ;)

Jo men så er vi jo igen ved din Internetforbindelse ;)

det du kan gøre er at smide xmlHttp.abort(); på


function showHint(str) {

    if (str.length==0) {
        document.getElementById("txtHint").innerHTML="";
        return;
    }

    xmlHttp=GetXmlHttpObject()

    if (xmlHttp==null) {
        alert ("Your browser does not support AJAX!");
        return;
    }

  if (xmlHttp.readyState != 0) {
    xmlHttp.abort();
  }

    var url="search.php";
    url=url+"?q="+str;
    url=url+"&sid="+Math.random();
    xmlHttp.onreadystatechange=stateChanged;
    xmlHttp.open("GET",url,true);
    xmlHttp.send(null);
}

Ja det kan jeg egentlig godt se nu. Javascript og AJAX kører jo client-side så det afhænger af internetforbindelsens hastighed. Så fik vi da det på plads :-)

Hmm, med hensyn til min query så har jeg egentlig meget godt styr på MySQL og PHP, men ikke særlig meget om hvad der er hurtigst så et par små fif ville være lækkert.

Jeg har lavet en funktion der viser hvor mange resultater der er fundet og hvor mange der findes i databasen. Til det har jeg lavet to query's men kan det gøres på en klogere måde?

Min kode for den funktion ser således ud:

$query1 = mysql_query("SELECT id FROM artikler")or die(mysql_error());
$query2 = mysql_query("SELECT overskrift, indhold FROM artikler WHERE LOWER(overskrift) LIKE '%".$q."%' OR LOWER(indhold) LIKE '%".$q."%' ORDER BY id ASC")or die(mysql_error());
$ialt = mysql_num_rows($query1);
$antal = mysql_num_rows($query2);

Derudover vil jeg bede dig om at smide et svar så du kan få point.

Hvor skal jeg indsætte din rettelse af ajax koden :-)

eller med andre ord, hvilken del skal jeg erstatte med din?

se under din function showHint(str)

Jeg må sige at hvis jeg bruger din opdatering af ajax delen så bliver koden væsentlig langsommere. Jeg aner ikke hvorfor, men det gør den. :-)

Men jeg vil sige at hvis vi kan få ordnet PHP delen så er det nok, så behøver vi ikke pille mere ved AJAX delen. Det gør jeg en anden gang.

Som sagt kan jeg selv, men har du en klogere løsning på problemet jeg nævner foroven?

Læg venligst et svar.

Klogere løsning : ikke andet en at du kan fjerne LOWER() kan ikke se nogle ide i det da LIKE alivel søger i text format (Ikke i Binær)

Tak for det hele du :-)

Selv tak ;)