0

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

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

    private Dictionary<string, string> clientsBase = new()
    {
        { "[email protected]", "Алексей И.Г."},
        { "[email protected]", "Петрова М.Р"},
        { "[email protected]", "Иванов А.А" },
        { "[email protected]", "Иванов А.А" }

    };

    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
    Commented 5 мая 2022 в 12:02
  • Это ведь просто вернет элементы с уникальными values, нет?
    – Mark Kaz
    Commented 5 мая 2022 в 12:09
  • 1
    Можно вообще использовать структуру даннх типа Dictionary<string, string[]>, где ключом будет имя, а значением будет массив емейлов, тогда задача сведется к однострочнику безо всяких Linq запросов. Но вообще решение странное, ведь может быть несколько разных людей с одинаковыми фамилиями и инициалами.
    – aepot
    Commented 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
    Commented 5 мая 2022 в 13:14
  • 1
    @aepot да, знаю, я так специально написал чтоб свои руки не сували куда попало
    – Aarnihauta
    Commented 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
    Commented 5 мая 2022 в 12:40
  • 1
    .ToList() здесь лишний, без него оптимальнее получится.
    – aepot
    Commented 5 мая 2022 в 12:46
  • 1
    @Aarnihauta да даже если дело не дойдет до БД, надо сразу учиться не писать лишнее.
    – aepot
    Commented 5 мая 2022 в 12:57
  • 1
    Использование ToList иногда сильно помогает при отладке. Без него дебаггер может скакать по выражениям чёрт знает как. Commented 5 мая 2022 в 13:24
  • 1
    @alexsteeel да без разницы, какой объем. Бесполезное создание прокси-объектов - это лишний мусор в памяти. Чем больше мусора, тем больше у GC работы, а чем больше у GC работы, тем жирнее он будет выставлять паузы в выполении вашего кода для уборки. Это тенденция общей произовдительности приложения. Здесь вы создали сотню лишних объектов, в другом месте у вас начало "икать" медиа в приложении. Казалось бы, совершенно несвязанные друг с другом вещи. Культура написания кода с контролем аллокаций объектов пока еще никому не навредила. Всё хорошо в меру, само собой.
    – aepot
    Commented 5 мая 2022 в 13:24

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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