Что не так с использованием универсального неявного оператора?

Question

Если я использую неявный оператор в не универсальном классе строителя, все в порядке:

 public class ReligionBuilder
{
    private Religion _religion;

    public ReligionBuilder()
    {
        _religion = new Religion(){//some codes}

    }

    public ReligionBuilder AddToRepository()
    {
        Repository<Religion>.Add(_religion);
        return this;
    }
    public Religion Build()
    {
        return _religion;
    }

    public static implicit operator Religion(ReligionBuilder _builder)
    {
        return _builder.Build();

    }
}

Я могу использовать это:

 Religion religion=new ReligionBuilder().AddToRepository();

но если этот оператор в обобщенном классе, то что-то не так:

 public abstract class DataTestBuilderBase<T> : IDataTestBuilder<T>
{
    protected T TestData { get; set; }

    public virtual T Build()
    {
        return TestData;
    }

    public abstract IDataTestBuilder<T> AddToRepository();
    public abstract IDataTestBuilder<T> WithDefault();

    public static implicit operator T(DataTestBuilderBase<T> builder)
    {
        return builder.Build();
    }
}


 public class PersonDataTestBuilder : DataTestBuilderBase<Person>
{
    private Person _person;

    public PersonDataTestBuilder()
    {
        //some codes
    }
    public override IDataTestBuilder<Person> AddToRepository()
    {
       //some codes
        return this;
    }
}

использование:

 PersonDataTestBuilder _testBuilder = new PersonDataTestBuilder();
        Person person = _testBuilder.AddToRepository();

ошибка: невозможно преобразовать IDataTestBuilder в Person

в чем проблема?

Answer 1

AddToRepository возвращается только IDataTestBuilder<Person> (с точки зрения возвращаемого типа во время компиляции) - и нет неявного преобразования из этого в Person, Если вы измените тип возврата абстрактного метода на DataTestBuilderBase<T> тогда это должно работать - хотя, честно говоря, я бы не хотел использовать это неявное преобразование в любом случае. Я обычно довольно осторожен в предоставлении неявных преобразований - они часто могут сделать код менее понятным, как я полагаю, что они делают здесь.

Если вы действительно предоставляете AddToRepository на строителя все равно? Это похоже на неуместное действие для застройщика - я бы ожидал:

Person person = new PersonBuilder { /* properties */ }
                     .Build()
                     .AddToRepository();

РЕДАКТИРОВАТЬ: Просто, чтобы показать, что я имею в виду изменение типа возврата AddToRepositoryВот короткая, но полная программа для демонстрации. Работает нормально.

using System;

public abstract class BuilderBase<T>
{
    public abstract T Build();
    public abstract BuilderBase<T> AddToRepository();

    public static implicit operator T(BuilderBase<T> builder)
    {
        return builder.Build();
    }
}

public class TestBuilder : BuilderBase<string>
{
    public override string Build()
    {
        return "Built by Build()";
    }

    public override BuilderBase<string> AddToRepository()
    {
        return this;
    }
}

class Program
{
    static void Main(string[] args)
    {
        string x = new TestBuilder().AddToRepository();
        Console.WriteLine(x);
    }
}