Функция "Телефонный справочник"

[san]

Заказчик хочет, чтобы внутри интерфейса программы можно было отобразить пользователю содержимое файла. Требуется для просмотра диспетчером телефонного справочника(ТСОП).

• В главное меню добавить пункт "Телефонный справочник" (иконка menu_logs.png)

• Настройку Прочее-> "Текстовая кодировка лога программы" переименовать в "Кодировка текстовых файлов".

• В файл настроек добавить "Путь к файлу телефонного справочника", по умолчанию это phonebook.txt, расположенный в папке программы.

• При переходе в "Телефонный справочник" пользователю отображается содержимое текстового файла, Файл отображается в кодировке, указанной в настройке "Кодировка текстовых файлов".

• Если файл обновился, то при следующем переходе в "Телефонный справочник" файл должен быть перечитан.

• Если при попытке открыть файл произошла ошибка - выдать сообщение.

[dimag]

Сделать настройку шрифта телефонной книги в настройках, во разделе "внешний вид" в виде списка от 8 до 20.

[dimag]

r497
Добавлен новый пункт в главное меню и новый параметр в раздел внешний вид настроек для вывода телефонной книги пользователя, файл с телефонной книгой пользователя по умолчанию в файле phonebook.txt.

[dimag]

размер шрифта от 8 до 48

[alx]

Запустил программу. Открываю вкладку "Телефонный справочник" и вижу:

 1 111111
 2 222222
 3 333333
 4 444444

Почему в телефонном справочнике такое странное содержимое? Не будет ли более логично, если изначально файл справочника будет пустой?

[alx]

Какую роль в реализации отображения телефонного справочника играют две добавленные строчки

 pjsipsua.DeInit();
 sshsftp.DeInit();

в файле StartupDlg.cpp?

[dimag]

Replying to alx:

Запустил программу. Открываю вкладку "Телефонный справочник" и вижу:

 1 111111
 2 222222
 3 333333
 4 444444

Почему в телефонном справочнике такое странное содержимое? Не будет ли более логично, если изначально файл справочника будет пустой?

Можно и пустой файл, но нужно было проверить как будет выглядеть текст на экране.

[dimag]

Replying to alx:

Какую роль в реализации отображения телефонного справочника играют две добавленные строчки

 pjsipsua.DeInit();
 sshsftp.DeInit();

в файле StartupDlg.cpp?

Никакого, я добавил это для корректного закрытия программы при выходе из диалога инициализации.

[alx]

Replying to dimag:

Можно и пустой файл, но нужно было проверить как будет выглядеть текст на экране.

Но ведь ничто, наверное, не мешало после завершения проверки и до коммита удалить этот текст...

Replying to dimag:

Никакого, я добавил это для корректного закрытия программы при выходе из диалога инициализации.

Следовательно, если когда-нибудь заказчик попросит убрать функцию отображения телефонного справочника, и changeset:497 будет откачен, программа будет некорректно закрываться при выходе из диалога инициализации...

[dimag]

Replying to alx:

Replying to dimag:

Можно и пустой файл, но нужно было проверить как будет выглядеть текст на экране.

Но ведь ничто, наверное, не мешало после завершения проверки и до коммита удалить этот текст...

Replying to dimag:

Никакого, я добавил это для корректного закрытия программы при выходе из диалога инициализации.

Следовательно, если когда-нибудь заказчик попросит убрать функцию отображения телефонного справочника, и changeset:497 будет откачен, программа будет некорректно закрываться при выходе из диалога инициализации...

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

[alx]

Replying to dimag:

Но я всегда проверяю когда убираю некий функционал.

1. Вы делаете работу не для себя, а для Компании. Почему Вы уверены, что откат будете делать непременно Вы? Может быть это будет делать кто-то совсем другой, в то время как Вы будете, например, заняты другой работой, отдыхать в отпуске, или, скажем, вообще уже не будете здесь работать... Человек прочитает комментарий к changeset:497, убедится, что там нет ни слова о диалоге аутентификации и корректности выхода и ошибочно сделает вывод, что ни к каким побочным эффектам откат changeset:497 не приведет...

2. Вы всегда проверяете что? Вы всегда, исправив какую-то мелочь типа упомянутого убирания отображения справочника, выполняете весь спектр проверок работы программы, включая корректность инициализации/деинициализации всех библиотек? Извините, "Не верю" (с) Станиславский. Как же тогда объяснить, что вплоть до r497 в программе не было вызовов pjsipsua.DeInit(); и sshsftp.DeInit();? Плохо проверяли?

[dimag]

Не было нужно, я поменял работу с SSH, плюс к тому было добавленна поддержка повторного входа в программу по Alt+Q.

[alx]

Replying to dimag:

Не было нужно, я поменял работу с SSH, плюс к тому было добавленна поддержка повторного входа в программу по Alt+Q.

??? Вы добавили повторный вход в программу по Alt-Q в changeset:497? Можете привести пример (имя файла и номер строки) изменения в changeset:497, добавляющего поддержку повторного входа (кроме двух упомянутых мной строк)?

[dimag]

Это была первая попытка, сразу так переделать не удалось, и я всё стёр из кода. Но вот этот фрагмент остался.

[alx]

Replying to dimag:

Это была первая попытка, сразу так переделать не удалось, и я всё стёр из кода. Но вот этот фрагмент остался.

Может быть чтобы исключить подобные ошибки Вам стоит что-то изменить в технологии вашей работы? Например всегда начинать работу над каким-то новым тикетом/багом/фичей с svn checkout, чтобы быть уверенным в отсутствии каких-либо следов других экспериментов в свежей рабочей копии исходников? И вручную стирать что-либо из кода тогда не потребуется...

[dimag]

Можно попробовать, хотя не любитель размножать коммиты.

[alx]

Replying to dimag:

Можно попробовать, хотя не любитель размножать коммиты.

Я не предлагал Вам размножать коммиты. Я предлагал Вам вариант изменения вашей внутренней организации работы, который, возможно, поможет Вам решить вашу проблему.

[san]

Milestone deleted