Skip to main content
se eliminaron 36 caracteres en el cuerpo
Origen Enlace
MitaCloud
  • 2.7k
  • 9
  • 20

Aquí hay varios problemas:

  1. Bucle infinito al estar llamando a init dentro del setInterval.
  2. Como dentro de setInterval estás llamando a init otra vez, por cada segundo estarás creando otro setInterval diferente.
  3. No estás en el mismo contexto para utilizar this.

Una de las formas correctas de hacerlo sería la siguiente:

class Temporizador {
    constructor() {
        this.tiempo = 0;
    }

    getTime() {
        return this.tiempo;
    }

    init() {
        var self = this;
        setInterval(function() => {
            selfthis.tiempo++;
            document.getElementById('countdown').innerHTML = selfthis.tiempo;
        }, 1000);
    }
}

var temporizador = new Temporizador();
temporizador.init();
<div id="countdown">0</div>

Fíjate que dentro de setInterval tenemos dos lineas de código que nos modifica el valor del contador y muestra el resultado.

Aquí hay varios problemas:

  1. Bucle infinito al estar llamando a init dentro del setInterval.
  2. Como dentro de setInterval estás llamando a init otra vez, por cada segundo estarás creando otro setInterval diferente.
  3. No estás en el mismo contexto para utilizar this.

Una de las formas correctas de hacerlo sería la siguiente:

class Temporizador {
    constructor() {
        this.tiempo = 0;
    }

    getTime() {
        return this.tiempo;
    }

    init() {
        var self = this;
        setInterval(function() {
            self.tiempo++;
            document.getElementById('countdown').innerHTML = self.tiempo;
        }, 1000);
    }
}

var temporizador = new Temporizador();
temporizador.init();
<div id="countdown">0</div>

Fíjate que dentro de setInterval tenemos dos lineas de código que nos modifica el valor del contador y muestra el resultado.

Aquí hay varios problemas:

  1. Bucle infinito al estar llamando a init dentro del setInterval.
  2. Como dentro de setInterval estás llamando a init otra vez, por cada segundo estarás creando otro setInterval diferente.
  3. No estás en el mismo contexto para utilizar this.

Una de las formas correctas de hacerlo sería la siguiente:

class Temporizador {
    constructor() {
        this.tiempo = 0;
    }

    getTime() {
        return this.tiempo;
    }

    init() {
        setInterval(() => {
            this.tiempo++;
            document.getElementById('countdown').innerHTML = this.tiempo;
        },1000);
    }
}

var temporizador = new Temporizador();
temporizador.init();
<div id="countdown">0</div>

Fíjate que dentro de setInterval tenemos dos lineas de código que nos modifica el valor del contador y muestra el resultado.

Revisión ortográfica y eliminación de etiquetas no necesarias en el snippet
Origen Enlace
OscarGarcia
  • 28.5k
  • 3
  • 29
  • 64

Aquí hay varios problemas:

  1. Bucle infinito al estar llamando a init dentro del setInterval.
  2. Como dentro de setInterval estasestás llamando a init otra vez, por cada seguno estarassegundo estarás creando otro setInterval diferente.
  3. No estasestás en el mismo contexto para utilizar this.

Una de las formas correctas de hacerlo seriasería la siguiente:

class Temporizador {
    constructor() {
        this.tiempo = 0;
    } 

    getTime() {
        return this.tiempo;
    }

    init() {
        var self = this;
        setInterval(function() {
            self.tiempo++;
            document.getElementById('countdown').innerHTML = self.tiempo;
        }, 1000);
    }
}

var temporizador = new Temporizador();
temporizador.init();
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width">
  <title>JS Bin</title>
</head>
<body>
  <div id="countdown">0</div>
</body>
</html>

Fíjate que dentro de setInterval tenemos dos lineas de código que nos modifica el valor del contador y muestra el resultado.

Aquí hay varios problemas:

  1. Bucle infinito al estar llamando a init dentro del setInterval
  2. Como dentro de setInterval estas llamando a init otra vez, por cada seguno estaras creando otro setInterval diferente.
  3. No estas en el mismo contexto para utilizar this

Una de las formas correctas de hacerlo seria la siguiente:

class Temporizador {
    constructor() {
        this.tiempo = 0;
    }
    getTime() {
        return this.tiempo;
    }

    init() {
        var self = this;
        setInterval(function() {
            self.tiempo++;
            document.getElementById('countdown').innerHTML = self.tiempo;
        }, 1000);
    }
}

var temporizador = new Temporizador();
temporizador.init();
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width">
  <title>JS Bin</title>
</head>
<body>
  <div id="countdown">0</div>
</body>
</html>

Fíjate que dentro de setInterval tenemos dos lineas de código que nos modifica el valor del contador y muestra el resultado.

Aquí hay varios problemas:

  1. Bucle infinito al estar llamando a init dentro del setInterval.
  2. Como dentro de setInterval estás llamando a init otra vez, por cada segundo estarás creando otro setInterval diferente.
  3. No estás en el mismo contexto para utilizar this.

Una de las formas correctas de hacerlo sería la siguiente:

class Temporizador {
    constructor() {
        this.tiempo = 0;
    } 

    getTime() {
        return this.tiempo;
    }

    init() {
        var self = this;
        setInterval(function() {
            self.tiempo++;
            document.getElementById('countdown').innerHTML = self.tiempo;
        }, 1000);
    }
}

var temporizador = new Temporizador();
temporizador.init();
<div id="countdown">0</div>

Fíjate que dentro de setInterval tenemos dos lineas de código que nos modifica el valor del contador y muestra el resultado.

Origen Enlace
MitaCloud
  • 2.7k
  • 9
  • 20

Aquí hay varios problemas:

  1. Bucle infinito al estar llamando a init dentro del setInterval
  2. Como dentro de setInterval estas llamando a init otra vez, por cada seguno estaras creando otro setInterval diferente.
  3. No estas en el mismo contexto para utilizar this

Una de las formas correctas de hacerlo seria la siguiente:

class Temporizador {
    constructor() {
        this.tiempo = 0;
    }
    getTime() {
        return this.tiempo;
    }

    init() {
        var self = this;
        setInterval(function() {
            self.tiempo++;
            document.getElementById('countdown').innerHTML = self.tiempo;
        }, 1000);
    }
}

var temporizador = new Temporizador();
temporizador.init();
<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width">
  <title>JS Bin</title>
</head>
<body>
  <div id="countdown">0</div>
</body>
</html>

Fíjate que dentro de setInterval tenemos dos lineas de código que nos modifica el valor del contador y muestra el resultado.