Mapp کردن کلاس ها در فرم ویرایش

سلام استاد عزیز وقتتون بخیر خسته نباشید. استاد بنده  برای تمامی کلاس های پروژه کلاس Mapping درست کردم و برای Add و Edite از اون استفاده میکنم خواستم بودونم آیا ساخت کلاس Mapping برای تمامی کلاس ها کار صحیحی هست؟ و ایکه من توی فرم ویرایش هم از اون استفاده میکنم برای مثال :

private void btnSave_Click(object sender, EventArgs e)
{
    if (IsValidateInputs() == true)
    {
       var data = PersonMapper.MapToPerson(txtName.Text, txtFamily.Text, (int)nudAge.Value, txtEmail.Text, txtMobile.Text, rbTeacher.Checked ? true : false);
       
       
  data.PersonId = _personId;
        _personRepositories.UpdatePerson(data);
        _personRepositories.Save();


        //Person person = new Person();
        //person.PersonId = _personId;
        //person.Name = txtName.Text;
        //person.Family = txtFamily.Text;
        //person.Age = (int)nudAge.Value;
        //person.Mobile = txtMobile.Text;
        //person.Email = txtEmail.Text;
        //if (rbTeacher.Checked == true)
        //{
        //    person.IsTeacher = true;
        //}
        //else
        //{
        //    person.IsTeacher = false;
        //}
    
        this.DialogResult = DialogResult.OK;
    }



}

فقط استاد اون قسمت که RadioButton رو چک میکنیم که true یا false هست رو شرطش در اون جایی که دارم مقادیر رو پاس میدم به کلاس Mapping نوشتم.

استاد عذر میخوام بنده یه کار دیگه ای هم انجام دادم خواستم نظرتون بدونم ای درسته 

بنده در این بخش لیست دانشجویان که نام و نام خانوادگی و سن و موبایل هست رو براش یه متود جدا در ریپوزیتوری و یه View Model براش ساختم 

namespace DataLayer.ViewModel
{
    public class StudentCourseListItemViewModel
    {
        public int PersonId { get; set; }
        public string FullName { get; set; }
        public int Age { get; set; }
        public string Mobile { get; set; }
    }
}

       public List<StudentCourseListItemViewModel> GetStudentCourseList()
       {
           List<StudentCourseListItemViewModel> studentCourses = _Context.people.Where(p => p.IsTeacher == false)
                .Select(p => new StudentCourseListItemViewModel
                {
                    PersonId = p.PersonId,
                    FullName = p.Name + " " + p.Family,
                    Age = p.Age,
                    Mobile = p.Mobile

                }).ToList();
           return studentCourses;
       }

در این متودی که ساختم دیگه توی لیست دانشجویان اسم استاد قرار نمیگیره 

ممنون میشم نظرتون رو بگید که آیا اصولی بوده یا خیر