Jade Desktop Environment (diskuse)

[ehm]

TLDR:

1. Totální prasárna. (skvosty jako tohle, funkce checkTime, která kontroluje... něco, a tak).
2. Nepřináší to žádné nové myšlenky. Argumentace, že „uživatelé jsou zvyklí na web“, a proto je záhodno stavět desktop na webových technologiích, je extrémně debilní.
3. Webové technologie jsou odpad.
4. Dát do distribuční verze nějakého projektu fotku svého ksichtu je skoro na diagnózu.

Rád bych přidal vlastní obsahově hodnotnější zprávičku:

---

Uživatel ehm zahájil vývoj nového desktopového prostředí optimalizovaného pro extrémní výkon. Zdrojové kódy:

Makefile:

all:
    gcc -O3 *.c -o ehmde

main.c:

// TODO

[Bystroushaak]

Teda ty vole

function checkTime(time) {
    if (time < 10) {
        time = "0" + time;
    }
    return time;
}

To je hovnokód světové úrovně.

[koroptev]

v kontextu to mozna dava vetsi nesmysl ale takto vytrzene to resi v dnesnim js standardu nepritomny padding zleva.. a jina reseni jsou sice obecnejsi ale nijak zvlast elegantnejsi tim autora nijak nwobhajuju myslim ze psat dalsi desktopove prostredi je naprosta picovina stejne to nebude nikdo pouzivat tak jako nepouziva kde gnome nebo oststni shity na linuxu cest mrtve vyjimce unity ktera sla vlastni cestou nekopirovani win/linuxu a popravu skoncila v propadlisti dejin

[koroptev]

shit nekpoirovani win/macosu

[Bystroushaak]

v kontextu to mozna dava vetsi nesmysl ale takto vytrzene to resi v dnesnim js standardu nepritomny padding zleva.. a jina reseni jsou sice obecnejsi ale nijak zvlast elegantnejsi tim autora nijak nwobhajuju myslim ze psat dalsi desktopove prostredi je naprosta picovina stejne to nebude nikdo pouzivat tak jako nepouziva kde gnome nebo oststni shity na linuxu cest mrtve vyjimce unity ktera sla vlastni cestou nekopirovani win/linuxu a popravu skoncila v propadlisti dejin

• Nesmyslný název checkTime(), místo zeroPadding() nebo něčeho podobného.
• Někdy to vrací string, jindy číslo.
• Ani neřeším, že pro záporná čísla to bude vracet blbosti jako 0-1 místo -01, uznávám, že to v tom kontextu třeba nikdy nepoužije.

Jinými slovy funkce, která názvem lže o tom co dělá a ještě se chová nepředvídatelně co do vrácených datových typů. Well done.

[koroptev]

je to trivialni picovina ktera zda se ma pouziti jen v tom jednom souboru ktery ma +- 20 radek a ktera slouzi k tomu aby minuty a vteriny byly vzdy formatovany na 2 mista; jiste misto kam uprit mikrozrak vsech byrokratickych poucek o sw engineeringu; vsechny vyrcene vytky pomijeji fakt, ze je na prvni pohled videt, co to dela

[Bystroushaak]

No, jenže zbytek kódu, jak už podotkl ehm není o nic lepší.

[koroptev]

ten ehm ktery nepochopil co dela ta funkce? lol

[Bystroushaak]

Celý ten kód (ta funkce + ten zbytek toho souboru co do určování času) by se mimochodem dal úplně vyhodit a použít místo toho today.toTimeString().slice(0, 8).

[koroptev]

google fu challenge v tom kdo kreativneji spocte a + b ? lol kdo dalsi soutezi?

[Bystroushaak]

Asi takhle: https://github.com/codesardine/Jadesktop/pull/49

[koroptev]

a tohle setTimeout("timeDate()", 200) jsi tam nechal proc, kdyz uz jsi odstranoval picoviny?

[Bystroushaak]

Vznesl jsem to jako dotaz, jestli to má nějaký smysl, který nevidím, nebo jen nezná setInterval.

[koroptev]

lol a ty uvozovky a jejich vnitrek?

[Bystroushaak]

Viz to co tam psal.

[Bystroushaak]

Now i remember why i used setTimeout recursively
setInterval(timeDate, 200); does not update the clock properly and it lags behind

Jo, to jsem si tak nějak myslel, proto jsem to neupravoval. Mám takovou nějakou polo-zasutou a zapomenutou vzpomínku, že jsem na to kdysi taky narazil.

[ehm]

Jak jsi přišel na to, že nevím, co ta funkce dělá?

funkce checkTime, která kontroluje... něco

Narážel jsem na chybné pojmenování té funkce – nic to nekontroluje a už vůbec to nesouvisí s časem. Formátuje to číslo na dvě číslice.

[koroptev]

Ty jši tak šikovný, vytiskněme ťo a ďáme to neliďničku

[ehm]

Děkuji.

[Pavel Stárek]

Jistě, ale proboha, to ten (nyní) všude cpaný a opěvovaný JS nemá v sobě takovou základní chujovinu jakou je formátování čísel a řetězců? Tomu se mi snad ani nechce věřit.

[ehm]

vsechny vyrcene vytky pomijeji fakt, ze je na prvni pohled videt, co to dela

Situace, kdy je identifikátor nedeskriptivní, je v některých případech přípustná (omezený scope – např. iterační proměnné, pomocné vnitřní funkce apod.). To ale není tento případ, protože název funkce je zavádějící, pro což neexistuje omluva. Zatímco v případě nedeskriptivního názvu víš, že je potřeba se podívat na definici, u zavádějícího názvu tě to hned trknout nemusí. V případě daného autora se jedná o dlouhodobý vzorec chování, ne jednu chybu v bezvýznamném modulu. Viz třeba funkci fetch_icon odkazovanou níže, která nefetchuje vůbec nic – vrací cestu. Tam má tedy alespoň dokumentační komentář, nicméně ten je rovněž zavádějící. Funkce totiž nevrací jméno a cestu, ale cestu.

jiste misto kam uprit mikrozrak vsech byrokratickych poucek o sw engineeringu

Je to jedno z míst, kam jsem upřel zrak, a zdaleka ne jediné. Především ale mícháš dvě věci dohromady: Nemluvil jsem o tom, odkud to začít refactorovat, ale zběžně jsem se s tím projektem seznámil a soudím, že kvalita kódu je příšerná. Jó, kdybych měl smůlu a byl to fakt super projekt a ten týpek byl ve skutečnosti mistr vývojář a nikde jinde podobné boty neměl, jen zrovna tady v tomhle modulu mu to ujelo, tak se dá říct, že jsem to soudil moc předčasně. Tak to ale samozřejmě není, ostatně si ten kód projděte sami.

[ehm]

Tohle je taky extrémně kvalitní kousek kódu. :DD

[lertimir]

No možná se autor spletl z názvem. Just Another Debility Environment

[ehm]

:D

[mirec]

Aj toto je pekné ;)

[R]

4. Keby aspon bol pekny...

[JZD]

Zprávičku bych přesunul do sekce humor, podle některých komentářů tam evidentě patří.

[Andrej]

ale ma to animovane meduzy na pozadi!

[lertimir]

Great. to mi v KDE chybí.