[SOLUCIONADO] Problema en consola

[JUMASOL]

Hola

Tengo una función para efecto de scroll en un archivo scroll.js, que incluye esta línea 14:

Código:

scrollTop: $(target).offset().top

Luego he colocado una función para cambiar el contenido de unos enlaces así:

Código:

var elements = document.getElementsByClassName('menus');
for (var i = 0; i < elements.length; i++) {
elements[i].setAttribute("href", "#");
}

Es bien fácil y todo funciona bien. Pero luego miras la consola de js y salta este problema:

Código:

scroll.js:14 Uncaught TypeError: Cannot read property 'top' of null
at HTMLAnchorElement.jump (scroll.js:14)

Lo está causando la función para el cambio de link porque cuando la elimino desaparece el error.

Aún así, todo funciona correctamente. O igual no y luego falla en algún dispositivo.

¿Me podéis decir qué es esto?

[ArturoGallegos]

Como el error dice ocurre porque en X momento el valor que intetas recuperar no existe, esto lo solventas validando previamente que exista

no soy tan diestro en jquery pero creo que seria algo así

Código Javascript:

let top = $(target).offset() ? $(target).offset().top : 0;
scrollTop: top
 

o quizás con

Código Javascript:

let top = $(target).offset().top || 0;
scrollTop: top
 

[JUMASOL]

Funciona, pero en ambos casos arroja este error en consola:

Código:

Uncaught SyntaxError: Unexpected identifier

En cualquier caso, gracias por responder.

[prueba230683]

Si $(target).offset() devuelve null o undefined será porque el elemento "target" (que no sabemos nada de él porque no lo has mostrado) es o bien invisible (visibilty: hidden), o directamente no existe en el documento.

Igual te referías a $(event.target), que es bien distinto:

https://api.jquery.com/event.target/

En resumen, verifica cuál es el valor de esa variable target.

[JUMASOL]

El problema es como dices. Utilicé este script en otra web donde había un "top". Luego lo traje a mi muevo proyecto sin quitarlo.

Lo quité y ya funciona.

Un saludo y gracias.