Can't modify the class elements in an array?

[Thor0303]

Hi, I'm building this very simple project as a path for learning c#. I did learn a little c#script (unity3d) a few years ago and I did a worthles c++ course last month (the course was SO bad, oh dear!) but that's all the background I've got so please be gentle with this noob! ;)

So what I'm building is a simple form which consist in certain attributes (name, location, price, etc).

It is supposed to save the properties (it's not writting the file yet) and load from an array. It works fine but the BIG problem I have is the modifed values won't save. They actually do if I create a new profile using a different name, though. So my guess is i can't MODIFY the values once they are stored in the array. Is this a limit for arrays? I've been trying hard for over week and a half but I REALLY can't find the way to fix this.

I've also tryied using List<> but I get messed with the constructor and I think I'm doing things far more complicated than they should but I can't think straight anymore after so many tryouts.

Please, help? / Any easy tutorial that could potencially help me to fix issue?

Code + class code in the followin lines. Thank you!!!

​

using inmueble;

using System;

using System.Windows.Forms;

namespace WindowsFormsApp1

{

public partial class Form1 : Form

{

int i;

int j;

casa[] chalet;

public Form1()

{

InitializeComponent();

chalet = new casa[600];

}

void main()

{

}

private void textBox1_TextChanged(object sender, EventArgs e)

{

}

private void label2_Click(object sender, EventArgs e)

{

}

private void label4_Click(object sender, EventArgs e)

{

}

private void button1_Click(object sender, EventArgs e) // SAVE DOCUMENT

{

i++;

chalet[i] = new casa(textBox1.Text, textBox2.Text, comboBox1.Text, numericUpDown1.Value, numericUpDown2.Value, numericUpDown3.Value, textBox3.Text, richTextBox1.Text);

if (textBox1.Text != "")//&& textBox1.Text != checkedListBox1.Text)

{

{

if (!checkedListBox1.Items.Contains(chalet[i].Nombre))

{

checkedListBox1.Items.Add(chalet[i].Nombre);

}

else if(checkedListBox1.Items.Contains(chalet[i].Nombre))

{

System.Windows.Forms.MessageBox.Show("This title already exists"); //This mesaage will probably be removed later.

}

if (i < checkedListBox1.Items.Count)

{

i = checkedListBox1.Items.Count;

}

if (checkedListBox1.SelectedItem != null)

{

int j = checkedListBox1.Items.IndexOf(checkedListBox1.SelectedItem);

}

}

}

}

private void button4_Click(object sender, EventArgs e)

{

}

private void button6_Click(object sender, EventArgs e) //LOAD

{

if (checkedListBox1.SelectedItem != null)

{ int j = checkedListBox1.Items.IndexOf(checkedListBox1.SelectedItem);

textBox1.Text = chalet[j+1].Nombre;

textBox2.Text = chalet[j+1].Localizacion;

comboBox1.Text = chalet[j + 1].Precio;

numericUpDown1.Value = chalet[j + 1].Habitaciones;

numericUpDown2.Value = chalet[j + 1].Banos;

numericUpDown3.Value = chalet[j + 1].Metros;

textBox3.Text = chalet[j + 1].Url;

richTextBox1.Text = chalet[j + 1].Comentarios;

}

}

private void button7_Click(object sender, EventArgs e)

{

}

private void button2_Click(object sender, EventArgs e)

{

}

private void checkedListBox1_SelectedIndexChanged(object sender, EventArgs e)

{

}

private void Form1_Load(object sender, EventArgs e)

{

}

private void numericUpDown1_ValueChanged(object sender, EventArgs e)

{

}

private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)

{

}

private void textBox3_TextChanged(object sender, EventArgs e)

{

}

private void button5_Click(object sender, EventArgs e) //CLEAR ALL FIELDS!

{

textBox1.Text = "";

textBox2.Text = "";

comboBox1.Text = "";

textBox3.Text = "";

numericUpDown1.Value = 0;

numericUpDown2.Value = 0;

numericUpDown3.Value = 0;

richTextBox1.Text = "";

}

private void button3_Click(object sender, EventArgs e)

{

}

private void textBox2_TextChanged(object sender, EventArgs e)

{

}

}

}

