Написал простую прогу с использованием шаблонов, вроде всё правильно, но дело в том,что условие   не работает. 

Я специально поставил значения в х1 = 8, х2 = 8,чтоб вывелся результат только один,

А выдаёт 2 результата.







Это сообщение отредактировал(а) Syltan - 18.9.2009, 14:20

здесь ошибка. сам догадаешься в чём?

Это место работате нормально, и с else,та же история.
Даже если поставить  будет тот же самый вывод:



Странно,но это условие не работает:


Это сообщение отредактировал(а) Syltan - 18.9.2009, 14:24

теперь тут ошибка. сам догадаешься в чём?

если a больше b, то выход со значением a, иначе выход со значением b. И если a равна b - выход со значением ноль. 

чувствуешь нестыковку?

Это сообщение отредактировал(а) Cheloveck - 18.9.2009, 14:32

Хотите сказать, что вот так будет правильно:


А else в таких случаях не используют?

Это сообщение отредактировал(а) Syltan - 18.9.2009, 14:32

Зачем лишний else,тогда лучше вот так:


Но дело в том,что я хочу чтоб не выдавало результат = 0, чтоб,если одно число равно другому, тогда ничего на экран не выдавало.

Это сообщение отредактировал(а) Syltan - 18.9.2009, 14:37

Это сообщение отредактировал(а) Cheloveck - 18.9.2009, 14:38

Cheloveck, как-то ты код извращённо переписал. 
я думаю, что сначала в if'ах должны быть перечислены стандартный/правильный поток выполнения программы а уже после них - обработка ошибочных вариантов.



Это сообщение отредактировал(а) zim22 - 18.9.2009, 14:41

Он не лишний. Потом поймёшь.

И даже если написать вот так:


Результат программы будет таков:


А нужно вот такой чтоб был:


Это сообщение отредактировал(а) Syltan - 18.9.2009, 14:45

zim22, если говорить о правильном подходе, то это должно быть вообще так


Добавлено через 1 минуту и 35 секунд
ну, или T вместо int

ок. где ты вычитал об этом "правильном" подходе? в чём его преимущества перед моим? (свой я вычитал в Макконнеле "Совершенный код")

недостатки "правильного" подхода вижу такие: 
1) создание ненужной переменной result, без которой можно обойтись. 
2) тип у result кстати int. хотя это шаблонная функция и передавать ей можно всё-что угодно. хоть тип Booblik
3) инициализация переменной result нулём. а что если тип у переменной не int и конструктор преобразования не определён?
4) сравнение идёт по оператору БОЛЬШЕ, а не МЕНЬШЕ. который с меньшей степенью вероятности определён в классах, чем МЕНЬШЕ 

вот всё работает

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


Это сообщение отредактировал(а) Syltan - 18.9.2009, 15:00

значит перед выводом на экран проверяй что вернёт функция.

до поры, до времени

почитай как сравнивать double ссылка

Это сообщение отредактировал(а) zim22 - 18.9.2009, 15:12

именно об этом хотел сказать, но на работе инет отрубили.

Как говорят в Одессе, пишите, как хочете.    Я же предлагаю такой вариант:

об этом говорят все, return'ов должно быть как можно меньше, в идеале один. Это упростит отладку.



Оптимизатор позаботится об этой переменной. А так как ТС у себя возвращает объект по значению, всё равно "лишняя" переменная будет сконструирована.
 

поправился за 2 минуты до цитируемого поста.


У ТС тоже ноль возвращается. А при объектах типа Booblik это может быть ооочень чревато, так как мы создадим одну дырку от бублика, но это отдельная тема. Здесь мы говорим о встроенных типах, я думаю.


Ну, это уже не суть важно. Хорошо спроектированный объект имеет оба оператора.

Добавлено через 1 минуту и 40 секунд

В данном случае речь идёт именно о конструкции if-else. А так, я чаще всего в таких случаях пользуюсь той же конструкцией.

Нет, как я понял, речь не об этом, а о том, что не работает сравнение на равенство двух операндов. Но если даже и так, то мне ближе вариант, который предложил zim22, а именно: 

Спасибо Dov, вот этот вариант самый лучший, это я и имел ввиду.
 



Это сообщение отредактировал(а) Syltan - 18.9.2009, 20:11

M bsa

Syltan, если ответ на вопрос получен, то пометь тему решенной

Я не знаю как её пометить.

Наверху справа (под листочками "ответить", "новая тема", "опрос").