Czemu ten kod zawiesza program?

Question

Witam, napisałem program. Program działa do czasu, gdy nie zdejmę komentarza z części

	     		int dlugosci = GetWindowTextLength(NazwaPliku);
				LPSTR nazwa = (LPSTR)GlobalAlloc(GPTR, dlugosci + 1);
				GetWindowText(NazwaPliku, nazwa, dlugosci + 1);
				
	     		ifstream plik;
				plik.open(nazwa);
				string calosc, linia;
				while(!plik.eof())
				{
					getline(plik, linia);
					calosc += linia+'\n';
				}
				string tekst = calosc;
				int n = tekst.length();
				char * lista = new char[n+1];
				strcpy(lista, tekst.c_str());
				znak = lista[i];
				while(znak!="x") 
				{
					widths = widths + lista[i];
					i = i+1;
				}
				i = i+1;
				while(znak!="z") 
				{
					heights = heights + lista[i];
					i = i+1;
				}
				i = i + 1;
				width = atoi(widths.c_str());
				height = atoi(heights.c_str());

Część z problemowym kodem:

		case WM_COMMAND:
		{
			if(( HWND ) lParam == Wczytywanie )
	     	{
	     		int dlugosci = GetWindowTextLength(NazwaPliku);
				LPSTR nazwa = (LPSTR)GlobalAlloc(GPTR, dlugosci + 1);
				GetWindowText(NazwaPliku, nazwa, dlugosci + 1);
				
	     		ifstream plik;
				plik.open(nazwa);
				string calosc, linia;
				while(!plik.eof())
				{
					getline(plik, linia);
					calosc += linia+'\n';
				}
				string tekst = calosc;
				int n = tekst.length();
				char * lista = new char[n+1];
				strcpy(lista, tekst.c_str());
				znak = lista[i];
				while(znak!="x") 
				{
					widths = widths + lista[i];
					i = i+1;
				}
				i = i+1;
				while(znak!="z") 
				{
					heights = heights + lista[i];
					i = i+1;
				}
				i = i + 1;
				width = atoi(widths.c_str());
				height = atoi(heights.c_str());
				kliknieto = 1;
			    UpdateWindow(hwnd);

 

Answer 1

                znak = lista[i];
                while(znak!="x") 

Bo masz tu nieskończoną pętlę.

Co to w ogóle ma robić? Strasznie dziwnie ten kod wygląda i trudno powiedzieć jaki jest jego cel.

(btw nigdzie też nie widać gdzie w ogóle deklarujesz i ustawiasz jakąś wartość dla `i`  i innych zmiennych, czemu `i` nie jest tutaj lokalne?)

Ostatnio też Ci mówiliśmy że nie musisz robić tego dziwnego kopiowania stringa z strcpy https://forum.pasja-informatyki.pl/568400/konwersja-string-na-liste , więc nie wiem czemu wciąż uparcie próbujesz to robić.

Comments

Nawet 2 nieskończone pętle:

while(znak!="x") 
{
    widths = widths + lista[i];
    i = i+1;
}
i = i+1;
while(znak!="z") 
{
    heights = heights + lista[i];
    i = i+1;
}

Ale bez opisu i całości kody trudno stwierdzić jak to naprawić .

---

(... w sumie to mam wątpliwości czy to się w ogóle kompiluje? Bo `znak!="x"` to jest porównanie chara do literału stringowego, co się nie skompiluje przecież.)

---

Z tego co widzę w kodzie który dołączył kolegą nie ma definicji zmiennej znak więc możliwe że jest to poprawne jeśli kod mu się skompilował bez błędów.

---

zmienna znak jest globalna, dołąćzyłem tylko część case z kodu