ReSQL.ru
     
powered by simpleCommunicator - 2.0.61     © 2026 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Delphi [игнор отключен] [закрыт для гостей] / ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
10 сообщений из 10, страница 1 из 1
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985415
mkr
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mkr
Участник
Смотрю проект, при переключении страницы в pagecontrol, вызывается несколько функций, выполняющих определённые, базовые действия: пересоздают popupmenu делает requery. Так вот в этих popupmenu и requery находится куча if-ов выполняющих действие в зависимости от установившейся страницы.
Есть ли возможность, все эти действия как-то вынести отдельно?
в теории, все действия для каждой TabSheet можно вынести в onShow, есть ли минусы?
...
Рейтинг: 0 / 0
30.07.2020, 18:50
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985424
Фотография Victor Cookin
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Victor Cookin
Участник
mkr,

Совершенно всё равно где вызывать, разве если Pages могут появляться динамически, то с OnShow - сложнее.
...
Рейтинг: 0 / 0
30.07.2020, 19:30
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985426
DimaBr
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
DimaBr
Участник
Вместо IF-ов можно использовать CASE, но хрен редьки не слаще. С IF-ами хоть понятно к какой вкладке код
...
Рейтинг: 0 / 0
30.07.2020, 19:40
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985427
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
mkr
при переключении страницы в pagecontrol, вызывается несколько функций, выполняющих определённые, базовые действия: пересоздают popupmenu делает requery.

Когда коту нефиг делать, он пересоздаёт popupmenu...

mkr
в теории, все действия для каждой TabSheet можно вынести в onShow

В OnShow чего?

mkr
Так вот в этих popupmenu и requery находится куча if-ов выполняющих действие в зависимости от установившейся страницы. Есть ли возможность, все эти действия как-то вынести отдельно?

Есть куча возможностей отрефакторить невменяемый код. Но чтобы ответить, как именно это стоит сделать - нужно смотреть на этот код и на требования к интерфейсу, отвечать в общем виде бессмысленно.
...
Рейтинг: 0 / 0
30.07.2020, 19:41
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985592
Фотография _Vasilisk_
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
_Vasilisk_
Участник
У меня это сделано так:

1. К каждой вкладке привязан объект-наследник некоторого TCustomTabControl
2. При переключении вкладки ссылка на текущий объект сохраняется в FCurrentControl: TCustomTabControl
3. Все действия на вкладке вызывают методы

2 пункт можно убрать, если объект хранить в Tag вкладки
...
Рейтинг: 0 / 0
31.07.2020, 15:05
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985609
rgreat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rgreat
Участник
Вместо кучи if-ов можно использовать for.
...
Рейтинг: 0 / 0
31.07.2020, 15:25
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985774
Гаджимурадов Рустам
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гаджимурадов Рустам
Участник
Мои 5 копеек - что If-ы, что CASE, что другой спагетти-вариант
лучше (удобнее в сопровождении) независимых обработчиков
(OnShow и пр.) отдельных вкладок. Хотя разумнее рефакторить,
конечно, если есть время и нет задач поважнее. Например, надо
посмотреть, а чем там код для вкладок собсно отличается -
возможно, его можно упростить/объединить. Возможно, сам
интерфейс (пейджконтрол или некоторые вкладки) можно
упростить/улучшить. И т.д.
Posted via ActualForum NNTP Server 1.5
...
Рейтинг: 0 / 0
01.08.2020, 13:00
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985777
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
Гаджимурадов Рустам
Мои 5 копеек - что If-ы, что CASE, что другой спагетти-вариант

При этих словах Дейкстра перевернулся в гробу и подумал много тёплых ласковых слов про современных разработчиков.

Гаджимурадов Рустам
Например, надо посмотреть, а чем там код для вкладок собсно отличается

Провангую, что

Код: pascal
1.
2.
3.
4.
procedure RefreshQuery(AQuery: TQuery; ATabSheet: TTabSheet);
begin
  if Assigned(ATabSheet) and (ATabSheet.PageControl.ActivePage = ATabSheet) then RefreshQuery(AQuery);
end;


снимет половину их необходимости в if-ах. Про более серьёзные улучшения я даже не собираюсь начинать.
...
Рейтинг: 0 / 0
01.08.2020, 13:22
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985783
Гаджимурадов Рустам
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гаджимурадов Рустам
Участник
softwarer> При этих словах

ЯННП. Не спагетти или что ?

> Провангую, что

Нет смысла гадать. С этой кверей ведь всё равно
ещё что-то кроме открытия делается, наверняка -
заполнение параметров, загрузка чего-то там и пр.
Posted via ActualForum NNTP Server 1.5
...
Рейтинг: 0 / 0
01.08.2020, 13:50
| Ответить | Цитировать | Написать  
ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
    #39985788
Фотография softwarer
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
softwarer
Участник
Гаджимурадов Рустам
ЯННП. Не спагетти или что ?

if-ы - это как раз то, что предлагал Дейкстра для борьбы со спагетти.
...
Рейтинг: 0 / 0
01.08.2020, 14:06
| Ответить | Цитировать | Написать  
10 сообщений из 10, страница 1 из 1
Форумы / Delphi [игнор отключен] [закрыт для гостей] / ОФФТОП. Как избавиться от кучи if-ов при смене pagecontrol
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение   Полит. конфиден.  
NoSQL.ru       Кролег: Проекты, Новости, Чат       РЕД ФОРУМ       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=58&msg=39985777&tid=2038102]:
 0ms
get settings:
 8ms
get forum list:
 9ms
check forum access:
 2ms
check topic access:
 2ms
track hit:
 158ms
get topic data:
 6ms
get forum data:
 2ms
get page messages:
 31ms
get tp. blocked users:
 1ms
others: 237ms
total:  456ms
созд. / загр.: 456ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]