Comments on: Co jest ważniejsze optymalizacja czy zasady programowania obiektowego?

By: Hubert Marzec

Hubert Marzec — Sat, 20 Sep 2008 08:21:43 +0000

@pablo Tutaj jest mowa o dwóch rzeczach: usuniecie rekordu z bazy i usuniecie zdjęcia z systemu plików. Dalszej Twojej wypowiedzi to ja nie rozumiem. Cały kod odpowiedzialny za usuwanie zdjęcia (baza / system plików) jest modelu, więc nie wiem skąd Twoje sugestie odnośnie szukania kodu w kontrolerze…

By: pablo picasso

pablo picasso — Fri, 19 Sep 2008 20:24:44 +0000

@koziołek:
trigger usuwajacy zdjecia? perwersja. deamon? nierealne dla serwerow hostingowych (my world).
nie bardzo rozumiem o czym jest ta dyskusja?
$row->delete() powinno usuwac zdjecia + ew. miniaturke.
nie korzystasz z $row->delete() i usuwasz hurtowo zdjecia z bazy? metoda to obslugujaca powinna tez usuwac te zdjecia. tak, ta metoda (np. deleteById(array $ids) {…} ) powinna znalezc sie w modelu, jezeli mowa o MVC. nie wyobrazam sobie szukac kodu ktory usuwa zdjecia z dysku np. w kontrolerze (w ktorym?)…

By: Koziołek

Koziołek — Tue, 02 Sep 2008 22:34:05 +0000

Skoro kasowanie zdjęcia składa się z kilku etapów, a chcemy wykonać je w ramach jednego rzutu to istnieją ciekawe rozwiązania:
1. Trigger + wspólny katalog - usuwamy rekord, a następnie triggery odpowiednio propagują nam tą zmianę. Z dysku usuwamy całe katalogi przy założeniu, że wszystkie pochodne zdjęcia i zdjęcie główne znajdują się w jednym katalogu.
2. Trigger + kolejka + demon - usunięcie zdjęcia jest dwu etapowe. Najpierw jest oznaczane jako do usunięcia i informacja o nim jest wysyłana do osobnej tabeli. tabelę co pewien czas np. przy niskim obciążeniu serwera czyta demon i wykonuje sprzątanie fizyczne. Sam demon może chodzić jako osobny proces systemu na serwerze ciągle lub być uruchamiany z crona.

By: Marcin Kłeczek

Marcin Kłeczek — Sat, 30 Aug 2008 10:37:32 +0000

W tym konkretnym przypadku bez problemu można dodać tego selecta - jego czasochłonność jest minimalna (0.000003sec?) - zakładam że to proste pobranie po ID. Bardziej czasochłonne jest usuwanie czegoś, co nie istnieje (nie z bazy, a z plików). Co prawda można pominąć wszelkie sprawdzania błędów… tylko, później trzeba pisać skrypty sprawdzające “spójność” systemu DB/pliki.

Rozszerzając moją propozycję - zawsze warto rozszerzyć/rozbudować/ułatwić model kosztem prędkości. Co najmniej 2 powody -
A. Koszt procesora/pamięci/dysku maleje w szybkim tempie, koszt pracy programisty jest droższy
B. Wszelkie zmiany struktury w przypadku (niespodziewanej?) rozbudowy są dużo łatwiejsze i unikamy sytuacji “nie da się”, lub “jest baaaardzo czasochłonne”.

By: piotrf

piotrf — Fri, 22 Aug 2008 23:16:21 +0000

btw jesli taki jeden select jest dokuczliwy to moze lepiej pomyslec nad jakims cachem :P

By: piotrf

piotrf — Fri, 22 Aug 2008 23:13:09 +0000

Szymon: “Jedyną wadę jako widzę to fakt, iż wszystko jest trzymane w modelu (przez co szybko się rozrasta).” - ale przy wiekszym projekcie takie podejscie moze bardzo pomoc, jak dla mnie taki “photosrow” nie jest jedynie zmapowanym wpisem do bazy, to jest reprezentacja pojdynczej fotki, wiec tak ja trzeba traktowac. Wywolujac na obiekcie fotki metode Delete, usuwana jest fotka, i mnie nie powinno juz interesowac czy sa usuwane zdjecia z dysku czy inne rzeczy - chyba na tym polega idea OOP

By: Szymon

Szymon — Fri, 20 Jun 2008 08:49:18 +0000

I jeszcze jedno “stworzyli takie metody jak _postDelete czy _postInsert” zapewne po to je stworzyli - pytanie tylko czy są z tego dumni i czy sami tego używają ;) Aby była jasność nie jestem zagorzałym przeciwnikiem tego, ale do zwolenników tego rozwiązania też nie należę ;)

By: Szymon

Szymon — Fri, 20 Jun 2008 08:47:15 +0000

Hubert odnośnie usuwania 4 różnych rzeczy - to również można zrealizować za pomocą sposobu by timi ;) Jedyną wadę jako widzę to fakt, iż wszystko jest trzymane w modelu (przez co szybko się rozrasta).

By: Szymon

Szymon — Fri, 20 Jun 2008 08:42:49 +0000

@timi zgadzam sie z Tobą, wg mnie to jest lepsze rozwiązanie, a pobieranie danych aby je później usunąć - tylko dlatego aby skorzystać z *Row, to jest dopiero obejście problemu na siłę ;)

By: Hubert Marzec

Hubert Marzec — Fri, 30 May 2008 13:42:22 +0000

@Maksymus007 W tym przypadku nie chodzi o szybkość php tylko dodatkowe zapytanie sql, sam php nic do tego nie ma. Gdyby tylko trzeba było usunąć tylko rekord z bazy to oczywiście masz racje. Ale ponieważ trzeba jeszcze usunąć z dysku 4 różne wersje (rozmiary) tego zdjęcia do co innego…