Diskuze: Šifra

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

Aktivity

Lukáš

Tvůrce

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

Matúš Olejník

Člen

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 12.9.2017 20:48

Nahoru Odpovědět

12.9.2017 20:45

/* I am not sure why this works but it fixes the problem */

Lukáš

Tvůrce

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

Lukáš

Tvůrce

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

Matúš Olejník

Člen

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 */

Lukáš

Tvůrce

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

Matúš Olejník

Člen

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 */

Lukáš

Tvůrce

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

Lukáš

Tvůrce

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

Lukáš

Tvůrce

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

Matúš Olejník

Člen

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 14.9.2017 19:30

Nahoru Odpovědět

14.9.2017 19:30

/* I am not sure why this works but it fixes the problem */

Lukáš

Tvůrce

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 23.9.2017 19:18

Nahoru Odpovědět

23.9.2017 19:17

Matúš Olejník

Člen

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 */

Matúš Olejník

Člen

Matúš Olejník:23.9.2017 21:42

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

Editováno 23.9.2017 21:43

Nahoru Odpovědět

23.9.2017 21:42

/* I am not sure why this works but it fixes the problem */

Lukáš

Tvůrce

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.

Nejčastěji vyhledáváné

Diskuze: Šifra