[PHP]refaktoring funkcji- proszę o pomoc Opcje

[miccom]

Cześć.

Czy da się zrefaktoryzować / rozbić tą funkcje jeszcze bardziej?
Będę wdzięczny za pomysły i sugestie.

Chodzi mi o funkcję calculateCardboard z klasy OrderPositionService

https://github.com/miccom1977/ERP_system/bl...tionService.php

[com]

oczywiście że się da, ale nikt z nas po pierwsze nie zna logiki algorytmu tych wyliczeń, z takich na szybko sugestii zmienić piramidę if else w bardziej płaska strukturę, spróbować zmienić metodę tak aby miała 1 odpowiedzialność/ resztę rozbić i liczby jako stałe, by było wiadomo czy są te doD === 1 itp, dodatkowo masz masę literówek w komentarzach/ opisach nawet w Readme i czemu komentarze po polsku, zmienne niech mówią czym są a nie skróty które tylko ty znasz i już pewnie nawet nie pamiętasz części co znaczą, algorytmy zdefiniuj sobie w jednym miejscu bo jak coś się zmieni to w 100 będziesz poprawiał dokładnie to samo, tak na szybko tyle (IMG:style_emoticons/default/wink.gif)

tu tak samo indexy jako stale co coś mówia dla $distributionElements[1] i w innych miejscach (IMG:style_emoticons/default/wink.gif) tak samo /1000 jako stała, bo zawsze może się zmienić, a nawet jak nie to wtedy wiadomo przez co się dzieli i czemu

Ten post edytował com 10.09.2021, 12:26:28