While in Coroutine Unity Crash

I'm trying to loop through a bunch of files inside an URL folder, when i use a "for" loop, it works perfectly fine, but when i use a "do while" or "while" Unity crashes, i tested the CheckURL method and it works as expected.

Code (CSharp):

1.  public static IEnumerator DownloadIcons()
2.     {
3.  
4.         int i = 0;
5.  
6.         string url = Preloader.base_url + "/src/icons/" + i + ".png";
7.  
8.         while (CheckUrl(url))
9.         {
10.             UnityWebRequest request = UnityWebRequestTexture.GetTexture(url);
11.             yield return request.SendWebRequest();
12.  
13.             while (!request.downloadHandler.isDone)
14.                 yield return null;
15.  
16.             if (request.downloadHandler.isDone)
17.             {
18.  
19.                 Texture2D tex = ((DownloadHandlerTexture)request.downloadHandler).texture;
20.                 Preloader.list_icons.Add(SrcManager.WriteTexture(tex));
21.                 i++;
22.                 url = Preloader.base_url + "/src/icons/" + i + ".png";
23.                 yield return null;
24.             }
25.  
26.         }
27.  
28.  
29.     }
30.  
31.  
32.     public static bool CheckUrl(string url)
33.     {
34.  
35.         WebRequest webRequest = WebRequest.Create(url);
36.         WebResponse webResponse;
37.         try
38.         {
39.             webResponse = webRequest.GetResponse();
40.         }
41.         catch //If exception thrown then couldn't get response from address
42.         {
43.             return false;
44.         }
45.         return true;
46.  
47.     }

any ideas ?

 

You never stop the loop. You're effectively saying while(true) as long as there are no errors. You need to break out of the while loop later on, or return from the function entirely when the icon is downloaded.

 

Even though i'm pretty sure my checks are fine, you got a good point, i'd rather wait for an error from UnityWebRequest than doing this nonsense, i think i was a bit frustrated i wasn't thinking clearly.
Thanks a lot, i'll double check!

 

When (CheckURL) returns false, the loop should be stopped no ? am i getting it wrong ?