Как вывести значение вместо адреса объекта в памяти?

Question

[Emince]

Вот, собственно, код

import os


def catalog_search(path_to_catalog: str) -> list:
    fl_lst = []
    directory_ = os.walk(path_to_catalog, topdown=True)

    for address, dirs, files in directory_:
        for name in files:
            fl_lst.append(os.path.join(address, name))

    return fl_lst


def amount_file_string(list_: list) -> int:
    for i_file in list_:
        try:
            with open(i_file, 'r') as f:
                amount = f.readlines()
                yield len(amount)

        except FileNotFoundError:
            print(f'Файл не найден: {i_file}')


catalog = 'L:\\28.Python\\Python_Basic\\Module14'

file_list = catalog_search(catalog)
file_strings = (amount_file_string(file) for file in file_list)

print(f'Найдено {len(file_list)} файлов в каталоге {catalog}.')

for i_value in file_strings:
    try:
        print(f'Количество строк в файле {i_value}')
    except StopIteration:
        pass

Код работает, но не так надо.

Comments

[Vindicar]

В текущем виде код не может работать вообще, так как num_list не определена.
Приведи минимальный запускающийся пример кода.

[Emince]

Vindicar, поправил, num_list не нужна была, циклом иду по file_strings.

import os


def catalog_search(path_to_catalog: str) -> list:
    fl_lst = []
    directory_ = os.walk(path_to_catalog, topdown=True)

    for address, dirs, files in directory_:
        for name in files:
            fl_lst.append(os.path.join(address, name))

    return fl_lst


def amount_file_string(string_: str) -> int:
    for i_file in string_:
        try:
            with open(i_file, 'r') as f:
                amount = f.readlines()
                yield len(amount)

        except FileNotFoundError:
            print(f'Файл не найден: {i_file}')


catalog = 'L:\\28.Python\\Python_Basic\\Module14'

file_list = catalog_search(catalog)
file_strings = (amount_file_string(file) for file in file_list)

print(f'Найдено {len(file_list)} файлов в каталоге {catalog}.')

for i_value in file_strings:
    try:
        print(f'Количество строк в файле {i_value}')
    except StopIteration:
        pass

[Vindicar]

for i_value in file_strings:
    try:
        print(f'Количество строк в файле {i_value}')
    except StopIteration:
        pass

Во-первых, StopIteration ловить бессмысленно, оно не будет в норме выкинуто в этом коде.
Во-вторых, строка

file_strings = (file_num for file_num in amount_file_string(file_list))

тоже не имеет смысла - у тебя был генератор и остался генератор, ничего не изменилось от этого. Можно было просто
file_strings = amount_file_string(file_list)

[Михаил Кростелев]

А как вообще код работает? Что ожидаете увидеть и что видите в итоге?

[Emince]

Михаил Кростелев, ожидаю увидеть количество строк в каждом файле в виде числа, по факту, вижу ссылку на объект в памяти, при чем файлов в папке 12 и вижу 12 ссылок на объект.

Answer 1

[va_k]

Поправил amount_file_string. Эта функция по вашей задумке принимает список строк, а вы передаёте туда строку. Советую пользоваться pycharm.

import os


def catalog_search(path_to_catalog: str) -> list[str]:
    fl_lst = []
    directory_ = os.walk(path_to_catalog, topdown=True)

    for address, dirs, files in directory_:
        for name in files:
            fl_lst.append(os.path.join(address, name))

    return fl_lst


def amount_file_string(list_: list[str]):
    for i_file in list_:
        try:
            with open(i_file, 'r') as f:
                amount = f.readlines()
                yield len(amount)

        except FileNotFoundError:
            print(f'Файл не найден: {i_file}')


catalog = 'L:\\28.Python\\Python_Basic\\Module14'

file_list = catalog_search(catalog)
file_strings = (file_num for file_num in amount_file_string(file_list))

print(f'Найдено {len(file_list)} файлов в каталоге {catalog}.')

for i_value in file_strings:
    try:
        print(f'Количество строк в файле {i_value}')
    except StopIteration:
        pass

Comments

[Emince]

Я пользуюсь pycharm, при замене list_ на string_ ни чего не изменилось

def amount_file_string(string_: str) -> int:
    for i_file in string_:
        try:
            with open(i_file, 'r') as f:
                amount = f.readlines()
                yield len(amount)

        except FileNotFoundError:
            print(f'Файл не найден: {i_file}')

[va_k]

Emince, В строке нет файлов
for i_file in string_:
Также функция не возвращает int. Посмотрите мой код. Возможно ваш pycharm сломан если он не подсвечивает ошибки.

[Emince]

va_k, Ваш код выдает ошибку

Traceback (most recent call last):
File "L:\28.Python\Python_Basic\0.test.py", line 33, in
for i_value in file_strings:
File "L:\28.Python\Python_Basic\0.test.py", line 29, in
file_strings = (file_num for file_num in amount_file_string(file_list))
File "L:\28.Python\Python_Basic\0.test.py", line 19, in amount_file_string
amount = f.readlines()
File "C:\Python39\lib\encodings\cp1251.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x98 in position 508: character maps to

Что то он не может перекодировать

[va_k]

Emince, сложно сказать, видно, что проблема с кодировкой. Попробуйте её указать явно with open(i_file, 'r', encoding='utf-8') as f:. Если ошибка останется, то файл у вас не в utf-8. Понятно что все файлы должны быть в utf-8 кодировке.

[Emince]

va_k, да действительно, проблема была в кодировке, отмечаю решением, но есть вопрос, в чем разница м/д ваши и моим кодом?

[va_k]

Emince, Ваша функция amount_file_string подразумевает получение списка строк, а вы передаёте туда строку. Я передаю туда список строк. Указывайте типы до конца, не list, а list[str] и.т.д. Попытайтесь настроить pycharm, он должен подсвечивать ошибки и предупреждения.

[Emince]

Ок, спс!