The latest articles on Forem by Фёдор Поцелуев (@__6e8fa9e6d9). https://forem.com/__6e8fa9e6d9 https://media2.dev.to/dynamic/image/width=90,height=90,fit=cover,gravity=auto,format=auto/https:%2F%2Fdev-to-uploads.s3.amazonaws.com%2Fuploads%2Fuser%2Fprofile_image%2F2125200%2Fa21c900d-5cdc-4002-897f-1ed4c8cd80c1.jpg https://forem.com/__6e8fa9e6d9 en Фёдор Поцелуев Wed, 25 Sep 2024 14:01:37 +0000 https://forem.com/__6e8fa9e6d9/tetris-game-development-seeking-coding-advice-and-feedback-41l2 https://forem.com/__6e8fa9e6d9/tetris-game-development-seeking-coding-advice-and-feedback-41l2 <p>Hey everyone!</p> <p>I’ve recently completed a Tetris game and am currently focusing on refining some parts of the code. I would be grateful for any general coding advice and feedback on how to improve the structure and readability of my code.</p> <p>In particular, I’ve been concerned about these specific areas:</p> <p>Functions in the <code>conflict_resolution.c</code> module:</p> <ul> <li>The functions <code>side_cells_crossing_prevention</code>, <code>side_boundaries_crossing_</code>, <code>bottom_top_boundaries_crossing_</code> of the <code>conflict_resolution.c</code> module don't seem to be clean and easy to read code. They used to be a single function. Is there a better way to structure these functions for clarity and maintainability?</li> </ul> <p>Duplicated logic in the <code>conflict_resolution.c</code> module:</p> <ul> <li>The <code>conflict_resolution.c</code> module has the functions <code>top_center_conflict</code>, <code>top_right_corner_conflict</code>, <code>right_center_conflict</code>, <code>bottom_right_corner_conflict</code>, <code>bottom_center_conflict</code>, <code>bottom_left_corner_conflict</code>, <code>left_center_conflict</code> and <code>top_left_corner_conflict</code> that seem to have quite a bit of duplicated logic. Additionally, the <code>handle_top_center_conflict</code>, <code>handle_right_center_conflict</code>, <code>handle_bottom_center_conflict</code>, <code>handle_left_center_conflict</code> functions could probably be refactored to reduce repetition. Any advice on how to better consolidate these would be really appreciated!</li> </ul> <p>Here's the link to the full code: <a href="https://github.com/FyodorPotseluev/tetris" rel="noopener noreferrer">https://github.com/FyodorPotseluev/tetris</a>.</p> <p>You're welcome to join the "Discussions" section of the repository.</p> <p>Any thoughts or suggestions would be super helpful! </p> c refactorit help