Настройка свойства только для чтения по методу

Question

У меня есть простая проблема, но я застрял в качестве новичка.

Мой метод SetGrade принимает параметр float, и я хотел бы, чтобы он возвращал char , а затем задал это значение для свойства Grade.

Я ничего не делаю правильно.

public Class Student { 
    private char grade; 

    public char Grade { get { return grade; } } 

    public char SetGrade(float score) { 
     char Mgrade; 
     if(score >= 90.0) { 
      return Mgrade = 'A'; 
     } 
     return Mgrade = 'F'; 
    } 
} 

Answer 1

Ваш синтаксис откусил:

public char SetGrade(float score) 
    { 
    if(score >= 90.0) 
    { 
     return 'A'; 
    } 
    return 'F'; 
    } 

Нет необходимости в Mgrade переменной, и вы должны просто вернуть персонаж хотел, вместо присваивания и возврата.

Answer 2

Не нужно назначать свой символ промежуточной переменной char. Просто верните правильный характер, как это.

public char SetGrade(float score) 
{ 
    if(score >= 90.0) 
    { 
     return 'A'; 
    } 
    return 'F'; 
} 

Answer 3

Существует множество проблем с этим кодом, но они могут быть не такими, какими вы считаете.

Прежде всего, Public неправ; C# требует public.

Во-вторых, использование местных Mgrade странно и не нужно, но достаточно интересно не на самом деле wrong; законно выполнять задание и возвращать за один шаг. Но в этом случае вам не нужно; только return 'A'; без местного задания.

В-третьих, метод неправильный, потому что не устанавливает Grade. Если вы собираетесь его установить класс, то он должен быть недействительным возвращения:

public void SetGrade(float score) 
{ 
    if(score >= 90.0) 
    { 
     this.grade = 'A'; 
    } 
    this.grade = 'F'; 
} 

Если вместо этого метод предназначен, чтобы быть переход от поплавков до символов, то оно должно быть статическим:

public static char ScoreToGrade(float score) 
{ 
    if(score >= 90.0) 
    { 
     return 'A'; 
    } 
    return 'F'; 
} 

Откровенно , Я был бы склонен делать то и другое:

public void SetGrade(float score) 
{ 
    this.grade = ScoreToGrade(score); 
} 

Там, у вас есть лучшее из обоих миров.

В-четвертых, это всего лишь стилистическая точка; Вы могли бы рассмотреть:

public char Grade { get; private set; } 

компилятор будет генерировать «невидимый» подкладочный поле для вас, так что вы не должны управлять сами. Этот синтаксис означает, что Grade может быть прочитан из любого места и написан с этого класса.