Como dito nos comentários pelo @Bigown@Maniero, fica difícil julgar se está correto ou não o mapeamento sem o contexto, o que pode ser feito é analisar e explicar o que o código está fazendo.
Não está errado este mapeamento:
public class Student
{
public Student()
{
}
public int StudentID { get; set; }
public string StudentName { get; set; }
//Não necessida da anotação [ForeignKey]
public int StandardId { get; set; }
public Standard Standard { get; set; }
}
public class Standard
{
public Standard()
{
}
public int StandardId { get; set; }
public string StandardName { get; set; }
public ICollection<Student> Students { get; set; }
}
No código acima, temos um mapeamento de 1:N, onde um Standard
possui vários Students
, onde a propriedade StandardId
é a foreing key do modelo Student
. Está correto, se o objetivo for esse. Utilizando o Entity Framework, não é necessário utilizar a anotação [ForeignKey]
, pois o nome da foreing keyforeing key é o nome da propriedade de navegação com o prefixo id
, ou seja, StandardId
(Standard + Id).
Em contra partida, o modelo Standard
possui uma coleção de Students
, onde o mesmo não necessita de ter uma foreing key para Students
, caracterizando a ligação de 1:N.