MyGNUHealth 1.0 (diskuse)

[mirec]

Příloha:

• profile_settings.png (4410 bytů)

Ok, programátori, ktorí sú ochotní niečo urobiť zdarma by sa našli, ale prečo takmer nikde v otvorenom softvéri nevidím prácu nejakého dizajnéra / UXáka?

Pozriem napr. nastavenie profilu. Absolútny chaos. Inputy len tak nejak nahádzané, nezarovnané. Tlačidlá o 2 pixely vyššie než inputy. 3 tlačidlá, ktoré sa tvária, že majú približne rovnaký význam. 2 z 3 inputov majú label. Inputy bez labelov sú inak tiež pekný antipattern, pretože po vyplnení si človek nemôže skontrolovať, či to skutočne napísal do správneho políčka.

[Peter Golis]

Keď ťa to zaujalo a vieš ako to urobiť lepšie, tak prečo nepriložíš ruku k dielu?

[mirec]

Nerobím ani dizajn, ani UX. Som obyčajný programátor, ale takú amatérskosť si všimnem aj ja. Okrem toho robím na desiatkach open source projektov, mám rozpracované úpravy, ktoré sú pre mňa omnoho dôležitejšie.

[Peter Golis]

A napadlo ťa že to je len trochu netradične zavesený inzerát na nábor zainteresovaného spolupracovníka ?

Asi nie, veď kopnúť si do druhého je také programátorské. Kde som to len videl ...

[biolog]

Když ten dialog mají v tak ošklivém stavu, asi je UX nezajímá. Kdyby je na to někdo upozornil, nejspíše by ho odbyli tím, že ty potíže nejsou nepřekonatelné, a poradili by, jak by nějaké těžkosti mohl vyřešit nebo zmírnit každý uživatel osobně (teda pokud by o té diskuzi nebo bug reportu uživatel věděl).

[xkucf03]

Na to nepotřebuješ designéra / UXáka – to má vidět i průměrně inteligentní jedinec, že to je hnusné a špatně použitelné. A programátor by měl být nadprůměrně inteligentní. Pak je otázka, jak je odfláknutý ten kód uvnitř, když GUI vypadá takhle. Když neměl čas udělat slušné GUI, tak asi bude bordel i pod tím. Kdyby najali UXáka, tak GUI bude možná naleštěné, ale uvnitř to bude stejná hrůza. Takhle aspoň víš, na čem jsi.

[trekker.dk]

Na druhou stranu je tu taky možnost, jestli kód uvnitř je v pořádku, ale toho člověka nebaví dělat UI.

[mirec]

Tak som sa trocha pozrel na zdrojáky. V prvom príspevku som písal, že sa mi nezdal význam tlačidiel, takže som si pozrel, čo tie tlačidlá vlastne robia.

text: qsTr("Set")
onClicked: {
    profile_settings.get_fedacct(userFedacct.text);
}

a

text: qsTr("Update Key")
enabled: (newPassword1.text.length > 3 && (newPassword1.text === newPassword2.text))
onClicked: {
    profile_settings.get_personalkey(userPassword.text,newPassword1.text,
                            newPassword2.text);
}

Prvý kód má aspoň správne whitespace. Ten druhý vyzerá ako nejaký nepodarený pokus o zarovnanie. Chýbajúce medzery medzi čiarkami sú len bonus.

Takže máme tu 2 tlačidlá, ktoré sa tvária, že niečo nastavujú, ale oba vyvolajú metódu s prefixom get_.

Nakoniec sa pozrime na jednu z metód:

@Slot(str)
def get_fedacct(self, userfedacct):
    if (userfedacct):
        self.update_fedacct(userfedacct)
        self.setOK.emit()

Takže metóda get nastavuje hodnotu, ale len ak je hodnota neprázdna. V prípade, že je prázdna nenastaví sa, takže užívateľ klikne, vidí vizuálnu odozvu tlačidla, ale nič sa nestane.

[trekker.dk]

Pokud první položka v kritice cizího kódu je stížnost na bílé znaky a zarovnání, bývá to hezká známka toho, že kritik nemá co kritizovat.

A ta poslední věc je chyba UI, takže to asi neměla být reakce na mě...

[mirec]

