[People] Réflexions sur le champ year_of_practice #26

New Issue

Sulley · 2020-10-26T11:03:20+01:00

Sulley commented

2020-10-26 11:03:20 +01:00

A quoi correspond le champ year_of_practice ? Comment se comportera-t-il dans un an ? Dans deux ans ? Est-ce qu'il ne faudrait pas une date de début, plutôt ? Idem pour la méthode actual_year_of_pratice. Comme elle se base sur le champ created_at, il suffit que l'enregistrement ne soit pas réalisé correctement pour que la méthode retourne une mauvaise information.

A quoi correspond le champ `year_of_practice` ? Comment se comportera-t-il dans un an ? Dans deux ans ? Est-ce qu'il ne faudrait pas une date de début, plutôt ? Idem pour la méthode `actual_year_of_pratice`. Comme elle se base sur le champ `created_at`, il suffit que l'enregistrement ne soit pas réalisé correctement pour que la méthode retourne une mauvaise information.

Sulley added the

question

label 2020-10-26 11:03:20 +01:00

Fred added this to the Revue de l'application `people` milestone 2020-10-28 09:26:34 +01:00

Fred commented

2021-06-18 15:49:05 +02:00

A priori, il faut savoir quand la personne a commencé et savoir depuis quand la personne est connue.

Sulley commented

2021-06-21 20:14:35 +02:00

	@property
	def actual_year_of_pratice(self):
		"""
		Renvoie le nombre d'année de pratique du gymnaste en se basant sur le
		nombre d'année qu'il avait déjà lors de son introduction dans le
		système + le nombre d'année qu'il est dans le système.
		"""
		period = pendulum.now() - pendulum.instance(self.created_at, "Europe/Brussels")
		return int(self.year_of_practice + period.in_years())

La variable year_of_practice permet donc de savoir combien de temps la personne a fait du trampoline AVANT d'être dans notre club.

Mais la variable et la méthode ne sont peut-être pas bien nommées.

``` @property def actual_year_of_pratice(self): """ Renvoie le nombre d'année de pratique du gymnaste en se basant sur le nombre d'année qu'il avait déjà lors de son introduction dans le système + le nombre d'année qu'il est dans le système. """ period = pendulum.now() - pendulum.instance(self.created_at, "Europe/Brussels") return int(self.year_of_practice + period.in_years()) ``` La variable `year_of_practice` permet donc de savoir combien de temps la personne a fait du trampoline AVANT d'être dans notre club. Mais la variable et la méthode ne sont peut-être pas bien nommées.

Fred commented

2021-06-22 13:56:10 +02:00

Attention que la timezone est déjà définie dans les paramètres de l'appli.
Tu dois pouvoir la récupérer via un settings.TIMEZONE ou quelque chose comme ça.

Attention que la timezone est déjà définie dans les paramètres de l'appli. Tu dois pouvoir la récupérer via un settings.TIMEZONE ou quelque chose comme ça.

Fred commented

2021-06-22 14:16:17 +02:00

Proposition pour la nomenclature: activity_started_at (pour le champ) et number_of_years_of_practice (pour la fonction/propriété) ?

Proposition pour la nomenclature: `activity_started_at` (pour le champ) et `number_of_years_of_practice` (pour la fonction/propriété) ?

Fred commented

2021-06-22 19:08:30 +02:00

	@property
	def actual_year_of_pratice(self):
		"""
		Renvoie le nombre d'année de pratique du gymnaste en se basant sur le
		nombre d'année qu'il avait déjà lors de son introduction dans le
		système + le nombre d'année qu'il est dans le système.
		"""
		period = pendulum.now() - pendulum.instance(self.created_at, "Europe/Brussels")
		return int(self.year_of_practice + period.in_years())
La variable year_of_practice permet donc de savoir combien de temps la personne a fait du trampoline AVANT d'être dans notre club.

Mais la variable et la méthode ne sont peut-être pas bien nommées.

Juste une question bête: tu parles de "notre" club. Pourquoi est-ce qu'il faut une représentation de "plusieurs" clubs, du coup ?

On en a parlé la semaine dernière, et j'ai un peu tiqué, mais pas relever: si un déploiement de Khana sert à un club, pourquoi faut-il en gérer/représenter plusieurs ?

> ``` > @property > def actual_year_of_pratice(self): > """ > Renvoie le nombre d'année de pratique du gymnaste en se basant sur le > nombre d'année qu'il avait déjà lors de son introduction dans le > système + le nombre d'année qu'il est dans le système. > """ > period = pendulum.now() - pendulum.instance(self.created_at, "Europe/Brussels") > return int(self.year_of_practice + period.in_years()) > ``` > > La variable `year_of_practice` permet donc de savoir combien de temps la personne a fait du trampoline AVANT d'être dans notre club. > > Mais la variable et la méthode ne sont peut-être pas bien nommées. Juste une question bête: tu parles de "notre" club. Pourquoi est-ce qu'il faut une représentation de "plusieurs" clubs, du coup ? On en a parlé la semaine dernière, et j'ai un peu tiqué, mais pas relever: si un déploiement de Khana sert à **un** club, pourquoi faut-il en gérer/représenter plusieurs ?

Sign in to join this conversation.