Не отрабатывает нормально цикл c# нид хелп

Raids

Client
Регистрация
12.09.2017
Сообщения
41
Благодарностей
19
Баллы
8
Есть код
Код:
        list_photo.Clear();
        reg1.Matches(post).Cast<Match>().ToList().ForEach(m=>list_photo.Add(m.Value));
        for (int f=0;f<list_photo.Count;f++)
        {
            parhe = list_photo[f];
            photo = String.Empty;
            project.SendInfoToLog(parhe,true);
            reg2.Matches(parhe).Cast<Match>().ToList().ForEach(m=>list_photo1.Add(m.Value));
            project.SendInfoToLog(Convert.ToString(list_photo1.Count),true);
            b = list_photo1.Count - 1;
            project.SendInfoToLog("b:" + b,true);
            project.SendInfoToLog("list_photo1[b]:" + list_photo1[b],true);
            photo = photo + "<img>" + list_photo1[b] + "</img>";
            list_photo1.Clear();
        }
первый круг отрабатывает нормально на втором не пихает данные в list_photo1
01.jpg
 

dr1veq

Пользователь
Регистрация
09.02.2018
Сообщения
39
Благодарностей
44
Баллы
18
Есть код
Код:
        list_photo.Clear();
        reg1.Matches(post).Cast<Match>().ToList().ForEach(m=>list_photo.Add(m.Value));
        for (int f=0;f<list_photo.Count;f++)
        {
            parhe = list_photo[f];
            photo = String.Empty;
            project.SendInfoToLog(parhe,true);
            reg2.Matches(parhe).Cast<Match>().ToList().ForEach(m=>list_photo1.Add(m.Value));
            project.SendInfoToLog(Convert.ToString(list_photo1.Count),true);
            b = list_photo1.Count - 1;
            project.SendInfoToLog("b:" + b,true);
            project.SendInfoToLog("list_photo1[b]:" + list_photo1[b],true);
            photo = photo + "<img>" + list_photo1[b] + "</img>";
            list_photo1.Clear();
        }
первый круг отрабатывает нормально на втором не пихает данные в list_photo1
Посмотреть вложение 27521
Добавьте просто проверку если индекс уходит в -1.
Код:
        list_photo.Clear();
        reg1.Matches(post).Cast<Match>().ToList().ForEach(m=>list_photo.Add(m.Value));
        for (int f=0;f<list_photo.Count;f++)
        {
            parhe = list_photo[f];
            photo = String.Empty;
            project.SendInfoToLog(parhe,true);
            reg2.Matches(parhe).Cast<Match>().ToList().ForEach(m=>list_photo1.Add(m.Value));
            project.SendInfoToLog(Convert.ToString(list_photo1.Count),true);
            b = list_photo1.Count - 1;
            project.SendInfoToLog("b:" + b,true);
            if (b >= 0 ){
            project.SendInfoToLog("list_photo1[b]:" + list_photo1[b],true);
            photo = photo + "<img>" + list_photo1[b] + "</img>";
            list_photo1.Clear();
                             }
        }
 

Raids

Client
Регистрация
12.09.2017
Сообщения
41
Благодарностей
19
Баллы
8
Добавьте просто проверку если индекс уходит в -1.
Код:
        list_photo.Clear();
        reg1.Matches(post).Cast<Match>().ToList().ForEach(m=>list_photo.Add(m.Value));
        for (int f=0;f<list_photo.Count;f++)
        {
            parhe = list_photo[f];
            photo = String.Empty;
            project.SendInfoToLog(parhe,true);
            reg2.Matches(parhe).Cast<Match>().ToList().ForEach(m=>list_photo1.Add(m.Value));
            project.SendInfoToLog(Convert.ToString(list_photo1.Count),true);
            b = list_photo1.Count - 1;
            project.SendInfoToLog("b:" + b,true);
            if (b >= 0 ){
            project.SendInfoToLog("list_photo1[b]:" + list_photo1[b],true);
            photo = photo + "<img>" + list_photo1[b] + "</img>";
            list_photo1.Clear();
                             }
        }
