[devel] [SCM] packages/hlint: heads/master

[Alexander Bokovoy]

2008/11/18 Alexander Bokovoy <ab на altlinux.org>:
> 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)) { ... }

Кстати, в Objective-C это выглядит естественнее с лингвистической точки зрения:

- (void)processFileResource:(ResourceHandle)handle
{
   if ([handle isFile]) {
    ....
  }
}


-- 
/ Alexander Bokovoy