Co je zle v tomto kode

Venco · Napísal **Venco**: 21.03.2015 11:07

Cawte to som zase ja. Vedeli by ste mi povedat co je zle v tomto programe? Program by mal precitat dve cisla a na zaklade delenia modulo vypisat ci su delitelne bez zvysku alebo nie. Niekde v riadku kde je prikaz if je chyba. Viete kde???

Kód:

#include <stdio.h>
#include <stdlib.h>

int main()
{
    int cislo1, cislo2;
    printf("Zadajte cele cislo\n");
    scanf("%d", &cislo1);
    printf("Zadajte druhe cele cislo\n");
    scanf("%d", &cislo2);
    if(cislo1 % cislo2 = 0) {
        printf("cislo %d je delitelne cislom %d bez zvysku", cislo1, cislo2);
    }
    else {
        printf("cislo %d nie je delitelne cislom %d bez zvysku", cislo1, cislo2);
    }
}

focko · Napísal **focko**: 21.03.2015 11:11

pri porovnávaní je rovná sa zdvojené, len jeden znak znamená priradenie
čiže:

Kód:

if (cislo1 % cislo2 == 0) {}

Venco · Napísal autor témy **Venco**: 21.03.2015 11:20

ahaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa, profesional som to...

dakujem

faraon · Napísal **faraon**: 21.03.2015 12:32

Jo, to je celkem běžná začátečnická chyba, která se občas povede i profesionálovi :filth:

Jeden ze způsobů jak se jí vyhnout je použití Yoda conditions, kde zpřeházíš pořadí, tak jako mistr Yoda měl zpřeházená slova ve větě. Místo
if (a=5)
napíšeš
if (5=a)
a překladač samozřejmě něco takového nemůže zkompilovat a vyhodí chybu. Zatímco porovnání
if (5==a)
se zkompiluje a pracuje správně. Pochopitelně to funguje jenom tam kde porovnáváš s konstantou, když mezi sebou prohodíš dvě proměnné tak se jedna z nich stejně přepíše. A navíc to vypadá divně a blbě se to čte.

Takže nejdůležitější věc ze všeho, zapni si ve svém vývojovém prostředí zobrazování všech warningů, a při každé kompilaci jim věnuj velkou pozornost, protože tě můžou upozornit na potenciálně nebezpečné konstrukce jako je zrovna tahle! Například mě při použití
gcc -pedantic -Wall program.c
vyběhne chyba
program.c: In function ‘main’:
program.c:7:5: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
takže hned vím že na řádku 7 je něco co musím zkontrolovat a případně opravit. Kdybych napsal jenom
gcc program.c
tak se mi program bez stěžování zkompiluje i z chybou, já se vůbec nic nedozvím, a budu se strašně divit proč to dělá to co to dělá a ne to co jsem já chtěl aby to dělalo...

Co je zle v tomto kode

Podobné témy