0

Возможно ли в foreach запихнуть два условия, что бы избежать foreach в foreach? Пример:

foreach(var t in ViewBag.News){
       foreach(var n in ViewBag.Like){
    }
}

это превратить в это:

 foreach(var t in ViewBag.News && var n in ViewBag.Like){}

Весь код страницы:

@using WebPhotoArchive.Models
@model IQueryable<Post>
@{
    ViewBag.Title = "Home Page";
    string Name = null;
    int id_link = 0;
}

@if (ViewBag.News == null)
{
    <p><h3><b>Подпишитесь на других пользователей, чтобы получать их публикации!</b></h3></p>
}
else
{
    foreach (var n in ViewBag.News)
    {
    <div class="row">
        <br />
        <div class="col-md-3"></div>
        <div class="col-md-6">
            <div class="thumbnail">
                @{ Name = n.UserDoName; id_link = n.UserDoId;}
                <input type="hidden" name="id_post" value="@n.Id" />
                <div class="collapse navbar-collapse">
                    <h4>@Html.ActionLink(Name, "/UserProfile/" + id_link, "Home", new { @class = "nav navbar-nav", @style = "text-decoration: none; color: #000000;" })</h4>
                </div>
                <hr />
                @Html.Raw("<img src=\"data:image/jpeg;base64," + Convert.ToBase64String(n.Photo) + "\" />") @*для уменьшения добавить в img style='width:300px; height:350px;'*@
                <p><strong>@Name:</strong><i>@n.Description</i></p>
                <div class="btn-group" id="Div1">
                    <p>
                        <a class="btn-group" href="@Url.RouteUrl(new {controller = "Home", action = "LikeSystem", id = n.Id })">
                            @foreach (var t in ViewBag.LikePost)
                            {
                                if (n.Id == t.LikingId)
                                {
                                    <img style='width:30px; height:30px;' src="~/Content/like_2.png" />
                                }
                                else
                                {
                                    <img style='width:30px; height:30px;' src="~/Content/like_1.png" />
                                }
                            }
                        </a>
                        <a class="btn-group" href="@Url.RouteUrl(new {controller = "Home", action = "Comment", id_post = n.Id })">
                            <img style='width:30px; height:30px;' src="~/Content/comments.png" />
                        </a>
                    </p>
                </div>
            </div>
        </div>
        <br />
        <div class="col-md-3"></div>
    </div>
    }
}

Контроллер:

public ActionResult Index()//новости
    {
        #region My Id, variable 'i' for get it
        int i = 0;
        var id = from c in db.Users where c.Login == User.Identity.Name select c.Id;
        foreach (var t in id)
        {
            i = t;
        }
        #endregion
        using (WebArchiveContext dbc = new WebArchiveContext())
        {
            var list_like = dbc.Likes.Where(l => l.LikerId == i);
            ViewBag.LikePost = list_like;
        }
        var news_following_id = db.Follows.Where(u => u.FollowerId == i).Select(u => u.FollowingId);
        IQueryable<Post> list_news = null;
        list_news = db.Posts.Where(u => news_following_id.Contains(u.UserDoId)).OrderByDescending(p => p.Time);
        if (list_news.Any())
        {
            ViewBag.News = list_news;
        }
        else ViewBag.News = null;
        return View();
    }
16
  • что такое ViewBag.News и что такое ViewBag.Like ?
    – tym32167
    21 июн 2020 в 20:26
  • @tym32167 Это гипотетически придуманные переменные, Новости и относящиеся к ним лайки. 21 июн 2020 в 20:26
  • тогда какого типа будет переменная t здесь? foreach(var t in ViewBag.News && var t in ViewBag.Like){}
    – tym32167
    21 июн 2020 в 20:28
  • Ошибочка, разные переменные должны быть foreach(var t in ViewBag.News && var n in ViewBag.Like){} 21 июн 2020 в 20:30
  • тогда нет, такого синтаксиса нет в C#. Один foreach на одну переменную. А какую вы задачу этим хотите решить?
    – tym32167
    21 июн 2020 в 20:31

