if-Abfrage von Sichtbarkeit (visibility)

[Schnâxélïòn]

hi,
ich wollte die Sichtbarkeit von einem Objekt abfragen und jenachdem ob es sichtbar ist oder versteckt mit einem link verstecken bzw anzeigen. In diesem script ist ein fehler drin aber ich weis nicht was für einer. Danke für eure Hilfe im vorraus.

<script type="text/javascript"> var visi; visi=document.getElementById("vid1").style.visibility;
function onoffvid1 () {
if ( visi == "hidden") { 
if (document.getElementById) {     document.getElementById("vid1").style.visibility = "visible";     }
 }
else { 
 if (document.getElementById) {     document.getElementById("vid1").style.visibility = "hidden";     } </script> 

[frodenius]

wenn du deinen code gescheit einrücken würdest, würdest du den fehler sofort sehen!!!

<script type="text/javascript"> 
var visi; 
visi=document.getElementById("vid1").style.visibility;
function onoffvid1 () {
    if ( visi == "hidden") { 
        if (document.getElementById) {     
            document.getElementById("vid1").style.visibility = "visible";     
        }
    }
    else { 
        if (document.getElementById) {     
            document.getElementById("vid1").style.visibility = "hidden";     
    } 

</script>

und nebenbei: da ist auch ein logischer fehler drin. zumindest fehlt noch was.

[Schnâxélïòn]

ok danke die }} hab ich wohl vergesse, aber was ist das sonst noch für ein fehler es wäre nett wenn man mir das sagen könnte

[wahsaga]

Original geschrieben von Schnâxélïòn
aber was ist das sonst noch für ein fehler es wäre nett wenn man mir das sagen könnte

Erst mal der übliche, schon x-fach diskutierte: Über element.style kann man nur die Werte von Eigenschaften abfragen, die per style-Attribut oder bereits zuvor per JavaScript über das style-Objekt gesetzt wurden - und keine, die sich bspw. aus den Regeln in einem per <style>- oder <link>-Element eingebundenen CSS ergeben haben.

Und darüber hinaus setzt du den Wert der Variablen, die du in den If-Bedingungen abfragst, nur ein einziges Mal vor Beginn der Funktion, und änderst ihn danach nicht mehr - also spiegelt er zwischenzeitliche Änderungen am Wert der Eigenschaft, die deine Funktion vornimmt, auch nicht wieder.

[Blackgreetz]

Entweder sehe ich den Sinn nicht, oder " if document.getElementById) " ist nicht gerade sinnvoll, weil du visi damit schon definierst.

mfg

[goth]

@Schnâxélïòn

Tippst Du gerne?

Warum machst Du nicht sowas?

<script type="text/javascript"><!--

function onoffvid1 () {
	document.getElementById("vid1").style.visibility = ( document.getElementById("vid1").style.visibility == 'visible' ? 'hidden' : 'visible' );
}

//--></script>

[Schnâxélïòn]

nein ich tippe nich gerne
aber ich habe gerade erst angefangen webseiten zu programmieren dh das hab/hatte ich nich so drauf.
auf jeden fall danke für eure hilfe aber ich habe anderweitig herausgefunden dass die variable in die funktion gehört dann funktionierts auch
gruß

[wahsaga]

Original geschrieben von Schnâxélïòn
aber ich habe anderweitig herausgefunden dass die variable in die funktion gehört

"Anderweitig herausgefunden" - na du bist ja'n Held ...

[asp2php]

OffTopic:

---

Die Leute unterschätzen die Fähigkeit von Google und somit immer wieder erwischt

---

[frodenius]

@goth:
<!-- ?? das ist einfach nur ein überflüssiger anachronismus.
es gibt seit 15 jahren keine browser mehr die <script> nicht kennen!!

[goth]

Naund? Gibt's 'ne Fehlermeldung oder Warnung??

Aber wenn Du meinst ... dann eben so:

<script type="text/javascript" language="javascript">
//<![CDATA[

function onoffvid1 () {
document.getElementById("vid1").style.visibility = ( document.getElementById("vid1").style.visibility == 'visible' ? 'hidden' : 'visible' );
}

//]]>
</script>