[RaVo]

#programowanie #spoj #cpp
No dalej, dlaczego to mi ciągle wyświetla jakiś śmieć na końcu?
http://pastebin.com/e3VTnBTC
http://pl.spoj.com/problems/PP0504B/

[Dymoslaw]

@RaVo: a nie prościej w c++

'
#include
#include
using namespace std;

int main(void) {
int n;
string s1, s2;
cin >> n;
for(int i = 0; i < n; i++) {
cin >> s1;
cin >> s2;

for(int j = 0; j < (int)min(s1.size(), s2.size()); j++)
cout << s1[j] << s2[j];
cout << endl;
}

return 0;
}
'

[RaVo]

@Dymoslaw: Ja zdaję sobie sprawę, że można to zrobić o wiele łatwiej, po prostu zastanawia się co robię źle, że wyświetla częsc dobrego wyniku i na końcu zawsze jest jakiś śmieć z pamięci.

[Dymoslaw]

może zapomniałeś o \0 ?

[nargil]

@RaVo: masz za maly bufor (czemu lend-1 ?!)

powinienes dac lend+1 i po pętli T[lend]='\0';

ps. jesli str2 jest krotszy, to potem drugi raz szukasz jego dlugosci.
sizet len = std::min(strlen(str1), strlen(str2);

ps2. rzutujesz sizet (zwracany przez strlen) na inta, wiec przy odpowiednio dlugim stringu wyjdzie ci z tego wartosc ujemna.

Poza tym popracuj nad stylem. Z jednej strony c++ (cin,cout,new,delete), a z drugiej ansi c (deklaracja

[A.....k]

@RaVo: W jednej pętli ciągle przydzielasz i zwalniasz pamięć. Pomyśl o buforze, może.