2 ответа 2

1

Смотрим на ваш цикл

@foreach (var t in ViewBag.LikePost)
{
    if (n.Id == t.LikingId)
    {
        <img style='width:30px; height:30px;' src="~/Content/like_2.png" />
    }
    else
    {
        <img style='width:30px; height:30px;' src="~/Content/like_1.png" />
    }
}

и что видим? Для каждого совпадающего поста вы ставите лайк, а для остальных картинку с отсутсвием лайка. Это неверно, так как для лайков, которые не относятся к посту, вы тоже что то нарисуете.

Как испроавить - заменить вложенный цикл, который вам не нужен, на условие:

@if (((IEnumerable<Like>)ViewBag.LikePost).Any(l=>l.LikingId == n.Id))
{
    <img style='width:30px; height:30px;' src="~/Content/like_2.png" />
}
else
{
    <img style='width:30px; height:30px;' src="~/Content/like_1.png" />
}

Dummy вариант

@{
    bool likeFound = false;

    foreach (var t in ViewBag.LikePost)
    {
        if (n.Id == t.LikingId)
        {
            likeFound = true;
            
            <img style='width:30px; height:30px;' src="~/Content/like_2.png" />

            break;
        }        
    }    

    if (!likeFound)
    {       
        <img style='width:30px; height:30px;' src="~/Content/like_1.png" />        
    }
}
12
  • Появилась ошибка: imgur.com/a/Uo3NyNG 21 июн 2020 в 21:11
  • Сделал каст в (IEnumerable<Like>)ViewBag.LikePost), где Like - ваш класс лайков. Сам не пишу на дотнете уже, потому такие ошибки могу допускать :)
    – tym32167
    21 июн 2020 в 21:16
  • Ошибка CS0012 Тип "ValueType" определен в сборке, на которую нет ссылки. Следует добавить ссылку на сборку "netstandard, Version=2.0.0.0, 21 июн 2020 в 21:18
  • так у вас asp.net core? Вы можете поменять @if (((IEnumerable<Like>)ViewBag.LikePost).Any(l=>l.LikingId == n.Id)) на @if (((IEnumerable<dynamic>)ViewBag.LikePost).Any(l=>l.LikingId == n.Id)) или добавить netstandard если первое не сработает
    – tym32167
    21 июн 2020 в 21:27
  • Нет, ASP NET MVC 5 На всякий попробовал это способ - не работает. При добавлении сборки так же ошибки 21 июн 2020 в 21:31
0

Давайте не будем возиться со страницами ASP.NET, а используем консольное приложение. Ведь главное - сделать одновременную итерацию по двум коллекциям сразу. Я правильно понял?

Наши гипотетические коллекции:

var ViewBagNews = new List<string> { "a", "b", "c", "d" };
var ViewBagLike = new List<int> { 1, 2, 3, 4 };

Стандартный способ прохода сразу по двум коллекциям, когда их размеры равны. Используем цикл for:

for (int i = 0; i < ViewBagNews.Count; i++)
{
    Console.WriteLine(ViewBagNews[i] + " " + ViewBagLike[i]);
}

Допустим, коллекции не реализуют интерфейс IList, поэтому обращение по индексу невозможно.

Используем другой стандартный способ с применением энумераторов:

using (var newsEnumerator = ViewBagNews.GetEnumerator())
using (var likeEnumerator = ViewBagLike.GetEnumerator())
{
    while (newsEnumerator.MoveNext() && likeEnumerator.MoveNext())
    {
        Console.WriteLine(newsEnumerator.Current + " " + likeEnumerator.Current);
    }
}
1
  • @DomenicRoss - Судя по комментариям и другому ответу, требовалось совсем не то, что указано в заголовке: "Два условия в Foreach". Если это так, напишите, удалю свой ответ. 21 июн 2020 в 21:53

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.