Приветствую вас! Имеется калькулятор, состоящий из следующих частей:

•  поле Edit, в которое вводятся опреанды и выдаётся результат
•  кнопки операций 
•  кнопка "равно"
•  кнопка очистки

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


Переменные в области private:


Код для кнопки очистки:


Подскажите пожалуйста, всё ли здесь корректно в части вычислений? Попробовал посчитать разные выражения - вроде нормально. Но не уверен, что всё предусмотрено.

Это сообщение отредактировал(а) Dilettante - 15.10.2012, 18:13

В части вычислений вроде всё корректно. Если позволишь, придерусь к стилю.
1) Для обработки кнопки "равно" лучше сделать отдельный обработчик, т.к. часть, отвечающая за обработку "равно" никак не связана с частью, отвечающей за обработку операций
2) В чём смысл держать double res; и wchar_t operation; членами класса ? Почему не объявить их прямо в функции-обработчике ?
3) Зачем нужно преобразование из enum'а в символ и обратно, если можно хранить очередь из enum'ом ?
4) (Необязательное) Чтобы отличать члены класса от переменных, объявленных в функции, у первых д.б. какой-нибудь признак. Иногда используют подчёркивание в конце имени, иногда префикс m_ - на твой вкус.

Можно просто писать this->MyVar. Так точно не запутаешься )

Я так понял если вводить такое выражение
1+6*7-3/8 = ????
какое число выведет твой калькулятор ?

2borisbn, artsb
Спасибо за замечания. Об отдельном обработчике для равенства я не подумал. Как-то включил его машинально к операциям. Вообще у меня изначально обработка всего поведения, связанного с полем TEdit, скопилась в одном методе.  
Насчёт очереди из символов, с одной стороны, я к ней ещё обращаюсь при выводе строки, хранящей последовательность операций: 

Ну а с другой - опять же, не подумал.
Совет про именование переменных класса учту.

2Dem_max
Выведет 5,75, т.к. не соблюдает приоритеты операций и считает такое: ((1+6)*7-3)/8. 

2All
Спасибо вам за внимание к моей проблеме! Попытаюсь сдать (лабораторная, как вы, возможно, догадались).

на здоповье. сравнивай так

а для их отображения сделай функцию перевода enum'а в символ и вызывай её

сама функция op_2_char, как ты понимаешь, будет совсем несложная ))

2borisbn

Да, удачно получается   Стоит так и сделать. Спасибо!  

Ну что тогда покритикую как просил - реализация фиговая.

2Dem_max

Тут вы правы. На большее не хватило. Но я надеюсь, что для моих нужд будет достаточно.  

Тут всё намного проще:

И теперь можно так:

Это будет работать быстрее, чем вызов функции.
Надеюсь, идея понятна.

>    Это будет работать быстрее, чем вызов функции.
Готов поспорить, что дааааааааапалеко не всегда. Сферических коней в вакууме не предлагать
  

Ну, смысла спорить я не вижу, т.к. тут всё очевидно, ИМХО.
При обращении к элементу массива ничего особенного не происходит. А вот при вызове функции, происходт много чего. Например, передача параметра; переключение на вызываемую функцию; работа в самой функции (скорее всего, там будет switch); возвращение значения из функции, что влечёт за собой создание временного объекта. Возможно, ещё что-то упустил. Как бы там ни было, на всё это нужно время.
Поправьте меня, если я не прав.

> Поправьте меня, если я не прав.
Теоретически (как в том анекдоте) Вы правы, а практически компилятор сделает из switch'а таблицу, и заинлайнит функцию. Проверьте. Сделайте два варианта и посмотрите ассемблерный код (с включенной оптимизацией, ессно)

Вот. Ключевой момент ) И тем не менее, даже если функция будет оптимизирована и заинлайнится, неужели простая операция с адресом (указатель на массив + смещение) будет длить дольше?

Добавлено через 2 минуты и 17 секунд
Кстати, обе операции (мой вариант и ваш вариант с функцией) будут отрабатывать очень быстро. Настолько, что разницу заметить очень тяжело.

Использование массива мне нравится (и кажется наиболее быстрым, т.к., наверное, O(1) не идёт из головы   ), хотя работу уже сдал с правками в стиле borisbn). 

borisbn, artsb, Dem_max, 
Спасибо вам за помощь и внимание к теме!