Не удаляет элементы списка, в чем ошибка?

Question

[Vadim kyklaed]

Всем привет!
есть проблема, есть список, делаю словарь ключом является символ из списка а его значением то сколько раз он содержится в списке.
мне нужно удалить из списка уникальные элементы то есть те которые встречаются один раз.
и нужно учитывать что "X" и 'x' это один и тот же символ.

Так вот проблема в том что из примера ниже не удаляются три числа 1, 8, 2 он их как будто пропускает и я не могу почему и где. Помогите понять в чем ошибка. Есть предположение что проблема возможна с кавычками и он просто пропускает их то есть если обращаться вот так [1] = error, а если так ['1'] = 1, но в тоже время [0] = 2 ноль он читает нормально хотя ключ в словаре тоже в кавычках.
Да код у меня ужасный. но какой есть, как умею.

check = ['X', 'H', 'e', 'V', 'm', 'l', 's', 1, 0, 'y', 'j', 'b', 'g', 'o',
        'R', 'U', 'O', 'p', 'p', 8, 'Y', 'B', 'Y', 'O', 'r', 'E', 't', 'I',
        'w', 'i', 'v', 'o', 2, 'd', 'Z', 'b', 'S', 'T', 'n', 0]
dct={}
for i in check:
    a="{0}".format(i)
    if a.islower() and not a.isdigit():
        if a.upper() in dct:
            dct[a.upper()]+=1
        else:
            dct[a.upper()]=1         
    else:
        if a in dct:
            dct[a]+=1
        else:
            dct[a]=1
        
for i in dct:
    if i in check:        
        if dct[i] == 1:
            check.remove(i)
    else:
        a="{0}".format(i.lower())
        if a in check:
            if dct[a.upper()] == 1 :
                check.remove(a)

                  
#print(dct['1'])
#print(dct['2'])
#print(dct['8'])

print(check)
print(dct)

Comments

[Дмитрий Зайцев]

for i in dct:
if i in check: если "1" in check, но у тебя там не "1", а просто 1

else:
a="{0}".format(i.lower()) и здесь ты повторяешь то же самое
if a in check:

Answer 1

[aRegius]

Привет!

>>> check = ['X', 'H', 'e', 'V', 'm', 'l', 's', 1, 0, 'y', 'j', 'b', 'g', 'o',
        'R', 'U', 'O', 'p', 'p', 8, 'Y', 'B', 'Y', 'O', 'r', 'E', 't', 'I',
        'w', 'i', 'v', 'o', 2, 'd', 'Z', 'b', 'S', 'T', 'n', 0]

# приводим к единому регистру
>>> check_low = [i.lower() if isinstance(i, str) else i for i in check]
>>> from collections import Counter

# считаем число вхождений элементов и результат приводим для удобства к списку кортежей
>>> check_pair = Counter(check_low).most_common()

# убираем уникальные
>>> check_pair = [(x, y) for (x, y) in check_pair if y != 1]

# создаем словарь
>>> check_dict = dict(check_pair)
>>> check_dict
{0: 2, 'y': 3, 'b': 3, 'v': 2, 't': 2, 'o': 4, 'p': 2, 'r': 2, 'e': 2, 'i': 2, 's': 2}

Answer 2

[Евгений]

• a="{0}".format(i) Зачем?
  
• if a.islower() and not a.isdigit(): Зачем?
  
• Если хотите показать свой код кому-нибудь используйте pep8
  

Answer 3

[Dimonchik]

без коллекций: ))))

check=[x.lower() if isinstance(x, str) else x for x in check] 
slovar=dict((i, check.count(i)) for i in check if check.count(i) > 1)

Comments

[aRegius]

Почему бы и нет, тоже отличный вариант :)

[Dimonchik]

тогда ТС вообще не поймет ))

Answer 4

[Вячеслав]

Классика.
i = 1
['X', 'H', 'e'] => remove[i] => ['X', 'e'], => i++ => remove[i]???
Удалили элемент и перешли вперед на один шаг, не учитывая что следующий элемент стал как бы текущим.
Итерироваться нужно по копии списка.

Comments

[Vadim kyklaed]

Но я ведь иду про ключам словаря и проверяю есть ли такое элемент в списке, если элемент есть то я проверю значение по ключу в словаре, если оно равно 1. то удаляю его из списка. и это делает в цикле for я прохожу по всему словарю и каждая итерация берет по 1 значению. как только значения кончаются цикл останавливается и я получаю свой print. Я наверно устал и не понимаю

[Vadim kyklaed]

Я сделал проверку по оригинальному списку, а удаление элемента их копии, ничего не вышло все тоже самое.