Tagi HTML w C++

[sido92]

Cześć,

próbuję uporać się z zadaniem Tagi HTML, ale niestety bezskutecznie. Na wyjściu, miedzy < > dostaję jakieś losowe znaki, zamiast wielkich liter. Niewłaściwie rzutuję typy, czy popełniłem jakiś inny błąd? Proszę o pomoc!

#include <iostream>

using namespace std;

int main()
{
    string kod;
    while(cin>>kod)
    {
        for (int i=0; i<kod.length()-1; i++)
        {
            if (kod[i] == '<')
            {
                while (kod[i] != '>')
                {
                    if (int (kod[i+1]) > 96 && kod[i+1] < 123)
                    kod[i+1] = int (kod[i-1]) - 32 ;
                    i++;
                }
            }
        }
    cout<<kod;
    }
    return 0;
}

[beginner1986]

Cześć,

1. Główny błąd jest w wewnętrznej pętli while. Chcesz tam zmienić kod małej litery na dużą metodą:
   X = X - 32,
   ale po lewej stronie tego równania Twój X to kod[i+1], a po prawej kod[i-1].
   Nie musisz rzutować znaku na int, bo zwyczajnie działając na typie char osiągniesz to samo.
2. Pobierasz tekst przez cin, czyli do stringa wczytujesz tekst, dopóki nie pojawi się jakiś biały znak, np. spacja lub enter, więc nie kontrolujesz, gdzie kończy się linia. Pomyśl o getline, bo każde słowo stanie się osobną liną.
3. Wypisując rozwiązanie linia po linii, pamiętaj na końcu każdej dać endl.
4. Żeby nie sprawdzać za każdym razem kodów ASCII liter możesz zamienić:
   if (int (kod[i+1]) > 96 && kod[i+1] < 123)
   na if (kod[i+1] >= 'a' && kod[i+1] <= 'z')
   Kompilator sam zamieni znaki na ich kody. To samo, ale wygodniej
5. Żeby pozbyć się warninga o porównaniu typu ze znakiem z typem bez znaku (w pętli for) zmień int na unsigned.
   Nie każda linia musię się kończy tagiem HTML więc sprawdzaj do czasu aż i < kod.length()

Powodzenia.

[sido92]

O kurcze, nie wiem skąd się wziął ten " - "... Dzięki za wyczerpującą odpowiedź i wszystkie dodatkowe uwagi, bardzo cenne info

Pozdrawiam!