Get set three dimensional array

Question

1.00/5 (4 votes)

See more:

What should be instead of "value"?

C#

bool[,,] A2;

bool[,,] A
{
  get { return A2[value,value,value]; }

  set  
    {
        if(A2[value,value,value]!=value] 
       {
         progressMade=true;
         A2[value,value,value]=value;
       }
    }
}

Posted 12-Jan-15 12:37pm

john1990_1

Add a Solution

2 solutions

Solution 2

I think this is kind of what you want; if it's not, I'd be curious to hear from you:

C#

public class IndexedArray
{
    public bool[,,] A2 { set; get; }

    public bool progressMade { set; get; }

    public List<int> lastChangedIndexes { set; get; }

    public IndexedArray(int d1, int d2, int d3)
    {
        progressMade = false;
        A2 = new bool[d1,d2,d3];
        lastChangedIndices = new List<int>();
    }

    // the indexer
    public bool this[int x, int y, int z]
    {
        get  { return A2[x, y, z]; }

        set
        {
            progressMade = A2[x, y, z] != value;

            if (progressMade)
            {
                lastChangedIndices .Add(x);
                lastChangedIndices .Add(y);
                lastChangedIndices .Add(z);

                A2[x, y, z] = value;
            }
        }
    }
}

Test:

C#

IndexedArray testArrayProp = new IndexedArray(3, 4, 5);

testArrayProp[1, 2, 3] = true;

bool check = testArrayProp[1, 2, 3];

var changedIndices = testArrayProp.LastChangedIndices;

I would not use this type of solution to the problem of how you manage an Array, and keep track of change ... if that's what you are really after here.

Of course, you can't inherit from 'Array, or use 'Array as a constraint in a Generic Class, because Array is sealed. Depending on the circumstances I would go for using Lists.

At the point you wanted to dynamic notifications of changes, you could implement INotifyPropetyChanged, rather than use the non-dynamic tracking variables used here.

Posted 13-Jan-15 1:09am

BillWoodruff

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

john1990_1 · Accepted Answer · 2015-01-12T12:56:00

Solution 1

C#

class AX
        {
            public bool this[int x, int y, int i]
            {
                get
                {
                    return A2[x, y, i];
                }

                set
                {
                    if (A2[x, y, i])
                        progressMade = true;

                    A2[x, y, i] = value;
                }
            }
        }

AX A = new AX();

static bool[, ,] A2 = new bool[9, 9, 9];

Posted 12-Jan-15 12:56pm

john1990_1

Updated 12-Jan-15 13:33pm

v3

Comments

Sergey Alexandrovich Kryukov 12-Jan-15 19:25pm

5ed.
—SA

john1990_1 12-Jan-15 19:32pm

?

Sergey Alexandrovich Kryukov 12-Jan-15 19:42pm

~~My vote of 5 for the answer.~~ By mistake; I did not look properly. See my last comment.
—SA

BillWoodruff 12-Jan-15 23:30pm

Hi Sergey, this is an answer by the OP to their own question that contains code that won't compile, that contains obvious errors, and code that would do nothing if it did compile. How can you possibly vote it #5 ?

Sergey Alexandrovich Kryukov 12-Jan-15 23:52pm

I was too fast, though that the idea so obvious that it would be impossible to create the code which cannot compile. Thank you for pointing out my mistake.
—SA

BillWoodruff 12-Jan-15 23:28pm

Are you aware your code as is will not compile ? Even if it did compile, the code you show that creates a new instance of 'AX does nothing, and contains nothing, and the second assignment of a boolean array to 'A2 has no relationship with 'AX.

This is not a soluton, and you should remove this, and add this code to your original question before you get more down-votes.

It appears you are experimenting with implementing a Class that uses an indexer, and that's a good thing to learn how to do.

Edit your original question, and try and describe exactly what you want to achieve; I, and, I am sure, other people will assist you, then.

Sergey Alexandrovich Kryukov 12-Jan-15 23:59pm

The class code will compile if you define the members A2 and progressMade. A2 also needs initialization. Role of progressMade is unclear, the logic looks pointless.
—SA

BillWoodruff 13-Jan-15 1:48am

Hi Sergey, and congratulation on being named an MVP again ! In this case it seems to me the code would have to substitute AX for A2 in the class definition to compile, as well as define 'progressMade. Hopefully the OP will clarify what they are after here, and we can assist.

Sergey Alexandrovich Kryukov 13-Jan-15 1:57am

Thank you. You too.
I don't see much sense in the code anyway, so the answer could better show something like
value = //... in setter and
return //... in getter
and refer to the MSDN page on indexed properties.
This is the old but valid tutorial: http://msdn.microsoft.com/en-us/library/aa288464%28v=vs.71%29.aspx.
—SA

john1990_1 13-Jan-15 7:37am

my code works perfectly for me and i'm using it! i initialize the objects in real time... this is for solving sudoku and progressMade is global

Sergey Alexandrovich Kryukov 13-Jan-15 11:04am

Very good. But you are not doing anything "in real time". Probably you meant to say "during run time".
—SA

john1990_1 13-Jan-15 11:23am

yes...

BillWoodruff 13-Jan-15 13:48pm

It's almost there now because the OP edited the version you and I saw.

Sergey Alexandrovich Kryukov 13-Jan-15 13:50pm

Right...
—SA

BillWoodruff 13-Jan-15 13:47pm

Glad to hear your code is working; my comments were based on the first version you posted, not this version which has been corrected, although the declaration of 'progressMade is still omitted.

I, and the other people here, really do want to assist you !

I do suggest you consider using INotifyPropertyChanged here since that facility is designed to do exactly what you are doing here: update when the value of a Property changes. If you want to see a sample of that, just ask.