Не взяли на работу из-за неправильно выполненного тестового задания, посмотрите?

@xahotaf452 · Регистрация: 18.03.2020

Студворк — интернет-сервис помощи студентам

Привет.
Устраиваюсь в одну контору на Junior Python 40 тыс.р до вычета. Сегодня ответили: “Бла бла бла…На данный момент не готовы сотрудничать по этой вакансии: есть замечания к тестовому”.
Выполнил тестовое за полтора дня - итого часов 8, учитывая, что ранее вообще на питоне не писал. Они проверяли 4 дня, у меня никогда так долго еще не проверяли тестовые, там по сути и проверять нечего.
Прилагаю ТЗ и написанный скрипт + пояснения. Там плевое дело.
Опытные товарищи, посмотрите пожалуйста, где же я наделал “замечаний”. Может действительно это никуда не потянет, или не взяли по другим причинам.

@kratorr · 18.03.2020, 17:36

xahotaf452, а как вы хотели сразу пройти, ранее не писавши на питоне ? Разбираться в коде не охото, но выглядит стрёмно.

Рыжий Лис · 18.03.2020, 18:03

Давайте по порядку:
1) нет функции main()
2) нет к-л консольного интерфейса (argparse)

Предусмотреть возможные сбои в сети или при записи на диск.

3) замалчивание исключений - плохая идея.
4) неумение пользоваться библиотекой requests

Python
1
2
r = requests.get('https://jsonplaceholder.typicode.com/users')
users = r.json()

5) нет инструкции по развёртыванию (req...txt)
6) нет шелл-банга
7) запускаете третий питон, а используете синтаксис второго

8) строки в питоне не мутируемые, так работает медленно:

Python
1
compl_tasks += task_title + '\n'

Лучше:

Python
1
2
3
compl_tasks.append(task_title)
#…
res = '\n'.join(compl_tasks)

9) в сам алгоритм не вникал, но как минимум вижу возможность перезатирания старого файла, если новый нельзя создать.

Добавлено через 52 секунды
10) ну и классика:

Python
1
'файлов ' + str(1)

https://pyformat.info/

@Garry Galler · 18.03.2020, 18:04

Код должен быть чистым, без необоснованных повторений, с выделенем функций

Это означает что вы должны всегда использовать функции.

Предусмотреть возможные сбои в сети или при записи на диск

Не сделано. Предусмотреть сбой это не значит "перехватить и геройски умереть". Это значит "перехватить и подождать, когда сбой починится".

Код сделать максимально эффективным

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

Предусмотреть крайние случаи (у пользователя нет задач, и т.п.).

Не сделано.

Код должен быть оформлен по pep8.

Не выполнено. (в коде без функций очень трудно сделать нормальное оформление)

Для подобного кода 80 строк маловато...

@xahotaf452 · 18.03.2020, 18:10 **[ТС]**

Это все на джуна?

@kratorr · 18.03.2020, 18:13

xahotaf452, конечно, вы же не думали пройти тестовое задание написав кривой скрипт и увидев питон первый раз в жизни))

Рыжий Лис · 18.03.2020, 18:16

Сообщение от Garry Galler

Это значит "перехватить и подождать, когда сбой починится".

Ну не совсем. Для сети может и есть смысл сделать несколько попыток с интервалом в несколько секунд (количество попыток задаётся аргументами в консоли).

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

Сообщение от xahotaf452

Это все на джуна?

Это далеко не всё. Там и знание теории баз данных, как работает сеть, маршрутизация и протоколы, использование системы контроля версий и прочее.

@Garry Galler · 18.03.2020, 18:25

Сообщение от Рыжий Лис

Для сети может и есть смысл

Я для сети и имел ввиду.

Сообщение от Рыжий Лис

сделать несколько попыток с интервалом в несколько секунд (количество попыток задаётся аргументами в консоли).

Хотя бы это. Но лично я все свои грабберы делал вечными. Они никогда не умирали :-) Только по предусмотренному хоткею.

Рыжий Лис · 18.03.2020, 18:32

Не по теме:

Сообщение от Garry Galler

Но лично я все свои грабберы делал вечными. Они никогда не умирали :-)

Я сегодня запускал с такими ключами:

Bash
1
wget -rc --read-timeout=10 -t 3 url

Потому что кривой сайт на некоторые адреса не отдавал данные и соединение что-то никак не рвалось по таймауту. И наоборот, закачка некоторых файлов иногда обрывалась.

Так что могут быть ньюансы :) Граббер может наткнуться на такой адрес-"ловушку" и повиснуть на бесконечное время

@Garry Galler · 18.03.2020, 18:47

Сообщение от Рыжий Лис

кривой сайт на некоторые адреса не отдавал данные

