Problem on sorting an array

Question

I have problem in sorting an array in ascending form and i don't know how to fix it.

string[] filePath = Directory.GetFiles(fbdialog.SelectedPath.ToString(), "*", SearchOption.AllDirectories);
Array.Sort(filePath);

Here are the values of an filePath.

"C:\\Documents and Settings\\20110909\\DAR-AAP070127-20110909.ods"
"C:\\Documents and Settings\\20110909\\DAR-ALA061452-09050909.xls"
"C:\\Documents and Settings\\20110819\\DAR-AAP070127-20110819.xls"

I want to look like this..

"C:\\Documents and Settings\\20110909\\DAR-AAP070127-20110909.ods"
"C:\\Documents and Settings\\20110819\\DAR-AAP070127-20110819.xls"
"C:\\Documents and Settings\\20110909\\DAR-ALA061452-09050909.xls"

Thanks in Advance.

Can you explain the logic behind your order?

Snowbear
– Snowbear

2011-09-23 09:27:49 +00:00
Commented Sep 23, 2011 at 9:27 — Snowbear
– Snowbear, Commented Sep 23, 2011 at 9:27
@Snowbear I think by filename

V4Vendetta
– V4Vendetta

2011-09-23 09:29:43 +00:00
Commented Sep 23, 2011 at 9:29 — V4Vendetta
– V4Vendetta, Commented Sep 23, 2011 at 9:29
I want to sort it by EmpNo(AAP070127).

Sam Casil
– Sam Casil

2011-09-23 09:40:10 +00:00
Commented Sep 23, 2011 at 9:40 — Sam Casil
– Sam Casil, Commented Sep 23, 2011 at 9:40

Tim Lloyd · Accepted Answer · 2011-09-23 10:05:06Z

1

To sort by "EmpNo" e.g. "AAP070127":

string[] sortedFiles = Directory
    .GetFiles(fbdialog.SelectedPath, "*", SearchOption.AllDirectories)
    .OrderBy(n => Path.GetFileName(n).Split('-')[1])
    .ToArray();

Update

And without Linq as you mention using C# 2.0. The following uses a custom comparer to compare only the "EmpNo" codes in your file names. The solution expects your file names to be well-formed i.e. for them to contain an "EmpNo" in the format of your example file names.

    [Test]
    public void Sort()
    {
        string[] files = Directory
            .GetFiles(fbdialog.SelectedPath, "*", SearchOption.AllDirectories);

        Array.Sort(files, new EmpNoFileComparer());
    }

    private class EmpNoFileComparer : IComparer<string>
    {
        public int Compare(string x, string y)
        {
            string empNoX = Path.GetFileName(x).Split('-')[1];
            string empNoY = Path.GetFileName(y).Split('-')[1];

            return empNoX.CompareTo(empNoY);
        }
    }

edited Sep 23, 2011 at 10:05

answered Sep 23, 2011 at 9:48

Tim Lloyd

38.6k10 gold badges106 silver badges132 bronze badges

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

7 Comments

abatishchev Over a year ago

It's better to use DirectoryInfo.GetFiles() and don't use SelectedPath.ToString()

Sam Casil Over a year ago

@abatishchev I'm using SelectedPath.ToString().

Tim Lloyd Over a year ago

@Sam I'm not sure I follow you?

Tim Lloyd Over a year ago

@Sam What problem are you having?

Tim Lloyd Over a year ago

@Sam I have provided an alternative answer which does not use Linq, as this is only available from C# 3.0 and you appear to be using C# 2.0. I would recommend upgrading if at all possible. C#2.0 is now quite old.

|

Snowbear · Accepted Answer · 2011-09-23 09:31:44Z

1

Sort by filename:

var result = filePath.OrderBy(p => Path.GetFileName(p));

answered Sep 23, 2011 at 9:31

Snowbear

17.3k3 gold badges45 silver badges67 bronze badges

Comments

abatishchev · Accepted Answer · 2011-09-23 09:51:10Z

1

Here's a version using some LINQ. Note also that if you only need the file names, not the files themselves, then DirectoryInfo.GetFiles() can be used rather than Directory.GetFiles().

var filePaths = new DirectoryInfo(fbdialog.SelectedPath.ToString())
    .GetFiles("*", SearchOption.AllDirectories)
    .OrderBy(f => f.Name)
    .Select(f => f.FullName);

edited Sep 23, 2011 at 9:51

abatishchev

101k88 gold badges303 silver badges443 bronze badges

answered Sep 23, 2011 at 9:33

Ian Nelson

59.1k20 gold badges79 silver badges104 bronze badges

Comments

abatishchev · Accepted Answer · 2011-09-23 09:47:48Z

0

using System.Linq; // requires .NET 3.5+

IEnumerable<string> r =
    new DirectoryInfo(fbdialog.SelectedPath) // no need for ToString()
        .GetFiles("*", SearchOption.AllDirectories)
        .Select(f => f.Name) // DAR-ALA061452-09050909.xls
        .Select(f => f.Substring(4, 9)) // ALA061452
        .OrderBy(f => f);

or shorter:

new DirectoryInfo(fbdialog.SelectedPath)
    .GetFiles("*", SearchOption.AllDirectories)
    .OrderBy(f => f.Name.Substring(4, 9));

edited Sep 23, 2011 at 9:47

answered Sep 23, 2011 at 9:29

abatishchev

101k88 gold badges303 silver badges443 bronze badges

7 Comments

Davide Piras Over a year ago

this could eventually work but not always, I think he should use System.Path.GetFileName() to isolate file name and sort by filename only ignoring the directory name...

abatishchev Over a year ago

@DavidePiras: First option for case if OP wants to order by date part.

abatishchev Over a year ago

The reason for downvote? If you don't understand something - ask.

Sam Casil Over a year ago

I want to order by EmpNo(AAP070127).

abatishchev Over a year ago

@Sam: Why don't use C# 2010 Express Edition? It's free as well and much more modern

|

Collectives™ on Stack Overflow

Problem on sorting an array

4 Answers 4

7 Comments

Comments

Comments

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

7 Comments

Comments

Comments

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related