jakoze chvalihodne a tak, pozor ale: tarbal obsahuje neprenositelne soucasti - Makefile*, vygenerovane ui_*, ...

Nasledujici body jsou jen takovym zamyslenim nad kodem. Jasne, vim, ze je to ucebni text a tak, ale co uz:

myslete pri psani. Viz "patch", ktery hlavni soubor zmensi asi na polovinu:
http://public.yarpen.cz/linuxsoft/ (diffy a src).

nepouzivejte impicitni pojmenovavani widgetu v designeru. Co je asi pushButton_5? Vyznate se v tom za par dni/tydnu/let?

S predchozim bodem souvisi i nestastne pouziti l_operator. Postrada srozumitelnost. Co je opreator 1? Nebylo by lepsi pouzit enum a pak cist:
if (l_operator == CKalkulacka::OperatorPlus) ...
?
Ja vim, takhle se to da dohledat hned, ale ve vetsim programu?

Jak se program zotavi ze stavu, kdy jsou vsechna tlacitka ve stavu disabled?

jen osobni preference: snake_case notace ve spolupraci s CamelCase knihovnou pusobi opravdu dramaticky.

Ale treba jen predbiham dalsi dily.

Toto je super článek, díky za něj. Sice souhlasím tady s kolegou, že některé věci nejsou úplně košér (např. se nemůžu dopátrat, co znamená pushButton_3), ale jinak dobré. Už se těším na pokračování.

Děkuji za upozornění. A děkuji za pochválení. Určitě potěšilo. Pokračování již mělo vyjít asi před 16 dni, jenžo ho bohužel nikdo zatím nevydal. Do systému jsem ho zadal, ale stále čeká na schválení šéfredaktorem.

Zdravím...
narazil jsem na rozdíly mezi funkčností v Linuxu a ve Windowsu. Pod Win 7 je všechno ok, ale pod Ubuntu mi to háže hlášku: /home/gepard/qt_ws/prvni/mainwindow.cpp:450: error: ‘sprintf’ was not declared in this scope. Když vytvořím oběkt QString, tak z něj můžu volat sprintf, ale jinak tato funkce není dostupná. Ten samý kód pod linuxem nezbildím.
Nevím, esy něco dělám špatně nebo kde je problém...

Linux Gepard 2.6.31-20-generic #58-Ubuntu SMP Fri Mar 12 04:38:19 UTC 2010 x86_64 GNU/Linux
Qt Creator 1.2.1 | Based on Qt 4.5.2 (64 bit) | Built on Sep 1 2009 at 17:30:24

switch(l_operator)
{
case 1: {
l_finish = l_first + l_second;
sprintf(a_buffer, "%g", (double)l_finish);
ui->output->setText(a_buffer);
ui->tlZpet->setDisabled(true);
break;
}
...