Comments 21
Постараюсь учесть в одном комментарии все замечания, которые хотелось бы оставить
0. Неформатированный кодл читать не возможно. Хабр поддерживает тег source.
1. Вся «литература» aka Документация по Qt Quick есть здесь или здесь, а еще в assistant
2. Русский текст в исходниках зло (конечно это приемлимо для «курсача», но никогда в продакшене), если хотите кнопочки на русском, пишите qsTr(«English text») потом переводите и подключайте транслятор.
3. Стилистика: имена переменных выбраны не удачно, вот вы, не зная, что делает программа, смогли бы сказать, чем отличается text_input1 от text_input2? Я нет. Но, если бы они назывались sourceWordInput и HammingCodeInput, было бы куда нагляднее.
4. Кнопка используется у вас два раза, если потребуется третья вам придется городить новый Rectangle,
гораздо лучше вынести кнопку в отдельный компонент (т.е. qml-файл), ято позволит ее использовать многократно.
5. Градиенты — штука безусловно клевая, но, к сожалению, сейчас жутко медленая, я (да и многие другие) рекомендую отказываться от градиента там, где это возможно, и заменять его изображением.
6. MainWindow не нужен, QDeclarativeView сам по себе является виджетом. Достаточно было в main.cpp написать:
Мало того, налицо нарушение архитектуры, вы в класс виджета выносите методы кодирования/декодирования, это гораздо лучше выглядело бы, вынесеным в отдельный класс, который можно было бы, как подключить qml плагином к приложению, так и (в данном простом случае) установить его инстанс как глобальное свойство.
В целом приятно что люди интересуются данной технологией, потому как она имеет огромный потенциал, но давайте все же самосовершествоваться и стараться писать хороший код с помощью хорошей технологии.
Спасибо.
0. Неформатированный кодл читать не возможно. Хабр поддерживает тег source.
1. Вся «литература» aka Документация по Qt Quick есть здесь или здесь, а еще в assistant
2. Русский текст в исходниках зло (конечно это приемлимо для «курсача», но никогда в продакшене), если хотите кнопочки на русском, пишите qsTr(«English text») потом переводите и подключайте транслятор.
3. Стилистика: имена переменных выбраны не удачно, вот вы, не зная, что делает программа, смогли бы сказать, чем отличается text_input1 от text_input2? Я нет. Но, если бы они назывались sourceWordInput и HammingCodeInput, было бы куда нагляднее.
4. Кнопка используется у вас два раза, если потребуется третья вам придется городить новый Rectangle,
гораздо лучше вынести кнопку в отдельный компонент (т.е. qml-файл), ято позволит ее использовать многократно.
5. Градиенты — штука безусловно клевая, но, к сожалению, сейчас жутко медленая, я (да и многие другие) рекомендую отказываться от градиента там, где это возможно, и заменять его изображением.
6. MainWindow не нужен, QDeclarativeView сам по себе является виджетом. Достаточно было в main.cpp написать:
QDeclarativeView view;
view.setResizeMode(QDeclarativeView::SizeRootObjectToView);
view.setSource("qrc:/main.qml");
view.show();
Мало того, налицо нарушение архитектуры, вы в класс виджета выносите методы кодирования/декодирования, это гораздо лучше выглядело бы, вынесеным в отдельный класс, который можно было бы, как подключить qml плагином к приложению, так и (в данном простом случае) установить его инстанс как глобальное свойство.
В целом приятно что люди интересуются данной технологией, потому как она имеет огромный потенциал, но давайте все же самосовершествоваться и стараться писать хороший код с помощью хорошей технологии.
Спасибо.
UFO just landed and posted this here
Ну как я понял, этого делать было нельзя ибо такое приложение было бы похожим на то что сдавал предыдущий человек.
UFO just landed and posted this here
Ну например можно из программы убрать кнопки и приклеиться к textChanged и пытаться кодировать/декодировать на лету :)
У нас к счастью интересовались.
Про Qt не понял, почему сразу перестает интересовать?
У нас к счастью интересовались.
Про Qt не понял, почему сразу перестает интересовать?
> А вообще преподаватели, интересующиеся кодом, пересчитываются на пальцах
*Вздыхая* даа…
Кстати в некоторых ВУЗах Qt преподают, очень сожалею, что мой к таким не относится…
*Вздыхая* даа…
Кстати в некоторых ВУЗах Qt преподают, очень сожалею, что мой к таким не относится…
Человеку и правда не нравилось, его слова «Все равно похоже»…
Так же попробую ответить одним постом:
0. Первый раз пишу статью, честно — не знал, что тег
0. Первый раз пишу статью, честно — не знал, что тег
нормально не подсветит!
1. Просят именно литературу, а не документацию. Урок, я думаю, лучше английской спецификации для новичков.
2. Ааааа тут просто nc, вы думаете я не работаю? Не оформляю официальный код как надо? Зачем загромождать пример, именно пример, сторонней информацией?
3. Эти вещи набросал в редакторе, очевидно. Я вроде бы сказал насчет 15-ти минут...
4. Так и было, кнопочку уже использовал и не раз, файл носил гордое название Button.qml, для этой цели там и сигнал кстати есть... Но опять же не хотелось ничего лишнего.
5. Учту, но медлительности, собственно, не замечал - даже на симбианах :)
6. Ответ все остальное сразу - скажите, что проще понять новичку - подключение плагина или обычный вызов слотов через свойство?... Ответ очевиден!
В целом, очень рад, что первый же коммент оказался таким полным, однако я лично считаю, что не стоит относиться к уроку, как к проекту на сдачу :) И холивар тоже кстати разводить не хочется насчет индивидуальных предпочтений. Спасибо.Урок на русском: developer.qt.nokia.com/wiki/GettingStartedQMLRussian
ну он же уроК :) Это же хорошо, когда есть урокИ?
Ну вот могу предложить вот это lorcode.org/wiki/Qt_in_Education_Course_Material
Но перевод пока что в процессе.
Но перевод пока что в процессе.
Прошу прощения, по QtQuick уже перевели статью. Вот, тут чисто архив с уроками Qt Quick ge.tt/8ilffC5
Приятно видеть, что я не зря старался. А вообще перевод заглох по причине нехватки людей. Если у кого то есть желание помочь — было бы замечательно, с полным переводом этих уроков можно было бы надеятся на внедрение Qt в образовательный процесс. Вся инфраструктура уже готова (), все статусы «в процессе» на самом деле равнозначны «нет перевода», т.к. взявшиеся люди видимо «забили». Если кто-нибудь желает взяться за перевод — пишите в личку, может и я снова займусь.
Я бы очень хотел помочь развитию проекта. Но у меня была сессия, сейчас начинается практика и к тому же нужно готовиться к сертификационному экзамену.
Вот после 30 июня я смогу помочь с переводами, самому хочется хорошо разбираться. Занимался 2 переводами в Full Circle.
Вот после 30 июня я смогу помочь с переводами, самому хочется хорошо разбираться. Занимался 2 переводами в Full Circle.
Приятно, что моя статья дала хоть и небольшой, но толчок развитию русскоязычного сообщества Qt :)
Я связь QML с С++ изучал по gutenberg.troll.no/4.7-snapshot/qtbinding.html, когда еще доки не было. Примеры отличные, как впрочем и вся дока Qt.
Я как-то писал немного более сложный пример для подобных целей.
Вот тут можно почитать. В основном про встраивание C++-класса в QML.
Вот тут можно почитать. В основном про встраивание C++-класса в QML.
Sign up to leave a comment.
Создание гибридного Qt Quick и C++ приложения