On 25/06/2012 20:41, Edek Pienkowski wrote:
> Dnia Mon, 25 Jun 2012 20:30:49 +0100, Bronek Kozicki napisal:
>
>> On 25/06/2012 19:38, Edek Pienkowski wrote:
>>> Powiedzmy, że mam metodę która ma ze 30 zmiennych i, co oczywiste w tym
>>> przypadku, jest za duża. Chcę ją podzielić.
>>>
>>> Problem polega na tym, że cokolwiek bym nie chciał wydzielić z tej
>>> metody zmienia te zmienne lokalne. Jeżeli zmieniałoby jedną, nie ma
>>> problemu:
>>> x1 = doSth(a,b,c,d,e);
>>
>> elementarne - zrób klasę i zamień zmienne lokalne na prywatne pola
>> klasy, a częsci funkcji na funkcje klasy. Potem upraszczaj.
>
> Powiedz od razu, że mam pisać klasy zamiast metod

niekoniecznie. Zależy od tego czy w ten sposób program robi się bardziej
zrozumiały, czy nie.

> - jest takich metod
> dobre kilka. I nie bardzo wiem, co miałbym upraszczać i dlaczego miałbym

no właśnie, dlaczego napisałeś "chcę ją podzielić" ? Mogę zgadywać że
przyczyna jest ta sama co zawsze - kod jest zbyt skomplikowany i w
związku z tym trudno jest go poprawiać. Ale, to mojej zgadywanie.
Prawdziwą przyczynę zapewne znasz sam.


B.