Pull to refresh

Comments 15

а почему бы для этих целей не использовать Google Translate API?
Думаю, это хорошая идея.
На самом деле, когда это начиналось, была мысль использовать AJAX API, но я уже, честно, не помню почему, она отпала. По-моему из-за ограничений… Но это надо копать глубже и дальше…
Но всеравно спасибо за подсказку, а то я уже совсем забыл про это.
А чего ошибки не обрабатываете? Хотя бы if (error) { emit errorOccurred(http->errorString()); return; } в textTranslated
А, думаете, в этом есть смысл?
Вообще говоря конечно не критично, но как же пользователь приложения узнает по какой причине Ваша программа не хочет работать если, например, нет связи с сервером? И, кстати, Вы забыли почистить память за version.
Да, согласен, есть такое…
Ну пока до 1.0 дойду, все такие заусенцы поправлю.
version = new QString("0.9.5");

Удалять, конечно, необязательно?

Ну и в целом есть над чем работать, код весьма уродлив.
Соприть не буду, но попросил бы пояснений.
Парсинг построенный на replace — не самая лучшая идея.

А самое прикольное, раз уж Вы пишете на Qt, это то, что весь textTranslated() можно уместить в несколько строк. Посмотрите, что вернет
QscriptEngine engine;
QscriptValue value = engine.evaluate(text);


Про удаление пояснять не надо?
S в Qscript* должна быть большая. Хабровский парсер косячит.
А памяти он при этом кушать будет не сликом ли много?
Ну, по-моему, немного неуместный вопрос для программы, которая отжирает 18 метров (под виндой) ничего при этом не делая :)
Использование Qtscript в данном случае не сильно повлияет, к тому же QscriptEngine будет создаваться только на время парсинга.
18 метров? 0_о
Никогда под виндой не проверял, но удивлен…
Ладно, спасбо за совет, попробую…
нтересно, что там на 18 метров накушивает???
Посмотрел поближе.
При повторном запуске стала отжирать 11 :) Что изменилось — не знаю.
Но в любом случае, накушивает там много чего.

Сами библиотеки Qt съедают метров 5.
А дальше вы создаете всё, что только можно, даже если это может никогда не понадобиться, либо необходимо на короткий срок.

1. Создаете все окна, вместо того что бы создавать их непосредственно при нажатии пункта меню. Я, например, ни разу так и не открыл настройки или «О программе».
2. Сразу создаете свой класс GoogleProto, вместо того, что бы создавать его во время перевода.
3. При закрытии главного окна его можно не просто прятать, а удалять. И создавать его заново при вызове из трея. Тогда, в приницпе, пункт 2 можно оставить без изменений. Естественно, код создания трей-иконки надо будет вынести.
Sign up to leave a comment.

Articles