506. Flamaster [FLAMASTE]

[matir85]

Czy ktoś wie gdzie popełniłem błąd w tym zadaniu ???
Oto mój kod:

#include <iostream>
#include <string>
#include <cstdlib>
#include <algorithm>

using namespace std;

string wyraz; // zmienna przechowujaca wczytany napis
int p;   // zmienna powtorzen cyklu

int main()
{
    cin >> p;
    if((1<=p)&&(p<=50))
    {
        for(int i=0; i<p; i++)
        {
            cin >> wyraz;
            transform(wyraz.begin(),wyraz.end(),wyraz.begin(),::toupper);    // zamiana na duze litery
            int dlugosc = wyraz.length();
            if((0<dlugosc)&&(dlugosc<=200))                                 // sprawdzenie warunku co do dlugosci stringa
            {
                int liczba = 1;                                             // zmienna przechowujaca liczbe znakow takich samych wystepujacych po sobie
                for(int j=0; j<=dlugosc; j++)
                {
                    if(wyraz[j]==wyraz[j+1])                                // liczenie znakow takich samych obok siebie
                        liczba++;
                    else
                    {
                        if(liczba <2)
                        {
                            cout << wyraz[j];                             // wypisanie znaku gdy jest pojedynczy
                            liczba = 1;
                        }
                        else if(liczba==2)
                        {
                            cout << wyraz[j-1]<<wyraz[j];                  // wypisanie znakow gdy sa dwa takie same
                            liczba = 1;
                        }
                        else
                        {
                            cout << wyraz[j] << liczba;                    // wypisanie skroconego zapisu jesli jest wiecej niz dwa znaki obok siebie
                            liczba = 1;
                        }
                    }
                }
                cout << endl;
            }
            else exit(0);
        }
    }
    else exit(0);
    return 0;
}

[sig]

Wklej cały kod bo tutaj ewidentnie są tylko początki linijek. Najlepiej sformatowany.

[sig]

Zamiast for(int j=0; j<=dlugosc; j++) powinno być for(int j=0; j<dlugosc; j++), w twojej wersji pętla wychodziła poza stringa. Nie musisz też sprawdzać warunków wejścia podanych w zadaniu, od tego jest autor testów. Ty po prostu "wiesz" że będzie nie więcej niż 50 testów i nie więcej niż 200 wyłącznie dużych liter w każdym. I możesz użyć takich zmiennych do których więcej się nie zmieści. Ale if-owanie czy wymuszanie dużych liter tylko niepotrzebnie komplikuje program

[matir85]

Dzięki za cenne rady. Poprawiłem te niedociągnięcia.
Mam jeszcze pytanie co do tego mojego for(int j=0; j<=dlugosc; j++). Napisałeś że wychodzi poza stringa. Czyli mam przez to rozumieć że ten ostatni element stringa czyli NULL ma być niewczytywany przez program ???

[narbej]

Stringa wczytujesz dokładnie tak jak to robisz [łącznie z zerem na końcu]. Ale w środku pętli
for (int j = 0; j <= dl; j++) wychodzisz poza te zero:
W środku pętli używasz j +1; .

PS
Po poprawce podanej przez sig'a uzyskasz/uzyskałeś AC, więc powinieneś usunąć z forum swój kod.

[bpiechota]

Witam,
dołączę do tematu.

Mój kod wygląda tak:

include

using namespace std;
string intToStr(int n)
{
string tmp;
if(n < 0)
{
tmp = "-";
n = -n;
}
if(n > 9)
tmp += intToStr(n / 10);
tmp += n % 10 + 48;
return tmp;
}
int main()
{

string wyraz;
string nowywyraz;

int j=0;

int ile;
cin>>ile;
int a=0;

while(a<ile)
{
    nowywyraz.erase();
        cin>>wyraz;
    //   cout<<"wyraz.lenght : "<<wyraz.length()<<endl;

    for(int i=0; i<wyraz.length(); i++)
    {
        j=1;
        if(wyraz[i]!=wyraz[i+2])
        {
            nowywyraz.push_back(wyraz[i]);
        }
        else //if(wyraz[i]==wyraz[i+2])
        {
            nowywyraz.push_back(wyraz[i]);
            while(wyraz[i]==wyraz[i+j])
            {

                j++;
             //      cout<<"j : "<<j<<endl;

            }
            i+=j-1;
            //   cout<<"intToStr(j) : "<<intToStr(j)<<endl;
            nowywyraz+=intToStr(j);
        }
        //    cout<<"wyraz : "<<wyraz<<endl;


    }
    cout<<nowywyraz<<endl;
    a++;
}

return 0;

}

Spoj mi go nie uznaję mimo że dobrze go przetestowałem.. Jakieś podpowiedzi?