Угадай фильм по скриншоту (Delphi, CommFort 5)

Аватара пользователя
vlad
Супермодератор
Супермодератор
Сообщения: 233
Зарегистрирован: 02 фев 2012, 11:42

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение vlad » 14 июл 2012, 19:06

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

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 14 июл 2012, 19:09

Конечно. Если предложишь что-нибудь нормальное сделаем.

Аватара пользователя
vlad
Супермодератор
Супермодератор
Сообщения: 233
Зарегистрирован: 02 фев 2012, 11:42

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение vlad » 14 июл 2012, 19:26

1. Сделайте автообновление плагина - пользователи скажут спасибо.
2. Добавьте сразу побольше трекеров, а то потом каждый раз перекомпилировать надо.
3. Добавьте статистику.

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 14 июл 2012, 19:41

1. Автообновление самого плагина невозможно (можно только уведомление о наличии новой версии выдать), а вот автоматическое обновление библиотек трекеров я раньше планировал. Но по функционалу всё решаю не только я, если Maximus согласен, то сделаю, это не сложно.
2. Они в библиотеках, всё перекомпилировать не надо.
3. Она есть (!стат, !топ, !мой топ).

Отчёт после тестирования:
1. Вывод лишней фразы "Не угадано" не обнаружен.
2. Зависания не замечены.
3. Длительных интервалов между фильмами нет.
4. Утечка памяти пока есть (2 часа - 6 Мб).

Аватара пользователя
vlad
Супермодератор
Супермодератор
Сообщения: 233
Зарегистрирован: 02 фев 2012, 11:42

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение vlad » 14 июл 2012, 20:03

Andrey писал(а):Утечка памяти пока есть (2 часа - 6 Мб).
Фигня, можно так оставить.

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 14 июл 2012, 20:05

vlad, а в сутки 6*12=72 Мб. А если больше? Всё же желательно исправить.

Аватара пользователя
Maximus
Супермодератор
Супермодератор
Сообщения: 1507
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Maximus » 14 июл 2012, 20:06

Другие трекеры потом добавим. Статистику игроков расширю, будет более полная.

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 14 июл 2012, 20:08

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

Аватара пользователя
Maximus
Супермодератор
Супермодератор
Сообщения: 1507
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Maximus » 14 июл 2012, 21:37

Если много претензий, то это хорошо, значит плагин востребован.

Аватара пользователя
vlad
Супермодератор
Супермодератор
Сообщения: 233
Зарегистрирован: 02 фев 2012, 11:42

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение vlad » 17 июл 2012, 00:43

Как там на данный момент? Вроде всё нормально, но какая версия сейчас крутится на тестовом сервере я не знаю.
Ну что там по поводу автообновлений.

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 17 июл 2012, 00:51

vlad, на тестовом сервере очень старая версия. На данный момент вроде бы удалось немного уменьшить утечку памяти (совсем пока не исправил), сейчас разбираюсь с преждевременной проверкой ответа.
vlad писал(а):Ну что там по поводу автообновлений.
Полностью автоматически обновить сам плагин нельзя. Ещё 2 дня назад плагин умел выдавать сообщение о наличии новой версии, а завтра сможет обновлять самостоятельно трекеры. И трекеры можно будет добавлять или удалять прямо из окна настроек.

Аватара пользователя
vlad
Супермодератор
Супермодератор
Сообщения: 233
Зарегистрирован: 02 фев 2012, 11:42

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение vlad » 17 июл 2012, 01:41

Andrey писал(а):На данный момент вроде бы удалось немного уменьшить утечку памяти (совсем пока не исправил)
Она и раньше небольшая была.
Andrey писал(а):сейчас разбираюсь с преждевременной проверкой ответа.
Это когда выводится строка "Не угадано. Правильный ответ:"?
Andrey писал(а):Ещё 2 дня назад плагин умел выдавать сообщение о наличии новой версии, а завтра сможет обновлять самостоятельно трекеры.
А где новая версия будет лежать?
Andrey писал(а):И трекеры можно будет добавлять или удалять прямо из окна настроек.
Это как?

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 17 июл 2012, 01:47

vlad писал(а):Она и раньше небольшая была.
А если больше суток?
vlad писал(а):Это когда выводится строка "Не угадано. Правильный ответ:"?
Да, возникает если процесс загрузки долго работает, но я уже знаю как исправить.
vlad писал(а):А где новая версия будет лежать?
Пока здесь, потом решим.
vlad писал(а):Это как?
Потом покажу.

Аватара пользователя
tony
Сообщения: 12
Зарегистрирован: 20 июл 2012, 15:52

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение tony » 20 июл 2012, 16:20

Andrey писал(а):
vlad писал(а):Ну что там по поводу автообновлений.
Полностью автоматически обновить сам плагин нельзя.
Извините что вмешиваюсь в тему, в которой не всё понимаю, но если ваш плагин представляет собой обычную dll то автообновление сделать можно.
Т.к. сервер не освободит используемую dll, то весь функционал можно вынести в другую, а в основной поместить функционал отвечающий за обновление и вызов функций и процедур. Да и так лучше будет.
Хотя ещё раз повторяюсь что это только предположение, я не знаю всех тонкостей создания дополнений к этой программе.
А что касается памяти то дам два совета:
1. Проверить на XP и ниже, если там нормально, то это superfetch. В таком случае всё нормально и память освободится когда она будет нужна.
2. Если на старых ОС также, то проверяйте какие объекты динамически создаются и какие удаляются. ООП используется?

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 20 июл 2012, 16:30

Плагин - это обычная dll с определёнными базовыми процедурами и функциями. Автообновление так сделать можно, но это не первоочередная задача.
tony писал(а): Проверить на XP и ниже, если там нормально, то это superfetch.
На xp также.
tony писал(а):Если на старых ОС также, то проверяйте какие объекты динамически создаются и какие удаляются. ООП используется?
Уже много раз смотрю это и каждый раз замечаю что что-то удалить забыл, сегодня например заметил что в оном месте не удаляется IdHTTP, а новый создаётся. ООП не используется.

Аватара пользователя
tony
Сообщения: 12
Зарегистрирован: 20 июл 2012, 15:52

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение tony » 20 июл 2012, 17:14

Значит проверяйте создание/удаление стандартных компонентов.

Аватара пользователя
vlad
Супермодератор
Супермодератор
Сообщения: 233
Зарегистрирован: 02 фев 2012, 11:42

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение vlad » 16 сен 2012, 20:00

Уже давно говорили о том что всё почти готово. Я правильно понимаю что проект закрыт?
Жаль что увидеть этот плагин мне не суждено...

Аватара пользователя
Andrey
Администратор
Администратор
Сообщения: 2345
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Andrey » 16 сен 2012, 21:05

vlad, я не знаю когда плагин будет готов, но увидишь его точно.

Аватара пользователя
Maximus
Супермодератор
Супермодератор
Сообщения: 1507
Зарегистрирован: 02 фев 2012, 11:42
Откуда: Воронеж
Контактная информация:

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение Maximus » 16 сен 2012, 22:03

vlad, всё будет хорошо
© Русское Радио :-)

Аватара пользователя
vlad
Супермодератор
Супермодератор
Сообщения: 233
Зарегистрирован: 02 фев 2012, 11:42

Re: Угадай фильм по скриншоту (Delphi, CommFort 5)

Сообщение vlad » 14 окт 2012, 09:52

Ну а сейчас?

Ответить