Главная страница
Top.Mail.Ru    Яндекс.Метрика
Текущий архив: 2011.07.10;
Скачать: CL | DM;

Вниз

Как бы Вы написали и почему?   Найти похожие ветки 

 
OW ©   (2011-03-24 09:11) [0]

function NotHasNew:Boolean;
..
If NotHasNew then

или

function HasNew:Boolean;
..
If  not(HasNew) then


 
И. Павел ©   (2011-03-24 09:19) [1]

Я за №2, т.к. если потребуется написать "not NotHasNew" то это уже будет перебор :)


 
Юрий Зотов ©   (2011-03-24 09:22) [2]

> Как бы Вы написали
Так, чтобы в теле самой функции не было лишней инверсии.

> и почему?
Функция может использоваться в разных местах в разном контексте - и если где-то ее результат не надо инвертировать, то получим хоть и мизерную, но все же экономию.

PS
Мне бы Ваши заботы...
:o)


 
brother ©   (2011-03-24 09:25) [3]

2


 
brother ©   (2011-03-24 09:25) [4]

отрицательные имена
> NotHasNew

стараюсь не использовать...


 
sniknik ©   (2011-03-24 09:30) [5]

написал бы isHasNew ... и условия формировал бы, старался, так
if  isHasNew then
(с положительным первым)
а если чаще проверка на "старость" то вместо isHasNew с not написал бы isHasOld, чтобы опять первым...
а если равновероятно, и это объект... то сделал бы 2 проперти с такими названиями.


 
Думкин ©   (2011-03-24 09:32) [6]

THasNewOrNotHasNew = (hasNew, notHasNew, hasOrNot,hasAndNot);

function HasNewOrNotHasNew : THasNewOrNotHasNew;

....

if (HasNewOrNotHasNew=notHasNew) then...


 
Anatoly Podgoretsky ©   (2011-03-24 09:35) [7]

> OW  (24.03.2011 09:11:00)  [0]

Второе легче читается.


 
Anatoly Podgoretsky ©   (2011-03-24 09:37) [8]

> sniknik  (24.03.2011 09:30:05)  [5]

А код вероятнее всего был бы одинаковым


 
Dennis I. Komarov ©   (2011-03-24 09:51) [9]

1. not всегда можно написать отдельно
2. HasNew тоже не очень-то информативно
З.Ы.
       Я бы написал вроде FileExists


 
OW ©   (2011-03-24 09:53) [10]

так то да..
второе..
только..
только всегда нужно знать если нет новых записей, и никогда не нужно если они есть
во всяком случае пока :)
а так - not NotHasNew - это да, аргумент за 2.

> isHasOld - м.б., действительно так


> Думкин ©   (24.03.11 09:32) [6]

не, так бросил писать, не успеваю потом переписывать :)

Спасибо, All!


 
Думкин ©   (2011-03-24 10:01) [11]


> > isHasOld - м.б., действительно так


isHasOnlyOld


 
Игорь Шевченко ©   (2011-03-24 10:35) [12]


> > isHasOld - м.б., действительно так


что за насилие над языком ?


 
Думкин ©   (2011-03-24 10:42) [13]


> Игорь Шевченко ©   (24.03.11 10:35) [12]

твоя моя не понимай


 
Anatoly Podgoretsky ©   (2011-03-24 11:05) [14]

> Думкин  (24.03.2011 10:42:13)  [13]

Так Is и Has в одном предложение


 
Думкин ©   (2011-03-24 11:07) [15]


> Anatoly Podgoretsky ©   (24.03.11 11:05) [14]

Я понял, потому и пишу. Надо, видимо, тоньше. Пойду тренироваться на кошках.


 
Dimka Maslov ©   (2011-03-24 11:18) [16]

ThereIsNoNewItems


 
TUser ©   (2011-03-24 11:20) [17]

пофиг


 
asail ©   (2011-03-24 11:30) [18]

IsOnlyOld ?


 
Anatoly Podgoretsky ©   (2011-03-24 12:03) [19]

> Dimka Maslov  (24.03.2011 11:18:16)  [16]

ThereAre?


 
clickmaker ©   (2011-03-24 14:05) [20]

я бы уточнил HasNewЧто


 
OW ©   (2011-03-24 15:11) [21]

переименовал
немного удобнее имхо читать

function HasOnlyOldPhone(IdSubject: Integer): boolean; overload;
function HasOnlyOldPhone(Subject: TSubject): boolean; overload;

if HasOnlyOldPhone(



Страницы: 1 вся ветка

Текущий архив: 2011.07.10;
Скачать: CL | DM;

Наверх




Память: 0.51 MB
Время: 0.009 c
2-1301921321
Елена
2011-04-04 16:48
2011.07.10
построение фигур в delphi


4-1248621510
Nikfel
2009-07-26 19:18
2011.07.10
Как перекрасить меню полностью


11-1234115025
Lexi
2009-02-08 20:43
2011.07.10
Где можно взять KOLMHToolTip.pas?


2-1301853911
Basilisk
2011-04-03 22:05
2011.07.10
Программное создание элементов


1-1258972053
Cobalt
2009-11-23 13:27
2011.07.10
XMLNode - List index out of bounds