Bug 21376 - предложения по оптимизации обработки запроса new
Summary: предложения по оптимизации обработки запроса new
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-browser-qt (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Sergey V Turchin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-03 11:39 MSD by inger@altlinux.org
Modified: 2009-10-15 14:19 MSD (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description inger@altlinux.org 2009-09-03 11:39:46 MSD 
Сейчас при создании виджета часть его атрибутов приходит сразу в команде new, а часть в последующих командах set. При этом список "избранных атрибутов" жёстко фиксирован.

Я предлагаю: убрать ограничения в команде new на вид присутствующих там атрибутов. То есть вместо того чтобы информацию по виджету передавать в десяти командах, передавать её в одной. Например, можно было бы сразу задать текст в метке (атрибут text).

В результате:
1. Сократится объём трафика между alterator-browser и alterator.
2. Сам протокол станет проще и понятнее.
3. Упростится код самого browser-qt. В том числе проще будет отлаживать. А то сейчас ошибочное размещение очередного нового атрибута не в  том классе приводит к сложно-улавливаемой ошибке.
4. упростится и станет более понятным другим код alterator.

Данное изменение обладает обратной совместимостью, поэтому browser-qt будет работать (если это ему вдруг потребуется) как с новым, так и со старым alterator'ом. (инструкция set всё-равно останется).
Comment 1 Sergey V Turchin 2009-09-03 14:42:58 MSD 
Это лучше не делать до бранчевания. А вообще, я за
Comment 2 Sergey V Turchin 2009-09-03 14:47:12 MSD 
(В ответ на комментарий №0)
> 2. Сам протокол станет проще и понятнее.
Не факт. Все равно остануться атрибуты, которые будут работать только в пределах того запроса, в котором делали new.

> 3. Упростится код самого browser-qt.
Не, вряд ли. Но, гибче немного станет.
Comment 3 inger@altlinux.org 2009-09-03 19:46:59 MSD 
(В ответ на комментарий №2)
> (В ответ на комментарий №0)
> > 2. Сам протокол станет проще и понятнее.
> Не факт. Все равно остануться атрибуты, которые будут работать только в
> пределах того запроса, в котором делали new.
Да, но это будет уже тайное знание браузера не интересное его клиентам ;)
> 
> > 3. Упростится код самого browser-qt.
> Не, вряд ли. Но, гибче немного станет.
Ну как минимум можно будет убрать перекладывания атрибутов из одного хеша в другой, так как сейчас делается для onNewResponse ;)

P.S. Я бы не обращал внимание на бранчевания ибо не понятно насколько оно затянется. Как вариант можно сделать сейчас в отдельном бранче git-а и оттестировать.
Comment 5 Sergey V Turchin 2009-10-08 16:57:15 MSD 
Я переделал, посмотри. Должно работать и в текущем виде и потом
Comment 6 Sergey V Turchin 2009-10-15 14:19:16 MSD 
alterator-browser-qt-2.12.0-alt1