Это решение, что бы цикл пошел дальше, но дело в том, что цикл не отрабатывает как нужно, не забирает все данные. А мне они нужны
1 проход он забирает данные потом через регулярку закидывает все в список
2 проход он забирает данные и они не пустые, но в список их уже не кладет
 

ssXXXss

Client
Регистрация
23.12.2014
Сообщения
7 379
Благодарностей
2 041
Баллы
113
выкини дома посуду а потом скажи себе а куда она делась )))
ты очищаешь список в цикле и ждёшь в нём результат
 

Raids

Client
Регистрация
12.09.2017
Сообщения
41
Благодарностей
19
Баллы
8
выкини дома посуду а потом скажи себе а куда она делась )))
ты очищаешь список в цикле и ждёшь в нём результат
да что бы добавить новые данные.

А как тогда сделать правильно?
 

ssXXXss

Client
Регистрация
23.12.2014
Сообщения
7 379
Благодарностей
2 041
Баллы
113

Raids

Client
Регистрация
12.09.2017
Сообщения
41
Благодарностей
19
Баллы
8
Логика такая api vk возвращает список фоток
каждая фотка заключена в <photo>тут данные фотки</photo>
а в данных фотки содержатся ссылки на само фото, последняя самая большая
поэтому беру данные о фотке и кладу их в список
Код:
reg1.Matches(post).Cast<Match>().ToList().ForEach(m=>list_photo.Add(m.Value))
дальше беру последнюю строку и дописываю в переменную
Код:
photo = photo + "<img>" + list_photo1[b] + "</img>";
дальше мне нужна следующая фотка, поэтому список очищается и в него я хочу положить новые данные, о новой фотке но они туда не кладуться.
 

ssXXXss

Client
Регистрация
23.12.2014
Сообщения
7 379
Благодарностей
2 041
Баллы
113
Логика такая api vk возвращает список фоток
каждая фотка заключена в <photo>тут данные фотки</photo>
а в данных фотки содержатся ссылки на само фото, последняя самая большая
поэтому беру данные о фотке и кладу их в список
Код:
reg1.Matches(post).Cast<Match>().ToList().ForEach(m=>list_photo.Add(m.Value))
дальше беру последнюю строку и дописываю в переменную
Код:
photo = photo + "<img>" + list_photo1[b] + "</img>";
дальше мне нужна следующая фотка, поэтому список очищается и в него я хочу положить новые данные, о новой фотке но они туда не кладуться.
ты в этом цикле ничего не дописываешь ни куда, ты в нём же всё очищаешь и перезаписываешь, ты нормально задачу свою объсни
ты какого вообще результата ждёшь от того что ты всё чистеешь, или ты думаешь очистив списов и перезаписав переменую где то должно отобразиться
плюс сравнение у тебя стоит b >= 0, ты при пустом списке пытаешься ещё какие действия с ним сделать, ну честно полная билеберда
 

Raids

Client
Регистрация
12.09.2017
Сообщения
41
Благодарностей
19
Баллы
8
ты в этом цикле ничего не дописываешь ни куда, ты в нём же всё очищаешь и перезаписываешь, ты нормально задачу свою объсни
ты какого вообще результата ждёшь от того что ты всё чистеешь, или ты думаешь очистив списов и перезаписав переменую где то должно отобразиться
плюс сравнение у тебя стоит b >= 0, ты при пустом списке пытаешься ещё какие действия с ним сделать, ну честно полная билеберда
Давай попробую по шагам.

Есть ответ api vk

Код:
<photo>
      <id>431572122</id>
      <album_id>-7</album_id>
      <owner_id>18431189</owner_id>
      <photo_75>https://pp.userapi.com/c630018/v630018189/46c38/5rvHxSswaqU.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c630018/v630018189/46c39/wPfv5ZenI-I.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c630018/v630018189/46c3a/u2h4hcYC0FE.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c630018/v630018189/46c3b/nVmxBUNKdp4.jpg</photo_807>
      <width>640</width>
      <height>640</height>
      <text/>
      <date>1473415437</date>
      <post_id>1369</post_id>
      <access_key>ee4612f1555ea11637</access_key>
     </photo>
