Rozwiązany: Szyfr cezara - malutki problem

[mariusz1024]

Bardzo proszę o małą pomoc przy tym kodzie.

#include <iostream>
#include <cstdlib>
#include <string>
using namespace std;
int main()
{
	int k=0, przesuniecie=0, dlugosc_slowa;				 //k - licznik
	string szyfr;
	cout << "Podaj slowo do zakodaowania: ";
	//cin >> szyfr;
	cout << endl;
	szyfr = "abcdefghijklmnopqrstuvwxyz";
	dlugosc_slowa = szyfr.size();
	cout << "Podaj przesuniecie: "; cin >> przesuniecie;
	while(k<dlugosc_slowa)
	{
        if(szyfr[k]>='a')
        {
            szyfr[k]+=przesuniecie;
        }
		k++;
	}
	cout << szyfr << endl;
	k = 0;
	while(k<dlugosc_slowa)
	{
		if(szyfr[k]<='z')
		{
			szyfr[k]=(szyfr[k]-przesuniecie);
		}
		k++;
	}
	cout << szyfr << endl;
	system("pause");
	return 0;
}

[wies.niak]

A Twój problem to...? (poza tym, że program nie działa tak, jakbyś chciał)

[Shaker]

Czemu zakładasz, że 'szyfr' jest ciągiem znaków, a potem robisz z niego tablicę nawet jej nie deklarując?

Tak to raczej nie zadziała, bo k- odpowiada za konkretny znak, a ty w ogóle nie rezerwujesz pamięci.

@down

Chyba, że tak. Nie wiedziałem, czy to zadziała.

[Lord Hrabula]

Na co ma deklarować pamięć? Stworzył stringa to ma stringa i się po prostu przez indeks 'k' odwołuje do konkretnych znaków.

Stosowne w tym topicu jest pytanie wies.niaka Niemniej zakładając, że pierwsza pętla ma tekst szyfrować, a druga dekodować, można śmiało założyć, że na końcu obu tekstów pojawią się nieliterowe szlaczki

Jeśli program ma działać tylko dla małych liter, to trzeba zmienić implementacje pętlelk i np. zrobić stworzyć warunki na wypadek przekroczenia zakresu alfabetu.

To się może przydać, a przynajmniej pomoże zrozumieć, czemu dostajesz w wyniku to co dostajesz http://pl.wikipedia.org/wiki/ASCII

[mariusz1024]

Zakres liter powinien być od a do z ale czasami pod koniec wywala znaki z poza alfabetu. Własnie dokładnie nie wiem jakie warunki trzeba stworzyć.

[t3tris]

On iteruje to tablicy ASCII. Ogranicz się do zakresu [97, 122], to otrzymasz tylko małe litery alfabetu.

[mariusz1024]

Coś takiego na przykład:

if(szyfr[k]>='a' && szyfr[k]<='z')
{
		    szyfr[k]=szyfr[k]+przesuniecie;
}

?

[t3tris]

Nie. Jeżeli szyfr[k]=='y', a przesunięcie ==2, to podstawisz sobie '{' zamiast 'a'. Ja spróbowałbym czegoś w ten deseń:

szyfr[k]=((szyfr[k]-97+przesuniecie)%26)+97;

[mariusz1024]

Prawie działa.

Przy odkodowywaniu zamiast 'z' wstawia '`'.

[t3tris]

To przesuń sobie przedział o jedno "w prawo". '`' ma kod 96, 'a' - 97.

[mariusz1024]

Szyfrowanie działa ale przy odszyfrowywaniu jest ten problem.

szyfr[k]=((szyfr[k]-97-przesuniecie)%26)+97 to mam zmienić?

[t3tris]

Usiądź z kartką, napisz sobie z 5 przykładów literek (najlepiej 2 z lewego końca alfabetu, 2 z drugiego końca i jedną ze środka), wybierz 2 różne przesunięcia, zakoduj, odkoduj, i na podstawie analizy wyników wymyśl jak zależne od siebie są wartości "przed przesunięciem" i "po przesunięciu".

[mariusz1024]

Problem już rozwiązany.

[[Ekspert] Hakken]

Problem rozwiązany, więc temat zamykam.

W razie potrzeby otwarcia tematu, proszę o kontakt przez PW.