Использовать console.ReadLine() внутри свойства set

Question

Я начинающий кодер, сейчас я изучаю C#, и мне было интересно, можно ли использовать Console.ReadLine() внутри заданной части свойства, а затем использовать его как метод для чтения пользовательского ввода, как показано ниже:

class Employee 
{ 
    protected int empID; 
    public int EmployeeID 
    { 
     set 
     { 
      Console.WriteLine("Please enter Employee ID:"); 
      this.empID = int.Parse(Console.ReadLine()); 
     } 
    } 
    //more code here 
} 
class Program 
{ 
    static void Main(string[] args) 
    { 
     Employee employee1 = new Employee(); 
     employee1.EmployeeID; 
     //more code here 
    } 
} 

или единственным вариантом является использование Console.ReadLine() непосредственно в «основной», следующим образом:

class Employee 
{ 
    protected int empID; 
    public int EmployeeID { set; } 
    //more code here 
} 
class Program 
{ 
    static void Main(string[] args) 
    { 
     Employee employee1 = new Employee(); 
     employee1.EmployeeID = int.Parse(Console.ReadLine()); 
     //more code here 
    } 
} 

Спасибо за все ответы!

---

Благодарим всех вас за ответы! Теперь я вижу, что это неправильный способ написать код, и я понимаю, почему. Я думал, что с помощью 'Console.ReadLine();' в свойстве «набор» будет легче получить значение от пользователя и не придется переписывать эту часть:»

Console.WriteLine("Please enter Employee ID:"); 
this.empID = int.Parse(Console.ReadLine()); 

каждый раз, когда я буду просить у пользователя ввод. Но теперь я понимаю, почему его не следует использовать.
Еще раз благодарю вас за ответы и приятный день!

Answer 1

Да, вы можете поставить Console.ReadLine() внутри комплекта. Но это очень неправильно.

Свойства C# скомпилированы подобно методу, поэтому вы можете поместить любой доступный код C# внутри свойства, компилятор позволит вам это сделать. (Проблема в коде заключается в том, что вы не пишете правильный вызов для набора).

Но, думая о хороших практиках и S.O.L.I.D, это очень неправильно. Второй фрагмент кода выглядит намного лучше.

Просто для любопытства (не используйте это):

Для первого кода работы вам нужно переписать строку:

employee1.EmployeeID; 

Чтобы что-то вроде этого:

employee1.EmployeeID = 0; // This syntax will call the property set you wrote. 

Просмотреть проблему сейчас? Когда вы читаете код, он, кажется, устанавливает ваше свойство с нулем (0), но вместо этого код читается с консоли, поэтому вы можете понять, что беспорядок будет, если вы его используете.

Answer 2

Вы направляетесь в неправильном направлении. Правильный способ - через метод Main.

Или, если вы хотите поставить функциональность в своем классе по какой-либо причине, это должно быть таким образом

class Employee 
{ 
    protected int empID; 
    public int EmployeeID 
    { 
     get { return empId; } 
    } 
    //more code here 
    public void AskEmployeeID() 
    { 
     Console.WriteLine("Please enter Employee ID:"); 
     this.empID = int.Parse(Console.ReadLine()); 
    } 
} 

Теперь вы можете вызвать эту функцию на Employee объекта в employee1.AskEmployeeID();

Answer 3

public class Employee 
{ 
    public int EmployeeID { get; set; } 
} 
class Program 
{ 
    static void Main(string[] args) 
    { 
     Console.WriteLine("Please enter Employee ID:"); 

     var empID = int.Parse(Console.ReadLine()); 

     var employee1 = new Employee 
     { 
      EmployeeID = empID 
     }; 
    } 
} 

Getters и Setters следует использовать только для установки/возврата любого значения, которое имеет свойство. Вы также можете создавать частные поля и устанавливать их разными способами. Однако вы не будете звонить Console.ReadLine() из класса. Класс представляет собой представление вашего объекта.