Factorisation d'un code

Et l'eau,

Je ne vois pas totalement le contexte.
C'est quoi : palmSunday, fiveLentSunday, fourthLentSunday, etc. ?
Des instances sous quelle forme ? Du moins comment obtiens-tu firstAdventSunday.hasSame(....) ?
Modifié par niuxe (06 Jan 2021 - 22:21)

Hello,

Quelque chose de ce style ?

// cet array peut certainement être raccourci, mais je ne sais pas comment tes variables sont générés, là c'est un premier jet
const eventData = [
  ['firstAdventSunday',firstAdventSunday],
  ['secondAdventSunday',secondAdventSunday],
  ['thirdAdventSunday',thirdAdventSunday],
  ['fourthAdventSunday',fourthAdventSunday]
  // etc
];

for ( let i = 0; i < eventData.length; i++ ) {
  
  if ( eventData[i][1].hasSame(date, 'day')) data.m = dataM[eventData[i][0]];

}

Sinon tu peux pusher tes variables à la création dans un tableau et c'est plus simple encore:

// imaginons que tu ais un tableau comme ceci
const eventVars = [
  [firstAdventSunday],
  [secondAdventSunday],
  [thirdAdventSunday],
  [fourthAdventSunday]
  // etc
];

// pour transformer le nom d'une variable en string
const varToString = varObj => Object.keys(varObj)[0];

for ( let i = 0; i < eventVars.length; i++ ) {
  
  if ( eventVars[i].hasSame(date, 'day')) data.m = dataM[varToString({eventVars[i]})];

}

Et l'eau Stryk,

J'ai pensé à la même chose que toi. Il est sur NodeJS. Donc il peut faire quelque chose de beaucoup plus dynamique. Mais le contexte est manquant, d'où mes questions. Après je peux me tromper.

Ok, je vous donne un peu plus de contexte. Pour la manipulation des dates j'utilise luxon.js, la fonction .hasSame() vient de là. Exemple de définition pour deux variables, "josephHusbandOfMary" et "divineMercySunday" :

// ces variables renvoient un objet date :
const march19 = DateTime.fromFormat('1903' + year, 'ddMMyyyy'),
        march19InHolyWeek = holyWeek.contains(march19) ? true : false,
        march19InLentSunday = (lent.contains(date) && march19.weekday === 7) ? true : false,
        josephHusbandOfMary = march19InHolyWeek ? palmSunday.plus({days: -1}) : (march19InLentSunday ? march19.plus({days: 1}) : march19)
// une variable plus simple à définir :
const divineMercySunday = easter.plus({days: 7})

// plus loin, dans la partie que j'aimerais factoriser, je compare les variables avec .hasSame(date, 'day') qui renvoie true ou false. Si true alors appel aux données stockées dans un JSON et converties en objet :
if (josephHusbandOfMary.hasSame(date, 'day')) data.m = dataM.josephHusbandOfMary
if (divineMercySunday.hasSame(date, 'day')) data.m = dataM.divineMercySunday

Exemple d'output pour Saint Joseph :

{
  m: {
    key: 'josephHusbandOfMary',
    name: 'Saint Joseph, chaste Époux de la Bienheureuse Vierge Marie',
    color: [ 'white' ],
    type: 1,
    priority: 3
  }
}

S'il le faut je mettrais tout le code mais j'avais peur que ça devienne illisible (environ 235 lignes, et on n'a pas les outputs).
Modifié par Olivier C (07 Jan 2021 - 13:08)

Hello Niuxe,

Oui tout à fait, après je donnais une méthode générique

@Olivier en fait il te suffit de faire un push comme évoqué pour te servir du deuxième code:

let eventVars = [];
// puis à chaque déclaration
eventVars.push(divineMercySunday);

Merci. Je vais essayer de regarder tout ça cet après-midi et ferais un retour.

Edit : bon ben finalement ça ne sera pas pour aujourd'hui. Pour le principe je ferais un retour - comme à mon habitude - dès que possible.
Modifié par Olivier C (08 Jan 2021 - 21:27)