The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Релиз ftp-сервера ProFTPD 1.3.4 и 1.3.3g с устранением крити..., opennews (??), 10-Ноя-11, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


22. "Релиз ftp-сервера ProFTPD 1.3.4 и 1.3.3g с устранением крити..."  +/
Сообщение от iZEN (ok), 10-Ноя-11, 19:55 
> 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'ы в готовом коде?

Ответить | Правка | К родителю #13 | Наверх | Cообщить модератору

56. "Релиз ftp-сервера ProFTPD 1.3.4 и 1.3.3g с устранением крити..."  +/
Сообщение от Ваня (?), 11-Ноя-11, 11:18 
> Давайте сразу перейдём от никуда не годного к более правильному решению

Цель: демонстрационный пример кода, содержащего ошибки. Какое "более правильное решение" ты мне предлагаешь?

> "а" может быть 0, делить на 0 нельзя
> уравнение не будет квадратным, решение по приведённой функции бессмысленно

А функции как-то всё равно бессмысленно или осмысленно :). Будет ERROR.
Кстати, здесь была первая намерено допущенная "ошибка": нужно не "/2*а", а "/(2*а)". Её заметили, это хорошо, не всё ещё забыли квадратные уравнения.

> "х1" и "х2" могут быть NULL или ссылаться на недоступную память.
> Це-проблемы?

"Программа обратилась к недопустимой памяти и будет закрыта". Ничего не напоминает?
Здесь была вторая намерено допущенная "ошибка": "x1=..." - это присвоение указателю, чтобы обратиться по значению, указатель нужно разименовать. Я специально оставил ошибки чтобы показать что даже внешне корректно написанный код может содержать ошибки.

> > 3. "а", "b" и "с" могут быть подобраны так, что операции над
> ними превысят предел точности double.
> Для вычислений, требующих точного результата, не используйте типы float и double. Правильный путь решения вычислительной задачи заключается в применении для расчётов типов BigDecimal, int или long © Джошуа Блох.

Глупость написал. Ещё раз: значения могут быть подобраны так, что превысят предел точности типа данных. Неограниченных типов данных не бывает. К тому же, приведённые тобой типы целочисленные, а задача требует вычислений с плавающей точкой.

Ответить | Правка | Наверх | Cообщить модератору

61. "Релиз ftp-сервера ProFTPD 1.3.4 и 1.3.3g с устранением..."  +/
Сообщение от arisu (ok), 11-Ноя-11, 14:29 
да хватит уже бугуртить. мы уже поняли, что новость про epic fail с upd-пакетами никак не может оставить тебя равнодушной.
Ответить | Правка | Наверх | Cообщить модератору

94. "Релиз ftp-сервера ProFTPD 1.3.4 и 1.3.3g с устранением крити..."  +/
Сообщение от Аноним (-), 11-Ноя-11, 17:49 
> Давайте сразу перейдём от никуда не годного к более правильному решению:
> http://ipg.h1.ru/lessons/ci/les31.html

Писец годное решение:    #include <windows.h>

Портабельность консольной программы прибита на ровном месте. За такое расстреливать надо.

> А вы оставляете assert'ы в готовом коде?

Некоторые в сомнительных оставляют. Лучше если программа сразу упадет и баг будет локализован и починен, чем когда вместо падения сразу будет полчаса глюкания, потом все навернется, но баг никто никогда не обнаружит, потому что бэктрейсить на полчаса назад - нереально напрочь.

Ответить | Правка | К родителю #22 | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру