[Tschubai]

bräuchte auch mal hilfe :



was stimmt da nicht oder wie wärs richtig?

wäre schön wenn das jemand korrigieren könnte !

danke im voraus

gruß

[Witi]

Alles was mir so auffällt:

Zeile 6,9: Variablen immer mit var deklarieren, sonst haben sie eine globale Sichtbarkeit, also

var flash = false

bzw

for (var a = 0) ...

Zeile 11: Was soll denn bitte

parseInt(navigator.plugins[a].name)

liefern? In Name findest du keine Zahl, ich denke die korrekte Überprüfung sollte so aussehen:

(navigator.plugins[a].name == 'Shockwave Flash')

Zeile 19: In der If-Bedingung würde ich noch ein break einbauen, da er sonst weiter das Array durchläuft.

[Tschubai]

erstmal danke für die schnelle antwort @ Witi !

sieht jetzt so aus :

05 <script type="text/javascript">
06 var flash = false;
07 if ((navigator.plugins) && (navigator.plugins.length > 0))
08 {
09 for (var a = 0; a < navigator.plugins.length; a++)
10 if ((navigator.plugins[a].name.indexOf('Flash') != -1) &&
11 (navigator.plugins[a].name == 'Shockwave Flash'))
12 {
13 var flash = true;
14 break;
15 }
16 {
17 if ((navigator.mimeTypes) && (navigator.mimeTypes.length > 0))
18 for (var a = 0; a < navigator.mimeTypes.length; a++)
19 if (navigator.mimeTypes[a].type.indexOf("application/x-shockwave-flash")>= 0)}
20 var flash = true;
21 break;}
22 </script>

mozilla findet jetzt keine fehler mehr,
phase5 und notepad++ auch nicht .
funktionalität ist gegeben.

scheint also io !?

anmerkung :
bin halt mit java nicht so firm^^

gruß

[Tschubai]

@ Witi ,

war wohl bissel vorschnell ... es sind doch noch fehler drin gewesen.
habs noch mal umgebaut und die variable für die flashversion entfernt :

<script type="text/javascript">
var flash = false;
if ((navigator.plugins) && (navigator.plugins.length > 0))
{
for (var a = 0; a < navigator.plugins.length; a++)
if ((navigator.plugins[a].name.indexOf('Flash') != -1) &&
(navigator.plugins[a].name == 'Shockwave Flash'))
{
var flash = true;
break;
}
}
</script>

so funzt es jetzt

danke nochmal für die hilfe

gruß