Struppi: Erweiterung der DHTML.JS

Beitrag lesen

Hier die Suchfunktion:

Da sind einige Seltsamkeiten drin.

function searcher(suchwortges)
{
if (NS4) {
  alert("Vorerst nur fuer MS IExplorer. An einer Loesung fuer andere Browser wird gearbeitet ...");
  return;
}

Warum du einen Browser - den es sowieso nicht mehr gibt - ausschließt, obwohl du JS benutzt den eigentlich nur ein Browser kann. Interessant wäre auch zu wissen, wie du NS4 ermittelst

//Suchwörter trennen und anzahl festhalten
meldung = "Suchen nach ... ";
suchwort = suchwortges.split(" ");  //in einzelne Wörter trennen
for (var i = 0; i < suchwort.length; i++) {
suchwortanz++;
meldung = meldung + "Wort " + suchwortanz + ": [" + suchwort[i] + "], ";
}

Wozu die Variabel suchwortanz? Das hat den gleichen Wert wie i + 1
Verkettungen kannst du auch kürzer schreiben:
meldung += "Wort " + suchwortanz + ": [" + suchwort[i] + "], ";

/* Alle P-Tags suchen, und in 2 Arrays speichert (mit und ohne TAG) */
anzp = document.all.tags("p").length-3;

getElementsByTagName() kennst du ja schon.
Und das -3 deuten daraufhin, dass du das anders lösen willst.

for (var k = 0; k <= anzp; k++) {

nur ein Stil Hinweis: Du kannst hier durchaus wieder i als Schleifenvariabel verwenden, k kommt immer dann zum Einsatz, wenn du in einer Schleife bist und i belegt ist.

if (document.all.tags("p")[k].name != null)  {

Das p Element darf keine Eigenschaft name haben nur die Universalattribute

document.write("<HTML><HEAD><link rel=stylesheet type="text/css" href="xformate.css"></HEAD><BODY>");

Du solltest auch mit JS gernerierte Seiten niczht im Quirksmode anzeigen lassen. ausserdem kannst du dir das maskieren sparen wenn du einfahce uhnd doppelte Anführungszeichen verwendest.

document.write('<HTML><HEAD><link rel=stylesheet type="text/css" href="xformate.css"></HEAD><BODY>');

Struppi.

--
Javascript ist toll (Perl auch!)