Aplikacja napisana przeze mnie samoistnie się zamyka

Question

Witam, napisałem program w C++. Kompilator nie generuje żadnych błędów podczas kompilacji. Problem polega na tym, że gdy uruchamiam aplikacje ona się otwiera pojawia się puste okno i się zamyka (system nie generuje błędu "aplikacja nie odpowiada", ani żadnego innego). Aplikacja jest napisana z użyciem WinAPI. Program działa dobrze do czsu, gdy nie zdejme komentarza z tego fragmentu kodu:

case WM_PAINT:
		{
		
			
			PAINTSTRUCT ps;
		    HDC hdc = BeginPaint(hwnd, &ps);
		   while(znak!="A")
		    {
		    	
		    		r1 = lista[i];
		    		i = i+1;
		    		if(znak!=",")
		    		{
		    			r2=lista[i];
		    			i = i+1;
		    			if(znak!=",")
		    			{
		    				r3 = lista[i];
							i = i+1;	
						}
					}
					
					i = i+1;
		    		
		    		g1 = lista[i];
		    		i = i+1;
		    		if(znak!=",")
		    		{
		    			g2=lista[i];
		    			i = i+1;
		    			if(znak!=",")
		    			{
		    				g3 = lista[i];
							i = i+1;	
						}
					}
					
					i = i+1;
					
					b1 = lista[i];
		    		i = i+1;
		    		if(znak!=",")
		    		{
		    			b2=lista[i];
		    			i = i+1;
		    			if(znak!=",")
		    			{
		    				b3 = lista[i];
							i = i+1;	
						}
					}
					
					i=i+2;
		    	
		    	rl = r1+r2+r3;
		    	gl = g1+g2+g3;
		    	bl = b1+b2+b3;
		    	
		    	r = atoi(rl.c_str());
		    	g = atoi(gl.c_str());
		    	b = atoi(bl.c_str());
		    	
		    	SetPixel(hdc, x, y, RGB(r,g,b));
		    	x+1;
			}
		    EndPaint(hwnd, &ps); 
			break;
		}

 

Comments

BTW, linia 64  co to?

 x+1;

chyba chodziło Tobie o

x = x + 1;

lub

x++;

lub

x += 1;

 

wydaje Mi się, że ten zapis

case WM_PAINT:
        {
         
             
            PAINTSTRUCT ps;
            HDC hdc = BeginPaint(hwnd, &ps);
           while(znak!="A")
            {
                 
                    r1 = lista[i];
                    i = i+1;
                    if(znak!=",")
                    {
                        r2=lista[i];
                        i = i+1;
                        if(znak!=",")
                        {
                            r3 = lista[i];
                            i = i+1;    
                        }
                    }
                     
                    i = i+1;
                     
                    g1 = lista[i];
                    i = i+1;
                    if(znak!=",")
                    {
                        g2=lista[i];
                        i = i+1;
                        if(znak!=",")
                        {
                            g3 = lista[i];
                            i = i+1;    
                        }
                    }
                     
                    i = i+1;
                     
                    b1 = lista[i];
                    i = i+1;
                    if(znak!=",")
                    {
                        b2=lista[i];
                        i = i+1;
                        if(znak!=",")
                        {
                            b3 = lista[i];
                            i = i+1;    
                        }
                    }
                     
                    i=i+2;
                 
                rl = r1+r2+r3;
                gl = g1+g2+g3;
                bl = b1+b2+b3;
                 
                r = atoi(rl.c_str());
                g = atoi(gl.c_str());
                b = atoi(bl.c_str());
                 
                SetPixel(hdc, x, y, RGB(r,g,b));
                x+1;
            }
            EndPaint(hwnd, &ps); 
            break;
        }

można zapisać tak

    r1 = lista[i];
    if (znak != ",")
    {
        r2 = lista[++i];
        r3 = lista[++i];
    }
    
    i += 2;
    g1 = lista[i];
    if (znak != ",")
    {
        g2 = lista[++i];
        g3 = lista[++i];
    }
    
    i += 2;
    b1 = lista[i];
    if (znak != ",")
    {
        b2 = lista[++i];
        b3 = lista[++i];
    }                     

    i += 2;

 

---

Dodam jeszcze, że nie widać, by w ciele pętli zmienna znak była zmieniana, zatem ta pętla jak już zacznie się kręcić, to bez końca, ergo aplikacja się zawiesi.

---

Obsługa WM_PAINT powinna być możliwie krótka. Powinna służyć tylko do odmalowania powierzchni okna i nic więcej. Kontrola powinna być oddana jak najszybciej do pętli komunikatów.