1

Добрый день! Задача состоит в том, чтобы создать дерево в узлах которого будет запускаться тестируемая функция и если она не отработала за отведенное время - принудительно завершать исполнение. Т.к. тестируемых функций планируется много, то на мой взгляд, надо использовать либо делегаты либо ссылки на функции, которые описать где-нибудь в одном месте отдельно. Важно чтобы из потока можно было вернуть результат его выполнения или исключение.

public class TesterNode
{
    //ссылка на тестируемую функцию
    private Func<TestResult> TestFunc;
    private TestResult state = TestResult.NotRunning;

    public TestResult RunAction()
    {
        if (TestFunc != null)
        {
            state = TestResult.Running;

            //этот метод должен выполнять тестируемую функцию 
            //и если та не отработала за определенный промежуток времени
            //возвращать TestResult.Timeout
            var res = TestFuncAsync();
            if (res == TestResult.Success || res == TestResult.Warning)
            {
                foreach (var item in childs)
                {
                    item.RunAction();
                }
            }
            return res;
        }
        else
            return TestResult.Error;
    }
}
7
  • А в чем, собственно, проблема? Commented 1 мар. 2016 в 8:15
  • Не могу правильно написать TestFuncAsync. Пробовал через async/await но там есть только токен отмены операции, чтобы поток сам произвольно завершился, но это не приемлемый вариант, т.к. TestFunc использует внешний ресурс, который может зависнуть. Как сделать это через делегат - ума не приложу. А если использовать ссылку на функцию, то нужно писать что-то типа private Func<шт object, out void> TestFunc что является ошибкой синтаксиса. Вот и сижу, понять не могу как правильно это написать.
    – Tony
    Commented 1 мар. 2016 в 8:48
  • Вы же сами написали: завершить поток. А для того чтобы его завершить - надо сначала его создать! Commented 1 мар. 2016 в 8:51
  • Да, создать поток проблема:
    – Tony
    Commented 1 мар. 2016 в 8:54
  • 1
    Action<T> - это Func который ничего не возвращает
    – Grundy
    Commented 1 мар. 2016 в 9:52

1 ответ 1

2

Используйте замыкания. Адаптировать функцию под другой прототип - не так сложно как кажется.

public delegate TestResult TestFunction();
public static TestResult TestWithTimeout(TestFunction func, TimeSpan timeout)
{
     TestResult result = TestResult.Timeout;
     var thread = new Thread(() => {
       try {
         result = func();
       }
       catch (ThreadAbortException) {}
       catch {
         result = TestResult.Error;
       }
     });
     thread.Start();
     if (!thread.Join(timeout))
       thread.Abort();

     return result;
}
2
  • 1
    Ну... Вам стоит написать всё же, что Thread.Abort — это крайне плохая идея, так как может разрушить состояние практически любого глобального объекта. (Особенно если код висит во «внешней» функции.)
    – VladD
    Commented 1 мар. 2016 в 13:26
  • @VladD ну, вы уже написали об этом :) Конкретно же в данном случае - по-другому сделать и не получится. Commented 1 мар. 2016 в 14:17

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.