🚀 go-pugleaf

<wodzik@noreply.com> napisał:

> On 2010-01-03 03:34, Łukasz Rekucki wrote:

>> PS. Ogólnie twój kod nie jest zbyt pythoniczny (sposób użycia for ..
>> in, nazwy metod), a miejscami dość nieoptymalny, np:
>>
>> self.tablica.append(liczba)
>> self.tablica.sort()

> Pewnie dlatego, że dopiero zaczynam kodowanie w Pythonie, ale będę  
> wdzięczny za wszelkie sugestie.
> Rozumiem, że lepiej używać for coś in coś?

Tak, np.:

     def Print(self):
         if self.lisc:
             for wartosc in self.tablica:
                 print wartosc
         else:
             for i, wskaznik in enumerate(self.wskazniki):
                 wskaznik.Print()
                 if i < len(self.tablica):
                     print self.tablica[i]

Lub nawet:

     def Print(self):
         if self.lisc:
             print '\n'.join(map(str, self.tablica))
         else:
             for wskaznik, wartosc in itertools.iziplongest(
                                      self.wskazniki, self.tablica):
                 wskaznik.Print()
                 if wartosc is not None:
                     print wartosc


> I jak zoptymalizować ten kawałek z append i sort? czy wyszukać miejsce  
> gdzie ma się znaleźć liczba i użyć insert?

Patrz np. : http://docs.python.org/library/bisect.html (być może

> W sumie to podczas pisania nawet specjalnie się nie zastanawiałem nad  
> wydajnością, tylko moją wygodą ;]

I słusznie (przedwczesna optymalizacja to źródło [prawie] wszelkiego zła,
jak uważa np. GvR), ale bez przesady -- nie należy używac antyoptymalnych
konstrukcji, które mogą spowolnić program o rząd wielkości...

pozdr.
*j

-- 
Jan Kaliszewski (zuo)