|
ru.algorithms
- RU.ALGORITHMS ----------------------------------------------------------------
From : Sergiy Kanilo 2:5020/400 27 May 2003 22:19:51
To : Yuri Burger
Subject : Re: Hечеткое сравнение строк
--------------------------------------------------------------------------------
"Yuri Burger" <kruger@selena.net.ua> wrote in message
news:bautbp$qa1$1@news.lucky.net...
> // programmed by Burger J.A. (aka Kruger)
я не знаю как насчет реализованного алгоритма,
но с точки зрения стиля - полная мешанина всего
что можно придумать
безымянное namespace в данном случае практически
бесполезно
ведущие подчеркивания в идентификаторах не
рекомендуются
параметры d и maxD вполне ИМХО можно заменить одним,
хотя проверка _d в цикле и присвоение ему 0 в конце
выглядит несколько подозрительно
в первом темплейте в типе T указатель можно угадать
после третьего прочтения, (возможно подразумевались
итераторы, но +1 для стандартных итераторов недопустимо)
С++ допускает описание переменных практически в любом
месте кода и тут Сшный или FORTRANовский стиль ни к чему,
minmax нестандартен (стандартным есть <algorithm> )
и иклудится по-видимому только для одного min и одного max,
while(true) с последующим if()brake; и ++ для параметров в конце,
вы меня извините, но в этом for() видится за версту
трижды встречающийся кусок lf=...; if(best<lf){best=lf;} так и
просится в инлайновую функцию
введение s1 и s2 основано на предвосылке, что экономия
одного такта при компиялции без оптимизации важнее, чем
простота и ясность кода
static_cast<doube> - ИМХО это не то место, где стоит
его использовать
хотя сохранение исходных значений параметров и
рекомендовалось в литературе 15-20 лет назад,
но в данном случае это не нужно, сократит строк 5
кода и оградит от придумывания новых имен
пустые строки не сравниваются, а что стоило немного
расширить область параметров чтобы немного оградить
пользователя
Cheers,
Serge
--- ifmail v.2.15dev5
* Origin: Demos online service (2:5020/400)
Вернуться к списку тем, сортированных по: возрастание даты уменьшение даты тема автор
|
