Far 3 & SettingsControl

Место, где разработчики плагинов могут поделиться своим опытом.
User avatar
John Doe
Бюрократ
Posts: 11966
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 26 times
Been thanked: 133 times

Far 3 & SettingsControl

Post by John Doe » Wed 07 Nov, 2018 13:53

Latest post of the previous page:

DrKnS wrote:
Wed 07 Nov, 2018 13:31
Т.к. это заведомо неподдерживаемый сценарий, обречённый на неудачу, я не вижу проблемы в том, что автор этого сценария подождёт 5 секунд.
Тут конечно, имел в виду другие сценарии: если двух повторов оказалось недостаточно, то уже есть основания заподозрить, что что-то не по назначению используется (даже если с третьей попытки и получилось бы).

Ну вам виднее, я только предложил.

User avatar
John Doe
Бюрократ
Posts: 11966
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 26 times
Been thanked: 133 times

Far 3 & SettingsControl

Post by John Doe » Wed 07 Nov, 2018 14:10

zg wrote:
Wed 07 Nov, 2018 13:44
скорость.
За счёт чего выигрыш, и как его измерить?

На мой взгляд такое поведение только даёт лишнюю возможность допустить ошибку в названии подключа.
В добавок, оно неконсистентно: при работе с базами плагинов возвращается ошибка:

User avatar
DrKnS
Posts: 5002
Joined: Thu 04 Aug, 2005 06:44
Location: Kyiv
Has thanked: 4 times
Been thanked: 166 times

Far 3 & SettingsControl

Post by DrKnS » Wed 07 Nov, 2018 14:42

В контексте настроек фара SCTL_OPENSUBKEY сам по себе не имеет смысла - это такой трюк для доступа к истории диалогов.
Надо бы отразить это в документации.

User avatar
John Doe
Бюрократ
Posts: 11966
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 26 times
Been thanked: 133 times

Far 3 & SettingsControl

Post by John Doe » Wed 07 Nov, 2018 14:57

DrKnS wrote:
Wed 07 Nov, 2018 14:42
В контексте настроек фара SCTL_OPENSUBKEY сам по себе не имеет смысла - это такой трюк для доступа к истории диалогов.
Надо бы отразить это в документации.
Есть такой тикет.

Но даже так, мне кажется приведённые выше ргументы всё равно уместны

User avatar
DrKnS
Posts: 5002
Joined: Thu 04 Aug, 2005 06:44
Location: Kyiv
Has thanked: 4 times
Been thanked: 166 times

Far 3 & SettingsControl

Post by DrKnS » Wed 07 Nov, 2018 15:08

Читайте в этом случае SCTL_OPENSUBKEY как SCTL_REGISTER_NEW_FARSETTINGS_SUBFOLDER и когнитивный диссонанс возникать не будет.

User avatar
John Doe
Бюрократ
Posts: 11966
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 26 times
Been thanked: 133 times

Far 3 & SettingsControl

Post by John Doe » Thu 08 Nov, 2018 09:03

DrKnS wrote:
Wed 07 Nov, 2018 15:08
Читайте в этом случае SCTL_OPENSUBKEY как SCTL_REGISTER_NEW_FARSETTINGS_SUBFOLDER и когнитивный диссонанс возникать не будет.
Пользователь будет читать то, что сказано в Энциклопедии.
Если напишите там "SCTL_REGISTER_NEW_FARSETTINGS_SUBFOLDER", значит это и прочитает.

User avatar
John Doe
Бюрократ
Posts: 11966
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 26 times
Been thanked: 133 times

Far 3 & SettingsControl

Post by John Doe » Fri 09 Nov, 2018 12:25

0003673: SCTL_OPENSUBKEY для доступа к историям диалогов: константа для корня

В качестве документации пока что один changelog с примером, в котором в качестве корня для получения историй диалогов выступает 0.
Семь лет уже прошло с этого нововведения, не пора ли снять с него статус "экспериментального", и использовать выделенную константу с говорящим именем, вместо 0?

User avatar
zg
zero-gravity
Posts: 5507
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 73 times
Contact:

Far 3 & SettingsControl

Post by zg » Fri 09 Nov, 2018 12:38

John Doe wrote:
Fri 09 Nov, 2018 12:25
Семь лет уже прошло с этого нововведения, не пора ли
в посте с таким тоном как минимум должна присутствовать ссылка на пулреквест. где она?
John Doe wrote:
Fri 09 Nov, 2018 12:25
и использовать выделенную константу с говорящим именем, вместо 0?
так есть FSSF_ROOT.

User avatar
John Doe
Бюрократ
Posts: 11966
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 26 times
Been thanked: 133 times

Far 3 & SettingsControl

Post by John Doe » Fri 09 Nov, 2018 13:06

zg wrote:
Fri 09 Nov, 2018 12:38
так есть FSSF_ROOT.
В коде сейчас просто 0, и использования FSSF_ROOT как раз хотелось бы избежать, чтобы не создавалось впечатление, будто истории диалогов "находятся" в корне настроек фара.
Для меня это выглядит несколько неестественно, и в тикете я предлагаю добавить отдельную константу FSSF_DIALOGS_HISTORY_ROOT (которая численно может оставаться и 0).

User avatar
zg
zero-gravity
Posts: 5507
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 73 times
Contact:

Far 3 & SettingsControl

Post by zg » Fri 09 Nov, 2018 13:47

John Doe wrote:
Fri 09 Nov, 2018 13:06
В коде сейчас просто 0
разве?
John Doe wrote:
Fri 09 Nov, 2018 13:06
и использования FSSF_ROOT как раз хотелось бы избежать, чтобы не создавалось впечатление, будто истории диалогов "находятся" в корне настроек фара.
что плохого в таком впечатлении?

User avatar
John Doe
Бюрократ
Posts: 11966
Joined: Wed 27 Apr, 2005 20:42
Has thanked: 26 times
Been thanked: 133 times

Far 3 & SettingsControl

Post by John Doe » Fri 09 Nov, 2018 15:45

zg wrote:
Fri 09 Nov, 2018 13:47
разве?
Я о том коде, который в changelog.
zg wrote:
Fri 09 Nov, 2018 13:47
что плохого в таком впечатлении?
Мне помешало ухватить суть происходящего (поначалу).
Выглядит трюком.

buniak_a_h
Posts: 581
Joined: Sat 20 Apr, 2013 00:17
Location: Питер
Has thanked: 57 times
Been thanked: 13 times
Contact:

Far 3 & SettingsControl

Post by buniak_a_h » Fri 09 Nov, 2018 19:26

zg wrote:
Fri 09 Nov, 2018 13:47
John Doe wrote:
Fri 09 Nov, 2018 13:06
и использования FSSF_ROOT как раз хотелось бы избежать, чтобы не создавалось впечатление, будто истории диалогов "находятся" в корне настроек фара.
что плохого в таком впечатлении?
Вы мне когда-то присылали пример чтения из истории. И в этом примере действительно неожиданным было (а)использование Settings и (б) имя константы FSSF_ROOT. Как-то понятия "история" и "settings" (установки, настройки) на интуитивном уровне не коррелируют. Посему поддерживаю введение отдельной константы для истории.

Post Reply

Return to “Разработка плагинов”