Почему не считаются символы в строке?

Question

[Emince]

Добрый день! Необходимо сделать подсчёт символов в строке примитивами, код:

def count_letters()

text = str(input('Введите текст: '))
number = str(input('Какую цифру ищем? '))
letter = str(input('Какую букву ищем? '))


def count_letters(text, number, letter):
    result = list(text)
    num_count = 0
    text_count = 0
    index = len(text)
    for i in range(0, index + 1, 1):
        if number == result:
            num_count += 1
        elif letter == result:
            text_count += 1
    print('Количество цифр', number, ':', num_count)
    print('Количество цифр', letter, ':', text_count)

count_letters(text, number, letter)

Не могу понять, где ошибка...

Comments

[Kirgus]

Кое-что забыл написать в своём ответе.
Обрати внимание на то, что я не принтю результат прямо в функции count_letters. Функция лишь возвращает вычисленный результат. Это называется чистая функция. Одно из важных понятий чистого кода.

Answer 1

[Kirgus]

На каждой итерации цикла for вы сравниваете строку number со списком result и строку letter со списком result, что странно. Задайте себе вопрос: может ли буква равняться списку букв? Это совершенно разные типы данных.

Инпуты не надо оборачивать в str, т.к. input и так возвращает всегда значение типа str.

Исправленный и более удобный вариант:

def count_letters(text, letters):
    letter_counters = [0 for _ in range(len(letters))]
    for current_letter in text:
        for index, letter in enumerate(letters):
            if current_letter == letter:
                letter_counters[index] += 1
    return letter_counters

text = input('Текст: ')
number = input('Какую цифру ищем: ')
letter = input('Какую букву ищем: ')

number_count, letter_count = count_letters(text, (number, letter))

print('Количество цифр', number, ':', number_count)
print('Количество букв', letter, ':', letter_count)

Comments

[dayvion]

Тут если не менять ваш код, надо было просто писать в for-е не result а result [i], потому что как сказали в другом ответе чы сравниваете всë слово а не элемент массива

[Kirgus]

dayvion, да, но изначальная функция слишком узко работает, принимает зачем-то два параметра-символа вместо одного либо коллекции символов.

Answer 2

[Алан Гибизов]

Ошибка в том, что не используется result в цикле for

Ну и накорябал свой велосипед:

result = {'цифр': [0, None], 'букв': [0, None]}
text = input('Введите текст: ')
for key in result:
    result[key][1] = input(f'Какую {key}у ищем? ')
for symbol in text:
    for key in result:
        result[key][0] += int(symbol == result[key][1])       
for key in result:
    print(f'Количество {key} {result[key][1]} : {result[key][0]}')

Comments

[Алан Гибизов]

Мне показалось скучно так просто это всё считать.
Вот более другой вариант:

class Element (object):
    def __init__(self, name):
        self.name = name
        self.value = None
        self.counter = 0
    def __repr__(self):
        return f'Количество {self.name} {self.value} : {self.counter}'

class Text (object):
    elements_names = ['цифр', 'букв']
    def __init__(self):
        self.text = None
        self.elements = dict(
            [(name, Element(name) ) for name in self.elements_names])
    def set_text(self):
        self.text = input('Введите текст: ')
    def set_elements(self):
        for key in self.elements:
            self.elements[key].value = input(f'Какую {key}у ищем? ')
    def count_element(self, element):
        text = self.text
        for symbol in text:
            element.counter += int(symbol == element.value)
    def count_elements(self):
        for key in self.elements:
            self.count_element(self.elements[key])
    def __repr__(self):
        result = '\n'.join([str(self.elements[key]) for key in self.elements])
        return result

text = Text()
text.set_text()
text.set_elements()
text.count_elements()
print(text)

[Emince]

Алан Гибизов,
К сожалению классы еще не проходили )

[Алан Гибизов]

Emince, тогда обратите внимание на первый мой код. В нем, в отличие от вашего, не используются повторяющиеся строки.
Т.е. вместо двух одинаковых строк запроса значений - цикл. Вместо двух одинаковых строк вывода - цикл.

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

[Emince]

Алан Гибизов, если честно, я его не могу понять ( я только учусь...