Python-сообщество

Привет!
Закончил писать копию сапёра.
Выкладываю файлы.
Потестите на ошибки.
Буду рад послушать ваши мнения.

Прикреплённый файлы:
Saper1.zip (43,1 KБ)

Проверил код. Сразу скажу, что про PEP8 или банальный рефакторинг ты не слышал. Код полон повторений инструкций, в которых лишь меняются какие-то незначительные части. Ну прочти ты про “for”, и сорцы сократятся и упростятся в 4-5 раз. Также, делай отступы в 2 строки между классами и методами вне класса и 1 строку между методами класса. А то очень не читабельно. А так вроде легитно.

 # Life loop
while alive:
    if (fun > boredom) and money:
        pass_day(fun, boredom, money)
        continue
    else: break

А можно чуть конкретней.
В каком месте мне нужно что-то поменять.
Я поэтому и выложил программу, потому что понимаю, что там много недочётов.
Надеюсь вы укажете в каком месте?

у меня версия 1.9.6
но я не думаю, что дело в этом.
атрибут BUTTON_RIGHT обязательно будет и в ранней версии

panama555
атрибут BUTTON_RIGHT обязательно будет и в ранней версии

Отредактировано py.user.next (Фев. 10, 2020 13:00:18)

> А можно чуть конкретней. В каком месте мне нужно что-то поменять.

После того как в последний раз я прокомментировал чью-то игру, у человека случился истерический припадок. Но я надеюсь что ты не такой впечатлительный…

 if massive[y][x] == 0:
     picture = "d0.png"
elif massive[y][x] == 1:
      picture = "d1.png"
elif massive[y][x] == 2:
      picture = "d2.png"
....

 if number=="0":
       self.image.blit(self.long_image,(0,0),(28,0,28,50))
elif number=="1":
        self.image.blit(self.long_image,(0,0),(56,0,28,50))
....
elif number=="-":
        self.image.blit(self.long_image,(0,0),(0,0,28,50))

 for y in range(16):#заполняем пустыми клетками группу 
        for x in range(16):
            gr_kletka.add(Kletka((x * 35 + 23, y * 35 + 120),"kletka1.png",0))

 gr_kletka.add( Kletka((x * 35 + 23, y * 35 + 120),"kletka1.png",0) for x in  in range(16) for y in range(16) )

 massive = [[0 for i in range(stolb)] for j in range(stroki)]#матрица для хранения координат бомб

 massive = [ [0]*stolb for _ in range(stroki) ]#матрица для хранения координат бомб

Отредактировано Rodegast (Фев. 10, 2020 13:36:46)

py.user.next
Сам код ужасный. Это ты ощутишь, когда начнут сыпаться баги и недочёты всякие и код надо будет поправлять, чтобы их исправить. Естественно, при исправлениях код начнёт превращаться в свалку всяких правок. И ты это не сможешь предотвратить, даже если будешь соблюдать какую-то чистоту, потому что код изначально написан так, что он не пригоден к правке. Он очень быстро придёт к путанице.

Rodegast
Но я надеюсь что ты не такой впечатлительный…

Rodegast
А хуже всего то что ты не пользуешься ООП…

panama555
Вот это я бы исправил, если бы знал как. Вроде пытаюсь писать программу опираясь на методы объектов, но, как видите, видимо не получается. Сам понимаю, что что-то не так.

panama555
Довольно простая на вид программа разрослась до неприятных размеров и стала довольно запутанной.

Отредактировано py.user.next (Фев. 11, 2020 00:54:57)

panama555
Как правильно писать ООП? Я уже задавал тут этот вопрос, но, видимо, ответ для меня не очевидный

1. пжлст, форматируйте код, это в панели создания сообщений, выделите код и нажмите что то вроде
2. чтобы вставить изображение залейте его куда нибудь (например), нажмите и вставьте ссылку на его url
…
есчщо

Отредактировано AD0DE412 (Фев. 11, 2020 07:13:33)