Using foreach loop for the first time and can't seem to get this right even though it says everywhere online that the code is correct [duplicate]

Question

Dont understand what's the problem here, I have been trying to trouble shoot this for half an hour. First time using foreach loop, and I am getting a cannot convert type 'char' to 'char[]' what does this mean? pretty new to programming.

namespace X_O_spel { class Program { static void Main(string[] args) { char[,] spelBoard = new char[,] { {' ', '|', ' ', '|', ' '}, {'-', '+', '-', '+', '-'}, {' ', '|', ' ', '|', ' '}, {'-', '+', '-', '+', '-'}, {' ', '|', ' ', '|', ' '},}; foreach(char[] row in spelBoard) { foreach(char c in row) { Console.Write(c); } } } } }

Does this answer your question? Multidimensional vs Jagged array — V0ldek
– V0ldek, Commented Dec 23, 2019 at 19:42

Joelius · Accepted Answer · 2019-12-23 19:50:46Z

You're using a multidimensional array when you really want to be using a jagged array. Change char[,] to char[][] and you're code works:

char[][] spelBoard = new[]{ new[]{' ', '|', ' ', '|', ' '}, new[]{'-', '+', '-', '+', '-'}, new[]{' ', '|', ' ', '|', ' '}, new[]{'-', '+', '-', '+', '-'}, new[]{' ', '|', ' ', '|', ' '}}; foreach(char[] row in spelBoard) { foreach(char c in row) { Console.Write(c); } }

More information:

@Khaled Glad I could help. You should upvote and accept the answer to mark your question as solved if it helped you :)

John Lord · Accepted Answer · 2019-12-26 14:55:55Z

if you want to get a char array from your result, i believe you would have an easier time just storing rows of strings.

 string[] spelBoard = new string[] {" | | ", "-+-+-", " | | ", "-+-+-", " | | "}; foreach(string row in spelBoard) { Console.WriteLine(row); }

and of course if you need to, you could still treat row like a char array.

 foreach(char c in row) { Console.Write(c); }

0xSwego · Accepted Answer · 2019-12-23 19:47:50Z

You're using a multidimensional array as if it were a jagged array.

Multidimensional: matrix.

Jagged: Array of arrays.

A jagged array is an array whose elements are arrays. The elements of a jagged array can be of different dimensions and sizes.

You can solve declaring

char[][] spelBoard = new char { new char[] {' ', '|', ' ', '|', ' '}, new char[] {'-', '+', '-', '+', '-'}, ...};

Ashkan Mobayen Khiabani · Accepted Answer · 2019-12-23 19:48:28Z

 { {' ', '|', ' ', '|', ' '}, {'-', '+', '-', '+', '-'}, {' ', '|', ' ', '|', ' '}, {'-', '+', '-', '+', '-'}, {' ', '|', ' ', '|', ' '}}

is not char[,] but it is char[][], so change your code like:

 char[][] spelBoard = new char[][] { {' ', '|', ' ', '|', ' '}, {'-', '+', '-', '+', '-'}, {' ', '|', ' ', '|', ' '}, {'-', '+', '-', '+', '-'}, {' ', '|', ' ', '|', ' '}};

Rafaqat Ali · Accepted Answer · 2019-12-23 19:50:08Z

Here is complete working example:

 class Program { static void Main(string[] args) { char[][] spelBoard = { new[]{' ', '|', ' ', '|', ' '}, new[]{'-', '+', '-', '+', '-'}, new[]{' ', '|', ' ', '|', ' '}, new[]{'-', '+', '-', '+', '-'}, new[]{' ', '|', ' ', '|', ' '}}; foreach (char[] row in spelBoard) { foreach (char c in row) { Console.Write(c); } } Console.ReadLine(); } }

Collectives™ on Stack Overflow

Using foreach loop for the first time and can't seem to get this right even though it says everywhere online that the code is correct [duplicate]

5 Answers 5

2 Comments

3 Comments

1 Comment

1 Comment

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

2 Comments

3 Comments

1 Comment

1 Comment

1 Comment

Linked

Related