There's a lot to address here. First and foremost, DO NOT post code just as a picture. There may be cases where a screenshot can add value but, even then, you should ALWAYS post the code as text and formatted as code. We can't copy and paste code to quote it or test it for ourselves from a screenshot and they are also hard to view on mobile devices.
I'll ignore a number of the programming faux pas there as this is probably an assigned exercise and you can't do things differently. The reason your code is not working is that you are using two different arrays. You create one at the class level, then create another array in the first event handler and populate it, then use the original array to populate the
ListBox. You didn't put anything in that array so of course you have nothing to put in the
ListBox.
Now I want to copy some of your code to show you what you did wrong and I can't because you posted a picture. Let that be the last time. This:
Dim My_Original() As String = {}
will work but has issues. Firstly, don't use
Dim to declare member variables. ALWAYS explicitly specify an access level for everything and that should pretty much ALWAYS be
Private for fields. Secondly, you're assigning an empty array to the field that you will never use. Don't create objects you don't need. If you're going to assign an object to the variable later, don't assign anything to it when you declare it. Finally, it's a smaller thing but the current recommendation is to put the parentheses on the type rather than the variable when declaring an array variable. Only put the parentheses on the variable if you are initialising the variable by specifying an upper bound. With all that in mind, your code should be this:
Private My_Original As String()
That variable name is terrible too. It doesn't describe what the variable is for and, while you can use whatever naming conventions you want, there's rarely a good reason to not use the recommended camel casing, e.g.
Private userInputValues As String()
The real issue is here:
Dim My_original(elems - 1) As String
That is declaring a new local variable and assigning an array of the specified size to it. Why would you want another variable? You already have a variable at the class level. That's where you should be assigning the new array object you just created:
My_Original = New String(elems - 1) {}
You then populate the array assigned to that field and thus you'll get the input back from the array to add to the
ListBox.
I think I just realised why you were pointlessly assigning an empty array to the field when you declared it. Without that, you'd get an exception when you tried to get the data from it to populate the
ListBox. If that's the case, that should have been a clue that you weren't actually setting that field in the first event handler. You avoided the worked arounf the problem of the exception without solving the actual problem that caused it.
