string subscript out of range - szyfr Cezara - C++

Question

Piszę program szyfrujący plik txt za pomocą szyfru Cezara. Przy próbnym załączeniu dla indeksu = 1 pojawia mi się błąd "string subscript out of range" przy linii którą oznaczyłem w kodzie. Czy mógłby ktoś spojrzeć gdzie mógłby być błąd?

void szyfr_cezara(fstream& in, fstream& out, int klucz)      //implementacja szyfru Cezara
{
    string linia_input;   //tu bedziemy przechowywac poszczegolna linie tekstu z pliku wejsciowego
    string linia_output;    //jw. z pliku wyjsciowego

    while (getline(in, linia_input))    //czytamy plik wejsciowy linia po linii
    {
        for (int i = 0; i < linia_input.length(); i++)   //czytamy kazda litere z wczytanej linii i szyfrujemy
        {
            char znak = linia_input[i];
            if (czy_litera(znak))     //sprawdzamy czy wczytany znak jest litera
            {
                if (islower(znak))   //sprawdzamy czy wczytana litera jest mala
                    linia_output[i] = (znak + klucz) % 26;    // <- tu jest error
                else
                {
                    litera = tolower(znak);
                    linia_output[i] = (znak + klucz) % 26;
                    linia_output[i] = toupper(linia_output[i]);
                }
            }
            else continue;
            cout << linia_output;
        }

        out << linia_output << "\n";    //zapisujemy zaszyfrowana linie do pliku wyjsciowego
    }

}

 

Comments

Nie usuwaj treści wpisu, nie wolno tego robić, bo pozostają puste wątki w których są jakieś odpowiedzi i rozmowa, ale zupełnie nie wiadomo o co chodzi. Zapytałeś o coś, otrzymałeś odpowiedź (czasem jest ona mniej a czasem bardziej trafna czy rozwiązująca dany problem), a więc ktoś poświęcił swój czas, aby chociaż spróbować pomóc. Skoro już ktoś to zrobił to warto, aby opis problemu i sposób rozwiązania pozostały. Dzięki temu osoby mające podobny problem w przyszłości będą mogły od razu znaleźć tu rozwiązanie, a nie wyrwaną z kontekstu rozmowę.

Właśnie dlatego usuwanie wszystkiego o co się zapytało zaraz po otrzymaniu pomocy przeczy idei forum i nie można tego robić. Rozumiemy, że czasem są sytuacje wyjątkowe, gdy dane pytanie może być problematyczne, ale w takiej sytuacji należy je edytować tak, aby usunąć problematyczny fragment wypowiedzi czy kawałek kodu, a nadal pozostawić pełny sens umożliwiający zrozumienie o co chodzi. W szczególnych przypadkach, gdy to rzeczywiście konieczne i nic innego nie da się zrobić, można poprosić nas o usunięcie całego tematu, ale nie traktujemy tego jako zwyczaj.

Answer 1

linia_output jest pusty, a operujesz na nim, jakby było w nim miejsce na wartości, które wpisujesz.

Użyj operatora +=, aby dodać nową literę na koniec.

Comments

sprawdziłem, i program rzeczywiście zaczął działać w tym sensie, że się nie wysypuje, natomiast do pliku i w konsoli zamiast poprawnie zaszyfrowanego tekstu zaczęły mi się wypisywać śmieciowe znaki typu trefl, nuty, etc. gdzie tu jest błąd?

---

Zapewne tam, gdzie wykonujesz operacje modulo. Powinieneś ją robić nie dla kodów znaków, a dla ich pozycji w alfabecie.

---

1. Jaki zakres wartości otrzymasz wykonując działanie X%26?
2. Jaki zakres mają litery w systemie ASCII?
Te dwa pytania powinny naprowadzić cię na rozwiązanie.

---

@j23, @tangarr, trochę się z tym pobawiłem i rzeczywiście pomogło, natomiast teraz mam problem z tym że do wyjściowego pliku zapisywana jest w jednej linii nie tylko linia która była w oryginalnym pliku, ale i wszystkie poprzednie, domyślam się że tutaj trzeba użyć out.ignore(), ale nie wiem zbytnio w którym miejscu  

---

Po linii 26 wstaw linia_output.clear();