Nové výhodné předplatné namísto kreditů je tu! Více informací. Upravili jsme také naše provozní podmínky.
NOVINKA: Staň se datovým analytikem a získej jistotu práce, lepší plat a nové kariérní možnosti. Více informací:
Avatar
Lukáš
Tvůrce
Avatar
Lukáš:12.9.2017 16:26

Ahoj, potřeboval bych pomoct opravit chybu v mém kodu od cézarovi šifry. Vždy když zadám větu s mezerami, tak se mi program úplně zblázní a každé slovo zašifruje zvlášť. Output viz. obrázek a zdroják zde: https://www.itnetwork.cz/dev-lighter/968

 
Odpovědět
12.9.2017 16:26
Avatar
Matúš Olejník
Člen
Avatar
Odpovídá na Lukáš
Matúš Olejník:12.9.2017 20:45

Chyba je v tom, že

scanf()

číta vstup po prvý biely znak a potom ti to tam začne robiť neplechu.
Preto ak chceš aby ti to čítalo aj biele znaky použi

scanf("%[^\n]s", zprava);

toto ti však nezabezpečí pretečenie buffera a preto použi

fgets (zprava, 100, stdin);

No a kedže pred tým načítavaš pomocou scanf() do premennej vyber a potom zadáš enter tak by sa ti tento enter načítal už do pola zprava a preto stačí ak dáš za

scanf("%d", &vyber);

ešte

getchar();

ktorý ti ten enter zožerie :)

Editováno
Nahoru Odpovědět
12.9.2017 20:45
/* I am not sure why this works but it fixes the problem */
Avatar
Lukáš
Tvůrce
Avatar
Lukáš:12.9.2017 22:09

Teď mi to sice funguje s mezery, ale zase mi to místo mezer dělá vykřičníky.

 
Nahoru Odpovědět
12.9.2017 22:09
Avatar
Lukáš
Tvůrce
Avatar
Odpovídá na Matúš Olejník
Lukáš:12.9.2017 22:17

Teď mi to sice funguje s mezery, ale zase mi to místo mezer dělá vykřičníky.

 
Nahoru Odpovědět
12.9.2017 22:17
Avatar
Matúš Olejník
Člen
Avatar
Odpovídá na Lukáš
Matúš Olejník:12.9.2017 22:55

Presne tak pretože ascii hodnota medzery je 32 a ty si dal posun 1 a teda znak s ascii hodnotou 33 je výkričník :) Ak nechceš šifrovať medzeru pridaj podmienku ktorá kontroluje či aktuálny znak je medzera

for (i = 0; zprava[i] != '\0'; i++)     {
        //ak je načítaný znak medzera posuň sa v cykle ďalej
        if (zprava[i] == ' ')
                continue;

        zprava[i] = zprava[i] + posun;
        if (zprava[i] > 'z')
                zprava[i] = zprava[i] - 26;
}
Nahoru Odpovědět
12.9.2017 22:55
/* I am not sure why this works but it fixes the problem */
Avatar
Lukáš
Tvůrce
Avatar
Odpovídá na Matúš Olejník
Lukáš:12.9.2017 22:58

To mi taky mohlo dojít.. a ještě poslední blbůstku, na konci věty mám tento znak. Ten tam je proč?

 
Nahoru Odpovědět
12.9.2017 22:58
Avatar
Matúš Olejník
Člen
Avatar
Odpovídá na Lukáš
Matúš Olejník:12.9.2017 23:15

Aha hej prepáč, takže ešte jedna podmienka pretože fgets() ti načíta aj nový riadok ktorého ascii hodnotu tiež potom posúvaš čiže treba skontrolovať či je na konci zpravy nový riadok a ak hej nahradiť ho NUL charakterom

fgets(zprava, 100, stdin);
        if (zprava[strlen(zprava) - 1] == '\n')
                zprava[strlen(zprava) - 1] = '\0';
printf("Zadejte posun\n");
scanf("%d", &posun);
Nahoru Odpovědět
12.9.2017 23:15
/* I am not sure why this works but it fixes the problem */
Avatar
Lukáš
Tvůrce
Avatar
Lukáš:12.9.2017 23:20

