Как отрефакторить это чудо?) https://dpaste.org/JEMz

Question

Django [ru]

Sho

Как отрефакторить это чудо?) https://dpaste.org/JEMz

#backend #django #programming #python #russian

0

29.09.2020

10 ответов

17 просмотров

Sho Автор вопроса

вот как прошел мой рефакторинг) буду рад если укажите, что-то еще было https://dpaste.org/JEMz 1 круг https://dpaste.org/VcLb 2 круг https://dpaste.org/Equz

0

29.09.2020

Sho Автор вопроса

сделал по совету) спасибо про файлнейм, вначале проверка на наличие ключа, а дальше ее использование, а логики тут кроме респонсов и не было же?!)

0

29.09.2020

Artem Gubatenko

https://dpaste.org/Equz#L18,24 я бы придрался к этим двум строкам 18-я: думаю, лучше написать if в несколько строк, чем через - / 24-я: стараюсь ловить конкретные исключения и избегать ловить Exception

0

29.09.2020

Artem Gubatenko

и в switcher, ключи можно взять из модуля status. Чтоб по красоте и единообразно 😄

0

29.09.2020

Sho Автор вопроса

Когда, на все остальные эксепшены один вид респонса, ловить Exception критично?

0

29.09.2020

Artem Gubatenko

кстати: в 18 строке, отступ не лишний?

0

29.09.2020

Sho Автор вопроса

Оно в дпасте, может быть лишним

0

29.09.2020

Artem Gubatenko

а что там может еще выскочить, кроме FileNotFoundError? Если где-то в кастомных "кишочках" что-то может выскочить, лови там же и выкидывай кастомное исключение. А уже кастомное ловишь на верху и формируешь response. ИМХО

0

29.09.2020

Sho Автор вопроса

Несколько, связанных с файловой системой, от OSError, IOEror и тд, также мб от того что нет ключа нюнейм) киеррор

0

29.09.2020

Не выходи из комнаты. · Accepted Answer

Не выходи из комнаты.

Можешь вынести респонзы в отдельный модуль и импортировать их. Непонятно, зачем ты делаешь request.data[filename], если до этого идёт проверка на наличие ключа такого. И вообще я бы всю логику вынес в отдельный слой.

0

29.09.2020

170 похожих чатов

Как отрефакторить это чудо?) https://dpaste.org/JEMz

10 ответов

Похожие вопросы