代码第一外键关联MVC实体

Question

I have an Html sanitizer (more like an xss detector) code I'm working on and would like to log failures caused during code xss sanitation/detection. So in entity, I have the following model for my UserProfile

[Table("UserProfile")]
public class UserProfile
{
    [Key]
    [DatabaseGeneratedAttribute(DatabaseGeneratedOption.Identity)]
    public int UserId { get; set; }
    public string UserName { get; set; }
    public virtual List<Security> SecurityViolations { get; set; } //is this correct???
}

and for my security class model, I have the following:

[Table("Security")]
public class Security
{
    [Key]
    public int Id { get; set; }
    [Required]
    public string SecMsg { get; set; }
    public string Trace { get; set; }
    public int ErrorCode{ get; set; }
    public int UserId { get; set; } 
    public virtual UserProfile UserProfile { get; set; } 
}

To clarify my idea, when a security failure happens, the Security model is filled in with information but I like to tie all that information to the user that generated the errors so I establish a foreign key relation (or try to at least), to the UserProfile UserId primary key column.

Therefore, the code to log to my Security table is as follows:

using (CCSecurity sec_db = new CCSecurity())
            {
                Security sec_execption = new Security();
                sec_execption.SecMsg = ex.Message;
                sec_execption.Trace = ex.StackTrace;
                sec_execption.UserId = uid;  //uid is 12, a PK entry in UserProfile
                try
                {
                    sec_db.Security.Add(sec_execption);
                    sec_db.SaveChanges();
                }
                catch (Exception e)
                {
                    Debug.WriteLine(e.StackTrace.ToString());
                }

            }

So then during one of my unit test execution, I get the following exception:

"The property 'UserId' cannot be configured as a navigation property. The property must be a valid entity type and the property should have a non-abstract getter and setter. For collection properties the type must implement ICollection where T is a valid..."

I'm new to entity and googling the above turned many suggestion that did not solve my problem. How can I have a foreign key in my Security class that references a primary key in my UserProfile class and still have it well documented in code (ie the ForeignKey property field denotes foreign key in Security class) ?

Edit:

public class CCSecurity : DbContext
{
    public DbSet<Security> Security { get; set; }
    public CCSecurity()
        : base("TBCdb")
    {}
}

Edit 2

The auto migration generated the Security Table as follow's:

CREATE TABLE [dbo].[Security] (
[Id]        INT            IDENTITY (1, 1) NOT NULL,
[SecMsg]    NVARCHAR (MAX) NOT NULL,
[Trace]     NVARCHAR (MAX) NULL,
[ErrorCode] INT            NOT NULL,
[UserId]    INT            NOT NULL,
CONSTRAINT [PK_dbo.Security] PRIMARY KEY CLUSTERED ([Id] ASC),
CONSTRAINT [FK_dbo.Security_dbo.UserProfile_UserId] FOREIGN KEY ([UserId]) REFERENCES [dbo].[UserProfile] ([UserId]) ON DELETE CASCADE

);

Seem ok but it throws the error on SaveChanges():

"The INSERT statement conflicted with the FOREIGN KEY constraint "FK_dbo.Security_dbo.UserProfile_UserId". The conflict occurred in database \\"TBCdb\\", table \\"dbo.UserProfile\\", column 'UserId'.\\r\\nThe statement has been terminated."

Latest Edit So I started a new project console app to test if something weird is happening on my main project. I basically copied my two models in the new project (Security and UserProfile) and got it to work... I'm using a file db and don't know what could be wrong in my main project and what can be different in the new console app project. I'm running a Unit Test on my main project and I wonder if that has anything to do with it.

Thanks!

Answer 1

As the error message indicates, your foreign key needs to be associated with a valid entity property. So, instead of declaring the foreign key as follows:

[ForeignKey("UserId")]
public int? UserId { get; set; }

You need to declare it as:

[ForeignKey("UserProfile")]
public int UserId { get; set; }

Answer 2

You don't need UserId property, just remove it and use UserProfile. Entity Framework will handle this automatically.

sec_execption.UserProfile = getUserProfileById(uid);

getUserProfileById might look like:

return sec_db.UserProfiles.Where(u => u.UserId == uid).FirstOrDefault();

PS

public virtual List<Security> SecurityViolations { get; set; } // is ok

but

public virtual ICollection<Security> SecurityViolations { get; set; } // is better

Answer 3

Try it like below to have one-to-many relationship between Users and Securtiy. As mentioned below virtual keyword is all required to indicate to EF a relationship between entities.

public class Security
{
    public int Id { get; set; }
    public string SecMsg { get; set; }
    public string Trace { get; set; }
    public int ErrorCode { get; set; }

    public int UserId { get; set; } //This is just to tell EF to name the FK column as UserId rather than UserProfile_UserId

    public virtual UserProfile UserProfile { get; set; } //This is good enough to tell EF that there is a relationship
}

public class UserProfile
{
    [Key]
    public int UserId { get; set; }
    public string UserName { get; set; }

    public virtual ICollection<Security> Securities { get; set; } //Good enough to indicate the relationship
}