У меня все ходы записаны :-) Так что проверялось абсолютно все. Вплоть до соответствия заголовка страницы.
Не говоря уж о пустых ответах или ответах, чье время ожидания превысило допустимый таймут.

Рыжий Лис · 18.03.2020, 19:02

Не по теме:

Сообщение от Garry Galler

Не говоря уж о пустых ответах или ответах, чье время ожидания превысило допустимый таймут.

В том-то и подвох, что запрос отправлялся, а ответа не дождёшься. Подозреваю, что ещё TCP keep-alive работал от веб-сервера. Если парсер однопоточный - 100% lock.

@Garry Galler · 18.03.2020, 19:16

Сообщение от Рыжий Лис

запрос отправлялся, а ответа не дождёшься

Ну для этого как раз и таймаут нужен. В requests он предусмотрен как параметр. Если же его не использовать, то ожидание ответа будет равно 2 часам (на винде), после чего соединение будет наконец разорвано.

Рыжий Лис · 18.03.2020, 19:40

Не по теме:

Сообщение от Garry Galler

то ожидание ответа будет равно 2 часам (на винде), после чего соединение будет наконец разорвано.

Что-то ты привираешь:

--read-timeout=seconds
Set the read (and write) timeout to seconds seconds. The "time" of
this timeout refers to idle time: if, at any point in the download,
no data is received for more than the specified number of seconds,
reading fails and the download is restarted. This option does not
directly affect the duration of the entire download.

Of course, the remote server may choose to terminate the connection
sooner than this option requires. The default read timeout is 900
seconds.

Это у wget. А для сервера соединение - это ценный ресурс. Он не будет оставлять неиспользуемые соединения и будет сбрасывать их.

apache timeout - default is 300 (in version 2.4 it is 60).

Синтаксис: proxy_send_timeout время;
Умолчание:

proxy_send_timeout 60s;

Добавлено через 3 минуты

Не по теме:

для nginx немного ошибся:

Синтаксис: send_timeout время;
Умолчание:

send_timeout 60s;

Контекст: http, server, location

Задаёт таймаут при передаче ответа клиенту. Таймаут устанавливается не на всю передачу ответа, а только между двумя операциями записями. Если по истечении этого времени клиент ничего не примет, соединение будет закрыто.

@Garry Galler · 18.03.2020, 19:51

Сообщение от Рыжий Лис

Это у wget.

А я про дефолт на винде.

KeepAliveTime
Значение KeepAliveTime контролирует частоту попыток TCP/IP проверить состояние простаивающего соединения. Если за указанное время деятельность отсутствует, будет отправлен контрольный пакет. Если сеть в порядке и партнер активен, то он отправит ответ. Если вам требуется более оперативно узнавать о потере партнера, уменьшите это значение. Если простаивающие соединения с продолжительными периодами бездействия являются обычным явлением и потеря партнера происходит нечасто, можно увеличить это значение для снижения дополнительной нагрузки.

По умолчанию Windows отправляет контрольный пакет при отсутствии активности в простаивающем соединении дольше, чем 7 200 000 миллисекунд (2 часа). Зачастую более предпочтительным является значение 1 800 000 миллисекунд, позволяющее обнаруживать половину закрытых соединений за 30 минут.

Добавлено через 1 минуту

Сообщение от Рыжий Лис

А для сервера соединение - это ценный ресурс.

Значит тебе просто не попадались сервера, которые не сбрасывали соединение через 900 сек.
Кстати, посмотри лекции Светлова (разработчика asynсio и автора aiohttp) - он как раз про эти 2 часа там где-то упоминает.

Новые блоги и статьи Все статьи Все блоги /
модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .	Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ
Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .	Использование значений реквизитов справочника в документе, с определенными условиями и правами Maks 07.04.2026 1. Контроль срока действия договора Алгоритм из решения ниже реализован на примере нетипового документа "ЗаявкаНаРаботу", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если. . .	Доступность команды формы по условию Maks 07.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: сделать доступной кнопку (команда формы "ЗавершитьСписание") при. . .

@kratorr 102 / 75 / 38 Регистрация: 08.11.2017 Сообщений: 272
	18.03.2020, 17:36
	xahotaf452, а как вы хотели сразу пройти, ранее не писавши на питоне ? Разбираться в коде не охото, но выглядит стрёмно. 0

@xahotaf452 0 / 0 / 0 Регистрация: 18.03.2020 Сообщений: 2
	18.03.2020, 18:10 [ТС]
	Это все на джуна? 0

@kratorr 102 / 75 / 38 Регистрация: 08.11.2017 Сообщений: 272
	18.03.2020, 18:13
	xahotaf452, конечно, вы же не думали пройти тестовое задание написав кривой скрипт и увидев питон первый раз в жизни)) 0