<photo>
      <id>422017904</id>
      <album_id>-7</album_id>
      <owner_id>18431189</owner_id>
      <photo_75>https://pp.userapi.com/c631418/v631418189/442eb/GU3rYb9Thq0.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c631418/v631418189/442ec/j32gCYpTm80.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c631418/v631418189/442ed/BuuiKMGDA1Q.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c631418/v631418189/442ee/psBhBO80F_A.jpg</photo_807>
      <width>640</width>
      <height>640</height>
      <text/>
      <date>1467438233</date>
      <post_id>1320</post_id>
      <access_key>53333fd4f156e8bd3b</access_key>
     </photo>
<photo>
      <id>414922583</id>
      <album_id>-7</album_id>
      <owner_id>18431189</owner_id>
      <photo_75>https://pp.userapi.com/c615823/v615823189/1225c/v2FtDAesIws.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c615823/v615823189/1225d/7pGeUV9VTTE.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c615823/v615823189/1225e/z6iXXFGrRW4.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c615823/v615823189/1225f/DTUlF6x2jLo.jpg</photo_807>
      <width>750</width>
      <height>600</height>
      <text/>
      <date>1463075950</date>
      <post_id>1297</post_id>
      <access_key>7808fff162930006c3</access_key>
     </photo>
<photo>
      <id>409611983</id>
      <album_id>-7</album_id>
      <owner_id>18431189</owner_id>
      <photo_75>https://pp.userapi.com/c630321/v630321189/20d7e/MpzOnvmE-6k.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c630321/v630321189/20d7f/YZ3BAfrA8Ps.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c630321/v630321189/20d80/EykPT3JO4JQ.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c630321/v630321189/20d81/9i1npBH81sU.jpg</photo_807>
      <width>640</width>
      <height>640</height>
      <text/>
      <date>1459781310</date>
      <post_id>1246</post_id>
      <access_key>e0a5ade8748ae3f73b</access_key>
     </photo>
<photo>
       <id>456239393</id>
       <album_id>-2</album_id>
       <owner_id>100</owner_id>
       <photo_75>https://pp.userapi.com/c621511/v621511463/582c8/o1L-HdkfNgU.jpg</photo_75>
       <photo_130>https://pp.userapi.com/c621511/v621511463/582c9/sCQ-XXobDVM.jpg</photo_130>
       <photo_604>https://pp.userapi.com/c621511/v621511463/582ca/5uRUwIfxMms.jpg</photo_604>
       <width>150</width>
       <height>80</height>
       <text/>
       <date>1516076204</date>
      </photo>
<photo>
      <id>456241128</id>
      <album_id>-7</album_id>
      <owner_id>-129531439</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c834403/v834403749/cda6e/nFz23hAuHb0.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c834403/v834403749/cda6f/tzaK-LXIsI4.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c834403/v834403749/cda70/7QnDsyyz5o8.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c834403/v834403749/cda71/7Mfvh3fT_IA.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c834403/v834403749/cda72/5uyMQPxHxE4.jpg</photo_1280>
      <width>764</width>
      <height>1080</height>
      <text/>
      <date>1519157226</date>
      <post_id>493</post_id>
      <access_key>76945ae1bda564f971</access_key>
     </photo>
<photo>
      <id>456243839</id>
      <album_id>-7</album_id>
      <owner_id>372356438</owner_id>
      <photo_75>https://sun1-4.userapi.com/c840730/v840730542/5c7e7/R1zouf0d84g.jpg</photo_75>
      <photo_130>https://sun1-4.userapi.com/c840730/v840730542/5c7e8/blUPfcabKLY.jpg</photo_130>
      <photo_604>https://sun1-4.userapi.com/c840730/v840730542/5c7e9/dUNyKYhKZjY.jpg</photo_604>
      <width>428</width>
      <height>426</height>
      <text/>
      <date>1519129084</date>
      <post_id>1056</post_id>
      <access_key>811836a3a8fc43cf7e</access_key>
     </photo>
