[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