> void sqrtroot(double a,double b,double c,double &x1,double &x2)
> {
> double d=b*b-4*a*c;
> if(d>=0)
> {
> x1=(-b+sqrt(d))/2*a;
> x2=(-b-sqrt(d))/2*a;
> }
> }
> А теперь разберём все ошибки :).Давайте сразу перейдём от никуда не годного к более правильному решению: http://ipg.h1.ru/lessons/ci/les31.html
> 1. "а" может быть 0, делить на 0 нельзя.
Не может — уравнение не будет квадратным, решение по приведённой функции бессмысленно.
> 2. "х1" и "х2" могут быть NULL или ссылаться на недоступную память.
Це-проблемы?
> 3. "а", "b" и "с" могут быть подобраны так, что операции над
> ними превысят предел точности double.
Для вычислений, требующих точного результата, не используйте типы float и double. Правильный путь решения вычислительной задачи заключается в применении для расчётов типов BigDecimal, int или long © Джошуа Блох.
> 4. ситуация когда d<0 не обрабатывается, что чревато ошибками в других функциях.
Классический пример неучёта всех без исключения требований ТЗ и сути проблемы.
> А теперь посмотрим на функцию sqrt, вы уверены что она не
> написана столь же "безграмотно", как и приведённая мной?
"Доверяй, но проверяй"?
> Ну, кто возьмётся переписать код с исправлением ВСЕХ ошибок? Только, пожалуйста, не
> предлагайте решения через try...catch, если вы работаете на i7 с 10
> Гб ОП, то не всем так повезло.
А причём тут 10 Гб ОЗУ?
> А затем увеличьте размер кода с приведённых 4 (всего лишь) до ~100.000
> и сделайте выводы. Обратите внимание во сколько раз увеличитсяся размер кода
> если выполнять все проверки.
А вы оставляете assert'ы в готовом коде?