0

Мне нужно вернуть Ключи словаря, с одинаковыми values, средствами linq.

Я понимаю принцип работы написанного запроса так: Из списка ключей -> взять список значений таких ключей из словаря -> добавить в коллекцию те ключи, значения которых удовлетворяют условию. В чем ошибка?

    private Dictionary<string, string> clientsBase = new()
    {
        { "al1992@gmail.com", "Алексей И.Г."},
        { "crusher777@gmail.com", "Петрова М.Р"},
        { "yayaN25@gmail.com", "Иванов А.А" },
        { "elena.k.@gmail.com", "Иванов А.А" }

    };

    public string GetClientEmail(string name)
    {
        var fltEmails = clientsBase.Keys.Select(v => clientsBase[v])
                                        .Where(n => n.Equals(name));

        StringBuilder builder = new StringBuilder();
        foreach (var fltEmail in fltEmails) builder.Append(fltEmail);

        return builder.ToString();
    }
3
  • clientsBase.GroupBy(x => x.Value);
    – Aarnihauta
    5 мая 2022 в 12:02
  • Это ведь просто вернет элементы с уникальными values, нет?
    – Mark Kaz
    5 мая 2022 в 12:09
  • 1
    Можно вообще использовать структуру даннх типа Dictionary<string, string[]>, где ключом будет имя, а значением будет массив емейлов, тогда задача сведется к однострочнику безо всяких Linq запросов. Но вообще решение странное, ведь может быть несколько разных людей с одинаковыми фамилиями и инициалами.
    – aepot
    5 мая 2022 в 12:44

2 ответа 2

3

Можно например так, но конечно способ @alexsteeel выглядит интуитивно понятнее

public string GetEmailsByName(Dictionary<string, string> items, string name)
{
    var emails = items.GroupBy(x => x.Value == name)
                      .First(x => x.Key)
                      .Select(x => x.Key);

    var sb = new StringBuilder();

    foreach (var item in emails)
        sb.Append(item);

    return sb.ToString();
}
2
  • Здесь есть один минус по сравнению с принятым ответом, если имени нет в выборке, будет исключение.
    – aepot
    5 мая 2022 в 13:14
  • 1
    @aepot да, знаю, я так специально написал чтоб свои руки не сували куда попало
    – Aarnihauta
    5 мая 2022 в 13:15
1

Например, так:

public string GetClientEmail(string name)
{
    var fltEmails = clientsBase
        .Where(v => v.Value == name)
        .Select(v => v.Key);

    StringBuilder builder = new StringBuilder();
    foreach (var fltEmail in fltEmails) builder.Append(fltEmail);

    return builder.ToString();
}

В варианте из вопроса проблема в том, что сначала выбираются ключи, потом по ключам выбираются имена и уже эти имена фильтруются. А доступа к ключам на этом этапе уже нет.

12
  • return string.Join(",", fltEmails);
    – aepot
    5 мая 2022 в 12:40
  • 1
    .ToList() здесь лишний, без него оптимальнее получится.
    – aepot
    5 мая 2022 в 12:46
  • 1
    @Aarnihauta да даже если дело не дойдет до БД, надо сразу учиться не писать лишнее.
    – aepot
    5 мая 2022 в 12:57
  • 1
    Использование ToList иногда сильно помогает при отладке. Без него дебаггер может скакать по выражениям чёрт знает как. 5 мая 2022 в 13:24
  • 1
    @alexsteeel да без разницы, какой объем. Бесполезное создание прокси-объектов - это лишний мусор в памяти. Чем больше мусора, тем больше у GC работы, а чем больше у GC работы, тем жирнее он будет выставлять паузы в выполении вашего кода для уборки. Это тенденция общей произовдительности приложения. Здесь вы создали сотню лишних объектов, в другом месте у вас начало "икать" медиа в приложении. Казалось бы, совершенно несвязанные друг с другом вещи. Культура написания кода с контролем аллокаций объектов пока еще никому не навредила. Всё хорошо в меру, само собой.
    – aepot
    5 мая 2022 в 13:24

Ваш ответ

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

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