CPP vprašanje o prirejanju

[VolkD]

Je tale koda :

Koda: Izberi vse

        wcl=wcl+dif;
        if (wcl>=24) wcl=0;
        if (wcl<0) wcl=23;

po funkcionalnosti enaka tej:

Koda: Izberi vse

        wcl=+dif;
        if (wcl>=24) wcl=0;
        if (wcl<0) wcl=23;

???

[Cloud]

Najbolje da oba primera scompilaš in primerjaš kodo v assemblerju

[transalp]

Namesto "+=" je v drugem izseku kode zapisano "=+" ...

[VolkD]

Najbolje da oba primera scompilaš in primerjaš kodo v assemblerju

Koda je različna že po dolžini.

[VolkD]

Namesto "+=" je v drugem izseku kode zapisano "=+" ...

To bo !
Če dam "+=" je pa koda enaka.
Hvala za rešitev zagonetke.
Zanimivo, da prevajalnik ne generira nobenega opozorila ampak gladko prevede. Precej zahrbtna zadeva.

[GJ]

Namesto "+=" je v drugem izseku kode zapisano "=+" ...

To bo !
Če dam "+=" je pa koda enaka.
Hvala za rešitev zagonetke.
Zanimivo, da prevajalnik ne generira nobenega opozorila ampak gladko prevede. Precej zahrbtna zadeva.

Le zakaj bi dal prevajalnik opozorilo?
Znak "+" se v primeru, če stoji pred variablo ali številom vedno smatra kot predznak pozitiven, če pa napišeš "=-" pa se variabla/število za zankom "-" negira.

LP GJ

[VolkD]

Le zakaj bi dal prevajalnik opozorilo?

Že, že,...
Saj je tudi tole:

Koda: Izberi vse

   if (i=0) {
  nekaj
}

Povsem legitimno, a je hkrati zelo pogosta napaka. In prevajalnik seveda da opozorilo.

[MitjaN]

Prevajalnik pač počasi integrirajo funkcionalnosti statičnih analizatorjev. Pred n leti večina prevajalanikov niti trznila ni za primer:

Koda: Izberi vse

if (i=0) {
  nekaj
}


To so ujeli samo statični analizatorji (pc-lint, ...). Sedaj večina prevajalnikov izda opozorilo. Čeprav je to sintaktično popolnoma pravilna koda.

Aktualen primer

Koda: Izberi vse

wcl=+dif;


je tudi sintaktično pravilna koda. Samo prevajalnik ni natreniran, da bi izdal opozorilo.
Poizkusi z (pazi na presledke):

Koda: Izberi vse

wcl =+ dif;


Mogoče bo v tem primeru našel težavo.

Moje pogled na to (in verjamem, da se marsikdo s tem ne bo strinjal) je, da se kodo piše za to, da se jo lažje bere. Tako se izogibam uporabe operatorjev, ki tudi dodeljujejo (assignement operators), ker ima moja glava težave ob ponovnem branju take kode.

[VolkD]

Poizkusi z (pazi na presledke):

Koda: Izberi vse

wcl =+ dif;


Mogoče bo v tem primeru našel težavo.

Tole sem šel pa takoj preizkusiti. Imam namreč težavo, da za raziko od mnogih, pišem kodo brez teh presledkov, ker se mi presledki zdijo čisto odveč.

Žal presledki niso rešitev. Obnašanje je povsem enako.

[Sigi]

Ali imaš warninge na prevajalniku nastavljene na "all"?
Ker vsaj za prirejanje namesto primejave v if pogoju bi skoraj moral javit opozorilo

edit:
GCC za if(state_timer=19) {... javi opozorilo:
warning: suggest parentheses around assignment used as truth value [-Wparentheses]
Za state_timer =+ 20 ne javi ničesar, ne glede na najvišji nivo opozoril.
Ravno tako ne trzne na state_timer =- variabla

Pa ja, sem videl ta to že veš

[VolkD]

Ja warning je na all.
Očitno tega nima za napako, na katero je treba biti pozoren. Je pametno vedeti in biti pozoren.