Question

I can’t make my function work to copy my listview from the previous form.

Form1 Code:

private void customerInformationToolStripMenuItem_Click(object sender, EventArgs e)
    {
        if (Customers.SelectedItems.Count != 0)
        {

            var myformmessagedialog = new MessageBoxForm
            {
                name = Customers.SelectedItems[0].SubItems[0].Text,
                address = Customers.SelectedItems[0].SubItems[3].Text,
                telephone = Customers.SelectedItems[0].SubItems[4].Text,
            };
            myformmessagedialog.LoadListView(Customers.Items);
            myformmessagedialog.ShowDialog();
        }

    }

Form2 (Messageboxform) code:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace ClaudsPizzeria
{
public partial class MessageBoxForm : Form
{
    public MessageBoxForm()
    {
        InitializeComponent();
    }


    public string name;
    public string address;
    public string telephone;

    public void LoadListView(System.Windows.Forms.ListView.ListViewItemCollection          items)
    {
        orderListView.Clear();
        orderListView.AddRange(items);
    }

    private void MessageBoxForm_Load(object sender, EventArgs e)
    {

        lblName.Text = name;
        lbladdress.Text = address;
        lbltelephone.Text = telephone;
    }
}
}

ERROR I GET:

Error   1   'System.Windows.Forms.ListView' does not contain a definition for '      AddRange' and no extension method 'AddRange' accepting a first argument of type 'System.Windows.Forms.ListView' could be found (are you missing a using directive or an assembly reference?)   C:\Users\Claudiu\Documents\Visual Studio 2010\Projects\ClaudsPizzeria\ClaudsPizzeria\MessageBoxForm.cs  27  27  ClaudsPizzeria

All I want to do is copy my listview from my main form to my messagebox form and I get that. I’m new to c# so I’m sorry if I ask stupid questions.

Answer 1

I believe you meant:

orderListView.Items.Clear();
foreach(ListViewItem x in items) {
    orderListView.Items.Add(x);
}

You access the items of a ListView through the Items property. You may also want to wrap it in orderListView.BeginUpdate() and orderListView.EndUpdate() to reduce flickering.

Edit: If you’re displaying them in another ListView, you need to clone them. So:

orderListView.Items.Clear();
foreach(ListViewItem x in items) {
    orderListView.Items.Add((ListViewItem) x.Clone);
}