What Wrong With This EF Code First Model..?

Question

0.00/5 (No votes)

See more:

This Is The Exception Em Recieving.
"The property 'MeterID' 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<t> where T is a valid entity type"

C#

public class Employee
{
    public int EmployeeID { get; set; }
    public String FirstName { get; set; }
    public String LastName { get; set; }
    public float Salary { get; set; }
    public int Age { get; set; }
    public String MobileNo { get; set; }
    public String Designation { get; set; }

    [ForeignKey("QuaterID")]
    public int QuaterID { get; set; }
    public DomesticQuater Quater { get; set; }


}

public class DomesticQuater
{
    public int ID { get; set; }
    public Int64 QuaterNo { get; set; }
    public String QuaterType { get; set; }
    public int Block { get; set; }
    public String Area { get; set; }
    public String Street { get; set; }
    public String AddtionalAddress { get; set; }

    [ForeignKey("MeterID")]
    public int MeterID { get; set; }
    public virtual ElectricMeter Meter { get; set; }

}

public class ElectricMeter
{
    public int MeterID { get; set; }
    public int MeterNo   { get; set; }
    public DateTime InstalledDate { get; set; }
    [Column("MeterType")]
    public bool IsDomestic { get; set; }

}

[Table("CommercialConsumers")]
public class CommercialShop
{
    public int ID { get; set; }
    public Int64 OwnerNIC { get; set; }
    public string OwnerName { get; set; }
    public String Block { get; set; }
    public String Area { get; set; }
    public String Street { get; set; }
    public String AddtionalAddress { get; set; }

    [ForeignKey("MeterID")]
    public int MeterID { get; set; }
    public virtual ElectricMeter Meter { get; set; }

}


public class DemoBillingContext : DbContext
{


    public virtual DbSet<Employee> Employees { get; set; }
    public virtual DbSet<ElectricMeter> Meters { get; set; }
    public virtual DbSet<DomesticQuater> Quaters { get; set; }
    public virtual DbSet< CommercialShop>CommercialConsumers { get; set; }

    protected override void OnModelCreating(DbModelBuilder modelBuilder)
    {
                base.OnModelCreating(modelBuilder);
    }
}

Here In Page Load Em Trying To put Some Data..

C#

protected void Page_Load(object sender, EventArgs e)
        {


            DemoBillingContext context = new DemoBillingContext();

            //context.Meters.Add()
            ElectricMeter m = new ElectricMeter()
                                {
                                    MeterNo = 56589255,
                                    IsDomestic = true,
                                    InstalledDate = DateTime.Now
                                };
            DomesticQuater q = new DomesticQuater()
                {
                    QuaterNo = 563,
                    Area = "Loco",
                    Block = 3,
                    QuaterType = "OfficerQuater",
                    Street = "45",
                    MeterID = 1,AddtionalAddress="LOCO FACTROY 1 STREET 1 HOUSE 23/43",

                };
            Employee emp = new Employee
            {


                FirstName = "ARIF",
                LastName = "HUSSAIN",
                MobileNo = "03475060769",
                Designation = "Manager",
                Salary = 50000,
                QuaterID = 1,

            };

            context.Employees.Add(emp);

            foreach (var item in context.Employees)
            {

                Domestic.Text += String.Format("{0}---{1}---{2}---{3}--{4}",item.FirstName+" "+item.LastName,
                    item.Salary,
                    item.Quater.QuaterNo,
                    item.Quater.Area,
                    item.Quater.Meter.MeterNo,
                    item.Quater.Meter.IsDomestic);
                
            }
      


        }

Posted 3-Mar-15 6:40am

Arif H Shigri

Updated 3-Mar-15 6:42am

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Dave Kreskowiak · Accepted Answer · 2015-03-03T07:00:00

It appears that your class should look like this:

public class DomesticQuater
    {
        public int ID { get; set; }
        public Int64 QuaterNo { get; set; }
        public String QuaterType { get; set; }
        public int Block { get; set; }
        public String Area { get; set; }
        public String Street { get; set; }
        public String AddtionalAddress { get; set; }
 
        public virtual ElectricMeter Meter { get; set; }
    }

You don't need the "int MeterId" line at all. EF will put it in there for you. You have the same problem in your CommercialShop and Employee classes.

Your ElectricMeter class also has a problem. The ID column name should be the name of the class with "Id" appended to it for a conventional configuration:

public class ElectricMeter
    {
        public int ElectricMeterID { get; set; }
        public int MeterNo { get; set; }
        public DateTime InstalledDate { get; set; }
        [Column("MeterType")]
        public bool IsDomestic { get; set; }
 
    }

OR you can explicitly tag it as the key to the table, but just make sure you don't get MeterId and MeterNo confused in your code.

public class ElectricMeter
    {
        [Key]
        public int MeterID { get; set; }
        public int MeterNo   { get; set; }
        public DateTime InstalledDate { get; set; }
        [Column("MeterType")]
        public bool IsDomestic { get; set; }
 
    }