V
эт пасхалка ?
Size: a a a
2021 August 09
NK
немножко не понял, не могли бы чуть-чуть развернуть?
V
ну для фласка указал версию? а для других пакетов?
NK
Это помарка, не сомневаюсь что(к сожалению) одна из многих=) Исправлю обязательно, спасибо
А
так и для остальных
NK
@bomj_1337 , @Kippi2 аааа, всё, понял что вы имеете ввиду. Поправлю, спасибо
V
тут pep8, лучше абсолютный путь, но это дело каждого
https://github.com/nickkirbus/monty_hall_problem/blob/master/app/api/v2/services/game_emulator.py#L5
https://github.com/nickkirbus/monty_hall_problem/blob/master/app/api/v2/services/game_emulator.py#L5
V
https://github.com/nickkirbus/monty_hall_problem/blob/master/app/api/common/models/user_scraps.py#L26
вот это дело можно один запросом организовать, а ты select + delete
как говорил мой тимлид до кода можно вечно доёбываться, в целом мне нравится аккуратность проекта, так что продолжай)
пусть даже местами проблемы есть
вот это дело можно один запросом организовать, а ты select + delete
как говорил мой тимлид до кода можно вечно доёбываться, в целом мне нравится аккуратность проекта, так что продолжай)
пусть даже местами проблемы есть
NK
Хорошая мысль, спасибо. Как думаете, если перепишу метод https://github.com/nickkirbus/monty_hall_problem/blob/master/app/api/common/models/base_model.py#L17 как
def delete(self):
db.session.query(self).filter(self.id == id).delete()
db.session.commit()
и затем использовать is_successful = UserScraps.delete(scrap_id) Было бы более приемлемо?
def delete(self):
db.session.query(self).filter(self.id == id).delete()
db.session.commit()
и затем использовать is_successful = UserScraps.delete(scrap_id) Было бы более приемлемо?
V
там по-моему delete умеет возвращать !количество! изменённых объектов,
поэтому если количество отличается от 0 - всё хорошо,
если 0 то понятно уж
поэтому если количество отличается от 0 - всё хорошо,
если 0 то понятно уж
V
T
если количество 0 тоже может быть всё хорошо
V
да обычная тупиковая задача, либо кто-то раньше удалил либо условие кривое
NK
Спасибо за фидбэк!+)
ПП
Есть проблемы
ПП
pip3 в докере не нужен
ПП
В тестах импорты не в начале файла
ПП
Зачем пустые иниты?
NK
Как это не нужен?