DG
ну например этот метод нейм: говорит о http verb метода, говорит об иерархии неймспейса (название его по сути дублирует)
return_purchase?
Size: a a a
2020 October 26
IS
С докстрингой и тп
DG
С докстрингой и тп
Денег на карту кинешь?
IS
Какие деньги лол
DG
Ну не хватало мне ещё докстринги за людей придумывать ;(
EB
purchase - а где хотя бы одно упоминание что это за сущность? этого слова дажи в API нет
DG
purchase - а где хотя бы одно упоминание что это за сущность? этого слова дажи в API нет
А что такое posting?
EB
не зная доменной области продукта я бы не проводил аналогию posting == purchase
EB
так что предложение дерьмо, как минимум потому что вводит синоним который никак самим автором не подтвержден, и нет прямой аналогии с неймспейсами существующего API
DG
так что предложение дерьмо, как минимум потому что вводит синоним который никак самим автором не подтвержден, и нет прямой аналогии с неймспейсами существующего API
Дело в том, что код больше читается, чем пишется. И надо его писать так, чтоб каждая обезьяна, которая в него залезла, могла понять что там происходит.
EB
претензии по неймингу самого API имхо за пределами критики кода на скрине. это уже задача разработчиков апи
VM
не зная доменной области продукта я бы не проводил аналогию posting == purchase
+
B
претензии есть, а предложений нет
DG
Ну а что +? Переопредели post в классе, и подпихивай туда headers, и не импортируй всю либу requests.
VM
Ну а что +? Переопредели post в классе, и подпихивай туда headers, и не импортируй всю либу requests.
Спасибо за предложение, но мой вопрос был совсем в другом.
DG
Спасибо за предложение, но мой вопрос был совсем в другом.
А почему нельзя двумя именованными параметрами передать?
IS
Ну а что +? Переопредели post в классе, и подпихивай туда headers, и не импортируй всю либу requests.
Можно переопределить просто request, без утомительного переопределения всех запросов
EB
а зачем перегружать интерфейс метода и передавать только какие-то атрибуты обьекта, зачем какие-то там именнованные параметы, когда можно передать сам обьект?
EB
тем более, что posting наверняка по своим полям совпадает с каким-то обьектом Модели в базе данных
DG
Можно переопределить просто request, без утомительного переопределения всех запросов
Вот тут не понял, можешь чуть подробнее описать?