TVT_AddNode - HiAsm

Форумы Делаем компонентыTVT_AddNode ← Ctrl1Ctrl →

Этот топик читают: Гость

Ответов: 10

Рейтинг: 0

#1: 2009-04-23 16:36:03 ЛС | профиль | цитата

По моему в коде компонента должно быть так:


#pas
begin
  d := ReadMTData(_Data, _data_Data); 
  err := _prop_TreeView.AddNode(d);
  case err of
    ADD_ERR_SUCCESS: _hi_onEvent(_event_onAddNode, d);
    ADD_ERR_PARENT_NF: _hi_onEvent(_event_onAddFailed, d); 
  end;    
end;

а не так:


#pas
case err of
    ADD_ERR_SUCCESS: _hi_onEvent(_event_onAddNode );
    ADD_ERR_PARENT_NF: _hi_onEvent(_event_onAddFailed, d); 
  end;

При варианте , который идет с HiAsm в поток НЕ выдаются добавленные данные...
Поправьте, если не так...
НЕ ТУДА! Переместите топик в делфи!!!

карма: 1

Google vip	#1.1контекстная реклама от партнеров

Администрация

Ответов: 15294

Рейтинг: 1518

#2: 2009-04-23 17:23:54 ЛС | профиль | цитата

а аргументов никаких не будет, почему это должно быть так?

карма: 26

https://hiasm.com

Ответов: 10

Рейтинг: 0

#3: 2009-04-23 17:28:23 ЛС | профиль | цитата

Потому что если не добавить d то в поток (в случае успеха вставки нода) ничего не выдается, пустые данные. А с d выдаются данные, которые успешно обработаны (параметры добавленного нода). И с ними можно работать дальше (например я их заношу в БД).
Проверил на практике.

карма: 1

Администрация

Ответов: 15294

Рейтинг: 1518

#4: 2009-04-23 17:44:11 ЛС | профиль | цитата

на самом деле добавление в дерево должно проходить последней инстанцией. Из расчета на это и строился элемент. Внести предложенную правку безусловно - нельзя, это приведет к потере совместимости. Поэтому нужно ставить Hub и проблема будет решена.
------------ Дoбавленo в 17.49:
на всякий случай поясню - когда с данными что-то делается(занесение в базу) после их отображения на экране пользователя(добавление в дерево), то сразу же возникает риск ложного отображения в случае краха последующей операции над ними. Например - добавили данные в дерево, а в базу внести не смогли - идентификаторы совпали или еще чего-то случилось. Получили рассинхронизацию реального положения дел с отображаемым.

карма: 26

https://hiasm.com

Голосовали:

Konst

Форумы Делаем компонентыTVT_AddNode ← Ctrl1Ctrl →


Сообщение ...
Прикрепленные файлы (файлы не залиты)