У меня есть:
txt = input('What is your sentence? ')
list = [0]*128
for x in txt:
list[ord(x)] += 1
for x in list:
if x >= 1:
print(chr(list.index(x)) * x)
Насколько я понимаю, это должно просто выводить каждую букву в предложении, например:
))
111
3333
etc.
Для строки aB)a2a2a2) вывод правильный:
))
222
B
aaaa
Для строки aB)a2a2a2 вывод неверен:
)
222
)
aaaa
Я чувствую, что все мои базы покрыты, но я не уверен, что не так с этим кодом.
list в качестве имени переменной, так как это имя встроенной функции. В любом случае, list.index(x) на самом деле не имеет смысла. В контексте x — это целое число, и это целое число может появляться в нескольких местах в списке. Во многих случаях index(x) не вернет индекс, который вам действительно нужен. Вместо этого используйте enumerate. 02.04.2021 list, это то же имя, что и у встроенного типа, и это может привести к странным беспорядкам. 02.04.2021 Когда вы делаете list.index(x), вы ищете в списке первый индекс, который появляется значение. На самом деле это не то, что вам нужно, вам нужен конкретный индекс значения, которое вы только что прочитали, даже если это же значение встречается где-то еще раньше в списке.
Лучший способ получить индексы по побочным значениям из последовательности — использовать enuemerate:
for i, x in enumerate(list):
if x >= 1:
print(chr(i) * x)
Это должно дать вам желаемый результат, но есть несколько других вещей, которые облегчат чтение и понимание вашего кода. Прежде всего, использование list в качестве имени переменной — очень плохая идея, так как это затенит имя встроенного типа list в вашем пространстве имен. Это очень сбивает с толку любого, кто читает ваш код, и вы даже сбиваетесь с толку, если хотите использовать обычный list для какой-то цели и не помните, что уже использовали его для своей собственной переменной.
Другая проблема также связана с именами переменных, но она немного более тонкая. Оба ваших цикла используют переменную цикла с именем x, но значение значения каждый раз разное. Первый цикл обрабатывает символы во входной строке, а второй — подсчет каждого символа. Использование значимых переменных сделало бы ситуацию намного яснее.
Вот комбинация всех предложенных мной исправлений вместе:
text = input('What is your sentence? ')
counts = [0]*128
for character in text:
counts[ord(character)] += 1
for index, count in enumerate(counts):
if count >= 1:
print(chr(index) * count)
chвместо полного написанияcharacter, потому что его легче набирать и читать, но оно достаточно символично, чтобы понять суть. 02.04.2021xиyв качестве координат илиiиjдля индексов). 02.04.2021