c++ te same funkcje blokują się wzajemnie

pytanie zadane 9 lipca 2015 w C i C++ przez katolik6 Dyskutant (8,140 p.)
edycja 9 lipca 2015 przez katolik6

Mam funkcję:

bool TForm1::strzal(int x, int y, int xd, int yd, int zasieg){//startowy, docelowy, zasięg
TImage *pocisk;
pocisk=new TImage(Form1);
if((xd==x)&&(y==yd))return 0;
int xr=0, yr=0, d, xb=xd-x, yb=yd-y, xw=1, yw=1;//reszta, dłuższy, boki, wektor
if(yb<0)yb=-yb;
if(xb<0)xb=-xb;
if(xd<x)xw=-1;
if(yd<y)yw=-1;
if(xb>yb)d=xb; else d=yb;

pocisk->Visible=true;
pocisk->Parent=Form1;
pocisk->AutoSize=true;
pocisk->Picture=bullet;
sndPlaySound("gun.wav", SND_ASYNC);
//przesuwanie<br>
for(int i=0; i<zasieg; i++){
    xr+=xb%d;
    if(xr>=d){xr-=d; x+=xw;}
    x+=(xb/d)*xw;
    yr+=yb%d;
    if(yr>=d){yr-=d; y+=yw;}
    y+=(yb/d)*yw;
    pocisk->Left=x-1;
    pocisk->Top=y-1;
//kolizja<br>
    if(trafienie(beczka1, x, y)){buum(beczka1); delete pocisk; return true;}//funkcja trafienie odejmie hp
    if(pixel[x][y]){delete pocisk; return false;}
    Application->ProcessMessages(); Sleep(10);
}}

i gdy wywołuję ją dwa razy, tak, że druga jest wywoływana zanim pierwsza zwróci wartość, to pierwsza się ,,zamraża'' na czas wykonania drugiej (co oczywiście nie jest porządane), widać to po obiekcie pocisk. Czy ktoś wie czym to jest spowodowane (może

Application->ProcessMessages(); Sleep(10);

, jednak nie umiem zatrzymać pętli inaczej bez wstrzymywania całej reszty programu, więc nie mam jak tego sprawdzić) i jak można ten problem obejść?

P.S. Zauważyłem, że blokują też wyłączenie programu.

2 odpowiedzi

odpowiedź 9 lipca 2015 przez hit02 Nałogowiec (33,970 p.)
wybrane 11 lipca 2015 przez katolik6

Najlepsza

Czy uruchamiasz funkcje w oddzielnych wątkach? Funkcja Sleep() zamraża cały wątek.

Warto by określić dokładnie, w którym miejscu funkcje się zamrażają. Możesz spróbować debuggerem lub wypisywać coś na konsolę i sprawdzać, czy tam już jest zamrożone, czy nie.

komentarz 10 lipca 2015 przez katolik6 Dyskutant (8,140 p.)

wywołuję ją w OnClicku przeźroczystego TImage rozciągniętego na całą formę. Funkcja okodowuje tor lotu pocisku , więc dość dokładnie widać kiedy kula się zatrzymuje: dokładnie w chwili wystrzału drugiej.
Co do wielowątkowości: pojedynczy OnClick na pewno ma oddzielny wątek, ale nie wiem jak jest w tej sytuacji, czyli dwukrotne jednoczesne jego wywołanie, ale wygląda na to, że to ten sam wątek.

komentarz 10 lipca 2015 przez hit02 Nałogowiec (33,970 p.)

Nie wiem, jak to wygląda dokładnie w tej sytuacji, więc pewnie da się to zrobić lepiej, ale ja bym poprostu zamiast wywoływać funkcję strzal() wywołał jakiś wrapper, ktury by wywoływał strzal() w oddzielnym wątku.

Możesz też pokombinować z synchronizacją czasu tj. nie uruchamiać wszystkich pocisków w oddzielnych wątkach, bo zbyt wydajne to to nie jest, ale stworzyć listę pocisków i w pętli robić obliczenia dla każdego z nich na podstawie czasu, który minął.