Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

ZU4ZYK #13

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

ZU4ZYK #13

wants to merge 2 commits into from

Conversation

Sandorx17
Copy link

No description provided.

if (Board[i, j].State == player.color &&
j <= col - 4)
{
if (Board[i, j + 1].State == player.color &&
Copy link
Contributor

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 &&
Copy link
Contributor

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";
Copy link
Contributor

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();
Copy link
Contributor

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">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Szép munka.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants