search

leraved / prog_Vedernikova

0 stars 0 forks source link

flying_balls #8

Open leraved opened 5 years ago

leraved commented 5 years ago

@Antosha417, летающие шары https://github.com/leraved/prog_Vedernikova/commit/553cb7b0b4396a84b57ae8928cf3accdae9b7ce8#diff-7b01658d8d448a1f32f0fdf61ea90706

antosha417 commented 5 years ago

Убери копипасту из tick_handler

antosha417 commented 5 years ago

Пока ставлю тебе 0,8 за задачу, чтобы поднять поправь код.

leraved commented 5 years ago

@antosha417, постаралась исправить копипаст https://github.com/leraved/prog_Vedernikova/commit/e165334baf4704efb3c132e416a981ec756f48fd#diff-7b01658d8d448a1f32f0fdf61ea90706

antosha417 commented 5 years ago

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

         if x < 0:
             dx = -dx;
             x = 0
         elif x > 300-R:
             dx = -dx
             x = 300-R

         if y < 0:
             dy = -dy
             y = 0
         elif y > 300-2*R:
            dy = -dy
            y = 300-2*R
leraved commented 5 years ago

@antosha417, описала некоторые переменные и их значение; добавила комментарии https://github.com/leraved/prog_Vedernikova/commits/master/flying_balls.py

leraved commented 5 years ago

@antosha417, https://github.com/leraved/prog_Vedernikova/commits/master/flying_balls.py