Почему не возвращает переменную part хотя условие выполняется?

Question

public static string Searcher(string path, string pattern)
    {
        var parts = new List<DirectoryInfo> { };

        DirectoryInfo dir = new DirectoryInfo(path);

        try
        {
            DirectoryInfo[] dirs = dir.GetDirectories();
            foreach (DirectoryInfo di in dirs)
            {

                parts.Add(di);
            }
        }
        catch { };

        foreach (DirectoryInfo part in parts.ToList())
        {

            if(part.ToString().Contains(pattern) == true)
            {
                return(part.ToString());
            }




            else
            {

                Searcher(part.ToString(), pattern);

            }

        }


        return null;


    }

вот так вызываю метод

Console.WriteLine(Searcher(d.Name, "___data___"));

Как и какими средствами находить ошибки в коде на C#? — tym32167, Mar 16 '22 at 19:49
И все-таки попробуйте отладку. Там сразу увидите, что у вас рекурсивный метод и, возвращая part из какого-то уровня рекурсии, вы всего-лишь выходите на один уровень выше (в рекурсии) и ничего не делаете с этим знанием. — vitidev, Mar 16 '22 at 20:25
я проделал откладку но так и не понял как это исправить? — ShaDy_AN, Mar 16 '22 at 20:39

score 0 · Answer 1 · answered Mar 17 '22 at 12:14

Проблема была в том, что Вы, как и указывал @vitidev, не использовали результат рекурсивного вызова метода. Также есть несколько замечаний:

Входной параметр у Вас называется path, что наталкивает на мысль, что это путь к директории, но в тоже время Вы кажется передаете в метод имя d.Name.
Использование блока try...catch кажется можно избежать, достаточно проверить, что директория существует.
Код испытывает серьезные проблемы с форматированием.

        public static string Searcher(string path, string pattern)
        {
            var parts = new List<DirectoryInfo>();
            var dir = new DirectoryInfo(path);
            if (!dir.Exists) 
                return null;
        parts.AddRange(dir.GetDirectories());
        foreach (var part in parts.Select(x =&gt; x.ToString()))
        {
            if (part.Contains(pattern))
                return part;
            var result = Searcher(part, pattern);
            if (result != null)
                return result;
        }
        return null;
    }

спасибо теперь всё работает – ShaDy_AN Mar 17 '22 at 14:51 — ShaDy_AN, Mar 17 '22 at 14:51

Почему не возвращает переменную part хотя условие выполняется?

1 Answers1