search

tetek / PWrBiblioteka

Client for PWr University library
2 stars 2 forks source link

Lokalizacja bibliotek do pllist #4

Closed bartoszhernas closed 11 years ago

bartoszhernas commented 11 years ago

Żeby ściągnąć jedną bibliotekę musze wykonać 2 zapytania. Jedno o jej dane z serwera PWR a drugie do GoogleMapsAPI o dane lokalizacyjne. Te dane się raczej nie zmieniają za często więc dobrym pomysłem będzie zapisanie wszystkich bibliotek jakoś na stałe w PropertyList.

Dopisze to potem bo teraz nawet jak wrócimy do listy książek i wejdziemy ponownie to znów pobiera nam wszystko od nowa.

Można się też zastanowić nad tym aby to cache było ważne tylko np. miesiąc lub tydzień.

tetek commented 11 years ago

nie tak szybko;p Zerknałem właśnie do kodu i widzę pełno wycieków pamięci. (nie wiem czy wcześniej pracowałeś bez ARC?) ale są tutaj poważne błędy. Hmm myślę, że łatwiej będzie mi tobie wytłumaczyć jak to zrobić dobrze niż poprawiać za każdym razem. Więc skomentuje kod w paru miejscach za chwile i wrzuce do repo żebyś zobaczył co jest na rzeczy;)

Wojtek Mandrysz

On Tuesday, January 8, 2013 at 9:56 PM, Bartosz wrote:

Żeby ściągnąć jedną bibliotekę musze wykonać 2 zapytania. Jedno o jej dane z serwera PWR a drugie do GoogleMapsAPI o dane lokalizacyjne. Te dane się raczej nie zmieniają za często więc dobrym pomysłem będzie zapisanie wszystkich bibliotek jakoś na stałe w PropertyList. Dopisze to potem bo teraz nawet jak wrócimy do listy książek i wejdziemy ponownie to znów pobiera nam wszystko od nowa.
Można się też zastanowić nad tym aby to cache było ważne tylko np. miesiąc lub tydzień.

— Reply to this email directly or view it on GitHub (https://github.com/tetek/PWrBiblioteka/issues/4).

bartoszhernas commented 11 years ago

Będzie super, dzięki wielkie :). Tak, pierwszy raz z ARC niestety. Jak przez 3 godziny miałem BAD_MEMORY_ACCESS to wszędzie gdzie się da dawałem retain modląc się że zadziała :P

tetek commented 11 years ago

Zrobiłem commit. Co do lokalizacji, to możesz zrobić zapisywanie do .plisty. Najlepiej napisać serializacje i deserializacje Library do NSDictionary.

bartoszhernas commented 11 years ago

Serializacje i deserializacje do NSDictionary znam już, bo robiłem to w swojej klasie Matrix w macierzach. Tylko powiedz mi czy tu są specjalne metody jakieś które mam nadpisać czy po prostu robię np. [Library getAsDictionary] i to potem zapisuję do pllist?

Co do komentarzy, ufff, widzę że wiele błędów. Sie postaram trochę teraz obronić bo jak wcześniej miałem bad_memory_access to gdzie się da wpisywałem retain i stąd też było autorelease na bloku (na StackOverflow ktoś to zaproponował).

Dzięki za te komentarze bo więcej i szybciej tak zrozumiem niż gdybym próbował sam to ogarniać. Idę szukać wycieków ;)

tetek commented 11 years ago

najlepiej w Library zrobić tak,

a logike to zapisywania, wyszukiwania czy jest zapisny itp w nowej klasie (np LibraryCache)

spoko, scommituj jak nie będzie leaków w profilerze;p

Wojtek Mandrysz

On Tuesday, January 8, 2013 at 10:59 PM, Bartosz wrote:

Serializacje i deserializacje do NSDictionary znam już, bo robiłem to w swojej klasie Matrix w macierzach. Tylko powiedz mi czy tu są specjalne metody jakieś które mam nadpisać czy po prostu robię np. [Library getAsDictionary] i to potem zapisuję do pllist? Co do komentarzy, ufff, widzę że wiele błędów. Sie postaram trochę teraz obronić bo jak wcześniej miałem bad_memory_access to gdzie się da wpisywałem retain i stąd też było autorelease na bloku (na StackOverflow ktoś to zaproponował).
Dzięki za te komentarze bo więcej i szybciej tak zrozumiem niż gdybym próbował sam to ogarniać. Idę szukać wycieków ;)

— Reply to this email directly or view it on GitHub (https://github.com/tetek/PWrBiblioteka/issues/4#issuecomment-12019713).

bartoszhernas commented 11 years ago

Cache zrobiłem inaczej bo prościej mi to było zaimplementować tak jak jest teraz. Podgląd biblioteki musiałby ktoś zrobić jakoś ładnie, bo mi nie bardzo idzie. Zobacz jak jest teraz najlepiej i oceń.