Более быстрый способ выполнения проверок аргументов метода

Question

Это в основном просто из любопытства, и это потенциально глупый вопрос. :)

У меня есть метод, как это:

public void MyMethod(string arg1, string arg2, int arg3, string arg4, MyClass arg5) 
{ 
    // some magic here 
} 

Ни один из аргументов не может быть пустым, и ни один из строковых аргументов не может равняться String.Empty.

Вместо меня имеющий большой список:

if(arg1 == string.Empty || arg1 == null) 
{ 
    throw new ArgumentException("issue with arg1"); 
} 

есть более быстрый способ просто проверить все строковые аргументы?

Извинения, если мой вопрос непонятен.

Спасибо!

Answer 1

Вы можете создать или использовать рамки для проверки контракта вашего метода, например, Code Contracts.

Также вы можете создавать различные полезные методы, такие как ThrowIfNullOrEmpty, которые будут инкапсулировать логику проверки аргументов.

Answer 2

if (string.IsNullOrEmpty(arg1)) 
{ 
    throw new ArgumentException("issue with arg1"); 
} 

Также есть code contracts в .NET 4.

Answer 3

Вы можете использовать String.IsNullOrEmpty:

if (String.IsNullOrEmpty(arg1)) 
    throw new ArgumentException("issue with arg1"); 

Если вы на Framework 4, есть также String.IsNullOrWhiteSpace метод (возможно, вы хотите бросить, если кто-то называет этот метод с строка, содержащая пробелов только).

Вы можете сделать метод утилиты, чтобы проверить аргумент и бросить, но это кратчайшая нотация, которую вы получите. В C# нет синтаксиса, который позволяет вам выразить компилятору, что нули являются неприемлемыми и язык забрасывает исключения для вас.

С каркасом 4 у вас также есть CodeContracts - но синтаксис для его использования остается вызовом метода.

Answer 4

Ну, во-первых, вы можете использовать if (!String.IsNullOrEmpty(arg1)), чтобы уменьшить выражение if.

Во-вторых, вам нужна проверка аргументов в вашей сборке релизов, или было бы достаточно выполнить проверку arg в вашей сборке отладки? Это зависит от того, откуда поступают данные. Если эта функция вызывается вашим собственным кодом, и аргументы обрабатываются вашим кодом вызова, вероятно, достаточно проверить аргументы только в ваших сборках отладки. В этом случае используйте Debug.Assert(!String.IsNullOrEmpty(arg1), "msg"); и т. Д. Утверждения лишены версий релиза.

В-третьих, проверить DevLabs: Code Contracts

Answer 5

Реальная проблема здесь состоит в том, что ваш метод имеет слишком много аргументов. Это делает вас громоздким для написания кода проверки. Делает это громоздким по клиентскому коду. Подумайте о написании вспомогательного класса, в котором аргументы, которые вы сейчас проходите, являются свойствами. Теперь ваш метод может принимать только один аргумент. И проверка может быть выполнена в самом классе, а не в вашем методе.

Канонический пример этого шаблона в Framework является классом Process.Вспомогательный класс ProcessStartInfo позволяет использовать его.

Answer 6

Там хорошая библиотека называется CuttingEdge.Conditions, что обеспечивает свободно интерфейс для этого:

public void MyMethod(string arg1, string arg2, int arg3, string arg4, MyClass arg5) 
{ 
    Condition.Requires(arg1, "arg1").IsNotNull().IsNotEmpty().StartsWith("Foo"); 
} 

Это обеспечивает безопасные условия типа для всех видов проверок, в том числе нулевых проверок, проверки диапазона на номера, строка проверки (в том числе StartsWith выше) и т. Д. Если у вас есть несколько проверок параметров, это может значительно сократить и упростить вашу проверку и, самое главное, сделать проверку аргументов и исключений согласованными во всем приложении.

Answer 7

Насколько я мог понять ваш вопрос, как об этом:

static void Main(string[] args) 
{ 
    Test(null, "", "", ""); 
} 

static void Test(MyClass arg1, params string[] args) 
{ 
    if (args.Count(a => String.IsNullOrEmpty(a)) == args.Length) 
    { 
     throw new Exception("All params are null or empty"); 
    } 
}