Как сделать корректную проверку на то, что символ не принадлежит строке?

Question

[atsin4469]

Есть вот такой код:

alphabet = 'abcdefghijklmnopqrstuvwxyz'

def unpack(arg):
    d = {}
    for index, item in enumerate(arg):
        d[item] = d.get(item, index) + 1
    return d

alphabet_unpacked = unpack(alphabet)

def alphabet_position(text):
    text_lowercase = text.lower()
    result = ''
    i = 0
    while i < len(text_lowercase):
        if text_lowercase[i] not in alphabet:
            continue
        result += str(alphabet_unpacked.get(text_lowercase[i]))
        i += 1
    return result

Он не работает из-за строк:

if text_lowercase[i] not in alphabet:
    continue

Как их переписать, чтобы можно было проверить, что символа нет в alphabet?

Comments

[o5a]

Уже написали, ошибку можно поправить, добавив i += 1 перед continue
Или использовать второй параметр у get, выкинув вообще кусок с continue

while i < len(text_lowercase):
        result += str(alphabet_unpacked.get(text_lowercase[i],'')) # если нет в словаре, просто передаем пустую строку
        i += 1

И в целом конечно много чего можно подсократить.

Но зачем вообще такая конвертация (символы в склеенные числовые индексы), она ведь даже необратимая? Какую задачу пытались решить?

Answer 1

[dmshar]

Сама проверка тут ни при чем - она написана правильно.

Но вот это:

if text_lowercase[i] not in alphabet:
            continue

- классический бесконечный цикл.
Вы взяли первый элемент строки, проверили, он в "alphabet" не присутствует, вы переходите к следующей итерации НЕ ИЗМЕНИВ значения i. По сути - опять проверяете тот-же символ. И так до бесконечности.

Просто перенесите i += 1 перед if .

Правда, у вас еще и предел цикла в while задан с ошибкой, и вообще - программа странноватая. Но это уже другой вопрос.

Comments

[atsin4469]

Отличный ответ, но насчет while не понял
Вот задание:

In this kata you are required to, given a string, replace every letter with its position in the alphabet.
If anything in the text isn't a letter, ignore it and don't return it.

"a" = 1, "b" = 2, etc.

Example
alphabet_position("The sunset sets at twelve o' clock.")
Should return "20 8 5 19 21 14 19 5 20 19 5 20 19 1 20 20 23 5 12 22 5 15 3 12 15 3 11" (as a string)

Вот итоговый код, который проходит все тесты:

alphabet = 'abcdefghijklmnopqrstuvwxyz'

def unpack(arg):
    d = {}
    for index, item in enumerate(arg):
        d[item] = d.get(item, index) + 1
    return d

alphabet_unpacked = unpack(alphabet)

def alphabet_position(text):
    text_lowercase = text.lower()
    result = ''
    i = 0
    while i < len(text_lowercase):
        if text_lowercase[i] not in alphabet:
            i += 1
            continue
        result += ' ' + str( + alphabet_unpacked.get(text_lowercase[i]))
        i += 1
    return result[1:]

Расскажите о своем мнении, как сделать код лучше?

[dmshar]

atsin4469, - "как сделать код лучше?" - Вот так:

alphabet = "abcdefghijklmnopqrstuvwxyz'. "
text="The sunset sets at twelve o'clock."
text_lowercase =text.lower().translate(str.maketrans("", "", '!"#$%&\'()*+,-./:;<=>?@[\\]^_`{|}~ '))
lst_of_numbers =  []
for chrt in text_lowercase:
    lst_of_numbers.append(str(alphabet.find(chrt)+1))
print (' '.join(lst_of_numbers))

Результат:

20 8 5 19 21 14 19 5 20 19 5 20 19 1 20 20 23 5 12 22 5 15 3 12 15 3 11

Лучше?

[atsin4469]

dmshar, выглядит проще, но некоторые моменты я еще не изучал, поэтому не все понятно. Вернусь к этому позже. Спасибо

Answer 2

[Drill]

atsin4469,

Расскажите о своем мнении, как сделать код лучше?

alphabet = 'abcdefghijklmnopqrstuvwxyz'
text = "The sunset sets at twelve o'clock."

def alphabet_position(text):
    text = ' '.join(char for char in text.lower() if char.isalpha())
    return text.translate({ord(c): str(i) for i,c in enumerate(alphabet,1)})

print(alphabet_position(text))
In [4]: 
20 8 5 19 21 14 19 5 20 19 5 20 19 1 20 20 23 5 12 22 5 15 3 12 15 3 11

Answer 3

[Ищю в поисковиках]

Зачем всё усложнять, когда задача решается довольно быстро:

alphabet = set('abcdefghijklmnopqrstuvwxyz')
def alphabet_position(text):
    lower_text = text.lower()
    result = ''
    for char in lower_text:
        if char not in alphabet:
            result += char
    return result

Зачем использование словарей, вместо множеств; использование while; непонятные конвертации.

Comments

[dmshar]

Попробуйте запустить вашу программу на контрольном примере -

text="The sunset sets at twelve o' clock."

и посмотрите, что у вас получиться в результате.

[Ищю в поисковиках]

dmshar, Ну, видимо утро и поездка на метро не сильно помогают в понимании чужого когда без знаний того, что он должен делать XD

Если сейчас теперь правильно понимаю, то задача не удалить символы алфавита из строки, а преобразовать буквы в порядковый номер их в алфавите.

Если это так, то Drill написал ответ выше.

> и посмотрите, что у вас получиться в результате.

Результат соответствует тому, что я ожидал на момент написания кода на коленке с телефона в метро.

Сейчас, когда результат более понятен, можно было так написать:

def alphabet_position(text):
    return ' '.join(str(ord(x)-ord('a')+1) for x in text if x >= 'a' and x <= 'z')