#programowanie Co to za czary (╯°□°）╯ #javascript (@DOgi)

Marmite

Marmite 5 l.

@DOgi: ktoś chyba nie słyszał o else if xD

c_z_a_r_o

c_z_a_r_o 5 l.

@DOgi: Glide API, ServiceNow.

Marmite

Marmite 5 l.

@GodlessMike: albo o mapach, albo o milionie innych sposobów na napisanie tego - ale akurat w takiej konkretnej implementacji najbardziej rzuca się w oczy brak else if xD

GodlessMike

GodlessMike 5 l.

@Marmite: Albo jeszcze lepiej: switch/case

DOgi

DOgi Autor 5 l.

@c_z_a_r_o: no tak, a dokładniej Client Script, ale bardziej chodziło mi o ten dziwny algorytm ;d

@Marmite: zdecydowanie i nie mogłem przejść obok tego obojętnie, bo ktoś zamienił Low z High xD

GodlessMike

GodlessMike 5 l.

@DOgi: O Panie, ale tam są 3 przypadki, byłoby czytelniej i łatwiejsze w późniejszej modyfikacji, dodasz kolejny case i po problemie.

c_z_a_r_o

c_z_a_r_o 5 l.

@DOgi: Jak masz możliwość to zrób audyt rekordu tego Client Scriptu, wtedy dowiesz się, kto tego dotykał i w jaki sposób modyfikował. Może się mylę, ale to pachnie beżowym majfrendem. ;)

GodlessMike

GodlessMike 5 l.

@DOgi: yep, w razie konieczności dodania kolejnych przypadków, bądź zmianą bieżących nie będzie trzeba się zagłębiać w kolejne ify i pisać spaghetti code.

DOgi

DOgi Autor 5 l.

switch/case

@GodlessMike: switch case dla zakresów jest trochę rzadko spotykane, no ale w sumie...

GodlessMike

GodlessMike 5 l.

@DOgi: Niekoniecznie bo może akurat urgency będzie w stanie Low przy założeniach <1 oraz >7 gdzie powyżej 7 to jest jest zwykły filesafe .

DOgi

DOgi Autor 5 l.

@GodlessMike: Minor (3) * Low (3) = Minor Low (9), czyli Urgency powinno być Low, a nie na High. Więc w sumie failsafe był, ale trochę zły ;d Ale znając życie ktoś w trakcie implementacji zmienił tablicę kalkulacji.