code cleaning

Author: bison--

README.md

@@ -1,4 +1,11 @@
-pySnake
-=======
+# pySnake
+
 
 tiny snake in Python pygame
+
+
+## FEATURES
+
+* Joystick and GamePad support
+* fullscreen and window mode
+* fully customizable

conf.py

@@ -0,0 +1,18 @@
+# screen settings
+#SCREEN_WIDTH = 1024
+#SCREEN_HEIGHT = 600
+SCREEN_WIDTH = 1152
+SCREEN_HEIGHT = 864
+FULLSCREEN = False
+
+# window settings
+WINDOW_POSITION_X = 0
+WINDOW_POSITION_Y = 0
+WINDOW_BORDER = False
+
+# game settings
+BLOCKSIZE = 25
+SNACKS = 10
+SNACK_COLOR = (99, 255, 99)  # 'green'
+SNAKE_COLOR = (245, 101, 44)  # ORANGE
+BG_COLOR = (55, 55, 55)  # GRAY

joystick.py

@@ -1,5 +1,6 @@
 import pygame
 
+
 class joystick(object):
 	def __init__(self):
 		self.joystick_names = []
@@ -12,22 +13,23 @@ def __init__(self):
 			self.joystick_names.append(pygame.joystick.Joystick(i).get_name())
 
 		# By default, load the first available joystick.
-		if (len(self.joystick_names) > 0):
+		if len(self.joystick_names) > 0:
 			self.myJoystick = pygame.joystick.Joystick(0)
 			self.myJoystick.init()
 
 	def joystickAvailable(self):
-		if self.myJoystick != None:
+		if self.myJoystick is not None:
 			return True
 		else:
 			return False
 
 	def getAction(self):
 		action = ""
-		if self.myJoystick != None:
+		if self.myJoystick is not None:
 			try:
 				xAx = 0
 				yAx = 0
+
 				# sometimes 2 axis, sometimes 6, wtf?!
 				if self.myJoystick.get_numaxes() == 2:
 					xAx = 0
@@ -39,25 +41,25 @@ def getAction(self):
 					self.doMove = 2
 				elif self.myJoystick.get_axis(xAx) < 0:
 					self.doMove = 4
-				elif  self.myJoystick.get_axis(yAx) > 0:
+				elif self.myJoystick.get_axis(yAx) > 0:
 					self.doMove = 3
-				elif  self.myJoystick.get_axis(yAx) < 0:
+				elif self.myJoystick.get_axis(yAx) < 0:
 					self.doMove = 1
 
-				if self.myJoystick.get_button(0) and self.joyButtonDown == False: # speed up
+				if self.myJoystick.get_button(0) and self.joyButtonDown is False:  # speed up
 					action = "speedUp"
-				elif self.myJoystick.get_button(1) and self.joyButtonDown == False: # speed down
+				elif self.myJoystick.get_button(1) and self.joyButtonDown is False:  # speed down
 					action = "speedDown"
-				elif (self.myJoystick.get_button(9) and self.joyButtonDown == False)\
-					or (self.myJoystick.get_button(3) and self.joyButtonDown == False): # (re)start
+				elif (self.myJoystick.get_button(9) and self.joyButtonDown is False)\
+					or (self.myJoystick.get_button(3) and self.joyButtonDown is False):  # (re)start
 					action = "restart"
-				elif (self.myJoystick.get_button(8) and self.joyButtonDown == False):
+				elif self.myJoystick.get_button(8) and self.joyButtonDown is False:
 					action = 'quit'
 				#else:
 				# make sure NO button is down for reset
 				someJoyButtonDown = False
 				for i in range(0, self.myJoystick.get_numbuttons()):
-					if (self.myJoystick.get_button(i)):
+					if self.myJoystick.get_button(i) is True:
 						someJoyButtonDown = True
 				self.joyButtonDown = someJoyButtonDown
 

popUp.py

@@ -1,24 +1,25 @@
 import pygame
 import time
+
+
 class popUp(object):
 	def __init__(self, screen):
 		self._screen = screen
-		self.fnt = pygame.font.SysFont("MS Comic Sans" , 30)
+		self.fnt = pygame.font.SysFont("MS Comic Sans", 30)
 		self.color = (245, 101, 44)  # orange ;)
 		self.popUps = []
 		self.defaultDuration = 3
 
-
 	def singlePopUp(self, txt, duration=0):
 		if duration == 0:
 			duration = self.defaultDuration
 		self.popUps = []
-		self.popUps.append({"txt":txt, "start":time.time(), "duration":duration})
+		self.popUps.append({"txt": txt, "start": time.time(), "duration": duration})
 
 	def addPopUp(self, txt, duration=0):
 		if duration == 0:
 			duration = self.defaultDuration
-		self.popUps.append({"txt":txt, "start":time.time(), "duration":duration})
+		self.popUps.append({"txt": txt, "start": time.time(), "duration": duration})
 
 	def drawPopUps(self):
 		if len(self.popUps) == 0:
@@ -37,6 +38,6 @@ def drawPopUps(self):
 		# http://stackoverflow.com/questions/2770886/pygames-message-multiple-lines
 		if fullText != "":
 			xPos = (self._screen.get_width() / 2)
-			self._screen.blit(self.fnt.render(fullText, 1, self.color), ( xPos - (self.fnt.size(fullText)[0] / 2), 5))
+			self._screen.blit(self.fnt.render(fullText, 1, self.color), (xPos - (self.fnt.size(fullText)[0] / 2), 5))
 
 		return True