c# openfiledialog

[Loeri]

Wanneer ik Load klik in mijn menu roep deze een OpenFileDialog op.
Nu wordt deze juist getoond en kan ik een file laden met de volgende code:

Code: Selecteer alles

public int[] LoadFileDataAccess(int[] getallen)
        {
            int[] loadLines;
            loadLines = new int[getallen.Length];
            // Create an instance of the open file dialog box.
            OpenFileDialog openDialog = new OpenFileDialog();

            // Set filter options and filter index.
            openDialog.Filter = "Dat Files (.dat)|*.dat";
            openDialog.FilterIndex = 1;
            openDialog.ShowDialog();


            FileStream openFile = (FileStream)openDialog.OpenFile();
            StreamReader sr = new StreamReader(openFile);

                int x = 0;
                while (!sr.EndOfStream)
                {
                    loadLines[x] = Convert.ToInt32(sr.ReadLine());
                    x += 1;
                }
            
            openFile.Close();
            return loadLines;
        }

Maar waneer ik de OpenFiledialog krijg en direct op Annuleren/Cancel druk krijg ik de exception: {"Index was outside the bounds of the array."} System.Exception {System.IndexOutOfRangeException} bij:

Code: Selecteer alles

FileStream openFile = (FileStream)openDialog.OpenFile();

Iemand een idee hoe ik dit kan fixen?

[tb0ne]

Als ik mij niet vergis helpt het je resultaat in een if else block te steken...

if (dialog.ShowDialog() == System.Windows.Forms.DialogResult.OK)
{
//Do stuff
}
else
{
//Cancelled
}

[gm123]

De luiemans oplossing is het in een try block steken en die exception catchen.

[cptKangaroo]

Na deze lijn moet je ook eerst kijken of openFile wel een object bevat vooraleer er een StreamReader op los te laten:
FileStream openFile = (FileStream)openDialog.OpenFile();

[Loeri]

Bedankt iedereen! Ik denk dat kangaroo de meest logische oplossing eheft gegeven.

Dus ik denk dat het zo is?
Werkt allesinds, thx!

Code: Selecteer alles

if (openDialog.FileName != "")
                {
                FileStream openFile = (FileStream)openDialog.OpenFile();
                StreamReader sr = new StreamReader(openFile);
                int x = 0;
                while (!sr.EndOfStream)
                {
                    loadLines[x] = Convert.ToInt32(sr.ReadLine());
                    x += 1;
                }

                openFile.Close();
                }
            else
                {
                //Cancelled
                }

[cptKangaroo]

Test eens met een foute filename in te geven

Persoonlijk zou ik die OpenFileDialog enkel gebruiken om er een path(s) mee te selecteren, en een CheckFileExists. Het openen van de file kan je dan rechtstreeks met de filestream doen want die dialog is immers van Windows zelf en dus in principe iets minder "managed".

[driesve]

De luiemans oplossing is het in een try block steken en die exception catchen.

Ik hoop dat je zelf weet dat ik absoluut geen goede manier van programmeren is. Het opvangen (en doorgooien) van excepties is zeer traag en nefast voor de "flow" van je programma.

if (dialog.ShowDialog() == System.Windows.Forms.DialogResult.OK)

Dit lijkt mij inderdaad de manier om te kijken of de gebruiker heeft geannuleerd of niet.

if (openDialog.FileName != "")

Ik zou de bovenstaande manier gebruiken en vervolgens de filename controleren met

Code: Selecteer alles

if (String.IsNullOrWhitespace(openDialog.FileName)) {
if (File.Exists(openDialog.FileName)) {
...
}}

[gm123]

De luiemans oplossing is het in een try block steken en die exception catchen.

Ik hoop dat je zelf weet dat ik absoluut geen goede manier van programmeren is. Het opvangen (en doorgooien) van excepties is zeer traag en nefast voor de "flow" van je programma.

Nee, echt?

Voor hobbyprojectjes maakt dit trouwens niets uit.

[Loeri]

Was voor schoolproject
Maar nu kan ik niets meer veranderen, heb al verdedeging gedaan.

[ubremoved_539]

Ik hoop dat je zelf weet dat ik absoluut geen goede manier van programmeren is. Het opvangen (en doorgooien) van excepties is zeer traag en nefast voor de "flow" van je programma.

Oei... dan zijn zowat alle moderne talen slecht ontworpen (al vraagt een exceptie genereren inderdaad wat meer CPU)

Probeer in Java of C# maar eens een deftig stukje code te schrijven zonder... je weet toch dat de compiler je daartoe zelfs verplicht !