Перегрузка оператора на C# - где должен идти код кода?

Question

Немного концептуального вопроса:

Я создаю свою пользовательскую структуру в виде вектора Vector3 (3 значения int), и я работал с перегрузкой стандартных операторов (+, -, *, /, == etc ...)

Поскольку я создаю библиотеку для внешнего использования, я пытаюсь соответствовать правилам FxCop. Поэтому они рекомендуют использовать методы, которые выполняют одну и ту же функцию.

Например. .Add(), .Subtract() и т. Д.

Чтобы сохранить дублирование кода, один из этих методов (перегрузка оператора или фактический метод) будет вызывать другой.

Мой вопрос, который должен назвать, какой?

Является ли это (и это только пример кода):

)

public static MyStruct operator +(MyStruct struc1, MyStruct struct2) 
    { 
     return struc1.Add(struct2); 
    } 

    public MyStruct Add(MyStruct other) 
    { 
     return new MyStruct (
      this.X + other.X, 
      this.Y + other.Y, 
      this.Z + other.Z); 
    } 

или:

B)

public static MyStruct operator +(MyStruct struc1, MyStruct struct2) 
    { 
     return new MyStruct (
      struct1.X + struct2.X, 
      struct1.Y + struct2.Y, 
      struct1.Z + struct2.Z); 
    } 

    public MyStruct Add(MyStruct other) 
    { 
     return this + other; 
    } 

Я действительно не уверен, либо предпочтительнее, но я ищу некоторые мнения :)

Answer 1

Я бы пошел с A). Так как перегрузка оператора не соответствует требованиям CLS (не все языки .NET поддерживают перегрузку операторов), ее можно рассматривать как синтаксический сахар вокруг Добавить метод.

Answer 2

Я бы проголосовал за вариант B, так как это более интуитивное место, чтобы посмотреть, какие значения/свойства фактически используются в операторе.

Answer 3

Либо. Это действительно неважно. Я бы ожидал, что код будет встроен в любом случае, так что идите с тем, что считаете более читаемым.

Answer 4

Поскольку это struct, наследование не является проблемой (в противном случае «А» делает virtual проще) - так что я пошел бы с «B», просто потому, что я ожидаю, чтобы позвонить более +, чем я называю Add ... меньше косвенности.

Для class с подлинной потребностью в + и наследовании (маловероятно), я бы пошел «А».

Answer 5

Определенно A.

Методы - это настоящая вещь. Метод может получить дополнительные параметры, они могут быть помечены как виртуальные (=> могут быть переопределены), перегружены и, как правило, больше характер языка. Прежде чем искать практическую причину, я думаю, вам нужно это почувствовать. , кроме того, языковой протокол CIL не является обязательным для поддержки перегрузки оператора и такой явной победы в методе.

Answer 6

Я бы также пошел с A по многим причинам, указанным выше, в частности, потому что я рассматриваю перегрузку оператора как синтаксический механизм, который обертывает фактический метод.