[jaggi]

jak już wrzuciłem tę pseudo gierkę to od razu pseudo program(y) do konwersji napisów do filmów ;p

http://pastebin.com/brHanny0

http://pastebin.com/DBSAzdvr

#programowanie #cpp #nuda #codereview

[a.....y]

@jaggi: u mnie w Visual Studio nic z twoich nie działa

[jaggi]

@asdasty: a u mnie z TDM-GCC 4.7.1 działa wszystko ( a przynajmniej ostatnio jak kompilowałem to działało)

[jaggi]

@asdasty: chyba że nie masz jakichś bibliotek albo masz inne wersje czy cuś

[sylwke3100]

@jaggi: Ja nie jestem jakiś pro programistą ale jaki sens jest rozdzielanie tego na dwie linjki ?zamiast

i=0;

for(;i<line.length();i++)

Zamiast :

for(int i =0;i

a jeszcze lepiej bo można by to było zrobić mniej więcej tak

for(auto char: line)

Poza tym to :

line="{"+sf1+"}"+"{"+sf2+"}"+line.erase(0, i+1);

dziwne wydaje mi się 

Poza tym jak już używasz tablic i C++ to mógłbyś wektorów używać 

Zamiast takiego dziwactwa:

string* tab=new string[x];

A i nie sprawdzasz

[KrzaQ2]

@sylwke3100: Pokutujące przyzwyczajenie z C89. http://ideone.com/eGolOo

[sylwke3100]

@KrzaQ2: Wydaje mi się że jak ktoś używa i sygnuje się na C++sowca to powinien się z tego leczyć. Takie nawyki są złe tak jak "goto"

[KrzaQ2]

@sylwke3100: Ja nie mówię, ƶe to dobre, po prostu wyjaśniam skąd się to wzięło (w sumie jak ktoś pisze kod w C pod Windows to wciąƶ tak musi pisać - MSVC nie wspiera C99, o C11 nie wspominając) :P

A co do goto - wbrew pozorom czasem moƶe być przydatne, chociaƶ w tym momencie nie mogę sobie przypomnieć ƶadnej dobrej do tego sytuacji.

[jaggi]

@sylwke3100:

i=0;

for(;i<line.length();i++)

używam tego samego "i" w dwóch kolejnych pętlach; tylko dlatego tak napisałem

line="{"+sf1+"}"+"{"+sf2+"}"+line.erase(0, i+1);

tutaj po prostu zamieniam {xxx}{xxx}jakistekst na {yyy}{yyy}jakistekst

jakoś nie wiem czemu tak zrobiłem ;p

for(auto char: line)

tak szczerze to z tym się nie spotkałem, ja nie jestem jakoś na bierząco. piszę jak mi się nudzi, jak jestem chory i siedzę w domu zamiast iść do szkoły

[sylwke3100]

@jaggi: Czyli robisz mały błąd bo nie zerujesz zmiennej a ją w kolejnej pętli używasz co może powodować że przekroczysz nieumyślnie rozmiar stringa gdyż inkrementujesz ze starą wartością

Od zamiany tekstu masz string::replace

http://www.cplusplus.com/reference/string/string/replace/

Poza tym warto by było zdobyć trochę wiedzy tym bardziej o obektówce.

[jaggi]

@sylwke3100: właśnie o to chodzi żeby nie zerować tej zmiennej bo jest mi potrzebna wartość jaką miała po wyjściu z poprzedniej pętli i nie ma prawa przekroczyć rozmiaru stringa bo osiągnięcie przez tą zmienną rozmiaru.stringa jest warunkiem kończącym jedną i drugą pętle, więc w razie czego ta 2 po prostu się nie wykona. Wiem że jest replace; tylko że tutaj było mi tak wygodniej bo z replace musiałbym jeszcze sprawdzać pozycje

[sylwke3100]

@jaggi: Wydaje mi się że naprawdę sobie pokomplikowałeś ten kod niepotrzebnie.

Tak nawiasem

for(;i
{

if(line[i]=='{')

{

i++;

while (line[i]!='}')

{

temp+=line[i];

i++;

}

break;

}

}

Moża by zapisać tak :

http://ideone.com/2wGhyc

Wtedy być F1 i F2 oraz kolejne z tablicy wyciągał i mniej pętli to szybciej a znając życie @KrzaQ2 by to lepiej napisał