[devel] [SCM] packages/hlint: heads/master
Alexander Bokovoy
=?iso-8859-1?q?ab_=CE=C1_altlinux=2Eorg?=
Вт Ноя 18 22:22:43 MSK 2008
2008/11/18 Alexey I. Froloff <raorn на altlinux.ru>:
> * Slava Semushin <php_coder@> [081118 20:00]:
>> - IndirectSourceFile(fileExist): removed public method.
>> - IndirectSourceFile(isFileExists): added new public method.
>
>> - bool fileExist() const {return fileWasOpened;}
>> + bool isFileExists() const {return fileWasOpened;}
>
> Операция "remove/add" с similarity index равным 100% называется
> "rename" или "move". К тому же по правилам английского языка эта
> функция должна называться "doesFileExst()".
На API логических функций нужно смотреть в контексте их использования.
Их обычно их именуют так, чтобы получить читаемый английский текст.
Сравните:
if (isFileExists(file)) { .... }
и
if (FileExists(file)) { .... }
Префикс is обычно используется для записи is-a отношений, где
выполняется проверка принадлежности объекта определенному классу:
if (isFile(handle)) { .. }
if (!isNapkins(paper)) { ... }
--
/ Alexander Bokovoy
Подробная информация о списке рассылки Devel