[C#] ListView..Items[0].Selected probleempje

[Rarz]

Ben bezig een Winform applicatie te maken waarin je door je drive/directory structuur kan stappen en verschillende type mediafiles af te spelen via WMP. Het is verre van klaar.

Ik heb een probleem met het zetten van de selected item in een ListView. Bij default is er niets geselecteerd als de control verschijnt, en ik dacht, ach het is wel handig om de bovenste vast te selecteren. Kunnen de gebruikers zien dat er iets te selecteren valt.

Effe gekeken hoe het moet, dit zou het moeten doen:

C#:

lwFilesAndFolders.Items[0].Selected = true;
lwFilesAndFolders.Select();

En het lijkt ook te werken. Scherm komt naar voren, eerrste optie (C:\) is geselecteerd. Cursortoetsen werken zoals verwacht (up/down). Met enter stap je naar een andere drive of in een directory.

Control wordt opnieuw opgebouwd, en de eerste optie is wederom geselecteerd, zoals verwacht. Prima. Echter het probleem is dat je nu twee keer op cursor down moet drukken om de selectie naar beneden te krijgen. Daarna is het weer gewoon zoals verwacht...? Google heeft niets opgeleverd behalve een dozijn keren de verwijzing naar ListView.Select() (die er al in staat -- regel 108-109).

Ik heb de form code maar niet meegepast, die is niet zo spannend.

Het lijkt erop dat de selectie niet aankomt bij verdere navigatie. De control rendert het geselecteerde Item wel, maar selecteerd 'm niet echt. Dus de eerste keer dat naar beneden gaat wordt het geselecteerde item daadwerkelijk geselecteerd (wat onzichtbaar is voor de gebruiker) en de tweede keer naar beneden verplaatst de selectie zich echt.

Iemand suggesties hoe ik dit kan verhelpen?

C#:

namespace DirViewer
{
    public partial class Form1 : Form
    {
        private StringCollection folderCol;
        private ImageList imageList;
        private ListView lwFilesAndFolders;
        private Label lblCurrentPath;

        public Form1()
        {
            InitializeComponent();

            // Init ListView and folder collection
            folderCol = new StringCollection();
            PaintListView(@"C:\");
            folderCol.Add(@"C:\");

            axWindowsMediaPlayer1.uiMode = "none";
        }

        private void PaintListView(string root)
        {
            try
            {
                ListViewItem lvi;
                ListViewItem.ListViewSubItem lvsi;

                this.lblCurrentPath.Text = root;

                DriveInfo[] allDrives = DriveInfo.GetDrives();
                DirectoryInfo dir = new DirectoryInfo(root);
                DirectoryInfo[] dirs = dir.GetDirectories();
                FileInfo[] files = dir.GetFiles();

                this.lwFilesAndFolders.Items.Clear();
                
                this.lwFilesAndFolders.BeginUpdate();

                // Add available drives
                foreach (DriveInfo di in allDrives)
                {
                    if (di.IsReady)
                    {
                        lvi = new ListViewItem();
                        lvi.ForeColor = Color.Gray;
                        lvi.Text = di.Name;
                        lvi.Tag = di.Name;
                        lvi.ImageIndex = 1;

                        this.lwFilesAndFolders.Items.Add(lvi);
                    }
                }
                
                // Add parent of current directory
                if (root != dir.Root.Name)
                {
                    lvi = new ListViewItem();
                    lvi.ForeColor = Color.Gray;
                    lvi.Text = "[up one level]";
                    lvi.Tag = dir.Parent.FullName;

                    this.lwFilesAndFolders.Items.Add(lvi);
                }

                // Add directories
                foreach (DirectoryInfo di in dirs)
                {
                    if (!di.Attributes.ToString().Contains(FileAttributes.System.ToString()) &&
                        !di.Attributes.ToString().Contains(FileAttributes.Hidden.ToString())
                        )
                    {
                        lvi = new ListViewItem();
                        lvi.ForeColor = Color.Gray;
                        lvi.Text = di.Name;
                        lvi.ImageIndex = 0;
                        lvi.Tag = di.FullName;

                        this.lwFilesAndFolders.Items.Add(lvi);
                    }
                }

                // Add files
                foreach (FileInfo fi in files)
                {
                    lvi = new ListViewItem();
                    lvi.Text = fi.Name.Replace("_", " ");
                    lvi.ForeColor = Color.LightGray;
                    lvi.Tag = fi.FullName;

                    this.lwFilesAndFolders.Items.Add(lvi);
                }
                this.lwFilesAndFolders.EndUpdate();

                if (lwFilesAndFolders.SelectedItems.Count == 0 && lwFilesAndFolders.Items.Count > 0)
                {
                    lwFilesAndFolders.Items[0].Selected = true;
                    lwFilesAndFolders.Select();
                }

            }
            catch (System.Exception err)
            {
                MessageBox.Show("Error: " + err.Message);
            }

            this.lwFilesAndFolders.View = View.Details;
        }

        private void lwFilesAndFolders_ItemActivate(object sender, EventArgs e)
        {
            ListView lw = (ListView)sender;
            if (lw.SelectedItems.Count > 0)
            {
                string filename = lw.SelectedItems[0].Tag.ToString();

                axWindowsMediaPlayer1.Ctlcontrols.stop();

                DirectoryInfo di = new DirectoryInfo(filename);

                if (!di.Exists) // Not a directory, so it probably is a media file -- play
                {
                    axWindowsMediaPlayer1.Ctlcontrols.play();
                    axWindowsMediaPlayer1.fullScreen = true;
                }
                else
                {
                    PaintListView(filename);
                    folderCol.Add(filename);
                }
            }
        }

        private void lwFilesAndFolders_SelectedIndexChanged(object sender, EventArgs e)
        {
            ListView lw = (ListView)sender;

            if (lw.SelectedItems.Count == 1) // Event is triggered twice, make sure we get the one -with- the SelectedItem
            {
                FileInfo fi = new FileInfo(lw.SelectedItems[0].Tag.ToString());
                lblFileName.Text = "";
                if (Library.isValidFiletype(fi.Name))
                {
    // WMP stuff
                }
            }
        }
    }
}

[ben15243]

Ik neem aan dat het Control opnieuw wordt opgebouwd op het moment dat er een directory (en geen bestand) wordt geselecteerd.

Tijdens het opbouwen wordt de ListView leeggehaald (met Items.Clear()) en weer gevuld. Volgens mij verlies je tussen die twee acties de focus van het control. Het lijkt er dan op dat je de focus vervolgens terug krijgt door de eerste keer op cursor down te drukken.

Lang verhaal kort: probeer eens de focus op het control te zetten (met .Focus()) nadat je het SelectedItem hebt ingesteld.

[mindcrash]

Daarnaast heb je de Select() method niet nodig volgens mij.

Het volgende zou genoeg moeten zijn om het eerste item in de listview te selecteren:

C#:

listView.TopItem.Selected = true

En dan heb je volgens mij idd zoals de bovenbuurman zegt nog een extra regeltje code nodig om de focus daarop te krijgen om zo via het toetsenbord door de lijst te kunnen stappen.

[ Voor 30% gewijzigd door mindcrash op 13-05-2009 22:07 ]

[Rarz]

Bedankt, dat werkt.

C#:

lwFilesAndFolders.TopItem.Selected = true;
lwFilesAndFolders.TopItem.Focused = true;

Het is duidelijker en makkelijker dan handmatig door ListView.Items[] te lopen en de selected te zetten.

Doet precies waar ik naar zocht.