Seznam prohlédnutých diskusí

5.2.2008 23:21 | Abíčko

Lucinka šla před hodinou spát a mně se ještě nechtělo, tak jsem si sedl ke zdrojákům Abíčka a napsal implementaci požadavku #486 - seznam naposledy prohlédnutých diskusí. Je tu jedno omezení z důvodu zatížení serveru, systém si pamatuje jen posledních 75 diskusí, tak nebuďte překvapeni. Ať se vám nová funkce líbí.

Dále se můžete těšit na štítky, David Watzke se opřel do plnění databáze štítků, Filip Jirsák napsal javascript a AJAX pro editační dialog a já si pohrál se serverovou částí. Ještě to není hotovo, ale blížíme se k oficiálnímu zveřejnění.

       

Komentáře

[tired joe]

Dobrá práca.

[Filip Jirsák]

Lucinka šla před hodinou spát…

Čekal jsem, že to bude pokračovat „ale předtím ještě stihla napsat…“

[Heron]

systém si pamatuje jen posledních 75 diskusí

Předpokládám, že takových omezení má abc víc. Všiml jsem si, že i změna v diskusi (hvězdička vedle počtu komentářů) má také omezenou životnost. Co brání tomu to ukládat navždy?

[Leoš Literák]

Jedna se o stejne omezeni. Brani tomu efektivita zpracovani a konecne mnozstvi pameti na pocitaci. Jestli koupis pocitac s neomezenou pameti a OS, ktery to zvladne, pak neni problem neomezovat pocet poslednich prohlednutych diskusi.

Implementacne je to napsano tak, ze pri zalogovani si ulozim do session seznam prohlednutych diskusi a cislo posledniho komentare. Pri nacteni stranky pak u kazde diskuse kontroluju, zda cislo posledniho komentare je vetsi nez to ze sessiony. A je jasne, ze mit v sessione treba tisice udaju pro tisice navstevniku neni dobry napad. Zatim jsem opatrny, proto je to cislo 75, zacinali jsme s 50. Az budu mit po ruce nejake statistiky, kolik realne tyto udaje zabiraji v prubehu dne, muzu limit zase posunout. A taky muzu udelat bonus, ze treba uzivatele se skore vetsim nez XY budou mit limit treba 250.

[Heron]

Myslel jsem, že se tyto udaje tahají z DB, kde by to neměl být problém. Nějak mi (momentálně, mozek offline), nedochází, proč to taháš do session.

[Leoš Literák]

Optimalizace na vykon. Za prve netusim, ktere diskuse se na strance objevi. Takze bych je musel tahat samostatne, kazdou zvlast. Za druhe se snazim, at v idealni pripade je mozne zobrazeni stranky bez jedineho SQL dotazu (vse cache).

[Ilfirin]

WOW. Jen jsem zapl abc a nestačil jsem zírat. Tak jsem hned koukl do blogů a už o tom čtu. Výborný nápad!

[ronny]

Super to mi chybelo dekuji :)

[ronny]

akorat by nebylo spatne kdyby se v prohlednutych diskuzich nezobrazovali ty, ktere jsou v mych komentarich

[Leoš Literák]

To by bylo matouci chovani.

[ʚĭɞ 🍉]

Myslím, že ne --- s mým příspěvkem jedna kategorie, bez mého příspěvku druhá kategorie.

[Leoš Literák]

Ale ono je to jinak: diskuse, kterých se aktivně účastním a diskuse, které jsem si prohlédl. Mezi oběma skupinami je částečný překryv, ale každá je jasně definována a má svůj účel. Odstraněním duplicit vznikne matoucí chování (proč nevidím mezi navštívenými diskusi tu o XY, když jsem ji ráno komentoval?)

[ʚĭɞ 🍉]

Prohlédnu si přehled diskusí, v nichž jsem reagoval, pak vlezu na druhý přehled --- proč, KDP!, vidím některé diskuse znova?

[Leoš Literák]

Protoze mezi seznamem diskusi, ktere jsi videl a do kterych jsi pridal komentar, je prekryv, to je logicke.

Ja chapu, co chces, ale je to spatne. Aplikace se musi chovat konzistentne a ocekavane. Takze kdyz je to seznam naposledy navstivenych diskusi, tak tam musi byt vsechny, nemuzu vyhazet duplikaty s jinym seznamem, protoze nahodou obsahuje stejne diskuse. Kazdopadne jsem proti.

[ʚĭɞ 🍉]

A což takhle volitelné chování podle nastavení?

[ronny]

to by nebylo spatne

[Ilfirin]

Vyjímečně souhlasím.

[Leoš Literák]

Tak si to hodte do bugzilly, treba to nekdy nekdo naimplementuje.

[ronny]

No nevim. Napred projedu "Mé komentáře" a potom jdu po Prohl. diskuzich, takze ty me komentare vidim vlastne znovu :)

[trekker.dk]

Přesně tak. Taky díky.

[David Jaša]

Je tu jedno omezení z důvodu zatížení serveru, systém si pamatuje jen posledních 75 diskusí, tak nebuďte překvapeni.

Jak je na tom server s výkonem, je možné bezpečně zvednou limity, jak žádám v bugu #957?

[Leoš Literák]

Vykon jo, spise jde o prenosy, at server zbytecne netaha moc dat. Chce to dobre vyladit defaulty (mozna nekde ke 40-50) a nastavit rozumna maxima, at se zbytecne nepreplnuje cache (nekde ke 100-250).