Filling 2D array of strings with data and then adding it to ListView

Question

I'm trying to fill my listview in Windows forms with some sample data. I have a List of transactions whose properties are like following:

        public float NetAmount { get; set; }
        public DateTime TimeStamp { get; set; }
        public string TransactionID { get; set; }

This is how the so far sample data has been put to the listview like following:

 private void seedListView(List<Transactions> list)
        {
            List<string[]> _nesto = new List<string[]>();


            //Define
            var data = new[]
            {

                new []{"Lollipop", "392", "0.2"},
                new []{"KitKat", "518", "26.0"},
                new []{"Ice cream sandwich", "237", "9.0"},
                new []{"Jelly Bean", "375", "0.0"},
                new []{"Honeycomb", "408", "3.2"}
            };

            //Add
            foreach (string[] version in data)
            {
                var item = new ListViewItem(version);
                materialListView1.Items.Add(item);
            }
        }

With the data as data source for the listview I get the following output:

Lollipop  392 0.2
KitKat    518 26.2
HoneyComb 408 3.2

Now I need to do the same, but instead of that I'd like to have the listview filled with the items from the list that I passed into the function like this:

15,2 2016-02-01 125215FESSEFOKP

19.2 2016-02-01 125215FESSEFOKP

13.6 2016-02-01 125215FESSEFOKP

15.9 2016-02-01 125215FESSEFOKP

Where 15.2 is net amount , 2016-02-01 is the timestamp and 125215FESSEFOKP is TransactionID...

How could I do this dynamically ?

Edit: I've did it like this:

List<string[][]> _nesto = new List<string[][]>();
    string[][] data = null;
    foreach (var item in list)
    {
        data = new[] { new[] { item.NetAmount.ToString(), item.TimeStamp.ToString(), item.TransactionID.ToString() } };
        _nesto.Add(data);
    }
    ListViewItem it;
    foreach (var item in _nesto)
    {
        foreach (string[] version in item)
        {
            it = new ListViewItem(version);
            materialListView1.Items.Add(it);
        }
    }

Does anyone have a more cleaner solution than this one? This seems far too much code that I wrote for a simple thing like this :D

Instead of using string[] use object[] and that should do it. — Auguste
– Auguste, Commented Dec 9, 2016 at 18:17
see my answer below. creating this as arrays is a poor design choice in my opinion. this is why there are classes in programming. it represents an "object" and that is exactly what you are doing here with a transaction. it will make managing your code and making changes to it much easier versus dealing with it as an array — peterpep
– peterpep, Commented Dec 9, 2016 at 18:25

Olivier Jacot-Descombes · Accepted Answer · 2016-12-09 18:26:20Z

1

I would do it like this

foreach (Transaction t  in list)
{
    var item = new ListViewItem(new string[]{ 
        t.NetAmount.ToString("N1"),
        t.TimeStamp.ToString("d"),
        t.TransactionID });
    materialListView1.Items.Add(item);
}

edited Dec 9, 2016 at 18:26

answered Dec 9, 2016 at 18:23

Olivier Jacot-Descombes

114k14 gold badges149 silver badges202 bronze badges

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

4 Comments

User987 Over a year ago

hahahahah an overkill for my method! Thanks man! P.S. Take a look into my method that I wrote up there, yours is far far better !! :) Accepted ur answe as it is simply the best one !!

peterpep Over a year ago

i see you have a transaction class (i missed that when i posted my answer below). you can basically ditch several of those lines by just adding that to an override ToString in your class. and just do t.ToString (assuming the format you want to present it in is always the same)

Olivier Jacot-Descombes Over a year ago

Yes, but in a listview you want to add the different properties to different columns. You won't get that with ToString. Note It's a ListView, not a ListBox.

User987 Over a year ago

@OlivierJacot-Descombes done , It didn't allow me to accept right away. Cheers mate ! :)

peterpep · Accepted Answer · 2016-12-09 18:22:45Z

1

I think this is the way to do it:

Create a class called transaction. it will have properties netamount, transaction id, timestamp. Create an override ToString method so that it will output the data how you want (as per your question).
Have a list that will house all those transaction objects.
You set the listview.ItemsSource equal to that list.
listview.items.refresh();

it should populate your listview accordingly. this is the preferred way (at least to me) because you can just add and remove items from the list and you do not need to constantly be running foreach statements to make sure the data is correct in the listview.

answered Dec 9, 2016 at 18:22

peterpep

3243 silver badges11 bronze badges

2 Comments

peterpep Over a year ago

@User987 i just want to point out a quick issue with the way you are dealing with your problem. by having a list of transactions and not binding it (via ItemsSource) to the listView. it will make handling the objects much more difficult. For example, if you add more methods to your transaction class or need to handle the transaction object, you can not do that directly from the listview you are using since you are adding it as a string array. Whereas if you use ItemsSource and override the ToString, you will have access to the actual transaction object and can use it. not sure if this is clear

peterpep Over a year ago

@User987 For example, when you remove an string[] from your listview its going to still be in your list<transaction>. its going to get messy quickly

Auguste · Accepted Answer · 2016-12-09 18:30:22Z

1

Here's what I mean:

         object[] data = new object[]
           {
            new object []{15.2 , 2016-02-01, "125215FESSEFOKP"},
            new object []{19.2, 2016-02-01, "125215FESSEFOKP"},
            new object []{19.2, 2016-02-01, "125215FESSEFOKP"},
            new object []{19.2, 2016-02-01, "125215FESSEFOKP"}
         };

    You won't be able to use anonymous type in this case.

answered Dec 9, 2016 at 18:30

Auguste

2,2092 gold badges22 silver badges28 bronze badges

Collectives™ on Stack Overflow

Filling 2D array of strings with data and then adding it to ListView

3 Answers 3

4 Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related