leraved / prog_Vedernikova

0 stars 0 forks source link

graphics #6

Closed leraved closed 6 years ago

leraved commented 6 years ago

@Antosha417, графика https://github.com/leraved/prog_Vedernikova/commit/c0e05214f7e35efa351e80acac3fc716392b60ac#diff-ac3e3060afa0c85bc86f9001d02bede2

antosha417 commented 6 years ago

Пайчарм ругается на твой кодстайл и несоблюдение пеп8, поправь, пожалуйста.

antosha417 commented 6 years ago

Давай хорошо подумаем над названиями переменных. Почему у тебя цифры в названиях? Это не намекает тебе на то, что нужно использовать списки и циклы?

antosha417 commented 6 years ago

Очень много одинаковых строк кода (в народе копипасты) перепиши через циклы

antosha417 commented 6 years ago

забыла в конце написать

window.getMouse()
window.close()

программа сразу закрывается, не видно картинку

antosha417 commented 6 years ago

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

leraved commented 6 years ago

у меня проблемы с облаками. там круги разных радиусов, и нет четкого смещения. это как-то можно задать циклами?

antosha417 commented 6 years ago

Сейчас скажу

antosha417 commented 6 years ago

https://github.com/konovalovatanya26/prog_Konovalova/issues/9#issuecomment-439221063

antosha417 commented 6 years ago

На будущее пиши лучше в чат, там быстрее отвечу

leraved commented 6 years ago

@antosha417, задала облака рандомом https://github.com/leraved/prog_Vedernikova/commit/6e32b4e5e1839d4e268f9398136ec404a9798aa7#diff-ac3e3060afa0c85bc86f9001d02bede2

antosha417 commented 6 years ago

distance_x было бы лучше, я думаю. Закрывай issue