<photo>
      <id>456241214</id>
      <album_id>-7</album_id>
      <owner_id>-57661205</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c834102/v834102542/c5a44/HqiaWlX3oKc.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c834102/v834102542/c5a45/SPvPCSiEsWs.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c834102/v834102542/c5a46/R0VI9WXZy5w.jpg</photo_604>
      <width>428</width>
      <height>426</height>
      <text/>
      <date>1519144114</date>
      <post_id>12933</post_id>
      <access_key>c302a4e2b8f8dd712f</access_key>
     </photo>
<photo>
      <id>456239923</id>
      <album_id>-7</album_id>
      <owner_id>-99012821</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c840430/v840430760/58baa/KEJdy6Gf4D4.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c840430/v840430760/58bab/VjRuw-byaZQ.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c840430/v840430760/58bac/GeuWeTsbDnI.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c840430/v840430760/58bad/cxfu6fT0-pc.jpg</photo_807>
      <width>640</width>
      <height>529</height>
      <text/>
      <date>1518882967</date>
      <access_key>8e1ddff0febfae79c6</access_key>
     </photo>
<photo>
      <id>456239924</id>
      <album_id>-7</album_id>
      <owner_id>-99012821</owner_id>
      <user_id>100</user_id>
      <photo_75>https://sun1-4.userapi.com/c840535/v840535760/56b0c/mEx6qTYLOw4.jpg</photo_75>
      <photo_130>https://sun1-4.userapi.com/c840535/v840535760/56b0d/HzfiKL4udtM.jpg</photo_130>
      <photo_604>https://sun1-4.userapi.com/c840535/v840535760/56b0e/TpgyUT3TaQk.jpg</photo_604>
      <width>600</width>
      <height>399</height>
      <text/>
      <date>1518882967</date>
      <access_key>48f16a8ee973dabf66</access_key>
     </photo>
<photo>
      <id>456239925</id>
      <album_id>-7</album_id>
      <owner_id>-99012821</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c824502/v824502760/b3b4b/O2VbAqrQwos.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c824502/v824502760/b3b4c/YBAqpdDBJwE.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c824502/v824502760/b3b4d/4tiU6mkrKp8.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c824502/v824502760/b3b4e/y3Gt9-fCrvU.jpg</photo_807>
      <width>720</width>
      <height>480</height>
      <text/>
      <date>1518882967</date>
      <access_key>af0fa2932b3834f28a</access_key>
     </photo>
<photo>
      <id>456239926</id>
      <album_id>-7</album_id>
      <owner_id>-99012821</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c841530/v841530760/6bc89/AW8E3Dc6WRE.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c841530/v841530760/6bc8a/ANZlSqwSVF8.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c841530/v841530760/6bc8b/tBgadZAdyXU.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c841530/v841530760/6bc8c/Ky_768prgMg.jpg</photo_807>
      <width>640</width>
      <height>803</height>
      <text/>
      <date>1518882967</date>
      <access_key>f95ad69f3758087214</access_key>
     </photo>
<photo>
      <id>456240272</id>
      <album_id>-7</album_id>
      <owner_id>-5932009</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c830401/v830401475/6aa56/RRKb4h2OdHw.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c830401/v830401475/6aa57/nMqt5-7i6q4.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c830401/v830401475/6aa58/W0TVgdpW0Ks.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c830401/v830401475/6aa59/NuhD5QfqLhc.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c830401/v830401475/6aa5a/xmndt0Xgja4.jpg</photo_1280>
      <photo_2560>https://pp.userapi.com/c830401/v830401475/6aa5b/9liMAtXICss.jpg</photo_2560>
      <width>2048</width>
      <height>1364</height>
      <text/>
      <date>1519041503</date>
      <lat>59.94924928453</lat>
      <long>30.252859114701</long>
      <post_id>1796</post_id>
      <access_key>44a3959fb47e8b7735</access_key>
     </photo>
