17

I have entities derived from a base entity in my application which uses ef core code-first approach.

Base class

public abstract class BaseEntity<T> : IEntity<T>
{
    [Key]
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public T Id { get; set; }

    object IEntity.Id { get { return Id; } set { } }

    private DateTime? createdOn;
    [DataType(DataType.DateTime)]
    public DateTime CreatedOn { get => createdOn ?? DateTime.Now; set => createdOn = value; }

    [DataType(DataType.DateTime)]
    public DateTime? ModifiedOn { get; set; }

    public bool IsDeleted { get; set; }
    // Auto increment for all entities.
    public int OrderId { get; set; }
}

And an entity

public class UserEntity : BaseEntity<int>
{
    public string EmployeeId { get; set; }
    public string FullName { get; set; }
    public string Email { get; set; }
}

I can apply the .ValueGeneratedOnAdd() method on property OrderId in OnModelCreating for each entity however, is there a way to apply a general rule for all entities without repeatig yourself?

5
  • Can't you use [DatabaseGenerated(DatabaseGeneratedOption.Identity)] on OrderId as well?
    – Jogge
    Nov 13, 2018 at 7:55
  • There is already a priamary key column with identity option. So it is not allowed to have with this attribute anymore.
    – ibubi
    Nov 13, 2018 at 11:06
  • Unrelated to the question, but IEntity knows Id is a T so IEntity.Id needn't return object.
    – sellotape
    Nov 15, 2018 at 19:27
  • @sellotape please refer to this link and see Implementation paragraph there for details..
    – ibubi
    Nov 15, 2018 at 20:44
  • @ibubi - fair enough; I should have realised it's IEntity.Id, not IEntity<T>.Id...
    – sellotape
    Nov 15, 2018 at 20:52

4 Answers 4

32

With the lack of custom conventions, you could use the typical modelBuilder.Model.GetEntityTypes() loop, identify the target entity types and invoke common configuration.

Identification in your case is a bit complicated because of the base generic class, but doable by iterating down Type.BaseType and check for BaseEntity<>. Once you find it, you can retrieve the generic argument T which you'll need later.

If you don't want to use generic class implementing IEnityTypeConfiguration<TEntity>, then the idea is to put the implementation in generic constrained method like this

static void Configure<TEntity, T>(ModelBuilder modelBuilder)
    where TEntity : BaseEntity<T>
{
    modelBuilder.Entity<TEntity>(builder =>
    {
        builder.Property(e => e.OrderId).ValueGeneratedOnAdd();
    });
}

Passing the actual entity type TEntity to modelBuilder.Enity method is crucial, because otherwise EF Core will consider whatever you pass to be an entity type and configure TPH inheritance.

Calling the method requires reflection - finding the generic method definition, using MakeGenericMethod and then Invoke.

Here is all that encapsulated in a static class:

public static class BaseEntityConfiguration
{
    static void Configure<TEntity, T>(ModelBuilder modelBuilder)
        where TEntity : BaseEntity<T>
    {
        modelBuilder.Entity<TEntity>(builder =>
        {
            builder.Property(e => e.OrderId).ValueGeneratedOnAdd();
        });
    }

    public static ModelBuilder ApplyBaseEntityConfiguration(this ModelBuilder modelBuilder)
    {
        var method = typeof(BaseEntityConfiguration).GetTypeInfo().DeclaredMethods
            .Single(m => m.Name == nameof(Configure));
        foreach (var entityType in modelBuilder.Model.GetEntityTypes())
        {
            if (entityType.ClrType.IsBaseEntity(out var T))
                method.MakeGenericMethod(entityType.ClrType, T).Invoke(null, new[] { modelBuilder });
        }
        return modelBuilder;
    }

    static bool IsBaseEntity(this Type type, out Type T)
    {
        for (var baseType = type.BaseType; baseType != null; baseType = baseType.BaseType)
        {
            if (baseType.IsGenericType && baseType.GetGenericTypeDefinition() == typeof(BaseEntity<>))
            {
                T = baseType.GetGenericArguments()[0];
                return true;
            }
        }
        T = null;
        return false;
    }
}

Now all you need is to call it from inside your OnModelCreating override:

modelBuilder.ApplyBaseEntityConfiguration();
9
  • 1
    Thanks for this detailed answer.
    – ibubi
    Nov 15, 2018 at 20:43
  • @Ivan Stoev, Its not working for me in case of EF Core 3.0. Note: My BaseEnity is not Generic. Nov 21, 2019 at 12:46
  • 6
    @TanvirArjel If your base entity is not generic, then it should be simpler. Remove the T from Configure method (e.g. static void Configure<TEntity>(ModelBuilder modelBuilder) where TEntity : BaseEntity) and from MakeGenericMethod. Also IsBaseEntity method is not needed, use if (entityType.ClrType.IsSubclassOf(typeof(BaseEntity)) criteria for instance.
    – Ivan Stoev
    Nov 21, 2019 at 13:21
  • 1
    @IvanStoev Thats's because my entities don't inherit from BaseEntity (as they are indepedent from EF). So I would need to access all properties from modelBuilder like in EF6 (ie: modelBuilder.Properties<string>().Configure(x => x.HasColumnType("VARCHAR2"));)
    – sinsedrix
    Jun 9, 2020 at 14:27
  • 1
    @sinsedrix stackoverflow.com/questions/41468722/…
    – Ivan Stoev
    Jun 9, 2020 at 15:17
2

If you want every property named OrderId to be ValueGeneratedOnAddYou you could solve it like this without repeating yourself:

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    base.OnModelCreating(modelBuilder);

    foreach (var et in modelBuilder.Model.GetEntityTypes())
    {
        foreach (var prop in et.GetProperties())
        {
            if (prop.Name == "OrderId")
            {
                prop.ValueGenerated = Microsoft.EntityFrameworkCore.Metadata.ValueGenerated.OnAdd;
            }
        }
    }
}

Source:

https://stackoverflow.com/a/62380293/3850405

This can also be used to set maximum string length for all strings as an example:

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    base.OnModelCreating(modelBuilder);
    
    foreach (var property in modelBuilder.Model.GetEntityTypes()
        .SelectMany(t => t.GetProperties())
        .Where(p => p.ClrType == typeof(string)))
    {
        if (property.GetMaxLength() == null)
            property.SetMaxLength(450);
    }
}

Source:

https://stackoverflow.com/a/50852517/3850405

You can also set mark every entity that inherits from IEntity as IsTemporal:

protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    base.OnModelCreating(modelBuilder);

    foreach (var et in modelBuilder.Model.GetEntityTypes())
    {
        if (et.ClrType.IsAssignableTo(typeof(IEntity)))
        {
            et.SetIsTemporal(true);
        }
    }
}

Source:

https://stackoverflow.com/a/75496573/3850405

1

In EF6 you can use:

modelBuilder.Properties<int>().Where(p=>p.Name == "OrderId").Configure(c => c.HasDatabaseGeneratedOption(DatabaseGeneratedOption.Identity));
0
foreach (var item in modelBuilder.Model.GetEntityTypes())
{
    item.FindProperty("CreatedOn")?.SetDefaultValueSql("GETDATE()");
    item.FindProperty("ModifiedOn")?.SetDefaultValueSql("GETDATE()");
    item.FindProperty("AAStatus")?.SetDefaultValue("Alive");
}

i researched and found this thread, my solution after reading it is

Your Answer

Reminder: Answers generated by Artificial Intelligence tools are not allowed on Stack Overflow. Learn more

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.