506. Flamaster [FLAMASTE]

[BlackJack_sof]

Oto mój kod w Perlu, czemu dostaję WA? Wszystkie swoje testy i te przykładowe przechodzi

<>;
while(<>)
{
	chomp($_);
	while(length($_))
	{
		$ile = 0;
		$znak=substr($_,0,1);
		$ile++ while $_ =~ /$znak/g;
		$_ =~ s/$znak//g;
		print "$znak$ile" if($ile>2);
		print $znak x $ile if($ile<3);
	}
	print "\n";
}

[harold_sof]

Popatrz, jak Twój skrypt radzi sobie z danymi typu
AAABBBBAAAA
Co do stylu, to nigdy nie pisz "$_ =~ /re/" czy "$_ =~ s/foo/bar/;" (jest to powszechnie uważane za nieeleganckie), tylko po prostu "/re/" i "s/foo/bar/;".
A na koniec (jeżeli będziesz miał ochotę) spróbuj ograniczyć ciało pętli while (<>) do 2 instrukcji, w końcu to Perl

[manalishi]

Trochę się pobawiłem według rad (za które serdecznie dziękuję). Dla ułatwienia rozumowania pozbyłem się także tabel danych wejściowych i rozwiązań. Niestety rozwiązanie jest wciąż wadliwe. Teraz program liczy tylko liczbę znaków w stringu, nie liczbę tych samych znaków. Jakby

jowisz[k]==jowisz[k+1]

było wiecznie prawdziwe. Możliwe że program nie rozumie wyrażenia "j+=g" jako warunek pierwszej pętli.
Na dane wejściowe np NKKKKKDDDD odpowiada N10.

Usunąłem!

[manalishi]

Z tym zerem przy g to wynik jednej z paru prób szukania błędu. Przekopiowałem kod tutaj zapominając o tym gremlinie. Ale z operatorem && to skopałem po pełnej linii. Program w końcu działa. Serdecznie dziękuję za pomoc.

[robertrobert]

Witam,
mi rowniez to zadanie sprawia troche problemow. Przede wszystkim tablica jest jakby pusta chociaz mozna wczytac pojedyncze jej elementy (cout<< wyjscie[x] ) i nastepuje podmiana jesli string wyjscie="AAA...". Druga sprawa ze ilosc danej litery jest wypisywana przed litera chyba ze ciag liter nie jest poprzedzony innymi. Pomozecie?

#include <iostream>
#include <string>
using namespace std;
int ile, dlug, dlug2;
string wejscie;
int main()
{
    cin >> ile;
    for (int i=0; i<ile; i++) {
        cin >> wejscie;
        dlug = wejscie.length();
        string wyjscie;
        for (int j=0, z=0; j<dlug; j++, z++) {
            int k =1, c=j;
            wyjscie[z]= wejscie[j];
            do {
                if (c<dlug-1) {
                   if (wejscie[c]==wejscie[c+1])
                        k++;
                        c++;
                    }
                }while (wejscie[c]==wejscie[c+1]);
                if (k>2 && k<10) {
                    wyjscie[z+1]= k + 48;
                    z++;
                    j+=(k-1);
                    }
                else if (k>9 && k<100) {
                    int jed, dzie;
                    jed=k%10;
                    dzie=k/10;
                    wyjscie[z+1]= dzie + 48;
                    wyjscie[z+2]= jed + 48;
                    z+=2;
                    j+=(k-1);
                    }
                else if (k>99 & k<1000) {
                    int jed, dzie,setki;
                    jed=k%10;
                    setki= k/100;
                    dzie=((k-(setki*100))/10);
                    wyjscie[z+1]= setki + 48;
                    wyjscie[z+2]= dzie + 48;
                    wyjscie[z+3]= jed + 48;
                    z+=3;
                    j+=(k-1);
                    }
                }
                dlug2 = wyjscie.length();
                cout << dlug2 << "       ";
                cout << wyjscie << endl;
            }
    return 0;
}

[krystian01]

Napisz program od nowa w następujący sposób, sam tak go rozwiązałem(powiem ci tylko kilka porad żebyś miał zabawę):
1. Zrób tak by twój program zliczył ile jest różnych liter - sttttreewas - tutaj jest 6 róznych od siebie liter. Jak widzisz 's' się powtarza na początku i na końcu, ale tego nie możesz wziąć pod uwagę bo nie znajdują się koło siebie. Zakładasz, że są różne.
Przykładowo:
[bbone=CPP,2320] for (o = 1; o < heh.length(); ++o){
if (heh[o - 1] == heh[o])
continue;
else
++u;
}[/bbone]
2. Następnie zrób pętlę, która przeleci literę po literze i jeżeli będą koło siebie te same litery, to niech zlicza do jakiejś zmiennej(++z). Do tego potrzebne będzie dynamiczne alokowanie pamięci.
3. Stwórz jakąs zmienna która będzie przechowywała znak. Np. char bufer;
4. Poczytaj także o:
- pętli nieskończonej - for(;
- break;
- continue;
- dynamiczne alokowanie pamięci
PS.
Nawet nie starałem się tego dogłębnie czytać, ale widzę, że przy niektórych momentach dobrze myślisz.
dlug = wejscie.length();
wejscie[c]==wejscie[c+1]; // tutaj zmieniłbym inaczej. W pętli ustawiłbym wartosc poczatkowa 1, a później wejscie[c]==wejscie[c-1]

-- Cz sty 29, 2015 5:00 pm --

Błąd u ciebie kryje się już w tej linii:
[bbone=CPP,2321] if (k>2 && k<10) {
wyjscie[z + 1] = k + 48;
z++;
j += (k - 1);
}[/bbone]
Dalej nie sprawdzałem. Wykonujesz jakieś obliczenia, które nie są potrzebne.