Как обеспечить ошибку компиляции на изменение подписи, в которой используется ключевое слово «Params»

Question

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

public void Foo(params string[] args) { 
    bar(args[0]); 
    bar(args[1]); 
} 

Новые требования приводят к изменению, как это:

public void Foo(string baz, params string[] args) { 
    if("do bar".Equals(baz)) { 
    bar(args[0]); 
    bar(args[1]); 
    } 
} 

проблема в том, что, хотя я изменил подпись метода, ошибок в компиляции не происходит, что, конечно, правильно, но я хочу, чтобы там были ошибки компиляции для каждого вызова метода Foo, где аргумент baz не указан. То есть, если вызов Foo перед изменением был один:

Foo(p1,p2); //where p1 and p2 are strings 

теперь должна быть такой:

Foo(baz,p1,p2); 

Если он не будет изменен таким образом, p1 бы присваивается baz, а массив params args будет иметь длину 1 и исключение OutOfBounds.

Каков наилучший способ изменить подпись и обеспечить, чтобы все код вызова был соответствующим образом обновлен? (Реальный сценарий - это то, где Foo живет в сборке, совместно используемой многими проектами, автоматически создаваемыми на сервере сборки. Таким образом, ошибка компиляции будет простым способом обнаружить весь код, который необходимо затронуть, чтобы внести изменения.)

Редактировать: Как указал Даниэль Манн и другие, приведенный выше пример предполагает, что я не должен использовать параметры вообще. Поэтому я должен объяснить, что в моем реальном мире не всегда так, что args должен иметь два элемента, поскольку касается логики в Foo, args может содержать любое количество элементов. Итак, допустим, это Foo:

public void Foo(string baz, params string[] args) { 
    if("do bar".Equals(baz)) { 
    int x = GetANumberDynamically(); 
    for(int i = 0; i<x; i++) 
     bar(args[i]); 
    } 
} 

Answer 1

Вот решение. Не меняйте прежнюю сигнатуру метода, просто добавьте атрибут Obsolete с обоими указанными аргументами.

[Obsolete("Use Foo(string, params string[]) version instead of this", true)] 
public void Foo(params string[] args) { 
    bar(args[0]); 
    bar(args[1]); 
} 

Затем создайте новый метод с новой подписью.

public void Foo(string baz, params string[] args) { 
    if("do bar".Equals(baz)) { 
    bar(args[0]); 
    bar(args[1]); 
    } 
} 

Второй аргумент в атрибуте Obsolete обеспечивает ошибку компиляции. Без этого он просто вызывает предупреждение о компиляции. Более подробная информация об атрибуте доступна по адресу MSDN.

EDIT:

На основе обсуждения в комментариях ниже, Дэниел Манн выступил с интересной проблемой.

Это не решит проблему. А если вы назовете Foo («a», «b»)? В этом случае он по-прежнему вызовет не устаревший метод только с двумя аргументами и вызовет ту же проблему.

Я бы посоветовал, чтобы проверить, есть ли более один аргумент, передаваемый через args перед вызовом bar.

Answer 2

Самое простое решение - не использовать ключевое слово params, если у вас есть .

Очевидно, что вы ожидаете, что args содержит как минимум два параметра. Можно с уверенностью сказать, что это необходимо. Почему у вас нет такой подписи метода?

public void Foo(string baz, string requiredArgument1, string requiredArgument2, params string[] optionalArguments)

Это устраняет неоднозначность: Это всегда требуют, по крайней мере, 3-х аргументов.

Другой вариант, о котором я даже не думал, почему-то использовать именованные параметры. Очевидно, что весь код должен явно сделать это, но вы можете сделать это:

Foo(baz: "bar", args: new [] {"a", "b", "c"});