[DOgi]

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

[c_z_a_r_o]

@DOgi: Glide API, ServiceNow.

[Marmite]

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

[GodlessMike]

@Marmite: Albo jeszcze lepiej: switch/case

[DOgi]

@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

[Marmite]

@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

[DOgi]

switch/case

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

[GodlessMike]

@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 .

[GodlessMike]

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

[DOgi]

@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.

[DOgi]

@GodlessMike: Takie coś miałeś na myśli?

[c_z_a_r_o]

@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]

@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]

@GodlessMike: oki, no to dziękuję