Skip to main content
se añadieron 809 caracteres en el cuerpo
Origen Enlace
diegoveloper
  • 7.3k
  • 2
  • 11
  • 20

El problema es que estás creando el Timer cada vez que se hace rebuild, es decir, cada vez que usas el notifyListeners vuelve a entrar al método build, por lo tanto llama a cronometer entonces vuelve a crear otro Timer.

La manera correcta sería crear el Timer solo una vez, o también puedes crear una validación para que no vuelva a crear el timer.

Tu clase quedaría así:

class PriceProvider extends ChangeNotifier{
  int _time = 60;
  Timer? _timer;

  int get time{
    return _time;
  }

  void cronometer(){//método que activa el timer
     if(_timer == null){
    _timer = Timer.periodic(const Duration(seconds: 1), (Timer timer){
      print(DateTime.now());//Imprimo la fecha para que vean cada cuanto tiempo se ejecuta el código
      
      _time += -1;//decrementa time

      if(_time == 0){
        _time = 60;
      } 

      notifyListeners(); 
    });
   }
  } 

}

Otra forma de solucionarlo, como te decía, solo ejecutar el cronometer una vez.

class PriceWithClock extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    PriceProvider priceProvider = Provider.of<PriceProvider>(context);
    // no llamarlo aquí.
    //priceProvider.cronometer();
    return CircularPercentIndicator(
      radius: 100.0,
      lineWidth: 5.0,
      percent: 1-priceProvider.time/60,
      center: Text("00:${priceProvider.time}"),
     ),
    );
  }
}

Llamar al cronometer cuando se crea el Provider.

void main() async{
  runApp(    
    MultiProvider(
      providers: [
        ChangeNotifierProvider(create: (_) => PriceProvider()..cronometer()),
      ],
      child: MyApp(),
    ),
  );
} 

El problema es que estás creando el Timer cada vez que se hace rebuild, es decir, cada vez que usas el notifyListeners vuelve a entrar al método build, por lo tanto llama a cronometer entonces vuelve a crear otro Timer.

La manera correcta sería crear el Timer solo una vez, o también puedes crear una validación para que no vuelva a crear el timer.

Tu clase quedaría así:

class PriceProvider extends ChangeNotifier{
  int _time = 60;
  Timer? _timer;

  int get time{
    return _time;
  }

  void cronometer(){//método que activa el timer
     if(_timer == null){
    _timer = Timer.periodic(const Duration(seconds: 1), (Timer timer){
      print(DateTime.now());//Imprimo la fecha para que vean cada cuanto tiempo se ejecuta el código
      
      _time += -1;//decrementa time

      if(_time == 0){
        _time = 60;
      } 

      notifyListeners(); 
    });
   }
  } 

}

El problema es que estás creando el Timer cada vez que se hace rebuild, es decir, cada vez que usas el notifyListeners vuelve a entrar al método build, por lo tanto llama a cronometer entonces vuelve a crear otro Timer.

La manera correcta sería crear el Timer solo una vez, o también puedes crear una validación para que no vuelva a crear el timer.

Tu clase quedaría así:

class PriceProvider extends ChangeNotifier{
  int _time = 60;
  Timer? _timer;

  int get time{
    return _time;
  }

  void cronometer(){//método que activa el timer
     if(_timer == null){
    _timer = Timer.periodic(const Duration(seconds: 1), (Timer timer){
      print(DateTime.now());//Imprimo la fecha para que vean cada cuanto tiempo se ejecuta el código
      
      _time += -1;//decrementa time

      if(_time == 0){
        _time = 60;
      } 

      notifyListeners(); 
    });
   }
  } 

}

Otra forma de solucionarlo, como te decía, solo ejecutar el cronometer una vez.

class PriceWithClock extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    PriceProvider priceProvider = Provider.of<PriceProvider>(context);
    // no llamarlo aquí.
    //priceProvider.cronometer();
    return CircularPercentIndicator(
      radius: 100.0,
      lineWidth: 5.0,
      percent: 1-priceProvider.time/60,
      center: Text("00:${priceProvider.time}"),
     ),
    );
  }
}

Llamar al cronometer cuando se crea el Provider.

void main() async{
  runApp(    
    MultiProvider(
      providers: [
        ChangeNotifierProvider(create: (_) => PriceProvider()..cronometer()),
      ],
      child: MyApp(),
    ),
  );
} 
Origen Enlace
diegoveloper
  • 7.3k
  • 2
  • 11
  • 20

El problema es que estás creando el Timer cada vez que se hace rebuild, es decir, cada vez que usas el notifyListeners vuelve a entrar al método build, por lo tanto llama a cronometer entonces vuelve a crear otro Timer.

La manera correcta sería crear el Timer solo una vez, o también puedes crear una validación para que no vuelva a crear el timer.

Tu clase quedaría así:

class PriceProvider extends ChangeNotifier{
  int _time = 60;
  Timer? _timer;

  int get time{
    return _time;
  }

  void cronometer(){//método que activa el timer
     if(_timer == null){
    _timer = Timer.periodic(const Duration(seconds: 1), (Timer timer){
      print(DateTime.now());//Imprimo la fecha para que vean cada cuanto tiempo se ejecuta el código
      
      _time += -1;//decrementa time

      if(_time == 0){
        _time = 60;
      } 

      notifyListeners(); 
    });
   }
  } 

}