C++, Szyfr cezara. Konwersacja string na char.

Question

#include <iostream>

#include <string>



using namespace std;



int main()

{

    string tekst, zaszyfrowanytekst;

    int dlugosc, przesuniecie;

    string alfabet[]={"A" "B" "C" "D" "E" "F" "G" "H" "I" "J" "K" "L" "M" "N" "O" "P" "Q" "R" "S" "T" "U" "V" "W" "X" "Y" "Z"};

    cout << "Wpisz tekst do zaszyfrowania ";

    getline(cin, tekst);

    dlugosc = tekst.length();

    cout << "Wpisales " << tekst;

    cout << "\nPiata litera wpisanego slowa to " << tekst[4];

    cout << "\nDlugosc twojego tekstu wynosi " << dlugosc;

    cout << "\nPodaj przesuniecie ";

    cin >> przesuniecie;

    zaszyfrowanytekst=tekst;

    for (int i=0; i<dlugosc; i++){

        tekst[i]= toupper(tekst[i]);

        zaszyfrowanytekst[i] = alfabet[i+przesuniecie];

}

    cout << "Zaszyfrowany tekst " << zaszyfrowanytekst;

    return 0;

}

W 43 linii dostaje taki oto błąd error: cannot convert 'std::__cxx11::string' {aka 'std::__cxx11::basic_string<char>'} to '__gnu_cxx::__alloc_traits<std::allocator<char>, char>::value_type' {aka 'char'} in assignment| Czy ktoś wie w jaki sposób go rozwiązać? Szukam po różnych forach i niestety nie mogę znaleźć odpowiedzi.
 

Answer 1

Chyba coś potężnie pomieszałeś.

Po pierwsze, to:

={"A" "B" "C" "D" "E" "F" "G" ... }

jest równoważne

={"ABCDEFG..."}

Bo zgubiłeś przecinki.

Po drugie, co do samego zamysłu kodu, to przekombinowałeś z tą tablicą stringów. Każde z tych by miało miało sens:

string alfabet = "ABCDEFG...";
const char *alfabet = "ABCDEFG...";
char alfabet[] = "ABCDEFG...";
char alfabet[] = {'A', 'B', 'C', 'D', 'E', 'F', 'G', ...};

Ale nie tablica stringów którą napisałeś.

Comments

Khm.. nie do końca równoważne. W przypadku c-stringa, na końcu w tablicy jest '\0' a w przypadku "literkowania" tego znaku nie ma.

---

Ale nigdzie nie napisałem że moje 4 przykłady są równoważne. Napisałem tylko że spełnią zadanie w kontekście tego programu.

Te pierwsze dwie linie są równoważne.

Answer 2

        zaszyfrowanytekst[i] = alfabet->c_str()[i + przesuniecie];

Comments

To technicznie jest całkowicie poprawne, ale nie zmienia faktu że sama tablica stringów jest przekombinowana i to -> w ogóle by nie było potrzebne.

Również to .c_str() w ogóle nie jest potrzebne i można je po prostu wyrzucić.

---

Dzięki, a wiesz może czego zaszyfrowany tekst to kompletna głupota?