Провалил тестовое задание DrWeb. В чем косяк?

Question

[swkonstr]

В DrWeb дали тест:

На Python реализовать программу, осуществляющую копирование файлов в соответствии с конфигурационным файлом. Конфигурационный файл должен иметь формат xml. Для каждого файла в конфигурационном файле должно быть указано его имя, исходный путь и путь, по которому файл требуется скопировать.

Предлагаю решение:
https://github.com/swkonstr/drweb_test_python/comm...

Ответ:

К сожалению, в настоящее время мы не можем сделать Вам предложение.

Вот чешу репу и думаю, что не понравилось.

upd.
HR Анна, и тех. спецы из DrWeb, не пожалели времени и ответили конкретно по пунктам, за что им огромное спасибо.
Список косяков:
1. Большой try..except на весь код, скрывающий возможные проблемы (в любом случае выведется "Can't open configuration file.", независимо от того, что произошло)
2. Вообще любовь к безусловным try..except с теми же проблемами, что и в п. 1
3. Жестко заданный путь к конфигурационному файлу
4. Нет стандартной для питона проверки if __name__ == "__main__"
5. Не проверяется наличие всех необходимых элементов в xml
6. Не обрабатывается ситуация, когда файл с таким именем уже есть в целевой директории

Comments

[Александр]

хм... интересно. Эти требования подразумевались "по умолчанию" или где-то было указано, что, например, входные XML-данные не прошли валидацию. А были ли описаны требования к обработке ошибок (должен быть print или пусть вылетает с ошибкой)?

Из списка косяков я согласен с 1, 2 и 4. Что касается 3, 5, 6 — ТЗ требовало уточнений.

Поделитесь, какая вилка по ЗП предлагалась?

Answer 1

[Алексей Черемисин]

Главное, ваша программа не выполняет именно главного - копирования! move - перемещение.
shutil.move(input_path+name, output_path+name)
Надо или shutil.copy или shutil.copy2

Comments

[swkonstr]

Аааааа!!! Посыпаю голову пеплом!

Вот это в точку!
Спасибо!

[Алексей Черемисин]

Ну и конечно это не отменяет обработку командной строки , валидацию xml, и обработку ошибок. Я так понимаю, что задание было не на скорость, так что можно было еще немного постараться. Ну да ничего, все впереди.

Answer 2

[User23]

except:
при таком синтаксисе пропустит любую ошибку, например ошибку синтаксиса и даже ctrl+c, правильно указывать исключения, например так
except: KeyError
что бы не экранировать лучше использовать "сырые строки"
например print(r'\|\|\//?')
Код нужно разбить хотя бы на функции, а не все в одну кучу

Answer 3

[swkonstr]

Со сниппетом согласен. А вот с

выдранный откуда-то кусок кода, самостоятельно не работающий

категорически не согласен.
1. Не выдран. Написан за 2 часа неспешного изучения офф. доков.
2. Какие тесты с Вашей точки зрения тут должны присутствовать?
3. Самостоятельно, это без установки Python-a? В задании не было указания сделать stand-alone версию. У меня работает как в Sublime, так и под WinPython. (v3.6)
4. Не укажете на "грубые ошибки"? Мое понимание PEP8, SublimePythonIDE плагин и примеры на docs.python.org говорят об обратном.
5. А не будет ли наглостью попросить Вас показать, как должен выглядеть ответ не Juniora, а матерого программиста?

В любом случае спасибо за уделенное мне время.
:)

Comments

[Animkim]

Мельком посмотрел, не написана должность вакансии.
О том что ещё не сказали, нельзя так использовать try except, строки так складывать тоже нельзя у Лутца об этом где-то в сам начале написано.

[swkonstr]

OnYourLips: Спасибо.
1. Учитывая, что данный "скрипт" по сути собран из кусков кода предложенного в офф. доках, мне трудно понять что тут не так с синтаксисом. Понимаю, что и так отнял много времени, но за более глубокую порку на примере пары строк был бы признателен. Без подколок, просто пытаюсь понять. :)
2. Пошел копать. Спасибо!
3. Ну я в начале думал о таком варианте, но отбросил его так как решил, что о таком требовании написали бы в задании. Очевидно "не угадал" :)
4. Спасибо. Буду читать.
5. Без обид. Я не то что бы на что-то особо рассчитывал делая это задание, но отрицательный ответ на решение в общем то полностью соответствующее заданию меня обескуражил. Впрочем как уже выше указали, я успешно прощелкал "копирование", так что - позор мне.

з.ы.: Джанго крутил, но так и не понял, в чем его "прелесть" относительно проверенных html+php+java...

Спасибо!

[swkonstr]

Буду работать. Спасибо за критику!