Главная Загрузки Проекты Ссылки Гостевая Мысли вслух

Размышления на тему

Deanit, 
Интересно. Молодец. Изучал проект DMECalc
Но надеюсь ты воспримешь правильно критику :

Не обижайся, но в коде есть недочеты. Если ты собрался помочь новичкам, то стиль твоего кода должен быть таким, чтоб его нельзя было отличить от майкрософтовского, что б на тебя равнялись и за тобой повторяли. Что я имею ввиду :
а) в коде много "сигналов из космоса" например : 
Что значит число 33 ? Возраст Христа или максимальное количество возможных чисел ? Почему б это число не превратить в именованию константу (типа MAXIMUM_DIGITAL_COUNT) ?
б) Встречаются члены класса, с очень интригующими названиями как например Mem0, Mem1
в) Очень много "пустых" операторных скобок
г) Не ясное использование операндов в методе Resultat(). Почему б параметр functionIndex не превратить в перечисление ? 
И человеку который твой код изучает понятно и программе, а сейчас только понятно тебе и программе - это не правильно.
д) Почему б не использовать единый метод для всех цифровых кнопок ? И в этом методе привязаться к такому замечательному свойству объекта Control как Tag (см. ObjectInspector) ?

И код этого метода будет следующим :


	CODE	


private void NumberButton_Click(object sender, System.EventArgs e)

{

                        isFuncButton=false;

if(CalcScreen.Text.Length> MAXIMUM_DIGITAL_COUNT)

                              return;

string buttonCaption = (sender as Control).Tag.ToString();


if (!isFirstDigit)

CalcScreen.Text +=buttonCaption;

else 

{

                                        operand1=Convert.ToDouble(CalcScreen.Text);

CalcScreen.Text=buttonCaption;

isFirstDigit=false;

}


if (AfterEq)

{

operand1=0;

funcIndex=0;

AfterEq=false;

}

}




е) Существует скрытая угроза в твоем использовании isFuncButton. Посмотри куда я его перенес.

ё) Было бы правельние следующие строки разнести по отдельным методам :

Один метод :

	CODE	


{

                                        operand1=Convert.ToDouble(CalcScreen.Text);

CalcScreen.Text=buttonCaption;

isFirstDigit=false;

}



Второй метод :

	CODE	


{

if (!AfterEq)

                                   return;



operand1=0;

funcIndex=0;

AfterEq=false;

}




А в целом - не плохо. Не обращай внимания на необоснованные высказывания некоторых разработчиков. Следуй принципу : Программировать не на языке, а при помощи него.