What's wrong with my multidimensional - array declaration? C#

Question

private int[, ,] table = new int[4 , 5 , 5]{
    {{0,6,2,6,4},{2,2,4,2,8},{4,4,8,4,6},{6,6,2,6,4},{8,8,6,8,2}},
    {{0,2,8,8,4},{2,4,6,6,8},{4,8,2,2,6},{6,2,8,8,4},{8,6,4,4,2}},
    {{0,4,2,4,4},{2,8,4,8,8},{4,6,8,6,6},{6,4,2,4,4},{8,2,6,2,2}},
    {{0,8,8,2,4},{2,6,6,4,8},{4,2,2,8,6},{6,8,8,2,4},{8,4,4,6,2}}
};

I want this table:

k|l     0      1       2      3      4

0       06264  22428  44846  66264  88682
1       02884  24668  48226  62884  86442
2       04244  28488  46866  64244  82622
3       08824  26648  42286  68824  84462

thanks for help

just new int[4 , 5] or did i miss something :>?

Tyzak
– Tyzak

2010-01-30 11:11:22 +00:00
Commented Jan 30, 2010 at 11:11 — Tyzak
– Tyzak, Commented Jan 30, 2010 at 11:11
yup, it needs to be 3dimensional

Johannes Rudolph
– Johannes Rudolph

2010-01-30 11:11:55 +00:00
Commented Jan 30, 2010 at 11:11 — Johannes Rudolph
– Johannes Rudolph, Commented Jan 30, 2010 at 11:11

Peter Lillevold · Accepted Answer · 2010-01-30 11:18:44Z

2

There's nothing wrong with your declaration. It compiles fine and I can write something like this:

        for (int i = 0; i < 4; i++)
        {
            for (int j = 0; j < 5; j++)
            {
                for (int k = 0; k < 5; k++)
                {
                    var x = table[i, j, k];
                }
            }
        }

Update: sure you can shorten it a tiny bit. Since the array dimensions are given by your initial values, you can write it like this:

        private int[,,] table = new [,,]{
             <your numbers go here>
        };

answered Jan 30, 2010 at 11:18

Peter Lillevold

34k7 gold badges102 silver badges131 bronze badges

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

2 Comments

user262426 Over a year ago

Thanks, seems like a copy from VS to this post and then back again fixed the problem. :) (didn't actually change anything though.. =) )

Yvo Over a year ago

Now that we have Linq, I'd like to use Enumerable.Range. Perhaps nice for those who don't know it: msdn.microsoft.com/en-us/library/…

Ali Tarhini · Accepted Answer · 2010-01-30 11:14:12Z

0

you need a 2 dimensional array, its [4,5] array

answered Jan 30, 2010 at 11:14

Ali Tarhini

5,3686 gold badges45 silver badges66 bronze badges

Comments

Tomas Vana · Accepted Answer · 2010-01-30 11:14:17Z

0

Try skipping the new int[4 , 5 , 5] part, the compiler should be able to find out the dimensions. Otherwise I don't see any problem.

answered Jan 30, 2010 at 11:14

Tomas Vana

19k9 gold badges57 silver badges67 bronze badges

Comments

silk · Accepted Answer · 2010-01-30 11:19:29Z

0

Nothing is wrong. It works flawlessly for me. What problem exactly you have?

answered Jan 30, 2010 at 11:19

silk

2,73522 silver badges21 bronze badges

Comments

Mark Byers · Accepted Answer · 2010-01-30 11:23:11Z

The compiler can figure out the length of each dimension for you, so let it do all the hard work:

private int[,,] table = new int[,,]{
    {{0,6,2,6,4},{2,2,4,2,8},{4,4,8,4,6},{6,6,2,6,4},{8,8,6,8,2}},
    {{0,2,8,8,4},{2,4,6,6,8},{4,8,2,2,6},{6,2,8,8,4},{8,6,4,4,2}},
    {{0,4,2,4,4},{2,8,4,8,8},{4,6,8,6,6},{6,4,2,4,4},{8,2,6,2,2}},
    {{0,8,8,2,4},{2,6,6,4,8},{4,2,2,8,6},{6,8,8,2,4},{8,4,4,6,2}}
};

And this gives you the output you want, apart from the header:

    StringBuilder stringBuilder = new StringBuilder();

    for (int row = 0; row < table.GetLength(0); ++row)
    {
        stringBuilder.Append(row.ToString() + "\t");
        for (int column = 0; column < table.GetLength(1); ++column)
        {
            for (int valueIndex = 0; valueIndex < table.GetLength(2); ++valueIndex)
            {
                stringBuilder.Append(table[row, column, valueIndex].ToString());
            }
            stringBuilder.Append("\t");
        }
        stringBuilder.AppendLine();
    }

    string result = stringBuilder.ToString();
    Console.WriteLine(result);

Hannoun Yassir · Accepted Answer · 2010-01-30 11:24:03Z

0

Just change new int[4 , 5 , 5] with new[,,]

this is what you should have :

var table = new[,,]
            {
                {{0, 6, 2, 6, 4}, {2, 2, 4, 2, 8}, {4, 4, 8, 4, 6}, {6, 6, 2, 6, 4}, {8, 8, 6, 8, 2}},
                {{0, 2, 8, 8, 4}, {2, 4, 6, 6, 8}, {4, 8, 2, 2, 6}, {6, 2, 8, 8, 4}, {8, 6, 4, 4, 2}},
                {{0, 4, 2, 4, 4}, {2, 8, 4, 8, 8}, {4, 6, 8, 6, 6}, {6, 4, 2, 4, 4}, {8, 2, 6, 2, 2}},
                {{0, 8, 8, 2, 4}, {2, 6, 6, 4, 8}, {4, 2, 2, 8, 6}, {6, 8, 8, 2, 4}, {8, 4, 4, 6, 2}}
            };

edited Jan 30, 2010 at 11:24

answered Jan 30, 2010 at 11:18

Hannoun Yassir

21.4k23 gold badges80 silver badges114 bronze badges

Comments

slugster · Accepted Answer · 2010-01-30 11:28:10Z

0

That piece of code won't compile if you have it in the wrong place - if you have it inside a function then drop the private modifier. (If it is declared at class level then it is fine).

answered Jan 30, 2010 at 11:28

slugster

50k14 gold badges105 silver badges150 bronze badges

Collectives™ on Stack Overflow

What's wrong with my multidimensional - array declaration? C#

7 Answers 7

2 Comments

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

7 Answers 7

2 Comments

Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related