Pecka, už to funguje, radši to nechám ještě otevřený, zítra budu snad pokračovat, tak kdyby jsem zas něco nevěděl, tak se zeptám, zatim díky moc! :)

S pozdravem Lukáš

 
Nahoru Odpovědět
12.9.2017 23:20
Avatar
Lukáš
Tvůrce
Avatar
Lukáš:14.9.2017 16:26

Zdravím, tak mám další problém, funkce, co vrací písmeno a při dešifrování na z, mi nefunguje a prostě se ta funkce vynechá a od a se odečte posun, takže mi vyjde znak `. Nevíte v čem jsem udělal chybu?

Kod: if (zprava[i] >='a' && zprava[i]<='z')
{
zprava[i] = (zprava[i] - 'a' - posun) % 26 + 'a' ;
}

 
Nahoru Odpovědět
14.9.2017 16:26
Avatar
Lukáš
Tvůrce
Avatar
Odpovídá na Lukáš
Lukáš:14.9.2017 16:29

Tak už to mám, stačilo 'a' přepsat na 'z'.. dělám urychlené závěry, omlouvám se, ale než jsem sem napsal, tak jsem to řešil asi hodinu.. nemohl jsem přijít proč.

 
Nahoru Odpovědět
14.9.2017 16:29
Avatar
Matúš Olejník
Člen
Avatar
Odpovídá na Lukáš
Matúš Olejník:14.9.2017 19:30

V pohode, tak to býva že na chybu prídeš sám ak si to takto zhrnieš :-D

Editováno
Nahoru Odpovědět
14.9.2017 19:30
/* I am not sure why this works but it fixes the problem */
Avatar
Lukáš
Tvůrce
Avatar
Lukáš:23.9.2017 19:17

Ahoj, tak jsem tu opět, ale teď jsem se pustil do podstatně těžší šifry a to Vigenerovi, mám problém, že se mi vždy zašifrované písmeno zopakuje. Asi budou dělat bordel nějaké znaky které nejsou písmena.

TEXT: ***Ah-oj
***

Zde bych viděl problém
zdroják: https://www.itnetwork.cz/dev-lighter/973

Asi by to šlo opravit funkcí isalpha() Ale já s touto funkcí nikdy nepracoval, tak nevím jak na to. Možná vás napadne něco lepšího.

Editováno
 
Nahoru Odpovědět
23.9.2017 19:17
Avatar
Matúš Olejník
Člen
Avatar
Odpovídá na Lukáš
Matúš Olejník:23.9.2017 21:17

Áno je to kvôli tomu že príde znak ktorý nie je písmeno alebo medzera, vtedy ani jeden if neplatí a teda hodnota v premennej hodnota sa nezmení a vypíše sa to isté čo pred tým. Ak nevieš ako použiť nejakú novú funkciu stačí ju zadať do googlu a hneď vidíš plno príkladov a iných informácií :-) Funkcia isalpha(int a) vráti nenulovú hodnotu ak daný parameter (ASCII hodnota) je písmeno, inak vráti 0. Takže namiesto toho if kde kontroluješ či je znak medzera skontroluj rovno či nie je písmeno. Stačí teda

if(!isalpha((int) zprava[i])) //ak znak nie je číslo
   continue;
Nahoru Odpovědět
23.9.2017 21:17
/* I am not sure why this works but it fixes the problem */
Avatar
Matúš Olejník
Člen
Avatar
Odpovídá na Matúš Olejník
Matúš Olejník:23.9.2017 21:42

Sakra píšem to z mobilu a takto to dopadne :D, v tom komentári má byť //ak znak nie je písmeno

Editováno
Nahoru Odpovědět
23.9.2017 21:42
/* I am not sure why this works but it fixes the problem */
Avatar
Lukáš
Tvůrce
Avatar
Odpovídá na Matúš Olejník
Lukáš:23.9.2017 22:03

Super, díky moc :-)

 
Nahoru Odpovědět
23.9.2017 22:03
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 15 zpráv z 15.