Po wykonaniu funkcji dane w stringu cofaja sie

Odpowiedz Nowy wątek
2019-12-15 21:00

Rejestracja: 3 miesiące temu

Ostatnio: 2 miesiące temu

0

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);
}
edytowany 2x, ostatnio: Bartek Kuziel, 2019-12-15 21:26

Pozostało 580 znaków

2019-12-15 23:17

Rejestracja: 2 lata temu

Ostatnio: 1 dzień temu

1

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.

Pozostało 580 znaków

2019-12-16 01:16

Rejestracja: 6 lat temu

Ostatnio: 2 tygodnie temu

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>

#duda2020

Pozostało 580 znaków

2019-12-16 06:30

Rejestracja: 3 miesiące temu

Ostatnio: 2 miesiące temu

0

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.

Pozostało 580 znaków

Odpowiedz

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0