C# Is there a compact way to get more than one coordinate in a 2d-array?

Question

Trying to make connect4 and I want to check if a column (vertical) is full, so the easy way to do it would be in an if with [i, 1] != O && [i, 2] != O && [i, 3] != O et cetera, but isn't there a more efficient way to go on about this?

A board looks like:

O O O O O O O O
O O O O O O O O
O O O O O O O O
O O O O O O O O
O O O O O O O O
O O O O O O O O

The for-loop:

for (int i = row - 1; i > -1; i--)
{
     if (board[i, column].ToString() == "O" && beurt % 2 == 0)
     {
         board[i, column] = (Veld)player1.color;
         beurt += 1;
         break;
     }
     else if(board[i, column].ToString() == "O" && beurt % 2 == 1)
     {
             board[i, column] = (Veld)player2.color;
             beurt += 1;
             break;
     }
     else if(???)
     {
             //???
     }
}

Is the first row on the bottom?

John Alexiou
– John Alexiou

2019-03-25 19:45:49 +00:00
Commented Mar 25, 2019 at 19:45 — John Alexiou
– John Alexiou, Commented Mar 25, 2019 at 19:45

Ashkan Mobayen Khiabani · Accepted Answer · 2019-03-25 14:42:43Z

4

bool isRowFull(char[,] array, int row)
{
    for(int i=0;i<array.GetLength(0); i++)
        if(array[row, i] == 'O') return false;
    return true;
}

And you can call it like:

bool isFull = isRowFull(board, 3); // check if row 3 is full

or

if(isRowFull(board, 3))
{
    //your code
}

To Get First NonFull row:

int FirstNonFull(char[,] array)
{
    for(int i=0;i<array.GetLength(1);i++)
        if(!isRowFull(array, i)) return i;
    return -1; // -1 indicating that all rows are full (not found)
}

edited Mar 25, 2019 at 14:42

answered Mar 25, 2019 at 14:14

Ashkan Mobayen Khiabani

34.3k35 gold badges111 silver badges187 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

zhrgci Over a year ago

You don't need any { } on the for-loop or if? Sorry I'm kinda new to this.

Broots Waymb Over a year ago

@Zheng-rongCai - Not if it's only one statement. I still put them a lot of the time for consistency/readability, but technically you don't need to here. It also gets confusing incredibly quickly if you nest multiple loops/ifs and don't use brackets...

Ashkan Mobayen Khiabani Over a year ago

as there is only one line of code inside for, its not necessary to have {} (but you could). if there were more lines of code {} were necessary

Ashkan Mobayen Khiabani Over a year ago

note that return true; is outside of for loop, meaning that if function didn't return anything inside for loop (no zeros exist) and reached this far, then it is full so return true

Idle_Mind Over a year ago

You should always put the brackets in, even for single lines. I see it all the time with my students; they have a single line, which works fine initially, but then later they decide to add more code and can't figure out where the bug is...

|

littlerufe · Accepted Answer · 2019-03-25 14:24:39Z

2

Similaraly you could use Linq to verify that All items at row equal O:

public static bool IsRowEmpty(char[,]board, int row)
{
    return Enumerable.Range(0, board.GetUpperBound(1)).All(col => board[row, col] == 'O');
}

answered Mar 25, 2019 at 14:24

littlerufe

37.2k5 gold badges33 silver badges48 bronze badges

Collectives™ on Stack Overflow

C# Is there a compact way to get more than one coordinate in a 2d-array?

2 Answers 2

9 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

9 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related