search

Maxime22 / WorkPlay

0 stars 0 forks source link

Fonction refactoring #46

Closed alexandrelux closed 3 months ago

alexandrelux commented 3 months ago

resolves https://github.com/Maxime22/WorkPlay/issues/45

alexandrelux commented 3 months ago

Je n'arrive pas à externaliser "calculateUserTime". Peut-être qu'une notion de React m'échappe.

Impovement à envisager pour le futur créer un Provider et un useContext pour le timer. Impovement à envisager pour le futur crée un Provider et un useContext pour les inputs.

Maxime22 commented 3 months ago

Tu commences à faire des trucs qui ont rien à voir dans la PR, genre le reset, je pense qu'on fera cette PR plus tard en vrai nan ?

alexandrelux commented 3 months ago

Tu avais codé la fonction reset en dur dans app.tsx.

j'ai juste déplacé ce code de App.tsx const resetInputs = () => { const newInputs = inputs.map(input => ({...input, value: '0'})); setInputs(newInputs); };

Dansutils en rajoutant la ligne "saveInputs(newInputs);" (car pour moi ct un oublie)

Ce qui est totalement dans le thème de refactoring de fonction...

Maxime22 commented 3 months ago

Tu avais codé la fonction reset en dur dans app.tsx.

j'ai juste déplacé ce code de App.tsx const resetInputs = () => { const newInputs = inputs.map(input => ({...input, value: '0'})); setInputs(newInputs); };

Dansutils en rajoutant la ligne "saveInputs(newInputs);" (car pour moi ct un oublie)

Ce qui est totalement dans le thème de refactoring de fonction...

Ouais tu as probablement raison, mais je l'ai déjà fait dans ma PR le rajout du saveInputs je crois, faudrait qu'on refactorise ensemble au même moment sinon ça va être compliqué de relire ce qu'on a fait je pense, il vaut mieux d'abord écrire puis refactoriser, mais une PR de refacto change tout le code de plein de fichiers et crée plein de conflits nan ?