<photo>
      <id>456241669</id>
      <album_id>-7</album_id>
      <owner_id>27338364</owner_id>
      <photo_75>https://pp.userapi.com/c841023/v841023333/58800/fQVOiTtGtG8.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c841023/v841023333/58801/7NKdtVHiK4k.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c841023/v841023333/58802/SOrFRssgLsE.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c841023/v841023333/58803/raSj8qbtTnc.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c841023/v841023333/58804/TCbbNJwsV9o.jpg</photo_1280>
      <photo_2560>https://pp.userapi.com/c841023/v841023333/58805/MhJGCky0-qM.jpg</photo_2560>
      <width>2560</width>
      <height>1707</height>
      <text/>
      <date>1515950246</date>
      <access_key>7eab2f1b1626a9745c</access_key>
     </photo>
<photo>
      <id>456240244</id>
      <album_id>112533297</album_id>
      <owner_id>27338364</owner_id>
      <photo_75>https://pp.userapi.com/c638527/v638527537/5bd8f/NNhJMT5Q0Lg.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c638527/v638527537/5bd90/oX2XCDv3bJI.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c638527/v638527537/5bd91/pISGf-Zc4XE.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c638527/v638527537/5bd92/0Ppyr9S5o6U.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c638527/v638527537/5bd93/pUGvXJy6AdA.jpg</photo_1280>
      <photo_2560>https://pp.userapi.com/c638527/v638527537/5bd94/3OiTP4aqY6M.jpg</photo_2560>
      <width>2560</width>
      <height>1707</height>
      <text/>
      <date>1500827341</date>
      <access_key>d0818b656ba7a93e65</access_key>
     </photo>
<photo>
      <id>456240218</id>
      <album_id>112533297</album_id>
      <owner_id>27338364</owner_id>
      <photo_75>https://pp.userapi.com/c638527/v638527537/5bc80/BCmDwyvXNf4.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c638527/v638527537/5bc81/PSYlOj6n2xk.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c638527/v638527537/5bc82/QDAh7z19WA0.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c638527/v638527537/5bc83/Lj32lbQIYPM.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c638527/v638527537/5bc84/QUUF9Z8mAPY.jpg</photo_1280>
      <photo_2560>https://pp.userapi.com/c638527/v638527537/5bc85/J639V0yh3Jk.jpg</photo_2560>
      <width>2560</width>
      <height>1707</height>
      <text/>
      <date>1500826642</date>
      <access_key>fbd0508b003321feaa</access_key>
     </photo>
<photo>
      <id>456239824</id>
      <album_id>-7</album_id>
      <owner_id>27338364</owner_id>
      <photo_75>https://pp.userapi.com/c638621/v638621364/36412/fndys7yE0Vs.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c638621/v638621364/36413/te_yPpyz_0s.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c638621/v638621364/36414/tOAwgQ8RnF0.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c638621/v638621364/36415/HxJzdromx5M.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c638621/v638621364/36416/v_96S4bEkNw.jpg</photo_1280>
      <width>1080</width>
      <height>1080</height>
      <text/>
      <date>1493116205</date>
      <post_id>2505</post_id>
      <access_key>9ce54608e63d140038</access_key>
     </photo>
<photo>
      <id>456245508</id>
      <album_id>-7</album_id>
      <owner_id>-69734984</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c834101/v834101395/c12b2/eG9ABHxP3t8.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c834101/v834101395/c12b3/O1_zve-g__I.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c834101/v834101395/c12b4/Yi050tmJHF4.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c834101/v834101395/c12b5/wqoO1vhgRmY.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c834101/v834101395/c12b6/w74EkuUmOts.jpg</photo_1280>
      <width>960</width>
      <height>960</height>
      <text/>
      <date>1518910091</date>
      <post_id>8202</post_id>
      <access_key>89218a3da5247231f0</access_key>
     </photo>
<photo>
      <id>456246489</id>
      <album_id>162392081</album_id>
      <owner_id>166763402</owner_id>
      <photo_75>https://pp.userapi.com/c831208/v831208205/59da7/GP0RtrItERk.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c831208/v831208205/59da8/U3R2NaB4YJ4.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c831208/v831208205/59da9/GB6VpyNiL38.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c831208/v831208205/59daa/QHtGeeGlkxQ.jpg</photo_807>
      <width>640</width>
      <height>640</height>
      <text/>
      <date>1516954817</date>
      <access_key>11d898515fd6c1875f</access_key>
     </photo>
