Как получить значение с foreach и передать его методу?

Question

[Ness]

Есть такой метод:

public string GetPhotoUrl()
        {
            var posts = Api.Wall.Get(new WallGetParams { Domain = "some_domain", Count = 2 });
            foreach (var item in posts.WallPosts)
            {
                string phUrl = (item.Attachments[0].Instance as Photo).Sizes[0].Url.AbsoluteUri.ToString();
                MessageBox.Show(phUrl);
            }
            
            return "";
        }

Как мне с foreach получить значение и передать методу?

Comments

[freeExec]

Вы же получили phUrl и передали его методу MessageBox.Show, что вам ещё нужно?

[Ness]

freeExec, нужно вернуть методу GetPhotoUrl, а месседж бокс то я чекал, правильно ли оно передаёт

[freeExec]

Taifunov, Так а что не получается-то, я так и не пойму?

[Дмитрий Макаров]

у вас, в общем случае, получатся несколько значений
какое именно вам нужно?

Если нужно каждое значение, то один ответ,
если одно из них, то другой
а если нужно из всех этих значений сгенерировать что-то обобщающее, то - третий.

[Ness]

Решил вопрос так:

public List<string> GetPhotoUrl()
        {
            var phUrlList = new List<string>();
            var posts = Api.Wall.Get(new WallGetParams { Domain = "lol.community", Count = 2 });
            foreach (var item in posts.WallPosts)
            {
                string phUrl = (item.Attachments[0].Instance as Photo).Sizes[0].Url.AbsoluteUri.ToString();
                phUrlList.Add(phUrl);
            }

            return phUrlList;
        }

[Foggy Finder]

Taifunov, лучше оформите ответом.

Можно немного сократить код примерно следующим образом:

public List<string> GetPhotoUrl() => 
            Api.Wall
               .Get(new WallGetParams { Domain = "lol.community", Count = 2 })
               .WallPosts
               .Select(item => (item.Attachments[0].Instance as Photo).Sizes[0].Url.AbsoluteUri.ToString())
               .ToList();

[#]

Foggy Finder, лучше вы оформите ответом ))

[Foggy Finder]

#, могу и я, подождем что скажет автор вопроса

[Дмитрий Макаров]

Foggy Finder, я так понимаю, List тут непринципиален, а был добавлен чтоб в кучку строки складывать. Если всё же нужен List, то стоит оставить это решение за клиентом, использующим этот метод.

public IEnumerable<string> GetPhotoUrl() =>
            Api
                .Wall
                .Get(new WallGetParams
                {
                    Domain = "lol.community",
                    Count = 2
                })
                .WallPosts
                .Select(item => item.Attachments.FirstOfDefault()?.Instance)
                .OfType<Photo>()
                .Select(item => item.Sizes.FirstOfDefault()?.Url.AbsoluteUrl.ToString());

[Foggy Finder]

Дмитрий Макаров, согласен

[#]

Дмитрий Макаров, Foggy Finder, давайте уже кто то дайте ответ а не камент ))

Taifunov, вы тоже можете свое решение дать ответом. это нормально (мало того, тут многие любят так делать)

джентльмены предлагают лаконичный код в современном стиле (функциональном, LINQ). понятно что новичку он может быть далеко не сразу понятен. так что собственная победа проблемы автором, достойна представления публике

[#]

Foggy Finder, Taifunov, присоединяйтесь господа (я отметил один ответ, хотел бы отметить и ваши)

что бы вполне себе качественно закрыть тему

[Foggy Finder]

#, одного ответа вполне достаточно =)

[#]

Foggy Finder, скромность украшает, но ваша версия была раньше, опубликованная есть лишь легкая оптимизация ))

ps между прочим спорная. если не известны дальнейшие способы использования. императивное (явно указанное) преобразование IEnumerable к List, будет эффективнее в производительности (другой вопрос, объемы, затраты на извлечение из первоисточника, естественно, на 99% тут важнее читаемость и надежность кода )))

[Foggy Finder]

#, сдаюсь, убедили

[Ness]

#, извините, подзабыл заглянуть, но решения действительно лучше чем моё :D
Спасибо, всем, кто ответил!

Answer 1

[Дмитрий Макаров]

public IEnumerable<string> GetPhotoUrl() =>
            Api
                .Wall
                .Get(new WallGetParams
                {
                    Domain = "lol.community",
                    Count = 2
                })
                .WallPosts
                .Select(item => item.Attachments.FirstOfDefault()?.Instance)
                .OfType<Photo>()
                .Select(item => item.Sizes.FirstOfDefault()?.Url.AbsoluteUrl.ToString());

Answer 2

[Foggy Finder]

public List<string> GetPhotoUrl() => 
            Api.Wall
               .Get(new WallGetParams { Domain = "lol.community", Count = 2 })
               .WallPosts
               .Select(item => (item.Attachments[0].Instance as Photo).Sizes[0].Url.AbsoluteUri.ToString())
               .ToList();