Главная страница
    Top.Mail.Ru    Яндекс.Метрика
Форум: "Прочее";
Текущий архив: 2011.07.10;
Скачать: [xml.tar.bz2];

Вниз

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

 
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;
Скачать: [xml.tar.bz2];

Наверх





Память: 0.49 MB
Время: 0.002 c
2-1301675364
Gu
2011-04-01 20:29
2011.07.10
PhysicalDrive


9-1189378302
max_
2007-09-10 02:51
2011.07.10
текстура


2-1301635425
mefodiy
2011-04-01 09:23
2011.07.10
Не работает режим "full screen"


15-1300952862
oxffff
2011-03-24 10:47
2011.07.10
Сегодня полетит к нам Ижемский ТУ-154М


3-1261512342
XeON
2009-12-22 23:05
2011.07.10
SQLite3 без "врапперов"





Afrikaans Albanian Arabic Armenian Azerbaijani Basque Belarusian Bulgarian Catalan Chinese (Simplified) Chinese (Traditional) Croatian Czech Danish Dutch English Estonian Filipino Finnish French
Galician Georgian German Greek Haitian Creole Hebrew Hindi Hungarian Icelandic Indonesian Irish Italian Japanese Korean Latvian Lithuanian Macedonian Malay Maltese Norwegian
Persian Polish Portuguese Romanian Russian Serbian Slovak Slovenian Spanish Swahili Swedish Thai Turkish Ukrainian Urdu Vietnamese Welsh Yiddish Bengali Bosnian
Cebuano Esperanto Gujarati Hausa Hmong Igbo Javanese Kannada Khmer Lao Latin Maori Marathi Mongolian Nepali Punjabi Somali Tamil Telugu Yoruba
Zulu
Английский Французский Немецкий Итальянский Португальский Русский Испанский