Forbid string & object concatenation

Question

Я конвертирую кучу строк в api в классы. Я использую экстенсивно компилятор, чтобы помочь мне в этом, так как я могу заменить некоторые типы в core apis и следовать за ошибками типа.

Однако, есть разочарование источником ошибок при таком подходе:

// linqpad snippet 
void Main() 
{ 
    var t = "yay : " + (new Foo()); 
    t.Dump(); // "yay: namespace.Foo" instead of compilation error 
} 

class Foo { 
} 

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

Я хочу ошибку компиляции, потому что эти данные в конечном счете отображаются пользователю. Если я пропущу одну конкатенацию, пользователи будут видеть имена классов в приложении, что на самом деле не является поведением, которое я хочу.

Answer 1

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

Да - если вы используете Visual Studio 2015 или выше :)

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

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

Обратите внимание, что в некоторых случаях код, который вы указали, может быть полностью уместным - если Foo переопределяет ToString(), например. Вы действительно уверены вы хотите запретить это?

Answer 2

Одним из возможных решений (что может быть неуместно) было бы переопределить метод ToString в ваших классах и заставить их выбросить ошибку. Это не даст вам ошибки компиляции времени, но дало бы исключение времени выполнения:

public class Test 
{ 
    public override string ToString() 
    { 
     throw new NotImplementedException(); 
    } 
} 

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

[Obsolete("Don't use this!")] 
public override string ToString() 
{ 
    throw new NotImplementedException(); 
} 

Хотя это не будет иметь никакого влияния на код, который вы в курсе, вы должны получить предупреждения в любом месте вы явно вызвать ToString, для Например, это даст предупреждение компилятору:

var s = (new Test()).ToString();