Diskuze: C se chová divně, kde je chyba?

Naučíme tě pracovat na home-office.

Zjistit více...

V předchozím kvízu, Online test znalostí C++, jsme si ověřili nabyté zkušenosti z kurzu.

Aktivity

tomekmichalhu

Člen

tomekmichalhu:29.9.2015 16:47

dobrý den, můžete mi někdo říct kde je v tomhle kódu chyba?

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

/* run this program using the console pauser or add your own getch, system("pause") or input loop */
void main(int argc, char *argv[]) {

        //init
        int h, w, hw, ww, dead, alive;
        char s;
        dead = 32;
        alive = 64;
        h = 10;
        w = 10;
        int life[w][h];
        int lifen[w][h];

        //kill all cells
        for(hw = 0;hw<h;hw++){
                for(ww = 0;ww<w;ww++){
                        life[ww][hw] = dead;
                }
        }

        //create life
        life[2][3] = alive;
        life[3][2] = alive;
        life[3][4] = alive;

        //draw first cells
        system("cls");
        for(hw = 0;hw<h;hw++){
                printf("\nI");
                for(ww = 0;ww<w;ww++){
                        printf("%c",life[ww][hw]);
                }
                printf("I");
        }

        //infinite loop
        while(1){

        Sleep(1000);

        //compute
        for(hw = 0;hw<h;hw++){
                for(ww = 0;ww<w;ww++){
                        lifen[ww][hw] = life[ww][hw];
                }
        }

        for(hw = 0;hw<h;hw++){
                printf("\n"); //debug
                for(ww = 0;ww<w;ww++){

                        s=0;

                        if(ww < w-1 ){
                                if(life[ww+1][hw] == alive){
                                        s = s+1;
                                }
                        }
                        if(ww > 0){
                                if(life[ww-1][hw] == alive){
                                        s = s+1;
                                }
                        }
                        if(hw < h-1){
                                if(life[ww][hw+1] == alive){
                                        s = s+1;
                                }
                        }
                        if(hw > 0){
                                if(life[ww][hw-1] == alive){
                                        s = s+1;
                                }
                        }

                        printf("%i", s); //debug

                        if(life[ww][hw] = alive){
                                if(s<2 || s>3)lifen[ww][hw] = dead;
                        }else{
                                if(s == 3)lifen[ww][hw] = alive;
                        }

                }
        }

        for(hw = 0;hw<h;hw++){
                for(ww = 0;ww<w;ww++){
                        life[ww][hw] = lifen[ww][hw];
                }
        }

        //draw
        //system("cls");
        for(hw = 0;hw<h;hw++){
                printf("\nI");
                for(ww = 0;ww<w;ww++){
                        printf("%c",life[ww][hw]);
                }
                printf("I");
        }
        }

}

má to být jednoduchý dvoustatový celulární automat ale když se snažím spočítat "živé" buňky které se dotýkají aktuální buňky (řádky 56-79), vyhazuje to kraviny a v místech kde žádná buňka není a nikdy nebyla to s klidem napíše že má 2 sousedy.

Odpovědět

29.9.2015 16:47

Pjanus

Člen

Pjanus:29.9.2015 17:39

Z nějakého důvodu to způsobuje to uspání vlákna ale upřímně netuším proč...

Nahoru Odpovědět

29.9.2015 17:39

Pjanus

Člen

Pjanus:29.9.2015 17:44

Oprava chyba je někde v této části

for (hw = 0; hw < h; hw++){
        for (ww = 0; ww < w; ww++){
                life[ww][hw] = lifen[ww][hw];
        }
}

Nahoru Odpovědět

29.9.2015 17:44

tomekmichalhu

Člen

tomekmichalhu:29.9.2015 17:46

Ne, dal jsem to pryč a akorát to začalo spamovat ty příšernosti 100x rychlejc
edit: ne, tím to taky není.

Editováno 29.9.2015 17:49

Nahoru Odpovědět

29.9.2015 17:46

Pjanus

Člen

Pjanus:29.9.2015 17:49

Ne promiň už jsem na to přišel celou dobu jsem byl vedle

if (life[ww][hw] = alive)
{
        if (s < 2 || s>3)
                lifen[ww][hw] = dead;
        else
                lifen[ww][hw] = alive;
}
else
{
        if (s == 3)
                lifen[ww][hw] = alive;
        else
                lifen[ww][hw] = dead;
}

tady si v předchozí verzi nenastavoval žádný stav pokud to nesplňovalo tu podmínku

if (s < 2 || s>3)
if (s == 3)

a proto pokud nebyla splněna nebyla nastavená žádná hodnota respektive tam zůstavala po průběhu

Nahoru Odpovědět

29.9.2015 17:49

Pjanus

Člen

Pjanus:29.9.2015 17:57

A pořád jsem naprostý vůl tentokrát naposledy :-)

if (life[ww][hw] = alive)

tady ses upsal a já si toho nevšiml v podmínce tam přiřazuješ hodnotu takže to vlastně ani není podmínka, vlastně ani nevim kdy je splněna... Ale tam je ten problém a to co jsem říkal posledně je taky chyba takže to si taky oprav ;-)

Akceptované řešení
+20 Zkušeností
+2,50 Kč

Nahoru Odpovědět

29.9.2015 17:57

tomekmichalhu

Člen

tomekmichalhu:29.9.2015 17:58

no teď to píše jiný kraviny, ale pořád jsou to kraviny

Nahoru Odpovědět

29.9.2015 17:58

tomekmichalhu

Člen

tomekmichalhu:29.9.2015 17:58

super, už to valí dík

Nahoru Odpovědět

29.9.2015 17:58

Pjanus

Člen

Pjanus:29.9.2015 18:02

Mě to takhle už běží správně, zkus si upravit tuhle část kódu možná si na něco zapomněl nebo já jsem provedl změnu o které jsem se zapomněl zmínit

for (hw = 0; hw < h; hw++){
                        printf("\n"); //debug
                        for (ww = 0; ww < w; ww++){



                                s = 0;

                                if (ww < w - 1){
                                        if (life[ww + 1][hw] == alive){
                                                s = s + 1;
                                        }
                                }
                                if (ww > 0){
                                        if (life[ww - 1][hw] == alive){
                                                s = s + 1;
                                        }
                                }
                                if (hw < h - 1){
                                        if (life[ww][hw + 1] == alive){
                                                s = s + 1;
                                        }
                                }
                                if (hw > 0){
                                        if (life[ww][hw - 1] == alive){
                                                s = s + 1;
                                        }
                                }
                                printf("%i", s); //debug

                                if (life[ww][hw] == alive)
                                {
                                        if (s < 2 || s>3)
                                                lifen[ww][hw] = dead;
                                        else
                                                lifen[ww][hw] = alive;
                                }
                                else
                                {
                                        if (s == 3)
                                                lifen[ww][hw] = alive;
                                        else
                                                lifen[ww][hw] = dead;
                                }
                        }
                }

Nahoru Odpovědět

29.9.2015 18:02

Pjanus

Člen

Pjanus:29.9.2015 18:16

A poslední dodatek zkoušel jsem to a v tomhle případě je podmínka splněna pokud přiřazujeme jinou hodnotu než 0, jen tak pro úplnost.

Nahoru Odpovědět

29.9.2015 18:16

Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 10 zpráv z 10.

Nejčastěji vyhledáváné

Diskuze: C se chová divně, kde je chyba?