0

Siempre que entro a esta función y encuentra el id(1) no continua revisando si hay mas elementos con el id 3 y 5. ¿Cómo lo soluciono?

En la etiqueta a suelen solo listarse a veces las id 1,3 y 5. unas veces solo está el 3 u otras el 1 y 5 o el 5 solo.. varia.

                <ul class="dropdown-menu alert-dropdown"  onclick="javascript:cambia_enlace();">
                    <li ng-repeat="datalist in Notificaciones" >
                        <a id="{{datalist.ID_NOTIFICACIONES}}" href="">
                            <div>{{ datalist.TEXTO_NOTIFICACION }}</div>
                        </a>
                    </li>
                </ul>


        function cambia_enlace() {
            var id = document.getElementById('1')
            if (id == null) {
                var id2 = document.getElementById('3');
                if (id2 == null) {
                    var id3 = document.getElementById('5');
                    if (id3 == null) {
                    } else {
                        id3.href = "Principal.html#/Pago";
                    }
                } else {
                    id2.href = "Principal.html#/Abono";
                }
            } else {
                id.href = "Principal.html#/Inventario";
            }
        }
17
  • ¿Que es exactamente el 'elemento'?
    – Lithorell
    Commented el 21 jul. 2017 a las 6:08
  • @Lithorell mira arriba lo que hize y asi funciona, pero hay un problema..
    – RubxnMC
    Commented el 21 jul. 2017 a las 6:10
  • No tenemos la bola de cristal para adivinar cual es el problema si no lo explicas ;)
    – Lithorell
    Commented el 21 jul. 2017 a las 6:11
  • @Lithorell ya lo modifiqué.. :v perdón
    – RubxnMC
    Commented el 21 jul. 2017 a las 6:12
  • 1
    Pero es que eso es normal, cuando haces click revisa si esta la 1 y como esta usa esta. La funcion tienes que hacerla a nivel de enlace no a nivel de ul
    – Lithorell
    Commented el 21 jul. 2017 a las 6:41

2 respuestas 2

0

Lo primero, si la id es variable, no puedes buscar al elemento por ella. Yo te recomiendo variar la clase y mantener la id. De esta manera, podras buscar al elemento con facilidad. Una vez que lo tengas seleccionas su clase y la metes en un switch con un case por cada opcion.

He metido un html y una funcion inicializando para que sea mas comprensible y asi ejecutarlo.

function cambia_link(){
  
  var ida= "avariable";
  var elemento = document.getElementById(ida);
  var class_elemento = elemento.className;
  
  switch(class_elemento){
    case "a1":  
    elemento.href= "https://es.stackoverflow.com/questions/tagged/javascript";
    break;
    case "a2":
    elemento.href= "https://es.stackoverflow.com/questions/tagged/php";
    break;
    case "a3":
    elemento.href= "https://es.stackoverflow.com/questions/tagged/html";
    break;
  }
}

(function(){
cambia_link();
})();
<a id="avariable" class="a3" href="#"> Anchor variable</a>

0

Ya que como etiqueta has puesto jquery, una posible solución usando jquery y no javascriptpodría ser:

<ul class="dropdown-menu alert-dropdown">
     <li ng-repeat="datalist in Notificaciones" >
          <a id="{{datalist.ID_NOTIFICACIONES}}" class="enlace" href="">
               <div>{{ datalist.TEXTO_NOTIFICACION }}</div>
          </a>
     </li>
</ul>

La parte de jquerysería esta

$(document).ready(function(){
  $(".class").click(function(){
      var id= parseInt($(this).attr('id'));

      switch(id){
        case 1:  
          $(this).attr('href','Principal.html#/Inventario');
          break;
        case 2:
          $(this).attr('href','Principal.html#/Abono');
          break;
        case 3:
          $(this).attr('href','Principal.html#/Pago');
          break;
        default:
          $(this).attr('href','Principal.html#/Inventario');
          break;
      }
  });
)}

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.