пара замечаний по коду

Misha v.3 17.07.2006 14:34

1.

@doXmlNew[docName;root]
$self.$docName[...]

мне это очень не нравится, и вот почему:
^doXmlNew[doXmlNew;...]
неужели сложно было создать хеш-контекнер для хранения xdocs?

2.

@getXmlRpc[docName;flag]
$self.$docName[^xdoc::create{$request:body}]
^if($self.$docName is xdoc){
	$result(true)
}{
	$result(false)
}
^if($flag != 1){$result[]}

- ваш метод выдает или строку (пустую) или boolean переменную. не надо так делать (а есть методы которые выдают string|bool|hash).
- почему-бы сразу не проверить flag и в случае если он 1 - ничего не делать?
- мне очень не нравится строка '^if($flag != 1){$result[]}', которая присутствует почти в каждом методе (думаю не только мне не нравицца это)

3. локальные переменные - есть хорошо

---