<photo>
      <id>456246620</id>
      <album_id>-7</album_id>
      <owner_id>166763402</owner_id>
      <photo_75>https://pp.userapi.com/c824203/v824203150/9da96/68n7VH6beOM.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c824203/v824203150/9da97/piq0BcwQqXc.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c824203/v824203150/9da98/zbaGPh2bZJs.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c824203/v824203150/9da99/n_16U7Nczk0.jpg</photo_807>
      <width>564</width>
      <height>753</height>
      <text/>
      <date>1517118535</date>
      <access_key>03d7d25e987b043c3a</access_key>
     </photo>
<photo>
      <id>456246621</id>
      <album_id>-7</album_id>
      <owner_id>166763402</owner_id>
      <photo_75>https://pp.userapi.com/c824602/v824602150/9312c/tZmiMfWp1ic.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c824602/v824602150/9312d/1Oc6paIllOI.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c824602/v824602150/9312e/EDr7gjGhb2I.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c824602/v824602150/9312f/3sBhUOUq2rk.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c824602/v824602150/93130/gfbgAMk4cxo.jpg</photo_1280>
      <width>663</width>
      <height>1000</height>
      <text/>
      <date>1517118535</date>
      <access_key>2f11f25bff1636a30d</access_key>
     </photo>
<photo>
      <id>456239234</id>
      <album_id>251740502</album_id>
      <owner_id>-142324060</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c840333/v840333558/5638a/O1AjtlmLHcg.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c840333/v840333558/5638b/77a3Nmu7-xE.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c840333/v840333558/5638c/ouMTz8dClhs.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c840333/v840333558/5638d/xuQe_CAY1qw.jpg</photo_807>
      <width>800</width>
      <height>530</height>
      <text/>
      <date>1518806307</date>
      <access_key>16909aa87b3e2e239d</access_key>
     </photo>
<photo>
      <id>456239235</id>
      <album_id>251740502</album_id>
      <owner_id>-142324060</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c840333/v840333558/56392/1rgrQDAOrQg.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c840333/v840333558/56393/UkxxGSMfzD4.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c840333/v840333558/56394/iO58LaOE9RU.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c840333/v840333558/56395/8evmInZMh7Y.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c840333/v840333558/56396/Mqxi0QksSXA.jpg</photo_1280>
      <width>1200</width>
      <height>813</height>
      <text/>
      <date>1518806308</date>
      <access_key>18cef371433e6d1049</access_key>
     </photo>
<photo>
      <id>456239226</id>
      <album_id>251740502</album_id>
      <owner_id>-142324060</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c840333/v840333558/56347/XV4XYnW5hYw.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c840333/v840333558/56348/lBBqlYJZ1Ow.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c840333/v840333558/56349/L2cYBj4bb0k.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c840333/v840333558/5634a/0aA5n62n0fA.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c840333/v840333558/5634b/BGONWuRkJmE.jpg</photo_1280>
      <width>1080</width>
      <height>859</height>
      <text/>
      <date>1518806305</date>
      <access_key>635c2d2b2c1edced02</access_key>
     </photo>
<photo>
      <id>456239213</id>
      <album_id>251740502</album_id>
      <owner_id>-142324060</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c840333/v840333558/562d6/S0dODaB3U-0.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c840333/v840333558/562d7/N8FcogqZDq0.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c840333/v840333558/562d8/f0y07yShT-4.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c840333/v840333558/562d9/TEjMLwu7A8M.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c840333/v840333558/562da/ZvyfZ7WiLtY.jpg</photo_1280>
      <width>720</width>
      <height>1080</height>
      <text/>
      <date>1518806301</date>
      <access_key>ce4ed3158b20a42efe</access_key>
     </photo>
