Как можно использовать Path.Combine с более чем двумя аргументами? - PullRequest
Новая
       52

Как можно использовать Path.Combine с более чем двумя аргументами?

22 голосов
/ 17 апреля 2009

Я удивлен, что нет перегрузки, которая может принимать строковый массив. В любом случае, каков наилучший способ избежать вложенных вызовов в Path.Combine? 

pathValue = Path.Combine(path1, Path.Combine(path2, Path.Combine(path3, path4)))

Это кажется неэффективным, поскольку приводит к созданию четырех новых строк только для получения одной.

Ответы [ 3 ]

27 голосов
/ 17 апреля 2009

Если у вас уже есть массив или IEnumerable , вы можете сделать это в одну строку ... 

// I'm assuming that you've got an array or IEnumerable<T> from somewhere
var paths = new string[] { path1, path2, path3, path4, path5, path6 };

string result = paths.Aggregate(Path.Combine);

Если нет, то напишите свой собственный метод расширения для строки ... 

public static class PathExtension
{
    public static string CombinePathWith(this string path1, string path2)
    {
        return Path.Combine(path1, path2);
    }
}

... что позволит вам связать их вот так ... 

string result = path1.CombinePathWith(path2)
                     .CombinePathWith(path3)
                     .CombinePathWith(path4)
                     .CombinePathWith(path5)
                     .CombinePathWith(path6);
22 голосов
/ 17 апреля 2009

С точки зрения эффективности, проблема не в IMO, а в аспекте удобства использования. Лично я думаю, что должна быть перегрузка: 

Combine(string first, string second, string third, params string[] others)

Вам нужно иметь как минимум три, чтобы предотвратить конфликт с существующей двухпараметрической версией, если вы просто напишите Path.Combine("foo", "bar"), но это, безусловно, поможет сделать код более понятным. Почему бы не открыть запрос функции на Connect ?

Конечно, вы можете реализовать это самостоятельно (а в другом классе количество параметров не имеет большого значения): 

public static string CombinePaths(string first, params string[] others)
{
    // Put error checking in here :)
    string path = first;
    foreach (string section in others)
    {
        path = Path.Combine(path, section);
    }
    return path;
}
1 голос
/ 17 апреля 2009

Довольно просто реализовать это самостоятельно: 

public string Combine(params string[] paths)
{
    char[] pathSeparators = new char[] 
        { Path.AltDirectorySeparatorChar, Path.DirectorySeparatorChar, Path.VolumeSeparatorChar };

    if(paths == null) throw new ArgumentNullException("paths");
    if(paths.Length == 1) return paths[0];

    StringBuilder pathBuilder = new StringBuilder();

    foreach(string path in paths)
    {
        if(Path.IsPathRooted(path))
            pathBuilder = new StringBuilder(path);
        else
        {
            char last = pathBuilder.Length > 0 ?
                pathBuilder[pathBuilder.Length - 1] :
                path[path.Length - 1];

            if(Array.IndexOf(pathSeparators, last) == -1)
                pathBuilder.Append(Path.DirectorySeparatorChar);

            pathBuilder.Append(path);
        } // else
    } // foreach

    return pathBuilder.ToString();
}

[Test()]
public void CombinePaths()
{
    string result = Combine(@"C:\Program Files\", @"Microsoft.NET", @"ADOMD.NET\", "90", "msadomdx.dll");
    Assert.AreEqual(@"C:\Program Files\Microsoft.NET\ADOMD.NET\90\msadomdx.dll", result);
}
...