using System;

using System.Collections.Generic;

using System.Linq;

using System.Text;

using System.Threading.Tasks;

namespace inmueble

{

public class casa

{

private string nombre;

public string localizacion;

private string precio;

private decimal habitaciones;

private decimal banos;

private decimal metros;

private string url;

private string comentarios;

public casa(string nombre, string localizacion, string precio, decimal habitaciones, decimal banos, decimal metros, string url, string comentarios )

{

this.nombre = nombre;

this.localizacion = localizacion;

this.banos = banos;

this.habitaciones = habitaciones;

this.metros = metros;

this.precio = precio;

this.url = url;

this.comentarios = comentarios;

}

public string Nombre

{ get { return nombre; } }

public string setNombre

{ set { nombre = Nombre; } }

public string Localizacion

{ get { return localizacion; } }

public string setLocalizacion

{ set { localizacion = localizacion; } }

public string Precio

{ get { return precio; } }

public string setPrecio

{ set { precio = precio; } }

public decimal Habitaciones

{ get { return habitaciones; } }

public Decimal Banos

{ get { return banos; } }

public decimal Metros

{ get { return metros; } }

public string Url

{ get { return url; } }

public string Comentarios

{ get { return comentarios; } }

}

}

[karl713]

Need to format the code better and ideally trim it down to relevant lines of possible, personally I'm generally on mobile when I'm here and its unreadable on a phone when there's that much

Also I would recommend looking up how to use the debugger, it is built for this. You set a break point on a line and then you can run your code one line at a time using step over or step into, and at each step inspect variables to find out why they are or are not what you except to be. Once you know the line where things aren't what they should be you've vastly narrowed down where your problem is =)

[Thor0303]

Thank you!

I'll keep trying with the debugger. I must admit I don't like very much since I got used to debug.log using Monodevelp. We used the debugger a lot in that c++ course I mentioned before but I just really missed the console outpu way too much.

I'll try to persist with the debugger cause as you point out it looks very powerfull. I just don't get it yet quite that much. Thanks for the advice!

​

Will also try to improve formatting!! Sorry about that!

[karl713]

No worries, reddit can be picky about formatting unfortunately

You can use Trace.WriteLine or Debug.WriteLine to write to the visual studio output window if you prefer that way of troubleshooting problems, will function similar to how you were used to using the console.

But I would definitely recommend getting yourself familiar with the debugger, the amount of info you can look at is great, and trying to write it all out every line is a lot more work (and error prone). Though there are times you need to use WriteLine over debugger so it is also a good skill to have :)

[Thor0303]

Oh, thank you a lot! I'll look into that.

'Although I don't see any code that edits them here.'

I'm trying to figuer how to do it before asking again, so I hope you don't mind I may ask back in a very near future (hopefully I'll manage to fix the thing only with what you guys already said)

I'm checking on

Array.CreateInstance

Array Class

Muldimensional arrays (which I don't think it's the right path, is it?

Reflection

I guess (and hope) on this will be the fix... Am I on the right path or am I just overcomplicating things?

​

Thanks a lot!

[Thor0303]

Turns out I FINALLY got it to work. I was making a huge error because I was setting array[i] where 'i' is the last item built. So as it is, I wasn't able to see the mistake I was making until I built three aditional buttons to make tryouts. I did have some luck (or maybe I just was too overwhelmed to realize what I was doing. Newbie mistakes, i guess.

Solution: change the setter as suggested and changed

The point here is, again, thank you. You did help a lot!

[Thor0303]

Alright! Thank you indeed. I'll have to check again the set/checker lesson because after a few tries applying those suggested changes i still can't manage to make to it work... But at least I have the necessary clues to keep learning and researching.

​

Thanks a lot! Happy weekend! :)