[Javascript] Wisselen van images werkt niet

[Verwijderd]

Ik heb deze code:

code:

<script language="javascript">
function ToggleRowDisplay( objTarget, objTableBody, objImageName )
{
     try
     {
          // Variables
          var strDisplay = "block";
          var strValue = "min";
          var strImagePath = "images/min.gif";
          var strImageName = objImageName.value;
  
           // Toggle Display
          if( objTarget.value == "min" )
          {
               // Hide
               strDisplay = "none";
               strValue = "plus";
               strImagePath = "images/plus.gif";
          }

          // Toggle the table
          objTableBody.style.display = strDisplay;

          // Set button Value
          objTarget.value = strValue;
// deze werkt     document.imgsymptoom.src = strImagePath;
          eval("document." + strImageName + ".src = " + strImagePath);
     }
     catch( expError )
     {
          alert( expError.number + "   " + expError.description );
     }
}
</script>

vervolgens verderop in de HTML pagina:

code:

[img]"images/plus.gif"[/img]

en

code:

[img]"images/plus.gif"[/img]

Omdat ik meerdee images op deze manier wil veranderen bij een muisklik gebruik ik de naam van een img als variabele.
Als ik de document.imgnaam.src gewoon handmatig erin zet werkt het prima, maar de eval() functie krijg ik niet werkend en die heb ik toch echt nodig...

Wat doe ik hier fout?

[whoami]

Javascript hoort thuis bij de buren, het is immers clientside
p&w->w&g.

[Standeman]

Moet het toevallig niet zijn:

code:

eval("document." + strImageName + ".src") = strImagePath;

?

[RobIII]

Standeman schreef op vrijdag 19 augustus 2005 @ 16:01:
Moet het toevallig niet zijn:

code:

eval("document." + strImageName + ".src") = strImagePath;

?

eval is evil...
Gebruik document.getElementById gewoon en geef je images een id in plaats van een name.

[ Voor 20% gewijzigd door RobIII op 19-08-2005 16:06 ]

[Bram77]

Ik heb me niet echt verdiept in wat je allemaal met eval() kan, maar ik gebruik(te) het wel....voor iets heel anders als jij nu probeert.

Dit is een valide manier van gebruik van eval()

Javascript

code:

fucntion useEval(varForEval)
{
 x = 1;
 elmID = eval(varForEval+x); //elmID is nu bruikbaar voor b.v. document.element_1 i.t.t. varForEval
 document.elmID.innerHTML = "Dit is text...";
}

HTML

code:

<a href="javascript:void(0);" onClick="useEval('element_')">
<b id="element_1"></b>

Ik gebruik dit nu helemaal niet meer omdat je ook gewoon document.getElementBy('element_'+x).innerHTML kunt gebruiken. Dat werkt naar mijn smaak prettiger.


edit: Het is me toch niet helemaal duidelijk wat je wil bereiken. Kun je de pagina niet posten? Ik heb het idee dat je het jezelf moeilijker maakt dan nodig is.

[ Voor 37% gewijzigd door Bram77 op 19-08-2005 16:14 ]

[NMe]

JavaScript:

objTarget.src = strImagePath;

[ Voor 14% gewijzigd door NMe op 19-08-2005 16:09 ]

[RobIII]

-NMe- schreef op vrijdag 19 augustus 2005 @ 16:08:

JavaScript:

objTarget.src = strImagePath;

Scherp

Standeman schreef op vrijdag 19 augustus 2005 @ 16:13:
Wat ik meestal doe wanneer ik veel images moet swappen is document.images (= array van alle images in je document) opvragen en daar doorheen lopen..

Waarom niet gewoon de naam doorgeven? En dan dus inderdaad zoals -NMe- aangeeft, of met een document.getElementById ofzo. Wat jij doet is natuurlijk zonde van CPU cycles ja ja, relatief gezien niet natuurlijk, maar toch

[ Voor 61% gewijzigd door RobIII op 19-08-2005 16:14 ]

[Standeman]

Wat ik meestal doe wanneer ik veel images moet swappen is document.images (= array van alle images in je document) opvragen en daar doorheen lopen..

[crisp]

In de meest simpele vorm:

JavaScript:

//-- preload
var min = new Image(); min.src = 'http://mijnsite.com/images/min.gif';
var plus = new Image(); plus.src = 'http://mijnsite.com/images/plus.gif';

function ToggleRowDisplay(img, tableBodyID)
{
    var tableBody = document.getElementById(tableBodyID);

    if (img.src == min.src)
    {
        tableBody.style.display = 'none';
        img.src = plus.src;
    }
    else
    {
        tableBody.style.display = '';
        img.src = min.src;
    }
}

Geen elementen in global namespace aanspreken (IE-only), geen gebruik van deprecated name-attributen, geen eval en plaatjes netjes gepreload.

[ Voor 4% gewijzigd door crisp op 20-08-2005 00:16 ]

[Verwijderd]

Super! Het heeft enorm geholpen... was nodig aan koffie toe :-)

[Skaah]

crisp schreef op zaterdag 20 augustus 2005 @ 00:15:

JavaScript:

        tableBody.style.display = '';

Geweldig! Ik zit altijd te k*tten met dit soort constructies:

JavaScript:

tableBody.style.display = (navigator.appName.indexOf('Explorer') != -1) ? ('block') : ('table-row-group');

[ Voor 9% gewijzigd door Skaah op 20-08-2005 09:07 ]