506. Flamaster [FLAMASTE]

Wyniki wychodzą prawidłowe, lecz sędzia mówi o błędnej odpowiedzi. Proszę o jakieś wskazówski w jaki sposób mogę poprawić kod.

#include <iostream>
using namespace std;

int main() {
	int test;
	cin>>test;
		for(int i=0; i<test; i++)
		{
			string word; int licznik=0;
			cin>>word;
				for (int i=0; i<word.length(); i++)
				{
				    if(word[i]==word[word.length()])
				    {
				        if(licznik==0)		{cout<<word[i];}
					    else if(licznik<=1) {cout<<word[i-1]<<word[i]; licznik=0;}
					    else if(licznik>1)  {cout<<word[i]<<licznik; licznik=0;}
				    }
					else if(word[i]==word[i+1]) {licznik++;}
					else if(licznik==0) 		{cout<<word[i];}
					else if(licznik<=1) 		{cout<<word[i-1]<<word[i]; licznik=0;}
					else if(licznik>1)  		{cout<<word[i]<<licznik; licznik=0;}
				}
			cout<<endl;
		}
}

Mógłbym zrezygnować z if(word[i]==word[word.length()]) ale wtedy program będzie odwoływać sie do tabeli, która nie istnieje.

a czy zadałeś sobie trud przetestowania programu ? choć dla danych testowych podanych w zadaniu ?
jeżeli widzisz, że są identyczne to konieczna wizyta u okulisty

a jaki sens mam ten warunek, czy może kiedykolwiek być spełniony ? - jaką wartość ma word[word.length()] ?

Napisałem kod do zadania, u mnie wyrzuca poprawne odpowiedzi przy różnych danych wejściowych jednak sędzia je odrzuca, widzi może ktoś o co chodzi?

Nie wiem jak wstawić tutaj sformatowany kod :/

mój kod:

using namespace std;

int count(string str, char ch) {
int num = 0;
for (int i = 0; i < str.length(); i++) {
if (str[i] == ch) {
num++;
}
}
return num;
}

bool alreadyChecked(vector c, char s) {
for (int i = 0; i < c.size(); i++) {
if (c[i] == s) {
return true;
}
}
return false;
}

int main()
{
vector checks;
int t, it = 0, num;
string str;
char currChar = ‘/’;
cin >> t;
do {
cin >> str;
transform(str.begin(), str.end(), str.begin(), ::toupper);
for (int i = 0; i < str.length(); i++) {
if (!alreadyChecked(checks, str[i])) {
checks.push_back(str[i]);
num = count(str, str[i]);
if (num <= 2) {
for (int g = 0; g < num; g++) {
cout << str[i];
}
}
else {
cout << str[i] << num;
}
}
}
checks.clear();
cout << endl;
it++;
} while (it < t);
return 0;
}

i po raz kolejny muszę pisać:

1. nie zakładamy nowego wątku, gdy już istnieje wątek poświęcony temu zadania - to zadanie ma wiele wątków, dzięki niefrasobliwości takich jak ty
2. jeżeli jest wątek, czytamy - z dużym prawdopodobieństwem problem został dawno temu omówiony
3. wskazane jest przekazywanie kodu poprze ideone.com5 lub przynajmniej poprawnie go wklejając - tu zginęło formatowanie oraz fragmenty kodu - dzięki czemu stał się całkowicie nieczytelny

i dodatkowo - piszesz w niewłaściwym dziale - problemy związane z rozwiązywaniem zadań powinny być w http://discuss.spoj.com/c/pl-spoj-com/zbior-zadan4

