Help! Error SetValue - GLOBAL ARRAY it works

Question

Hi I´m trying to create a Global Array but my code have an error every time I try to debug I get this error : Global.SetValue(int int)': not all code paths return a value  This is my code: public Global() { arreglo = new int [3000]; } public static int [] Arreglo { get { return arreglo; } set { arreglo = value ; } } public static int SetValue( int pos int value) { arreglo[pos] = value; } public static int GetValue( int pos) { return arreglo[pos]; } } Can anyone help me with my problem? Thank you very much.

Kelvin Loke · Answer

Finally! You are right Alan! Thank you very much! So this is the GOOD code! : public static class Global { static int [] arreglo = new int [3000]; public static int [] Arreglo { get { return arreglo; } set { arreglo = value ; } } public static void SetValue( int pos int value) { arreglo[pos] = value; } public static int GetValue( int pos) { return arreglo[pos]; } } Thank you again for your help :). Take care

Alan · Answer

Sorry Kelvin I should have spotted that one in my first post. That line should be: static int[] arreglo = new int[3000]; Also the return type of the SetValue() method should be void rather than int. To answer your question Brandon the general advice is to use a get/set property plus a private/protected field rather than a public or internal field if: 1. The field can only be set to certain values and not any value which its type allows. 2. Some other subsidiary code needs to be executed when the value is set or accessed. 3. Even though neither 1. or 2. applies currently you think they might conceivably do so in the future. The reason for 3. is that if you change a public field to a public property at a later date it changes the interface of the containing class. A lot of programmers never use public read/write fields though I occasionally use them myself if the above criteria are satisfied.

Brandon Lewis · Answer

Sure thing. C# took global variables out due to the fact that they were unsafe because they could be accessed by code anywhere in the program. the closest thing Ive found is declaring the variable as an internal variable and sticking it in the Main() class so it can be used by all code within the main class. Good luck! Brandon

Kelvin Loke · Answer

Thanks for your help Brandon.. I can´t believe that creating a Global Array can be so difficult.. It shouldn´t be this way... I´ll continue tryin´ If I found the way to make it work I´ll report back thank you again to both of you. Regards :)

Brandon Lewis · Answer

Maybe if you fix the array issue and declare it as int[] arreglo; Im not sure... Sometimes if you tinker with it enough you can get it to work. Try taking class out of the public static class Global see if that helps.

Kelvin Loke · Answer

Hi I´ve tryed that one but it doesn´t work I still get the error: Invalid token ';' in class struct or interface member declaration Invalid token '=' in class struct or interface member declaration arreglo = new int [3000]; I also get this error: Array size cannot be specified in a variable declaration (try initializing with a 'new' expression)  but this error is easy to fix.. so.. any other idea? thank you!

Brandon Lewis · Answer

Or if all of your code is contained within the Main() class just put internal int arreglo = new int [3000]; right after you declare the Main() class. I hate not having a compiler here at work because I cant test these things out :( Brandon EDIT: Also Alan I have a question. When would it be applicable to use get and set? Ive never used them in anything Ive done and Ive built a few pretty hefty applications... I tend to only learn something new if I have to use it. Like writing reading and searching XML files.

Alan · Answer

It's not actually possible to create a global array in C# - it has to be inside a class or struct. So I'd try this: public static class Global { arreglo = new int [3000]; public static int [] Arreglo { get { return arreglo; } set { arreglo = value ; } } public static int SetValue( int pos int value) { arreglo[pos] = value; } public static int GetValue( int pos) { return arreglo[pos]; } }

Kelvin Loke · Answer

Hi thank you for your help I´ve tryed what yo said to me but now I get this error: Invalid token 'return' in class struct or interface member declaration Any idea? thank you for your time :).

Brandon Lewis · Answer

Not all code paths return a value means that somewhere in your code there is an accessible sequence that wont return a value and would be a dead end sequence. Try putting a return 0; statement between your last two block brackets and that should take care of it. public Global() { arreglo = new int[3000]; } public static int [] Arreglo { get { return arreglo; } set { arreglo = value; } } public static int SetValue(int pos int value) { arreglo[pos] = value; } public static int GetValue(int pos) { return arreglo[pos]; } //Here that way in case it skips int [] Arreglo itll still return some sort of a value return 0; } PS. Im at work right now so I cant test it out but at least you have an idea of whats wrong good luck! Brandon

Help! Error SetValue - GLOBAL ARRAY, it works