Переменные. Что лучше, несколько локальных или одна глобальная переменная?

[Женя32]

Здравствуйте!
Хотелось бы узнать, что лучше использовать, несколько локальных или одну глобальную переменную?

[Алексей1153]

Женя32, решение использовать глобальную переменную должно быть обосновано. Что именно требуется сделать и что хочется получить ?

[Женя32]

Сообщение от Алексей1153

Женя32, решение использовать глобальную переменную должно быть обосновано. Что именно требуется сделать и что хочется получить ?

В двух кнопках две переменных с одинаковым именем и типом. Лучше в каждой кнопке объявить локальную, или один раз глобальную?
Button1:

  if addBProF then
         begin
           Form1.BlockedProfile.Items.Add(strU);
           Form1.BlockedProfile.Items.SaveToFile('BlockPFile.mbm');
           Form1.BlockedProfile.Items.LoadFromFile('BlockPFile.mbm');
         end;

Button2:

if addBProF then
         begin
           Form1.BlockedProfile.Items.Add(strU);
           Form1.BlockedProfile.Items.SaveToFile('contacts.mbm');
           Form1.BlockedProfile.Items.LoadFromFile('contacts.mbm');
         end;

[Алексей1153]

Сообщение от Женя32

В двух кнопках две переменные

в кнопках не бывает переменных, речь идёт об обработчиках, то есть функциях.

Так о какой переменной идёт речь то ?

вижу два одинаковых участка кода, которые можно вынести в функцию, в которую нужно передать одну текстовую строку

[digitalis]

Зависит от конкретной ситуации. Например: по нажатию кнопки принять значение, по нажатию другой - что-то с этим значением делать. Тут переменная будет "обменником" между обработчиками - глобальной .

[evg_m]

В двух кнопках две переменных с одинаковым именем и типом.

если это ДВЕ переменных( по смыслу программы), то это локальные.

[BLACK_RAIN]

Извините, но не могу пройти мимо. Ответы просто бомбические. Вы просто жжёте напалмом

Сообщение от Алексей1153

в кнопках не бывает переменных, речь идёт об обработчиках, то есть функциях.

кэп не под своим аккаунтом зашел.
И кстати. В delphi обработчиками являются не функции, а процедуры.

Сообщение от Алексей1153

Так о какой переменной идёт речь то ?

о addBProF

Сообщение от Алексей1153

вижу два одинаковых участка кода, которые можно вынести в функцию, в которую нужно передать одну текстовую строку

Во-первых, они не одинаковые. Во-вторых, там же написано, что это два обработчика двух разных кнопок. Как вы смотрите?

Сообщение от Женя32

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

Если значение этой переменной задаётся в обработчике этой кнопки (а скорее всего так и есть), то лучше использовать локальные переменные. Не важно, сколько их - две или тысяча.
Глобальные переменные нужны только если вам надо использовать какое-то общее значение в разных процедурах/функциях.

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

Зависит от конкретной ситуации. Например: по нажатию кнопки принять значение, по нажатию другой - что-то с этим значением делать. Тут переменная будет "обменником" между обработчиками - глобальной .

Согласен. Это правильно. Но в данном случае вопрос ТС и ответ на него очевиден. ТС не подразумевал обмен данными между обработчиками. Он хотел просто обойтись одной глобальной переменной вместо того, чтобы в каждом обработчике создавать их локально.
На ютубе очень часто такое вижу. Они, почему-то, не любят локальные переменные.

[Arigato]

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

Во-первых, они не одинаковые. Во-вторых, там же написано, что это два обработчика двух разных кнопок. Как вы смотрите?

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

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

о addBProF

А может о strU?

С точки зрения кода:

           Form1.BlockedProfile.Items.SaveToFile('contacts.mbm');
           Form1.BlockedProfile.Items.LoadFromFile('contacts.mbm');

Во-первых, если это обработчик события класса TForm1, то не пишите там Forn1. Во-вторых, а зачем поле SaveToFile делать LoadFromFile? В надежде, что загрузится не то, что только что было сохранено в файл?

[BLACK_RAIN]

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

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

замечание-то верное, но вопрос автора был вполне однозначный. А какая именно переменная - не столь важно.

[Arigato]

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

вопрос автора был вполне однозначный

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