Po wykonaniu funkcji dane w stringu cofaja sie

Witam jest to moj pierwszy watek na forum wiec przepraszam za bledy jakie bede wykonywal. Mam problem poniewaz zainicjalizowalem funkcje "szyfrowanie" oraz "zgadywanie", po wykonaniu funkcji "szyfrowanie" i wydrukowaniu stringa w funkcji main na ekranie pojawia sie napis sprzed wykonania funkcji "szyfrowanie". Moze ktos mi pomoc i wytlumaczyc dlaczego sie tak dzieje? Z gory dziekuje za odpowiedz

#include <cstring>
#include <string>
#include <ctime>
#include <iostream>
using namespace std;
void losowanie(int tab[], int x);
void szyfrowanie(string haslo, int tab[], int x);
void zgadywanie(string x, string y, int z);
int main()
{
	int dl;
	string haslo;
	int* t_losowe, zaszyfrowana;
	cout << "podaj haslo: " << endl;
	getline(cin, haslo);
	dl = haslo.size();
	string zaszyfrowane(dl, '_');
	t_losowe = new int[dl];
	losowanie(t_losowe, dl);
	szyfrowanie(haslo, t_losowe, dl);
	cout << haslo << endl;
	zgadywanie(zaszyfrowane, haslo, dl);
	cout << zaszyfrowane << endl;
	for (int i = 0; i < 10; i++)
	{
		cout << t_losowe[i];
	}
	cout << "Brawo wykonales poprawnie zadanie :)" << endl;
}
void losowanie(int tab[], int x)
{
	srand(time(NULL));
	for (int i = 0; i < x; i++)
	{
		tab[i] =rand() % 10 - 5;
		cout << tab[i] << ", ";
	}
	cout << endl;
}
void szyfrowanie(string haslo, int tab[], int x)
{
	int sprawdzanie;
	for (int i = 0; i < x; i++)
	{
		haslo[i] = haslo[i] + tab[i];
		if (haslo[i] < 97)
		{
			sprawdzanie = haslo[i] - 97;
			haslo[i] = 122 + sprawdzanie;
		}
		if (haslo[i] > 122)
		{
			sprawdzanie = haslo[i] - 122;
			haslo[i] = 97 + sprawdzanie;
		}
		
	}
	cout << haslo << endl;
}
void zgadywanie(string x, string y, int z)
{

	do
	{
		char zmienna;
		cout << "Wpisz litere hasla" << endl;
		cout << x << endl;
		cin >> zmienna;

		for (int i = 0; i < z; i++)
		{
			if (zmienna == y[i])
				x[i] = y[i];
		}

	} while (y != x);
}

Witam.

Nie jestem pewien czy to zadziała bo to sprawdzałem na szybko.
Ale taki kod powinien rozwiązać problem.

#include <cstring>
#include <string>
#include <ctime>
#include <iostream>
using namespace std;
void losowanie(int tab[], int x);
string szyfrowanie(string haslo, int tab[], int x);
void zgadywanie(string x, string y, int z);
int main()
{
    int dl;
    string haslo;
    int* t_losowe, zaszyfrowana;
    cout << "podaj haslo: " << endl;
    getline(cin, haslo);
    dl = haslo.size();
    string zaszyfrowane(dl, '_');
    t_losowe = new int[dl];
    losowanie(t_losowe, dl);
    cout << szyfrowanie(haslo, t_losowe, dl);
    zgadywanie(zaszyfrowane, haslo, dl);
    cout << zaszyfrowane << endl;
    for (int i = 0; i < 10; i++)
    {
        cout << t_losowe[i];
    }
    cout << "Brawo wykonales poprawnie zadanie :)" << endl;
}
void losowanie(int tab[], int x)
{
    srand(time(NULL));
    for (int i = 0; i < x; i++)
    {
        tab[i] =rand() % 10 - 5;
        cout << tab[i] << ", ";
    }
    cout << endl;
}
string szyfrowanie(string haslo, int tab[], int x)
{
    int sprawdzanie;
    for (int i = 0; i < x; i++)
    {
        haslo[i] = haslo[i] + tab[i];
        if (haslo[i] < 97)
        {
            sprawdzanie = haslo[i] - 97;
            haslo[i] = 122 + sprawdzanie;
        }
        if (haslo[i] > 122)
        {
            sprawdzanie = haslo[i] - 122;
            haslo[i] = 97 + sprawdzanie;
        }

    }
    return haslo;
}
void zgadywanie(string x, string y, int z)
{

    do
    {
        char zmienna;
        cout << "Wpisz litere hasla" << endl;
        cout << x << endl;
        cin >> zmienna;

        for (int i = 0; i < z; i++)
        {
            if (zmienna == y[i])
                x[i] = y[i];
        }

    } while (y != x);
}

Jeśli nie będzie działać to napisz. Jak przeczytam to postaram się pomóc w zakresie mojej wiedzy.

Życzę miłego dnia.

Bartek Kuziel napisał(a):

po wykonaniu funkcji "szyfrowanie" i wydrukowaniu stringa w funkcji main na ekranie pojawia sie napis sprzed wykonania funkcji

Ponieważ Twoja funkcja przyjmuje string przez wartość, czyli tworzy sobie kopię i na niej operuje pozostawiając oryginał nietknięty.

Poza tym:

• Masz wyciek pamięci, bo alokujesz ją ale nie zwalniasz. Zamiast gołych tablic użyj std::vector i problem sam zniknie.
• Nadawaj sensowne nazwy zmiennym(argumentom funkcji przede wszystkim) tak, aby nie trzeba było wczytywać się w kod, żeby wiedzieć "co jest czym".
• Jeśli nie modyfikujesz argumentu funkcji to przyjmuj go jako stałą referencję. Przy okazji unikniesz zbędnego kopiowania.
• Nie musisz zapamiętywać długości stringa w osobnej zmiennej. Klasa string ma funkcję length, która poda Ci aktualną długość kiedy tylko zechcesz
• Nie używaj "magicznych liczb". Za tydzień możesz już nie pamiętać dlaczego użyłeś akurat takiej wartości a nie innej.
• Funkcje mają taką możliwość i zazwyczaj coś zwracają. Funkcja losująca liczby zwraca wylosowane liczby, funkcja szyfrująca tekst zwraca zaszyfrowany tekst. Dlaczego więc Twoje funkcje zwracają void czyli nic?
• No i rand(). Nie używaj rand, C++ ma <random>

Dziekuję Panowie za pomoc, @tajny_agent rady na pewno się przydadzą :), co do kodu który wysłał @pestka12 musiałem jeszcze tylko przypisać:

haslo = szyfrowanie(haslo, t_losowe, dl);

bo wcześniej to on mi to tylko drukował "zaszyfrowane" a nie przyjmował do stringa. Jeszcze raz dziękuję i życzę miłego dnia.