Ошибка словаря: невозможно преобразовать элемент последовательности обновления словаря № 0 в последовательность

Я пытаюсь подсчитать частоту из списка. Я получил ошибку типа

mylist = [1, 1, 1, 2, 3, 3, 4]
def frequency(x):
    counts = dict()
    if x in counts:
        counts[x] += 1
    elif len(counts) < 3:
        counts.update([x, 1])            
    return counts
for x in mylist:
    frequency(x)

Я получил TypeError: cannot convert dictionary update sequence element #0 to a sequence на counts.update([x, 1]). Я проверил, о чем эта ошибка, но не вижу, что нарушил ее.

Почему бы просто не использовать Counter?

Nick 10.04.2022 09:29

Между прочим, ваш код неверен, потому что он создает новый count каждый раз, когда вызывается frequency, который затем отбрасывается, когда вы return его, но не присваиваете ему значение в цикле. В принципе, как есть, ваш код ничего не делает (кроме выдачи ошибки).

BlackBeans 10.04.2022 09:35
Анализ настроения постов в Twitter с помощью Python, Tweepy и Flair
Анализ настроения постов в Twitter с помощью Python, Tweepy и Flair
Анализ настроения текстовых сообщений может быть настолько сложным или простым, насколько вы его сделаете. Как и в любом ML-проекте, вы можете выбрать...
7 лайфхаков для начинающих Python-программистов
7 лайфхаков для начинающих Python-программистов
В этой статье мы расскажем о хитростях и советах по Python, которые должны быть известны разработчику Python.
Установка Apache Cassandra на Mac OS
Установка Apache Cassandra на Mac OS
Это краткое руководство по установке Apache Cassandra.
Сертификатная программа "Кванты Python": Бэктестер ансамблевых методов на основе ООП
Сертификатная программа "Кванты Python": Бэктестер ансамблевых методов на основе ООП
В одном из недавних постов я рассказал о том, как я использую навыки количественных исследований, которые я совершенствую в рамках программы TPQ...
Создание персонального файлового хранилища
Создание персонального файлового хранилища
Вы когда-нибудь хотели поделиться с кем-то файлом, но он содержал конфиденциальную информацию? Многие думают, что электронная почта безопасна, но это...
Создание приборной панели для анализа данных на GCP - часть I
Создание приборной панели для анализа данных на GCP - часть I
Недавно я столкнулся с интересной бизнес-задачей - визуализацией сбоев в цепочке поставок лекарств, которую могут просматривать врачи и...
0
2
23
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Исправить

counts должен быть создан за пределами, поэтому раньше, если вы создаете его в методе, он каждый раз будет новым.

Вы можете обновить диктовку с помощью другой диктовки, а не списка

counts.update({x: 1})

Улучшить логику

  • просто сделайте counts[x] = 1 вместо этого, это дешевле
  • было бы лучше, если бы метод выполнял полное вычисление частоты из списка
counts = {}
def frequency(values):
    for x in values:
        if x in counts:
            counts[x] += 1
        else:
            counts[x] = 1
    return counts

mylist = [1, 1, 1, 2, 3, 3, 4]
counts = frequency(mylist)
print(counts)  # {1: 3, 2: 1, 3: 2, 4: 1}

Улучшить код

Существует метод, который может помочь вам получить более красивый код

  • collections.defaultdict который обрабатывает, если ключ отсутствует (установите 0, как мы ему сказали int)

    def frequency(values):
        counts = defaultdict(int)
        for x in values:
            counts[x] += 1
        return counts
    
  • мастер всего collections.Counter

    def frequency(values):
        return Counter(values)
    

Другие вопросы по теме