<photo>
      <id>456239210</id>
      <album_id>251740502</album_id>
      <owner_id>-142324060</owner_id>
      <user_id>100</user_id>
      <photo_75>https://pp.userapi.com/c840333/v840333558/562bd/iuV8xs4Ahes.jpg</photo_75>
      <photo_130>https://pp.userapi.com/c840333/v840333558/562be/WbLJ3GTwqfE.jpg</photo_130>
      <photo_604>https://pp.userapi.com/c840333/v840333558/562bf/wQymQheoECw.jpg</photo_604>
      <photo_807>https://pp.userapi.com/c840333/v840333558/562c0/tQjeCYXxT6Y.jpg</photo_807>
      <photo_1280>https://pp.userapi.com/c840333/v840333558/562c1/rLH-JYoWWJQ.jpg</photo_1280>
      <width>1183</width>
      <height>1024</height>
      <text/>
      <date>1518806301</date>
      <access_key>afb2c5fe11c164f73d</access_key>
     </photo>

я его пихаю в список, назовем его список 1 регуляркой:
Код:
var reg1 = new Regex(@"(?<=<photo>)[\w\W]*?(?=</photo>)");
получается каждая строка это фоточка со всеми параметрами
в том числе ссылки на саму фотку от самого маленького размера этой фотки до самого большого (мне она и нужна, самого большого размера, то есть самая последняя)

и вот я беру первую строку из списка 1 и пихаю все ссылки на фотку в список, назовем его список 2 регуляркой:
Код:
var reg2 = new Regex(@"https://pp\..*\.jpg");
дальше считаю количество строк в списке 2 вычитаю 1 потому что в списке строки с 0 начинаются и беру последнюю строку в списке 2, записываю ее в переменную:
Код:
photo = photo + "<img>" + list_photo1[b] + "</img>";
то есть в переменной photo получается <img>ссылка на фото</img><img>ссылка на фото</img><img>ссылка на фото</img> и т.д.

дальше, я очищаю список, что бы запихнуть в него следующие ссылки, беру 2 строку из списка 1 и пытаюсь снова запихнуть в него ссылки на следующее фото и тут затык в список ничего не кладется, хотя на первом круге цикла все ок. Даже если list_photo1.Clear(); поставить в начале цикла все равно 1 круг он отрабатывает, то есть в photo:<img>https://pp.userapi.com/c840323/v840323836/3eb19/44Okt6kyhDE.jpg</img>

но видимо, что не так в логике кода иначе он бы работал.... вот я как бы и спрашиваю, что не так.
 

Moadip

Client
Регистрация
26.09.2015
Сообщения
509
Благодарностей
824
Баллы
93
В итоге то цель какая? Вытащить все ссылки и сложить их куда то или что?
 

Raids

Client
Регистрация
12.09.2017
Сообщения
41
Благодарностей
19
Баллы
8

Moadip

Client
Регистрация
26.09.2015
Сообщения
509
Благодарностей
824
Баллы
93
C#:
var reg1 = new Regex(@"(?<=<photo>)[\w\W]*?(?=</photo>)");
var reg2 = new Regex(@"https[\w\W]*?jpg");

var temp = reg1.Matches(post).Cast<Match>().Select(m => m.Value).ToList();
var list = new List<string>();

foreach (var str in temp)
{
   var last = reg2.Matches(str).Cast<Match>().Select(m => m.Value).Last();
   list.Add(last);
}

var photo = list.Aggregate("", (current, str) => current + ("<img>" + str + "</img>"));
 
  • Спасибо
Реакции: Raids

Raids

Client
Регистрация
12.09.2017
Сообщения
41
Благодарностей
19
Баллы
8
C#:
var reg1 = new Regex(@"(?<=<photo>)[\w\W]*?(?=</photo>)");
var reg2 = new Regex(@"https[\w\W]*?jpg");

var temp = reg1.Matches(post).Cast<Match>().Select(m => m.Value).ToList();
var list = new List<string>();

foreach (var str in temp)
{
   var last = reg2.Matches(str).Cast<Match>().Select(m => m.Value).Last();
   list.Add(last);
}

var photo = list.Aggregate("", (current, str) => current + ("<img>" + str + "</img>"));
Вот это то что надо было огроменное спасибо
Код:
var photo = list.Aggregate("", (current, str) => current + ("<img>" + str + "</img>"));
 

Кто просматривает тему: (Всего: 1, Пользователи: 0, Гости: 1)