Удаление дубликатов в Python

Question

Удаление дубликатов в Python

1

Моя программа до сих пор

def RemoveDuplicates(text):
    result=[]
    used=set()
    for char in text:
        if char not in used:
            used.add(char)
            result.append(char)
    return ''.join(result)

text=input('Enter some text: ')
print(join(result))

Я новичок в Python, поэтому для меня это довольно сложная задача. Я знаю, что это не складывается, но где я допустил ошибку?

user1248367 08 март 2012, в 23:03

Источник

1

Не похоже, что вы когда-либо вызывали RemoveDuplicates в вашем примере кода.
Moishe Lettvin 08 март 2012, в 22:03
0

Ну, у Питона нет никакой магии. Вы не можете писать случайные вещи и надеяться, что это сработает.
Oleh Prypin 08 март 2012, в 22:04

Теги:

python

5 ответов

Ещё вопросы

Не похоже, что вы когда-либо вызывали RemoveDuplicates в вашем примере кода.
Ну, у Питона нет никакой магии. Вы не можете писать случайные вещи и надеяться, что это сработает.

kindall · Answer 1 · 2012-03-08T20-31-00.000Z

1

Вы никогда не называли вашу RemoveDuplicates().

Заменить:

text=input('Enter some text: ')
print(join(result))

с:

print(RemoveDuplicates(input("Enter some text:" )))

kindall 08 март 2012, в 20:31

0

Это сработало. Большое спасибо. Есть ли способы сократить его?
user1248367 08 март 2012, в 22:07
0

@ user1248367: Он может быть записан даже в одну строку, но с худшей производительностью. Функция кажется хорошей.
Oleh Prypin 08 март 2012, в 22:09
0

О, есть много способов написать это короче - в одну или две строки, на самом деле. Но это домашнее задание и то, что у тебя есть, так что я бы его сдал. :-)
kindall 08 март 2012, в 22:11

Показать ещё 1 комментарий

Thomas · Answer 2 · 2012-03-08T20-02-00.000Z

В дополнение к другим ответам: если вы используете Python 2, вам нужно использовать raw_input вместо input.

jwd · Answer 3 · 2012-03-08T20-25-00.000Z

0

Несколько вещей здесь:

Вы определили функцию RemoveDuplicates, но на самом деле не называете ее.
Вы используете set(), но затем проверяете дубликаты вручную. Набор, по определению, автоматически удалит дубликаты.
Вы не сказали нам, что вы на самом деле пытаетесь выполнить. Сделай это, тогда мы сможем помочь тебе больше.

jwd 08 март 2012, в 20:25

0

Хотя, поскольку он пытается удалить повторяющиеся символы из строки, set () не будет поддерживать порядок сортировки
jdi 08 март 2012, в 22:04
0

Сожалею. Я хочу, чтобы программа попросила пользователя ввести текст. На выходе должен быть текст с удаленными дубликатами. Введенный текст: "Stackoverflow", вывод: Stackoverflw
user1248367 08 март 2012, в 22:04
1

Я уверен, что set используется вручную, потому что необходимо сохранить порядок исходного текста.
kindall 08 март 2012, в 22:05
0

Я думаю, что if char not in used: test по-прежнему избыточен, и это именно то, что @jwd пытался сделать.
Marius 08 март 2012, в 22:20
0

@Marius: Ну, тест не требуется для того , чтобы сделать used.add(char) , но по - прежнему необходим для result.append(char) . Сначала я недоразумение.
jwd 08 март 2012, в 22:31

Показать ещё 3 комментария

Elliot Bonneville · Answer 4 · 2012-03-08T20-24-00.000Z

Ваша функция выглядит хорошо для меня, хотя я не эксперт. Я думаю, что проблема заключается в том, что вы на самом деле ничего не называете. Попробуйте вызвать его так: print(RemoveDuplicates(text)).

fraxel · Answer 5 · 2012-03-08T20-06-00.000Z

Одна небольшая проблема заключается в том, что вы забыли назвать свою функцию! Кроме того, если вы используете Python 2.x, пользовательский ввод должен быть собран с использованием raw_input.

Две небольшие исправления и ваш код отлично работают:

def RemoveDuplicates(text):
    result=[]
    used=set()
    for char in text:
        if char not in used:
            used.add(char)
            result.append(char)
    return ''.join(result)

text = raw_input('Enter some text: ')
print(RemoveDuplicates(text))

raw_input - это python 2. Он был переименован для input в python 3.x.
Да, возможно, опрометчивое предположение ...