Следует ли инициализировать переменную перед вызовом функции?

предполагая, что vPropValue является локальной переменной, то этот код

vPropValue := 0;
vPropValue := GetPropValue(vObject,'Height');

неотличима от

vPropValue := GetPropValue(vObject,'Height');

более простой пример может быть таким:

i := 0;
i := 1;

какой смысл назначать 0 to i и затем сразу же назначить 1 to i? Значит, ты никогда не напишешь этого. Вы напишете:

i := 1;

в вашем коде в верхней части этого ответа вы назначаете дважды одну и ту же переменную. Этот значение, назначенное в первом назначении, немедленно заменяется значением, назначенным во втором назначении. Поэтому первое назначение бессмысленно и должно быть удалено.

второй пример немного сложнее. Предполагая, что ваши функции написаны правильно и всегда присваивают их возвращаемое значение, и что vName является локальной переменной, то

vName := '';
vName := GetObjectName(vObject,'ObjectName');

неотличима от

vName := GetObjectName(vObject,'ObjectName');

причина, по которой я добавил дополнительное условие относится к причуде реализации возвращаемых значений функции, обсуждаемой ниже. Разница между этим случаем и приведенным выше случаем-тип возвращаемого значения. Вот это управляемый тип, string, тогда как в первом примере тип простой Integer.

опять же, учитывая условие о функции, всегда назначающей возвращаемое значение, первое назначение бессмысленно, потому что значение немедленно заменяется. Удалить первый назначение.

что касается функции в вашем редактировании, компилятор предупредит вас о ее ошибочной реализации, если вы включите подсказки и предупреждения. Компилятор сообщит вам, что не все пути кода возвращают значение.

function GetValue(vType:integer):integer;
begin
  if vType=1 then
    Result:=100
  else if (vType>2) and (vType<=9) then
    Result:=200;
end;

если ни одно из условий не выполняется, то переменной результата не присваивается значение. Эта функция должна быть:

function GetValue(vType:integer):integer;
begin
  if vType=1 then
    Result:=100
  else if (vType>2) and (vType<=9) then
    Result:=200
  else
    Result:=0;
end;

я не могу подчеркнуть, насколько важно, чтобы вы всегда возвращали значение из функции. На самом деле это ужасная слабость, что Delphi даже позволяет компилировать вашу функцию.

причина, по которой ваше двойное назначение иногда кажется вам полезным, связана с причудой реализации возвращаемых значений функции в Delphi. В отличие от почти всех других языков возвращаемое значение функции Delphi для некоторых более сложных типов на самом деле является . Так что эта функция

function foo: string;

на самом деле, семантически, то же самое, что это:

procedure foo(var result: string);

это действительно странное решение, принятое дизайнерами Delphi. В большинстве других языков, таких как C, C++, С#, Java и т. д., возвращаемое значение функции похоже на параметр by-value, передаваемый от вызываемого абонента вызывающему.

это означает, что вы можете, если хотите быть извращенным, передавать значения в функцию через ее возвращаемое значение. Например, рассмотрим этот код:

// Note: this code is an example of very bad practice, do not write code like this

function foo: string;
begin
  Writeln(Result);
end;

procedure main;
var
  s: string;
begin
  s := 'bar';
  s := foo;
end;

когда вы называете main, оно выведет наружу bar. Это довольно странно деталь реализации. Вы не должны полагаться на это. Позвольте мне повторить. Вы не должны полагаться на это. Не привыкайте инициализировать возвращаемые значения на сайте вызовов. Это приводит к недостижимому коду.

вместо этого следуйте простому правилу, гарантирующему, что возвращаемое значение функции всегда назначается функцией и никогда не считывается до его назначения.

более подробная информация о реализации возвращаемых значений функции предоставляется документация, С моим акцентом:

для возврата результата функции используются следующие соглашения ценности.

• порядковые результаты возвращаются, когда это возможно, в регистре ЦП. Байты возвращаются в AL, слова возвращаются в AX и double-words возвращаются в EAX.
• реальные результаты возвращаются в регистр верхнего стека сопроцессора с плавающей запятой (ST (0)). Результаты функции типа Currency , значение в ST (0) масштабируется на 10000. Например, значение валюты 1.234 возвращается в ST (0) как 12340.
• для строки, динамического массива, указателя метода или результата варианта эффекты такие же, как если бы результат функции был объявлен как дополнительный параметр var после объявленных параметров. В других слова, вызывающий передает дополнительный 32-разрядный указатель, который указывает на переменная, в которую возвращается результат функции.
• Int64 возвращается в EDX: EAX.
• указатель, класс, класс-ссылка, и порядок-указатель результат возвращается в eax.
• для статического массива, записи и набора результатов, если значение занимает один байт, оно возвращается в AL; если значение занимает два байта, оно возвращается в AX; и если значение занимает четыре байта, оно возвращается в РЕГИСТР EAX. В противном случае результат возвращается в дополнительный параметр var это передается функции после заявленные параметры.