-
Notifications
You must be signed in to change notification settings - Fork 15
ZU4ZYK #13
base: main
Are you sure you want to change the base?
ZU4ZYK #13
Conversation
if (Board[i, j].State == player.color && | ||
j <= col - 4) | ||
{ | ||
if (Board[i, j + 1].State == player.color && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ez ilyen if-ek tipik lehetnek metódusok mögött, valami olvasható, jó névvel.
if (Board[i, j].State == player.color && | ||
i <= row - 4) | ||
{ | ||
if (Board[i + 1, j].State == player.color && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jap, metódus, mert többször is ismétlődik ugyan az.
{ | ||
public static class GameBoardJsonSerializer | ||
{ | ||
static readonly string fileName = "board.json"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ez lehetetett volna sima const is :)
using System.Text; | ||
|
||
Console.OutputEncoding = Encoding.UTF8; | ||
GameBoard board = new(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kompakt :) Ilyenkor pl lehetne tagolni a részeket üres sorokkal.
@@ -0,0 +1,15 @@ | |||
<Project Sdk="Microsoft.NET.Sdk"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Szép munka.
No description provided.