Локальная переменная Декларация Issue

Question

Im получаю следующее сообщение об ошибке:

Cannot use local variable 'dob' before it is declared

Вот моя реализация

public class Person 
    { 
     ... 
     public string dob { get; set; } 
     ... 

     public int getAge() 
     { 
       DateTime origin = DateTime.Parse(dob); 
       return DateTime.Today.Year - origin.Year; 
     } 

     public string getFormattedDoB() 
     { 
       DateTime origin = DateTime.Parse(dob); 
       string dob = origin.ToString("d"); 
       return dob; 
     } 
    } 

Я не уверен, что сделать это, потому что он жалуется на это использование DOB в getFormattedDoB(), но не в getAge(), который приходит перед ним. Если бы кто-нибудь мог пролить свет на это, это было бы здорово

Answer 1

Вы указали локальную переменную в getFormattedDoB, называемую dob. Компилятор не может определить разницу между этим и членом dob. Попробуйте добавить «это», где вы имеете в виду переменную-член, а не местный:

DateTime origin = DateTime.Parse(this.dob); 

еще лучше, не использовать то же имя для локальной переменной.

Редактировать: Если вы действительно не намеревались устанавливать переменную-член, а не создавать новую. В этом случае удалите «строку», как предложил Эндрю.

Answer 2

Вы повторно использовали имя переменной «dob» в getFormattedDoB как локальную строку, которая запутывает компилятор. Существует 2 возможных решения:

1. Переименуйте локальный dob в getFormattedDoB (который вы действительно должны делать, потому что это хорошая практика).

2. Использование this.dob в следующей строке, чтобы указать переменную уровня класса (который вы, вероятно, следует также делать, потому что это также хорошая практика:

   DateTime происхождения = DateTime.Parse (this.dob);

Answer 3

проблема заключается в том, что у вас есть два dob s- собственности и локальной переменной. Область действия переменной декларации (string dob = ...) является весь блок (все между { и }). Поэтому компилятор считает, что на line:

DateTime origin = DateTime.Parse(dob); 

вы используете dobпеременную прежде, чем она объявлена, когда (мы предполагаем), вы на самом деле имел в виду dobсвойство.

Как уже упоминалось, вы должны переименовать это свойство. Стандартное соглашение об именовании в C# будет

public String DateOfBirth { get; set; } 
//(assuming that is what DOB stands for) 

или еще

public DateTime DateOfBirth { get; set; } 

Answer 4

Вы повторного объявления DOB лучше

string dob = origin.ToString("d"); 

должно быть

dob = origin.ToString("d");