Если результат функции используется несколько раз то следует ее значение присвоить переменной и использовать переменную?

@S0563506 · Регистрация: 11.09.2023

Author24 — интернет-сервис помощи студентам

Если результат функции используется несколько раз, то следует ее значение присвоить переменной и использовать эту переменную?

Как целесообразнее?

Пример 1

Delphi

1 2	for I:=0 to 100000 do a[I]:= Length('dfdfdfdfdfdfdfsdfdfdsfs');

или

Delphi

1
2
3

с:=  Length('dfdfdfdfdfdfdfsdfdfdsfs');
for I:=0 to 100000 do
a[I]:=с;

Пример 2

Delphi

A: = Length('dfdfdfdfdfdfdfsdfdfdsfs');
B: = Length('dfdfdfdfdfdfdfsdfdfdsfs');
C: = Length('dfdfdfdfdfdfdfsdfdfdsfs');
D: = Length('dfdfdfdfdfdfdfsdfdfdsfs');

или

Delphi

A: = Length('dfdfdfdfdfdfdfsdfdfdsfs');
B: = A;
C: = A;
D: = A;

@AzAtom · 26.10.2023, 21:54

Вообще, да. Но конкретно с Length длина строки хранится в памяти, потому, просто считывается точно так же, как и из переменной. Т.е., с Length не имеет смысла.

Verevkin · 27.10.2023, 09:38

Оптимизатор не допустит такого безобразия и сам выделит память под временную переменную.

@krapotkin · 27.10.2023, 10:47

еще как допустит)
мне несложно сделать переменную

@D1973 · 27.10.2023, 11:24

Как по мне - лучше объявить одну переменную с не слишком длинным именем, чем тягать постоянно имя функции и 100500 ее параметров...

@Beltar · 27.10.2023, 12:34

Сообщение от Verevkin

Оптимизатор не допустит такого безобразия и сам выделит память под временную переменную.

Delphi

x:=Random(0,100);
lbRnd.Items.Add(x.ToString);
x:=Random(0,100);
lbRnd.Items.Add(x.ToString);
x:=Random(0,100);

Добавлено через 9 минут
Делал я как-то отчет в Excel из Interbase базы с набором в 220 тыс. что ли строк. Считалась там статистика типа среднекв. откл. Все это дело строилось за 17 сек. на тогдашнем P4-3000. И я не подумавши брал в каждом обращении данные через AsFloat и так 6 раз каждое. А потом мне понадобилось добавить еще 1 параметр и пришлось поменять процедурку так, что все эти AsFloat стали передаваться 1 раз при вызове. В результате несмотря на добавление еще 1 листа выводимого в Excel время выполнения упало до 14-15. Ну подумаешь всего-то лишний миллион обращений к методу TIBQuery.AsFloat.

@krapotkin · 27.10.2023, 12:39

надеюсь это были FieldByName('xxx').AsFloat ? ))

@Beltar · 27.10.2023, 13:22

Нет, по индексу.

@Constcat · 27.10.2023, 15:46

Сообщение от S0563506

Если результат функции используется несколько раз, то следует ее значение присвоить переменной и использовать эту переменную?

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

@S0563506 · 06.11.2023, 09:09 **[ТС]**

Сообщение от Constcat

Если не предполагается, что результат функции может внезапно поменяться, то лучше использовать переменную.

Выше у меня немного другие примеры были.

Встречаются еще вот такие варианты. Значение-резултат функции будет одинаков. В обоих вызовах.
Вопрос в том, что мы получим результат функции GetLastModifiedFileDate в сравнении if в первом её вызове, а потом
она снова будет вызвана в Result := GetLastModifiedFileDate(SubFolder, APattern), и снова например будет просматривать скажем вес диск C:?
Или же каким то чудом компилятор поймет, что только что эта функция была вызвана с теми же параметрами и можно ее не запускать снова, а просто взять ранее полученное значение уже из памяти? Компилятор на такое способен или "тупо" как написно будет вызвана два раза процедура и она два раза будет просматривать содержимое директории SubFolder?

Delphi

1
2
3

          if Result < GetLastModifiedFileDate(SubFolder, APattern) then begin
            Result := GetLastModifiedFileDate(SubFolder, APattern);
          end;

Или правильнее сделать так:

Delphi

ATime := GetLastModifiedFileDate(SubFolder, APattern); 
          if Result < ATime  then begin
            Result := ATime ;
          end;

@Constcat · 06.11.2023, 10:46

Сообщение от S0563506

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

Нет, компилятор таким не занимается.
Оптимизация производится, если компилятор видит, что задействованные компоненты (параметры/данные) никак не зависят от каких-то внешних изменений. Т.е., выражение "2+3" компилятор заменит на "5" и "впишет" этот результат в бинарный код.

Сообщение от S0563506

и снова например будет просматривать скажем вес диск C:?

Да, будет просматривать. Тем более, что это - файловая система, которая не зависит от вашей программы. Между вызовами этой функции там много чего может поменяться.

Но в вашем примере лучше использовать переменную. Выражением

Delphi

1	ATime := GetLastModifiedFileDate(SubFolder, APattern);

вы получаете некий "срез" информации на текущий момент, который и нужно вернуть в Result
А иначе вы получите одно, а вернуть можете совсем другое.

@AzAtom Модератор 3756 / 2260 / 782 Регистрация: 15.11.2015 Сообщений: 8,991
	26.10.2023, 21:54	2
	Вообще, да. Но конкретно с Length длина строки хранится в памяти, потому, просто считывается точно так же, как и из переменной. Т.е., с Length не имеет смысла. 0

Verevkin Злостный нарушитель 9594 / 5188 / 1182 Регистрация: 12.03.2015 Сообщений: 24,500
	27.10.2023, 09:38	3
	Оптимизатор не допустит такого безобразия и сам выделит память под временную переменную. 0

@krapotkin 5868 / 4596 / 1448 Регистрация: 14.04.2014 Сообщений: 20,392 Записей в блоге: 20
	27.10.2023, 10:47	4
	еще как допустит) мне несложно сделать переменную 0

@D1973 Модератор 9626 / 6237 / 2422 Регистрация: 21.01.2014 Сообщений: 26,626 Записей в блоге: 3
	27.10.2023, 11:24	5
	Как по мне - лучше объявить одну переменную с не слишком длинным именем, чем тягать постоянно имя функции и 100500 ее параметров... 0

@krapotkin 5868 / 4596 / 1448 Регистрация: 14.04.2014 Сообщений: 20,392 Записей в блоге: 20
	27.10.2023, 12:39	7
	надеюсь это были FieldByName('xxx').AsFloat ? )) 0

@Beltar 541 / 241 / 27 Регистрация: 05.05.2022 Сообщений: 1,571
	27.10.2023, 13:22	8
	Нет, по индексу. 0

@Constcat 237 / 203 / 36 Регистрация: 19.02.2021 Сообщений: 1,431
	27.10.2023, 15:46	9
	Сообщение от S0563506 Если результат функции используется несколько раз, то следует ее значение присвоить переменной и использовать эту переменную? Если не предполагается, что результат функции может внезапно поменяться, то лучше использовать переменную. И отлаживать так проще. 0

	06.11.2023, 10:46