Więc tego powinieneś się najpierw nauczyć i dowiedzieć, zanim zaczniesz wklejać kod i zadawać pytania [w odpowiednim dziale forum]. Dodatkowo, komentarze pod zadaniami są do komentowania a nie do zadawania pytań. Jeżeli nie potrafisz przeczytać 4 pkt krótkich uwag pod zadaniem, zrozumieć ich i się dostosować, to czy nie mam prawa mieć wątpliwości, czy przeczytasz z wystarczającą uwagą i zrozumieniem jakąkolwiek podpowiedź? A podpowiedzi do tego zadania, jak wspomniał już @mariusz193 jest już całe mnóstwo.

No popatrz, a ja wkleiłem przykład spod zadania i “nie działa”: https://ideone.com/tUCbzm9 gdzie jest wynik 4 testu?

cin.sync() nie uchroni Cię przed czekającym w buforze znakiem końca lini. To już lepiej cin.get() po prostu tam wstawić i powinno być lepiej.

No jak to gdzie ? U mnie działa, wstawiłem cin.get() zamiast sin.sync() i tak samo “błędna odpowiedź”

Zadania zawsze testuj na ideone (lub używaj dokładnie tego samego kompilatora). Sędzia sprawdza Twój kod automatycznie generując wyjście i prównując je z wyjściem autora.

Jeśli nikt Ci nie pomoże sprawdzę później gdzie jest błąd. Możesz sprawdzić czy popdowiedź Mariusza rozwiązuje Twój problem: 506. Flamaster [FLAMASTE]

To jest inny problem

sprawdź dla danych, gdzie masz 111*A + 3*B

choć nie mam pewności, czy to jedyny problem

spojrzałem na kod, i zobaczyłem, że dla 111 kolejnych takich samych znaków da błędny wynik, dalej już nie sprawdzałem

Moje poprawki: cin.sync -> cin.get oraz:

for (int i=0; i<x; i++)
{
  for(int j = 0; j < napis[i].length(); ++j) 
  {
    if(isalnum(napis[i][j])) 
    {
      cout << napis[i][j];
    }
  }
  cout << endl;
}

Witam, pomoże ktoś? Nie znalazłem wartości, dla których odpowiedź się nie zgadza. Wrzucenie do pliku też nic nie zmienia. Macie jakiś pomysł?

Twój kod nie działa dla testów przykładowych.

Masz jakiś pomysł jak zrobić, żeby dla więcej niż 1 cyfrowych takich samych liter?
Bo zupełnie nie wiem jak to zrobić

W przypadku Twojego kodu oczywiście nie - nie pamiętam go po ok 9h od ostatniego czytania, nota bene na szybko. A kilka konstrukcji zdecydowanie utrudnia jego czytanie (np. for(same=2, licznik=i; slowo[licznik+1]==slowo[licznik+2]; same++, licznik++)).

Mam jednak dziwne przeczucie, że nie umiesz operować ASCII, i np. same+48 to tak na prawdę same+‘0’, zaś wyniki dla więcej niż 1-cyfrowych testów to po prostu graficzna reprezentacja kodów ASCII od 58 w górę. Jak zapewne się domyślasz, przyjrzenie się temu zagadnieniu powinno doprowadzić do AC

Dzięki, spróbuję uprościć ten kod i przyjrzeć się zagadnieniu kodu ASCII

Dlaczego sędzia nie akceptuje tego kodu?

#include <iostream>
using namespace std;
int a,b,c,z;
string w,g[1];
char s;
int main()
{
    cin>>c;
    string t[c];
    for(int i=0; i<c; i++)
        cin>>t[i];
    for(int i=0; i<c; i++)
    {
        w=t[i];
        s=w[0];
        a=1;
        b=w.length();
        z=0;
        for(int j=1; j<=b; j++)
        {
            if (w[j]==s)
                a++;
            else
            {
                if(a>2)
                {
                    for(int l=z; l<=j-a; l++)
                        cout<<w[l];
                    cout<<a;
                    z=j;
                }
                a=1;
                s=w[j];
            }
        }
        for(int l=z; l<=b; l++)
            cout<<w[l];
        cout<<endl;
    }
}

Bardzo proszę o pomoc.