search

iterate / mikrobloggeriet

1 stars 1 forks source link

_Extract_ litt repeterende kode så vi slipper å gjøre samme endring mange steder. #65

Closed jraregris closed 9 months ago

jraregris commented 9 months ago

Dro ut kode som virker som den er tenkt til å være lik.

Kanskje er artig nestesteg er å gjøre disse greiene om til en liste med data som mapper over denne funksjonen eller noe slikt.

jraregris commented 9 months ago

Originalt hadde vi duplisert kode fordi vi ikke nødvendigvis ville at kohort-seksjonene skulle se like ut. Se feks på URLOG. Skal jeg pirke, har du innført et navn som heter mikrobloggeriet.serve/cohort-section. Men Neno har jo også en kohortseksjon, selv om innholdet er annerledes?

Ja, her har vi fem ting som er like, og én ting som ikke er lik, men vi behandler det nå som seks forskjellige ting.

jraregris commented 9 months ago

Skal jeg pirke, har du innført et navn som heter mikrobloggeriet.serve/cohort-section. Men Neno har jo også en kohortseksjon, selv om innholdet er annerledes?

Samme argumentet for mikrobloggeriet.serve/doc-list.

Har du noe forslag til hva man kunne kalt dem i stedet? default-cohort-section?

teodorlu commented 9 months ago

default-cohort-section er bra!

Ja, her har vi fem ting som er like, og én ting som ikke er lik, men vi behandler det nå som seks forskjellige ting.

Enig, det skurrer litt.