Nechápem, aký je problém s poradím. Keď otvorím cudzí kód a začnem ho čítať a vidím, že za čiarkou chýbajú medzery, zarovnanie je tak nejak nekonzistentné tak to hneď skritizujem, pretože je to prvá vec, ktorú môžem skritizovať. Na čokoľvek ďalšie potrebujem väčší buffer. Hneď keď som sa dostal cez whitespace som videl, že metódy, ktoré updatujú databázu majú prefix get_, generovanie html bez escapovania, generovanie url bez escapovania, vytvorenie adresára mygh priamo v $HOME adresári bez možnosti zmeniť, bez použitia $XDG_CONFIG_HOME atď. To mám kvôli niektorým čitateľom, ktorým záleží na poradí invertovať poradie bodov?

[trekker.dk]

generovanie html bez escapovania, generovanie url bez escapovania, vytvorenie adresára mygh priamo v $HOME adresári bez možnosti zmeniť, bez použitia $XDG_CONFIG_HOME. To mám kvôli niektorým čitateľom, ktorým záleží na poradí invertovať poradie bodov?

Těkžo můžete změnit pořadí něčeho, co jste v předchozím příspěvku vůbec nezmínil A zrovna v téhle diskuzi to bylo něco, co za zmínku stálo.

[mirec]

Prvé 2 body tam boli. Mal som meniť ich poradie?

[trekker.dk]

Prvé 2 body tam boli.

"generovanie html bez escapovania, generovanie url bez escapovania"...?

[mirec]

Nerozumiem, to je akože odpoveď na otázku, alebo čo?

[trekker.dk]

Ptám se, protože v #6 "generovanie html bez escapovania, generovanie url bez escapovania" nikde nevidím.

[mirec]

Nerozumiem, #6 je o menu a ja vôbec neviem prečo by tam mali byť tieto výhrady.

Ja som chcel len odpoveď, či som mal invertovať poradie bodov v mojom prvom komentári ku kódu. Nič viac, nič menej. Z vašej streany nevidím žiadnu relevantnú výhradu okrem:

Pokud první položka v kritice cizího kódu je stížnost na bílé znaky a zarovnání, bývá to hezká známka toho, že kritik nemá co kritizovat.

[trekker.dk]

Ano, protože v #6 byla první kritika toho kódu o bílých znacích, tj. známka toho, že kritik nemá co kritizovat. Následovala zmínka o tom get_ - v podstatě prkotina v souladu s tím, že kritik nemá co kritizovat. A nějaká drobnost, která je problém UI, tedy mimo debatu o kvalitách kódu pod UI.

Teprve v #13 jste napsal o chybějícím escapování, což už je něco, co lze považovat za závažnou chybu kódu.

[mirec]

Vy ste programátor, že hodnotíte, čo je a čo nie je závažnou chybou? Požičiam si tento citát:

There are only two hard things in Computer Science: cache invalidation and naming things.

-- Phil Karlton

Zlé pomenovanie nie je prkotina. Základom kvalitného kódu je správne pomenovanie premenných a metód. Ak nie sú správne pomenované nedokážem zhodnotiť, čo kód robí. Musím pri úprave kódu pozerať do tela funkcií a do tela funkcií, ktoré volajú a do tela funkcií ktoré volajú volané funkcie atď. Neraz som zažil, že sme zabalili projekt kvôli mizernému pomenovaniu. Áno, chýbajúce escapovanie je do istej miery závažné, ale maximálne to rozbije zobrazenie, alebo prestane fungovať synchronizácia (ak hovoríme o tomto projekte, vo webovej aplikácii by takáto chyba bola fatálna). To sa dá opraviť jednoducho. Zlé pomenovania vo väčšom projekte môžu viesť k zahodeniu celého projektu, pretože sa v tom nikto nebude vyznať.

[trekker.dk]

Dobrá, shodněme se na tom, že se neshodneme

[Salamek]

No "obcas" se take stane ze funkcionalita ma proste vetsi prioritu nez vzhled (to je casto pripad techto one man show projektu)... hrat si s UX muze az jsou dulezite soucasti udelane a nic jineho neni na praci...

I ja mam par OSS projektu ktere maji hnusne UI ale funguji dle mych potreb/predstav a to mi staci, preci jen proc bych se mel parat s UI/UX kdyz mi za to nikdo neda ani korunu a me to pri pouzivani daneho SW neprinese vubec nic?

[nadtržítko]

Se ti to podpise na dusi. Jezis rekl, ze mame byti dokonali jako Otec nebesky, odflaknute GUI jde proti tomuto prikazani.

[Martin Tůma]

Tlačidlá o 2 pixely vyššie než inputy

Tohle je vlastnost KDE5 QT stylu. Ten dialog není žádnej zázrak (ono udělat dobrej dialog je mnohem víc práce, než si běžný diskutér na ABCLinuxu vůbec deokáže představit), ale zrovna tohle padá na hlavu lidí z KDE.