fix setup.py

New musics
Piece lock optimize

README.md

@@ -2,9 +2,11 @@
 
 Tetris clone made with Python and Arcade graphic library
 
+![Screenshot](https://malingrey.fr/images/fMd4EseZ/VtwJIMyQ.png)
+
 ## Requirements
 
-* [Python](https://www.python.org/)
+* [Python](https://www.python.org/) 3.6 or upper
 
 ## Install
 
@@ -30,6 +32,6 @@ Use key name from [arcade.key package](http://arcade.academy/arcade.key.html).
 ## Build
 
 ```shell
-python -m pip install cx-freeze
-python setup.py build
-```
+python -m pip install -r build-requirements.txt
+python setup.py bdist
+```

TetrArcade.py

@@ -1,34 +1,48 @@
 # -*- coding: utf-8 -*-
 import sys
-import locale
-import time
-import os
-
-try:
-    import configparser
-except ImportError:
-    import ConfigParser as configparser
-
 try:
     import arcade
 except ImportError as e:
-    sys.exit(str(e) + """
+    sys.exit(
+        str(e) + """
 This game require arcade library.
 You can install it with:
 python -m pip install --user arcade"""
     )
+import pyglet
 
-import tetrislogic
+import locale
+import time
+import os
+import itertools
+import configparser
+
+from tetrislogic import TetrisLogic, Color, State, Coord
 
 
 # Constants
+# Matrix
+NB_LINES = 20
+NB_COLS = 10
+NB_NEXT = 5
+
+# Delays (seconds)
+LOCK_DELAY = 0.5
+FALL_DELAY = 1
+AUTOREPEAT_DELAY = 0.300
+AUTOREPEAT_PERIOD = 0.010
+
+# Piece init coord
+MATRIX_PIECE_COORD = Coord(4, NB_LINES)
+NEXT_PIECE_COORDS = [Coord(NB_COLS + 4, NB_LINES - 4 * n - 3) for n in range(NB_NEXT)]
+HELD_PIECE_COORD = Coord(-5, NB_LINES - 3)
+
 # Window
 WINDOW_WIDTH = 800
 WINDOW_HEIGHT = 600
 WINDOW_MIN_WIDTH = 517
 WINDOW_MIN_HEIGHT = 388
 WINDOW_TITLE = "TETRARCADE"
-MINO_SIZE = 20
 BG_COLOR = (7, 11, 21)
 
 # Delays (seconds)
@@ -41,57 +55,84 @@ GHOST_ALPHA = 30
 MATRIX_BG_ALPHA = 100
 BAR_ALPHA = 75
 
-# Sprites
-WINDOW_BG_PATH = "res/bg.jpg"
-MATRIX_BG_PATH = "res/matrix.png"
-HELD_BG_PATH = "res/held.png"
-NEXT_BG_PATH = "res/next.png"
-MINOES_SPRITES_PATHS = {
-    "orange": "res/orange_mino.png",
-    "blue": "res/blue_mino.png",
-    "yellow": "res/yellow_mino.png",
-    "cyan": "res/cyan_mino.png",
-    "green": "res/green_mino.png",
-    "red": "res/red_mino.png",
-    "magenta": "res/magenta_mino.png",
-}
+# Mino size
+MINO_SIZE = 20
+MINO_SPRITE_SIZE = 21
 
-# User profile path
-if sys.platform == "win32":
-    USER_PROFILE_DIR = os.environ.get(
-        "appdata", os.path.expanduser("~\Appdata\Roaming")
-    )
+if getattr(sys, 'frozen', False):
+    # The application is frozen
+    PROGRAM_DIR = os.path.dirname(sys.executable)
 else:
-    USER_PROFILE_DIR = os.environ.get(
-        "XDG_DATA_HOME", os.path.expanduser("~/.local/share")
-    )
-USER_PROFILE_DIR = os.path.join(USER_PROFILE_DIR, "TetrArcade")
-HIGH_SCORE_PATH = os.path.join(USER_PROFILE_DIR, ".high_score")
-CONF_PATH = os.path.join(USER_PROFILE_DIR, "TetrArcade.ini")
+    # The application is not frozen
+    PROGRAM_DIR = os.path.dirname(__file__)
+RESOURCES_DIR = os.path.join(PROGRAM_DIR, "resources")
+
+# Sprites
+IMAGES_DIR = os.path.join(RESOURCES_DIR, "images")
+WINDOW_BG_PATH = os.path.join(IMAGES_DIR, "bg.jpg")
+MATRIX_BG_PATH = os.path.join(IMAGES_DIR, "matrix.png")
+HELD_BG_PATH = os.path.join(IMAGES_DIR, "held.png")
+NEXT_BG_PATH = os.path.join(IMAGES_DIR, "next.png")
+MINOES_SPRITES_PATH = os.path.join(IMAGES_DIR, "minoes.png")
+Color.PRELOCKED = 7
+MINOES_COLOR_ID = {
+    Color.BLUE: 0,
+    Color.CYAN: 1,
+    Color.GREEN: 2,
+    Color.MAGENTA: 3,
+    Color.ORANGE: 4,
+    Color.RED: 5,
+    Color.YELLOW: 6,
+    Color.PRELOCKED: 7,
+}
+TEXTURES = arcade.load_textures(
+    MINOES_SPRITES_PATH, ((i * MINO_SPRITE_SIZE, 0, MINO_SPRITE_SIZE, MINO_SPRITE_SIZE) for i in range(8))
+)
+TEXTURES = {color: TEXTURES[i] for color, i in MINOES_COLOR_ID.items()}
+
+# Music
+MUSIC_DIR = os.path.join(RESOURCES_DIR, "musics")
+MUSICS_PATHS = (entry.path for entry in os.scandir(MUSIC_DIR))
 
 # Text
 TEXT_COLOR = arcade.color.BUBBLES
-FONT_NAME = "res/joystix monospace.ttf"
+FONT_NAME = os.path.join(RESOURCES_DIR, "fonts/joystix monospace.ttf")
 STATS_TEXT_MARGIN = 40
 STATS_TEXT_SIZE = 14
 STATS_TEXT_WIDTH = 150
 HIGHLIGHT_TEXT_COLOR = arcade.color.BUBBLES
 HIGHLIGHT_TEXT_SIZE = 20
 
+# User profile path
+if sys.platform == "win32":
+    USER_PROFILE_DIR = os.environ.get("appdata", os.path.expanduser("~\Appdata\Roaming"))
+else:
+    USER_PROFILE_DIR = os.environ.get("XDG_DATA_HOME", os.path.expanduser("~/.local/share"))
+USER_PROFILE_DIR = os.path.join(USER_PROFILE_DIR, "TetrArcade")
+HIGH_SCORE_PATH = os.path.join(USER_PROFILE_DIR, ".high_score")
+CONF_PATH = os.path.join(USER_PROFILE_DIR, "TetrArcade.ini")
+
 
 class MinoSprite(arcade.Sprite):
+
     def __init__(self, mino, window, alpha):
-        super().__init__(MINOES_SPRITES_PATHS[mino.color], window.scale)
+        super().__init__()
         self.alpha = alpha
         self.window = window
+        self.append_texture(TEXTURES[mino.color])
+        self.append_texture(TEXTURES[Color.PRELOCKED])
+        self.set_texture(0)
 
-    def set_position(self, x, y):
+    def refresh(self, x, y, prelocked=False):
+        self.scale = self.window.scale
         size = MINO_SIZE * self.scale
-        self.left = self.window.matrix_bg.left + x * size
-        self.bottom = self.window.matrix_bg.bottom + y * size
+        self.left = self.window.matrix.bg.left + x * size
+        self.bottom = self.window.matrix.bg.bottom + y * size
+        self.set_texture(prelocked)
 
 
 class MinoesSprites(arcade.SpriteList):
+
     def resize(self, scale):
         for sprite in self:
             sprite.scale = scale
@@ -99,6 +140,7 @@ class MinoesSprites(arcade.SpriteList):
 
 
 class TetrominoSprites(MinoesSprites):
+
     def __init__(self, tetromino, window, alpha=NORMAL_ALPHA):
         super().__init__()
         self.tetromino = tetromino
@@ -108,17 +150,13 @@ class TetrominoSprites(MinoesSprites):
             self.append(mino.sprite)
 
     def refresh(self):
-        if self.tetromino.prelocked:
-            alpha = PRELOCKED_ALPHA
-        else:
-            alpha = self.alpha
         for mino in self.tetromino:
             coord = mino.coord + self.tetromino.coord
-            mino.sprite.set_position(coord.x, coord.y)
-            mino.sprite.alpha = alpha
+            mino.sprite.refresh(coord.x, coord.y, self.tetromino.prelocked)
 
 
 class MatrixSprites(MinoesSprites):
+
     def __init__(self, matrix):
         super().__init__()
         self.matrix = matrix
@@ -128,11 +166,27 @@ class MatrixSprites(MinoesSprites):
         for y, line in enumerate(self.matrix):
             for x, mino in enumerate(line):
                 if mino:
-                    mino.sprite.set_position(x, y)
-                    self.append(mino.sprite)
+                    mino.sprite.refresh(x, y)
+
+    def remove_line(self, y):
+        for mino in self.matrix[y]:
+            if mino:
+                self.remove(mino.sprite)
 
 
-class TetrArcade(tetrislogic.TetrisLogic, arcade.Window):
+class TetrArcade(TetrisLogic, arcade.Window):
+
+    NB_LINES = NB_LINES
+    NB_COLS = NB_COLS
+    NB_NEXT = NB_NEXT
+    LOCK_DELAY = LOCK_DELAY
+    FALL_DELAY = FALL_DELAY
+    AUTOREPEAT_DELAY = AUTOREPEAT_DELAY
+    AUTOREPEAT_PERIOD = AUTOREPEAT_PERIOD
+    MATRIX_PIECE_COORD = MATRIX_PIECE_COORD
+    NEXT_PIECE_COORDS = NEXT_PIECE_COORDS
+    HELD_PIECE_COORD = HELD_PIECE_COORD
+
     def __init__(self):
         locale.setlocale(locale.LC_ALL, "")
         self.highlight_texts = []
@@ -163,21 +217,25 @@ class TetrArcade(tetrislogic.TetrisLogic, arcade.Window):
         arcade.set_background_color(BG_COLOR)
         self.set_minimum_size(WINDOW_MIN_WIDTH, WINDOW_MIN_HEIGHT)
         self.bg = arcade.Sprite(WINDOW_BG_PATH)
-        self.matrix_bg = arcade.Sprite(MATRIX_BG_PATH)
-        self.matrix_bg.alpha = MATRIX_BG_ALPHA
-        self.held_bg = arcade.Sprite(HELD_BG_PATH)
-        self.held_bg.alpha = BAR_ALPHA
-        self.next_bg = arcade.Sprite(NEXT_BG_PATH)
-        self.next_bg.alpha = BAR_ALPHA
+        self.matrix.bg = arcade.Sprite(MATRIX_BG_PATH)
+        self.matrix.bg.alpha = MATRIX_BG_ALPHA
+        self.held.bg = arcade.Sprite(HELD_BG_PATH)
+        self.held.bg.alpha = BAR_ALPHA
+        self.next.bg = arcade.Sprite(NEXT_BG_PATH)
+        self.next.bg.alpha = BAR_ALPHA
         self.matrix.sprites = MatrixSprites(self.matrix)
         self.on_resize(self.init_width, self.init_height)
 
+        if self.play_music:
+            self.music = pyglet.media.Player()
+            playlist = itertools.cycle(
+                pyglet.media.load(path)
+                for path in MUSICS_PATHS
+            )
+            self.music.queue(playlist)
+
     def new_conf(self):
-        self.conf["WINDOW"] = {
-            "width": WINDOW_WIDTH,
-            "height": WINDOW_HEIGHT,
-            "fullscreen": False,
-        }
+        self.conf["WINDOW"] = {"width": WINDOW_WIDTH, "height": WINDOW_HEIGHT, "fullscreen": False}
         self.conf["KEYBOARD"] = {
             "start": "ENTER",
             "move left": "LEFT",
@@ -190,6 +248,10 @@ class TetrArcade(tetrislogic.TetrisLogic, arcade.Window):
             "pause": "ESCAPE",
             "fullscreen": "F11",
         }
+        self.conf["MUSIC"] = {
+            "play": True
+        }
+        self.conf["AUTO-REPEAT"] = {"delay": 0.3, "period": 0.01}
         self.load_conf()
         if not os.path.exists(USER_PROFILE_DIR):
             os.makedirs(USER_PROFILE_DIR)
@@ -201,65 +263,47 @@ class TetrArcade(tetrislogic.TetrisLogic, arcade.Window):
         self.init_height = int(self.conf["WINDOW"]["height"])
         self.init_fullscreen = self.conf["WINDOW"].getboolean("fullscreen")
 
+        for action, key in self.conf["KEYBOARD"].items():
+            self.conf["KEYBOARD"][action] = key.upper()
         self.key_map = {
-            tetrislogic.State.STARTING: {
+            State.STARTING: {
                 getattr(arcade.key, self.conf["KEYBOARD"]["start"]): self.new_game,
-                getattr(
-                    arcade.key, self.conf["KEYBOARD"]["fullscreen"]
-                ): self.toggle_fullscreen,
+                getattr(arcade.key, self.conf["KEYBOARD"]["fullscreen"]): self.toggle_fullscreen,
             },
-            tetrislogic.State.PLAYING: {
+            State.PLAYING: {
                 getattr(arcade.key, self.conf["KEYBOARD"]["move left"]): self.move_left,
-                getattr(
-                    arcade.key, self.conf["KEYBOARD"]["move right"]
-                ): self.move_right,
+                getattr(arcade.key, self.conf["KEYBOARD"]["move right"]): self.move_right,
                 getattr(arcade.key, self.conf["KEYBOARD"]["soft drop"]): self.soft_drop,
                 getattr(arcade.key, self.conf["KEYBOARD"]["hard drop"]): self.hard_drop,
-                getattr(
-                    arcade.key, self.conf["KEYBOARD"]["rotate clockwise"]
-                ): self.rotate_clockwise,
-                getattr(
-                    arcade.key, self.conf["KEYBOARD"]["rotate counter"]
-                ): self.rotate_counter,
+                getattr(arcade.key, self.conf["KEYBOARD"]["rotate clockwise"]): self.rotate_clockwise,
+                getattr(arcade.key, self.conf["KEYBOARD"]["rotate counter"]): self.rotate_counter,
                 getattr(arcade.key, self.conf["KEYBOARD"]["hold"]): self.swap,
                 getattr(arcade.key, self.conf["KEYBOARD"]["pause"]): self.pause,
-                getattr(
-                    arcade.key, self.conf["KEYBOARD"]["fullscreen"]
-                ): self.toggle_fullscreen,
+                getattr(arcade.key, self.conf["KEYBOARD"]["fullscreen"]): self.toggle_fullscreen,
             },
-            tetrislogic.State.PAUSED: {
+            State.PAUSED: {
                 getattr(arcade.key, self.conf["KEYBOARD"]["pause"]): self.resume,
-                getattr(
-                    arcade.key, self.conf["KEYBOARD"]["fullscreen"]
-                ): self.toggle_fullscreen,
+                getattr(arcade.key, self.conf["KEYBOARD"]["fullscreen"]): self.toggle_fullscreen,
             },
-            tetrislogic.State.OVER: {
+            State.OVER: {
                 getattr(arcade.key, self.conf["KEYBOARD"]["start"]): self.new_game,
-                getattr(
-                    arcade.key, self.conf["KEYBOARD"]["fullscreen"]
-                ): self.toggle_fullscreen,
+                getattr(arcade.key, self.conf["KEYBOARD"]["fullscreen"]): self.toggle_fullscreen,
             },
         }
 
+        self.AUTOREPEAT_DELAY = float(self.conf["AUTO-REPEAT"]["delay"])
+        self.AUTOREPEAT_PERIOD = float(self.conf["AUTO-REPEAT"]["period"])
+
         controls_text = (
             "\n\n\nCONTROLS\n\n"
             + "\n".join(
                 "{:<16s}{:>6s}".format(key, action)
-                for key, action in tuple(self.conf["KEYBOARD"].items())
-                + (("QUIT", "ALT+F4"),)
+                for key, action in tuple(self.conf["KEYBOARD"].items()) + (("QUIT", "ALT+F4"),)
             )
             + "\n\n\n"
         )
-        self.start_text = (
-            "TETRARCADE"
-            + controls_text
-            + "PRESS [{}] TO START".format(self.conf["KEYBOARD"]["start"])
-        )
-        self.pause_text = (
-            "PAUSE"
-            + controls_text
-            + "PRESS [{}] TO RESUME".format(self.conf["KEYBOARD"]["pause"])
-        )
+        self.start_text = "TETRARCADE" + controls_text + "PRESS [{}] TO START".format(self.conf["KEYBOARD"]["start"])
+        self.pause_text = "PAUSE" + controls_text + "PRESS [{}] TO RESUME".format(self.conf["KEYBOARD"]["pause"])
         self.game_over_text = """GAME
 OVER
 
@@ -270,47 +314,77 @@ AGAIN""".format(
             self.conf["KEYBOARD"]["start"]
         )
 
+        self.play_music = self.conf["MUSIC"].getboolean("play")
+
     def new_game(self):
         self.highlight_texts = []
         super().new_game()
+        self.matrix.sprites = MatrixSprites(self.matrix)
+        if self.play_music:
+            self.music.seek(0)
+            self.music.play()
 
     def new_tetromino(self):
         tetromino = super().new_tetromino()
         tetromino.sprites = TetrominoSprites(tetromino, self)
         return tetromino
 
-    def new_current(self):
-        self.matrix.sprites = MatrixSprites(self.matrix)
-        super().new_current()
-        self.ghost.sprites = TetrominoSprites(self.ghost, self, GHOST_ALPHA)
-        for tetromino in [self.current, self.ghost] + self.next:
+    def new_matrix_piece(self):
+        super().new_matrix_piece()
+        self.matrix.ghost.sprites = TetrominoSprites(self.matrix.ghost, self, GHOST_ALPHA)
+        for tetromino in [self.matrix.piece, self.matrix.ghost] + self.next.pieces:
             tetromino.sprites.refresh()
 
     def move(self, movement, prelock=True):
         moved = super().move(movement, prelock)
-        self.current.sprites.refresh()
+        self.matrix.piece.sprites.refresh()
         if moved:
-            self.ghost.sprites.refresh()
+            self.matrix.ghost.sprites.refresh()
         return moved
 
     def rotate(self, rotation):
         rotated = super().rotate(rotation)
         if rotated:
-            for tetromino in (self.current, self.ghost):
+            for tetromino in (self.matrix.piece, self.matrix.ghost):
                 tetromino.sprites.refresh()
         return rotated
 
     def swap(self):
         super().swap()
-        self.ghost.sprites = TetrominoSprites(self.ghost, self, GHOST_ALPHA)
-        for tetromino in [self.held, self.current, self.ghost]:
+        self.matrix.ghost.sprites = TetrominoSprites(self.matrix.ghost, self, GHOST_ALPHA)
+        for tetromino in (self.held.piece, self.matrix.piece, self.matrix.ghost):
             if tetromino:
                 tetromino.sprites.refresh()
 
     def lock(self):
-        self.current.prelocked = False
-        self.current.sprites.refresh()
+        self.matrix.piece.prelocked = False
+        self.matrix.piece.sprites.refresh()
         super().lock()
+        self.matrix.sprites.refresh()
+
+    def enter_the_matrix(self):
+        super().enter_the_matrix()
+        for mino in self.matrix.piece:
+            self.matrix.sprites.append(mino.sprite)
+
+    def remove_line(self, y):
+        self.matrix.sprites.remove_line(y)
+        super().remove_line(y)
+
+    def pause(self):
+        super().pause()
+        if self.play_music:
+            self.music.pause()
+
+    def resume(self):
+        super().resume()
+        if self.play_music:
+            self.music.play()
+
+    def game_over(self):
+        super().game_over()
+        if self.play_music:
+            self.music.pause()
 
     def on_key_press(self, key, modifiers):
         for key_or_modifier in (key, modifiers):
@@ -344,26 +418,23 @@ AGAIN""".format(
         arcade.start_render()
         self.bg.draw()
 
-        if self.state in (tetrislogic.State.PLAYING, tetrislogic.State.OVER):
-            self.matrix_bg.draw()
-            self.held_bg.draw()
-            self.next_bg.draw()
+        if self.state in (State.PLAYING, State.OVER):
+            self.matrix.bg.draw()
+            self.held.bg.draw()
+            self.next.bg.draw()
             self.matrix.sprites.draw()
 
-            for tetromino in [self.held, self.current, self.ghost] + self.next:
+            for tetromino in [self.held.piece, self.matrix.piece, self.matrix.ghost] + self.next.pieces:
                 if tetromino:
                     tetromino.sprites.draw()
 
             t = time.localtime(self.time)
             font_size = STATS_TEXT_SIZE * self.scale
-            for y, text in enumerate(
-                ("TIME", "LINES", "GOAL", "LEVEL", "HIGH SCORE", "SCORE")
-            ):
+            for y, text in enumerate(("TIME", "LINES", "GOAL", "LEVEL", "HIGH SCORE", "SCORE")):
                 arcade.draw_text(
                     text=text,
-                    start_x=self.matrix_bg.left
-                    - self.scale * (STATS_TEXT_MARGIN + STATS_TEXT_WIDTH),
-                    start_y=self.matrix_bg.bottom + 1.5 * (2 * y + 1) * font_size,
+                    start_x=self.matrix.bg.left - self.scale * (STATS_TEXT_MARGIN + STATS_TEXT_WIDTH),
+                    start_y=self.matrix.bg.bottom + 1.5 * (2 * y + 1) * font_size,
                     color=TEXT_COLOR,
                     font_size=font_size,
                     align="right",
@@ -382,8 +453,8 @@ AGAIN""".format(
             ):
                 arcade.draw_text(
                     text=text,
-                    start_x=self.matrix_bg.left - STATS_TEXT_MARGIN * self.scale,
-                    start_y=self.matrix_bg.bottom + 3 * y * font_size,
+                    start_x=self.matrix.bg.left - STATS_TEXT_MARGIN * self.scale,
+                    start_y=self.matrix.bg.bottom + 3 * y * font_size,
                     color=TEXT_COLOR,
                     font_size=font_size,
                     align="right",
@@ -392,18 +463,16 @@ AGAIN""".format(
                 )
 
         highlight_text = {
-            tetrislogic.State.STARTING: self.start_text,
-            tetrislogic.State.PLAYING: self.highlight_texts[0]
-            if self.highlight_texts
-            else "",
-            tetrislogic.State.PAUSED: self.pause_text,
-            tetrislogic.State.OVER: self.game_over_text,
+            State.STARTING: self.start_text,
+            State.PLAYING: self.highlight_texts[0] if self.highlight_texts else "",
+            State.PAUSED: self.pause_text,
+            State.OVER: self.game_over_text,
         }.get(self.state, "")
         if highlight_text:
             arcade.draw_text(
                 text=highlight_text,
-                start_x=self.matrix_bg.center_x,
-                start_y=self.matrix_bg.center_y,
+                start_x=self.matrix.bg.center_x,
+                start_y=self.matrix.bg.center_y,
                 color=HIGHLIGHT_TEXT_COLOR,
                 font_size=HIGHLIGHT_TEXT_SIZE * self.scale,
                 align="center",
@@ -428,23 +497,23 @@ AGAIN""".format(
         self.bg.center_x = center_x
         self.bg.center_y = center_y
 
-        self.matrix_bg.scale = self.scale
-        self.matrix_bg.center_x = center_x
-        self.matrix_bg.center_y = center_y
-        self.matrix_bg.left = int(self.matrix_bg.left)
-        self.matrix_bg.top = int(self.matrix_bg.top)
+        self.matrix.bg.scale = self.scale
+        self.matrix.bg.center_x = center_x
+        self.matrix.bg.center_y = center_y
+        self.matrix.bg.left = int(self.matrix.bg.left)
+        self.matrix.bg.top = int(self.matrix.bg.top)
 
-        self.held_bg.scale = self.scale
-        self.held_bg.right = self.matrix_bg.left
-        self.held_bg.top = self.matrix_bg.top
+        self.held.bg.scale = self.scale
+        self.held.bg.right = self.matrix.bg.left
+        self.held.bg.top = self.matrix.bg.top
 
-        self.next_bg.scale = self.scale
-        self.next_bg.left = self.matrix_bg.right
-        self.next_bg.top = self.matrix_bg.top
+        self.next.bg.scale = self.scale
+        self.next.bg.left = self.matrix.bg.right
+        self.next.bg.top = self.matrix.bg.top
 
         self.matrix.sprites.resize(self.scale)
 
-        for tetromino in [self.held, self.current, self.ghost] + self.next:
+        for tetromino in [self.held.piece, self.matrix.piece, self.matrix.ghost] + self.next.pieces:
             if tetromino:
                 tetromino.sprites.resize(self.scale)
 
@@ -499,12 +568,17 @@ High score could not be saved:
 
     def on_close(self):
         self.save_high_score()
+        if self.play_music:
+            self.music.pause()
         super().on_close()
 
 
 def main():
-    TetrArcade()
-    arcade.run()
+    try:
+        TetrArcade()
+        arcade.run()
+    except Exception as e:
+        sys.exit(e)
 
 
 if __name__ == "__main__":

build-requirements.txt

@@ -0,0 +1 @@
+arcade cx-freeze

setup.py

@@ -9,22 +9,35 @@ else:
     base = None
     icon = None
 
-setup(
-    name="TetrArcade",
-    version="0.1",
-    description="Tetris clone",
-    author="adrienmalin",
-    executables=[Executable(
-        script="TetrArcade.py",
-        icon=icon,
-        base=base
-    )],
-    options={
-        "build_exe": {
-            "packages": ["arcade", "pyglet"],
-            "excludes": ["tkinter"],
-            "include_files": "res",
-            "silent": True
-        }
-    },
+excludes = [
+    "tkinter",
+    "PyQt4",
+    "PyQt5",
+    "PySide",
+    "PySide2"
+]
+
+executable = Executable(
+    script = "TetrArcade.py",
+    icon = icon,
+    base = base,
+    shortcutName="TetrArcade",
+    shortcutDir="DesktopFolder"
+)
+
+options = {
+    "build_exe": {
+        "packages": ["arcade", "pyglet"],
+        "excludes": excludes,
+        "include_files": "resources",
+        "silent": True
+    }
+}
+setup(
+    name = "TetrArcade",
+    version = "0.3",
+    description = "Tetris clone",
+    author = "AdrienMalin",
+    executables = [executable],
+    options = options,
 )

test.py

@@ -1,15 +1,17 @@
 # -*- coding: utf-8 -*-
 
-from TetrArcade import TetrArcade, tetrislogic
+from TetrArcade import TetrArcade, State
 
 game = TetrArcade()
 game.new_game()
 game.move_left()
 game.move_right()
+game.swap()
 game.rotate_clockwise()
 game.rotate_counter()
 for i in range(12):
     game.soft_drop()
+game.matrix.sprites.refresh()
 game.on_draw()
-while game.state != tetrislogic.State.OVER:
+while game.state != State.OVER:
     game.hard_drop()

tetrislogic/__init__.py

@@ -1,5 +1,5 @@
 # -*- coding: utf-8 -*-
 from .consts import NB_LINES, NB_COLS, NB_NEXT
-from .utils import Movement, Rotation
+from .utils import Movement, Rotation, Color, Coord
 from .tetromino import Mino, Tetromino
 from .tetrislogic import TetrisLogic, State, Matrix

tetrislogic/consts.py

@@ -10,11 +10,10 @@ NB_NEXT = 5
 # Delays (seconds)
 LOCK_DELAY = 0.5
 FALL_DELAY = 1
-AUTOREPEAT_DELAY = 0.200  # Official : 0.300
-AUTOREPEAT_PERIOD = 0.010  # Official : 0.010
-
+AUTOREPEAT_DELAY = 0.300  # Official : 0.300 s
+AUTOREPEAT_PERIOD = 0.010  # Official : 0.010 s
 
 # Piece init coord
-CURRENT_COORD = Coord(4, NB_LINES)
-NEXT_COORDS = [Coord(NB_COLS + 4, NB_LINES - 4 * n - 3) for n in range(NB_NEXT)]
-HELD_COORD = Coord(-5, NB_LINES - 3)
+MATRIX_PIECE_COORD = Coord(4, NB_LINES)
+NEXT_PIECE_COORDS = [Coord(NB_COLS + 4, NB_LINES - 4 * n - 3) for n in range(NB_NEXT)]
+HELD_PIECE_COORD = Coord(-5, NB_LINES - 3)

tetrislogic/tetrislogic.py

@@ -2,8 +2,8 @@
 import random
 import pickle
 
-from .utils import Coord, Movement, Rotation, T_Spin, Line
-from .tetromino import Tetromino, T, I
+from .utils import Coord, Movement, Rotation, T_Spin
+from .tetromino import Tetromino, T_Tetrimino, I_Tetrimino
 from .consts import (
     NB_LINES,
     NB_COLS,
@@ -12,9 +12,9 @@ from .consts import (
     FALL_DELAY,
     AUTOREPEAT_DELAY,
     AUTOREPEAT_PERIOD,
-    CURRENT_COORD,
-    NEXT_COORDS,
-    HELD_COORD,
+    MATRIX_PIECE_COORD,
+    NEXT_PIECE_COORDS,
+    HELD_PIECE_COORD,
 )
 
 
@@ -30,11 +30,33 @@ class State:
     OVER = "OVER"
 
 
-class Matrix(list):
+class PieceContainer:
+
+    def __init__(self):
+        self.piece = None
+
+
+class HoldQueue(PieceContainer):
+    pass
+
+
+class Matrix(list, PieceContainer):
+
+    def __init__(self, *args, **kargs):
+        list.__init__(self, *args, **kargs)
+        PieceContainer.__init__(self)
+
     def cell_is_free(self, coord):
         return 0 <= coord.x < NB_COLS and 0 <= coord.y and not self[coord.y][coord.x]
 
 
+class NextQueue(PieceContainer):
+
+    def __init__(self):
+        super().__init__()
+        self.pieces = []
+
+
 class TetrisLogic:
 
     NB_LINES = NB_LINES
@@ -44,19 +66,18 @@ class TetrisLogic:
     FALL_DELAY = FALL_DELAY
     AUTOREPEAT_DELAY = AUTOREPEAT_DELAY
     AUTOREPEAT_PERIOD = AUTOREPEAT_PERIOD
-    CURRENT_COORD = CURRENT_COORD
-    NEXT_COORDS = NEXT_COORDS
-    HELD_COORD = HELD_COORD
+    MATRIX_PIECE_COORD = MATRIX_PIECE_COORD
+    NEXT_PIECE_COORDS = NEXT_PIECE_COORDS
+    HELD_PIECE_COORD = HELD_PIECE_COORD
     random_bag = []
 
     def __init__(self):
         self.load_high_score()
         self.state = State.STARTING
+        self.held = HoldQueue()
         self.matrix = Matrix()
-        self.next = []
-        self.current = None
-        self.ghost = None
-        self.held = None
+        self.matrix.ghost = None
+        self.next = NextQueue()
         self.time = 0
         self.autorepeatable_actions = (self.move_left, self.move_right, self.soft_drop)
         self.pressed_actions = []
@@ -88,8 +109,8 @@ class TetrisLogic:
         self.matrix.clear()
         for y in range(self.NB_LINES + 3):
             self.append_new_line_to_matrix()
-        self.next = [self.new_tetromino() for n in range(self.NB_NEXT)]
-        self.held = None
+        self.next.pieces = [self.new_tetromino() for n in range(self.NB_NEXT)]
+        self.held.piece = None
         self.state = State.PLAYING
         self.start(self.update_time, 1)
 
@@ -102,7 +123,7 @@ class TetrisLogic:
         return self.random_bag.pop()()
 
     def append_new_line_to_matrix(self):
-        self.matrix.append(Line(None for x in range(self.NB_COLS)))
+        self.matrix.append([None for x in range(self.NB_COLS)])
 
     def new_level(self):
         self.level += 1
@@ -113,20 +134,19 @@ class TetrisLogic:
             self.lock_delay = 0.5 * pow(0.9, self.level - 15)
         self.show_text("LEVEL\n{:n}".format(self.level))
         self.restart(self.fall, self.fall_delay)
+        self.new_matrix_piece()
 
-        self.new_current()
-
-    def new_current(self):
-        self.current = self.next.pop(0)
-        self.current.coord = self.CURRENT_COORD
-        self.ghost = self.current.ghost()
+    def new_matrix_piece(self):
+        self.matrix.piece = self.next.pieces.pop(0)
+        self.matrix.piece.coord = self.MATRIX_PIECE_COORD
+        self.matrix.ghost = self.matrix.piece.ghost()
         self.move_ghost()
-        self.next.append(self.new_tetromino())
-        self.next[-1].coord = self.NEXT_COORDS[-1]
-        for tetromino, coord in zip(self.next, self.NEXT_COORDS):
+        self.next.pieces.append(self.new_tetromino())
+        self.next.pieces[-1].coord = self.NEXT_PIECE_COORDS[-1]
+        for tetromino, coord in zip(self.next.pieces, self.NEXT_PIECE_COORDS):
             tetromino.coord = coord
 
-        if not self.can_move(self.current.coord, (mino.coord for mino in self.current)):
+        if not self.can_move(self.matrix.piece.coord, (mino.coord for mino in self.matrix.piece)):
             self.game_over()
 
     def move_left(self):
@@ -142,13 +162,11 @@ class TetrisLogic:
         self.rotate(Rotation.COUNTER)
 
     def move_ghost(self):
-        self.ghost.coord = self.current.coord
-        for ghost_mino, current_mino in zip(self.ghost, self.current):
+        self.matrix.ghost.coord = self.matrix.piece.coord
+        for ghost_mino, current_mino in zip(self.matrix.ghost, self.matrix.piece):
             ghost_mino.coord = current_mino.coord
-        while self.can_move(
-            self.ghost.coord + Movement.DOWN, (mino.coord for mino in self.ghost)
-        ):
-            self.ghost.coord += Movement.DOWN
+        while self.can_move(self.matrix.ghost.coord + Movement.DOWN, (mino.coord for mino in self.matrix.ghost)):
+            self.matrix.ghost.coord += Movement.DOWN
 
     def soft_drop(self):
         moved = self.move(Movement.DOWN)
@@ -165,38 +183,33 @@ class TetrisLogic:
         self.move(Movement.DOWN)
 
     def move(self, movement, prelock=True):
-        potential_coord = self.current.coord + movement
-        if self.can_move(potential_coord, (mino.coord for mino in self.current)):
-            if self.current.prelocked:
+        potential_coord = self.matrix.piece.coord + movement
+        if self.can_move(potential_coord, (mino.coord for mino in self.matrix.piece)):
+            if self.matrix.piece.prelocked:
                 self.restart(self.lock, self.lock_delay)
-            self.current.coord = potential_coord
+            self.matrix.piece.coord = potential_coord
             if not movement == Movement.DOWN:
-                self.current.last_rotation_point = None
+                self.matrix.piece.last_rotation_point = None
             self.move_ghost()
             return True
         else:
-            if prelock and not self.current.prelocked and movement == Movement.DOWN:
-                self.current.prelocked = True
+            if prelock and not self.matrix.piece.prelocked and movement == Movement.DOWN:
+                self.matrix.piece.prelocked = True
                 self.start(self.lock, self.lock_delay)
             return False
 
     def rotate(self, rotation):
-        rotated_coords = tuple(
-            Coord(rotation * mino.coord.y, -rotation * mino.coord.x)
-            for mino in self.current
-        )
-        for rotation_point, liberty_degree in enumerate(
-            self.current.SRS[rotation][self.current.orientation], start=1
-        ):
-            potential_coord = self.current.coord + liberty_degree
+        rotated_coords = tuple(Coord(rotation * mino.coord.y, -rotation * mino.coord.x) for mino in self.matrix.piece)
+        for rotation_point, liberty_degree in enumerate(self.matrix.piece.SRS[rotation][self.matrix.piece.orientation], start=1):
+            potential_coord = self.matrix.piece.coord + liberty_degree
             if self.can_move(potential_coord, rotated_coords):
-                if self.current.prelocked:
+                if self.matrix.piece.prelocked:
                     self.restart(self.lock, self.lock_delay)
-                self.current.coord = potential_coord
-                for mino, coord in zip(self.current, rotated_coords):
+                self.matrix.piece.coord = potential_coord
+                for mino, coord in zip(self.matrix.piece, rotated_coords):
                     mino.coord = coord
-                self.current.orientation = (self.current.orientation + rotation) % 4
-                self.current.last_rotation_point = rotation_point
+                self.matrix.piece.orientation = (self.matrix.piece.orientation + rotation) % 4
+                self.matrix.piece.last_rotation_point = rotation_point
                 self.move_ghost()
                 return True
         else:
@@ -211,20 +224,15 @@ class TetrisLogic:
     )
 
     def lock(self):
-        self.current.prelocked = False
+        self.matrix.piece.prelocked = False
         self.stop(self.lock)
 
         # Piece unlocked
-        if self.can_move(
-            self.current.coord + Movement.DOWN, (mino.coord for mino in self.current)
-        ):
+        if self.can_move(self.matrix.piece.coord + Movement.DOWN, (mino.coord for mino in self.matrix.piece)):
             return
 
         # Game over
-        if all(
-            (mino.coord + self.current.coord).y >= self.NB_LINES
-            for mino in self.current
-        ):
+        if all((mino.coord + self.matrix.piece.coord).y >= self.NB_LINES for mino in self.matrix.piece):
             self.game_over()
             return
 
@@ -233,12 +241,12 @@ class TetrisLogic:
             self.restart(self.repeat_action, self.AUTOREPEAT_DELAY)
 
         # T-Spin
-        if type(self.current) == T and self.current.last_rotation_point is not None:
+        if type(self.matrix.piece) == T_Tetrimino and self.matrix.piece.last_rotation_point is not None:
             a = self.is_t_slot(0)
             b = self.is_t_slot(1)
             c = self.is_t_slot(3)
             d = self.is_t_slot(2)
-            if self.current.last_rotation_point == 5 or (a and b and (c or d)):
+            if self.matrix.piece.last_rotation_point == 5 or (a and b and (c or d)):
                 t_spin = T_Spin.T_SPIN
             elif c and d and (a or b):
                 t_spin = T_Spin.MINI
@@ -247,18 +255,14 @@ class TetrisLogic:
         else:
             t_spin = T_Spin.NONE
 
-        for mino in self.current:
-            coord = mino.coord + self.current.coord
-            del mino.coord
-            if coord.y <= self.NB_LINES + 3:
-                self.matrix[coord.y][coord.x] = mino
+        self.enter_the_matrix()
 
         # Clear complete lines
         nb_lines_cleared = 0
         for y, line in reversed(list(enumerate(self.matrix))):
             if all(mino for mino in line):
                 nb_lines_cleared += 1
-                self.matrix.pop(y)
+                self.remove_line(y)
                 self.append_new_line_to_matrix()
         if nb_lines_cleared:
             self.nb_lines_cleared += nb_lines_cleared
@@ -293,41 +297,44 @@ class TetrisLogic:
         if self.goal <= 0:
             self.new_level()
         else:
-            self.new_current()
+            self.new_matrix_piece()
+
+    def enter_the_matrix(self):
+        for mino in self.matrix.piece:
+            coord = mino.coord + self.matrix.piece.coord
+            if coord.y <= self.NB_LINES + 3:
+                self.matrix[coord.y][coord.x] = mino
+
+    def remove_line(self, y):
+        self.matrix.pop(y)
 
     def can_move(self, potential_coord, minoes_coords):
-        return all(
-            self.matrix.cell_is_free(potential_coord + mino_coord)
-            for mino_coord in minoes_coords
-        )
+        return all(self.matrix.cell_is_free(potential_coord + mino_coord) for mino_coord in minoes_coords)
 
     T_SLOT_COORDS = (Coord(-1, 1), Coord(1, 1), Coord(-1, 1), Coord(-1, -1))
 
     def is_t_slot(self, n):
-        t_slot_coord = (
-            self.current.coord + self.T_SLOT_COORDS[(self.current.orientation + n) % 4]
-        )
+        t_slot_coord = self.matrix.piece.coord + self.T_SLOT_COORDS[(self.matrix.piece.orientation + n) % 4]
         return not self.matrix.cell_is_free(t_slot_coord)
 
     def swap(self):
-        if self.current.hold_enabled:
-            self.current.hold_enabled = False
-            self.current.prelocked = False
+        if self.matrix.piece.hold_enabled:
+            self.matrix.piece.hold_enabled = False
+            self.matrix.piece.prelocked = False
             self.stop(self.lock)
-            self.current, self.held = self.held, self.current
-            if type(self.held) == I:
-                self.held.coord = self.HELD_COORD + Movement.LEFT
-            else:
-                self.held.coord = self.HELD_COORD
-            for mino, coord in zip(self.held, self.held.MINOES_COORDS):
+            self.matrix.piece, self.held.piece = self.held.piece, self.matrix.piece
+            self.held.piece.coord = self.HELD_PIECE_COORD
+            if type(self.held.piece) == I_Tetrimino:
+                self.held.piece.coord += Movement.LEFT
+            for mino, coord in zip(self.held.piece, self.held.piece.MINOES_COORDS):
                 mino.coord = coord
 
-            if self.current:
-                self.current.coord = self.CURRENT_COORD
-                self.ghost = self.current.ghost()
+            if self.matrix.piece:
+                self.matrix.piece.coord = self.MATRIX_PIECE_COORD
+                self.matrix.ghost = self.matrix.piece.ghost()
                 self.move_ghost()
             else:
-                self.new_current()
+                self.new_matrix_piece()
 
     def pause(self):
         self.state = State.PAUSED
@@ -339,7 +346,7 @@ class TetrisLogic:
     def resume(self):
         self.state = State.PLAYING
         self.start(self.fall, self.fall_delay)
-        if self.current.prelocked:
+        if self.matrix.piece.prelocked:
             self.start(self.lock, self.lock_delay)
         self.start(self.update_time, 1)
 
@@ -361,7 +368,11 @@ class TetrisLogic:
         if action in self.autorepeatable_actions:
             self.auto_repeat = False
             self.pressed_actions.append(action)
-            self.restart(self.repeat_action, self.AUTOREPEAT_DELAY)
+            if action == self.soft_drop:
+                delay = self.fall_delay / 20
+            else:
+                delay = self.AUTOREPEAT_DELAY
+            self.restart(self.repeat_action, delay)
 
     def repeat_action(self):
         if self.pressed_actions:

tetrislogic/tetromino.py

@@ -1,5 +1,5 @@
 # -*- coding: utf-8 -*-
-from .utils import Coord, Rotation
+from .utils import Coord, Rotation, Color
 
 
 class Mino:
@@ -44,20 +44,20 @@ class Tetromino(list):
         return type(self)()
 
 
-class O(Tetromino, metaclass=MetaTetromino):
+class O_Tetrimino(Tetromino, metaclass=MetaTetromino):
 
     SRS = {
         Rotation.CLOCKWISE: (tuple(), tuple(), tuple(), tuple()),
         Rotation.COUNTER: (tuple(), tuple(), tuple(), tuple()),
     }
     MINOES_COORDS = (Coord(0, 0), Coord(1, 0), Coord(0, 1), Coord(1, 1))
-    MINOES_COLOR = "yellow"
+    MINOES_COLOR = Color.YELLOW
 
     def rotate(self, direction):
         return False
 
 
-class I(Tetromino, metaclass=MetaTetromino):
+class I_Tetrimino(Tetromino, metaclass=MetaTetromino):
 
     SRS = {
         Rotation.CLOCKWISE: (
@@ -74,34 +74,34 @@ class I(Tetromino, metaclass=MetaTetromino):
         ),
     }
     MINOES_COORDS = (Coord(-1, 0), Coord(0, 0), Coord(1, 0), Coord(2, 0))
-    MINOES_COLOR = "cyan"
+    MINOES_COLOR = Color.CYAN
 
 
-class T(Tetromino, metaclass=MetaTetromino):
+class T_Tetrimino(Tetromino, metaclass=MetaTetromino):
 
     MINOES_COORDS = (Coord(-1, 0), Coord(0, 0), Coord(0, 1), Coord(1, 0))
-    MINOES_COLOR = "magenta"
+    MINOES_COLOR = Color.MAGENTA
 
 
-class L(Tetromino, metaclass=MetaTetromino):
+class L_Tetrimino(Tetromino, metaclass=MetaTetromino):
 
     MINOES_COORDS = (Coord(-1, 0), Coord(0, 0), Coord(1, 0), Coord(1, 1))
-    MINOES_COLOR = "orange"
+    MINOES_COLOR = Color.ORANGE
 
 
-class J(Tetromino, metaclass=MetaTetromino):
+class J_Tetrimino(Tetromino, metaclass=MetaTetromino):
 
     MINOES_COORDS = (Coord(-1, 1), Coord(-1, 0), Coord(0, 0), Coord(1, 0))
-    MINOES_COLOR = "blue"
+    MINOES_COLOR = Color.BLUE
 
 
-class S(Tetromino, metaclass=MetaTetromino):
+class S_Tetrimino(Tetromino, metaclass=MetaTetromino):
 
     MINOES_COORDS = (Coord(-1, 0), Coord(0, 0), Coord(0, 1), Coord(1, 1))
-    MINOES_COLOR = "green"
+    MINOES_COLOR = Color.GREEN
 
 
-class Z(Tetromino, metaclass=MetaTetromino):
+class Z_Tetrimino(Tetromino, metaclass=MetaTetromino):
 
     MINOES_COORDS = (Coord(-1, 1), Coord(0, 1), Coord(0, 0), Coord(1, 0))
-    MINOES_COLOR = "red"
+    MINOES_COLOR = Color.RED

tetrislogic/utils.py

@@ -28,5 +28,12 @@ class T_Spin:
     T_SPIN = "T-SPIN"
 
 
-class Line(list):
-    pass
+class Color:
+
+    BLUE = 0
+    CYAN = 1
+    GREEN = 2
+    MAGENTA = 3
+    ORANGE = 4
+    RED = 5
+    YELLOW = 6