Python не выходит из цикла while, что делать?

Question

[HunLZy]

Я делаю сервер-клиентское приложение, где клиент может загружать файлы на сервер и скачивать от туда (что то типа облака), но при реализации загрузки файлов из сервера столкнулся с проблемой

o_file = open(name_file, 'wb')
	file_data = client.recv(524288)
	o_file.write(file_data)
	while file_data:
		file_data = client.recv(524288)
		o_file.write(file_data)
	o_file.close()
	client.close()

программа не выходит из цикла while, перепробовал все что возможно, но безрезультатно(сам файл скачивается правильно и его можно открыть, если завершить работу программы принудительно)

Comments

[Everything_is_bad]

ну так дебаж, смотри что у тебя получается внутри цикла

[mayton2019]

Надо посмотреть чему равен объект file_data когда читает хвост стрима.

while file_data:
		file_data = client.recv(524288)
                print("file_data = ", file_data)
		o_file.write(file_data)

[HunLZy]

mayton2019, сейчас попробую посмотреть

[HunLZy]

mayton2019,
data = b'asdsadknmadnqasokdknsDJKLSNK;SNAJKLNKL;DN;JKLSDASJKVBA;DKVBADJS;BVKVBDHJVBSJKLVACNSVBSJZHKVJVHNSJSCVBHDNV'

Вывел все то, что хранится в файле, но ничего больше не произходит

[Евгений Залецкий]

Так может выйдишь из цикла break?

[HunLZy]

Евгений Залецкий, пробовал, не помогло

while True:
		file_data = client.recv(524288)
		o_file.write(file_data)
		if not file_data:
			break

этот способ не помог

Answer 1

[mayton2019]

Мне кажется я понимаю в чем проблема автора. Socket - это не файл.
Это просто стриминговый протокол по которому можно ждать данных
бесконечно. Потому что никакого маркера конца нету.

Тут нужно договориться что либо сервер закрывает соединение после отдачи файла.
Либо клиент сам отваливается когда детектирует либо принятие числа байтов
либо специальный байт маркер EOT (как договоритесь).

Answer 2

[d'Ivan]

после открытия файла данные считываются лишь одинажды и затем код проверяет в цикле одно и то же:
while file_data:
что логично. Ведь данные не изменились.

Чтобы исправить, нужно изменить на

while True:
    file_data = client.recv(524288) # или указать размер поменьше
    if not file_data:
        break
    o_file.write(file_data)

И лучше писать так:

with open(name_file, 'wb') as o_file:
    while True:
        file_data = client.recv(524288)
        if not file_data:
            break
        o_file.write(file_data)

client.close()

Comments

[HunLZy]

не помогло, мне кажется, что клиент постоянно принимает что-то, но как это исправить я не знаю

[d'Ivan]

HunLZy, ну, клиент получает ровно столько, пока данные идут. Убедись, что с размером данных нет проблем на отправителе и он работает исправно.

[HunLZy]

Everything_is_not_so_bad, на отправление все гуд, а вот обратно уже проблемы начинаются

[mayton2019]

Попробуй так.

received_data = b''
    bytes_received = 0
    while bytes_received < file_size:
        chunk = connection.recv(1024)
        if not chunk:
            break
        received_data += chunk
        bytes_received += len(chunk)

[galaxy]

HunLZy,

мне кажется, что клиент постоянно принимает что-то, но как это исправить я не знаю

99% что клиент у тебя висит просто на recv(), сокет-то блокирующий?
И будет висеть, откуда он должен узнать, что передача данных закончена?
Либо изобретай свой мини-протокол для передачи файла (например, в первых нескольких байтах передавай длину файла), либо пусть сервер после передачи закрывает соединение.

[d'Ivan]

HunLZy, посмотри на сетевой трейсинг.
В Линуксе: strace -p PID
PID - ID процесса скрипта.
Если там ожидание с блокировкой, то galaxy прав.
Как минимум, нужно исправить код, как я указал. А ошибок может быть больше и не только в том куске кода.

[d'Ivan]

Если данные принимаются частями, то зачем указывать аж 524288, а не, скажем, по 64кб?

[HunLZy]

galaxy, блокирующий это наличие функции accept? Если да, то мой сокет блокирующий, хотя он только на серверной программе, нужен ли он клиенту?

[HunLZy]

Everything_is_not_so_bad, уже исправил на 4кб

[HunLZy]

mayton2019, хорошо, я попробую, позже напишу результат

[Hemul GM]

после открытия файла данные считываются лишь одинажды и затем код проверяет в цикле одно и то же:
while file_data:
что логично. Ведь данные не изменились

Бред. Данные он забирает внутри цикла повторно и while file_data проверит уже новые данные.

Проблема тут в том, что client.recv() не отпускает выполнение, пока данные не будут приняты размером с буфер

Answer 3

[Hemul GM]

Функция client.recv() будет висеть, пока данные не придут, причем размером с буфер. Работать с сокетом нужно не так