Главная страница
Top.Mail.Ru    Яндекс.Метрика
Текущий архив: 2006.07.02;
Скачать: CL | DM;

Вниз

мессага для listview   Найти похожие ветки 

 
redlord   (2006-05-24 18:38) [0]

привет всем
народ подскажите почему приведенный ниже код приводит к критичесской ошибке памяти в kernel32.dll

lvitem.cchTextMax:=20;
lvitem.iSubItem:=1;
lvitem.pszText:="";

hndcontrol:=form1.ListView1.Handle;
Count:=SendMessage(hndcontrol, LVM_GETITEMCOUNT, 0, 0);
for i := 0 to Count-1 do
begin
q:=SendMessage(hndcontrol,LVM_GETITEMTEXT, 1, intege(@LVItem));
if q <> 0 then
ListBox1.Items.Add(LVItem.pszText)
else
ListBox1.Items.Add("error");
end;


 
GanibalLector ©   (2006-05-24 18:49) [1]

1) Структуру LVITEM нужно инициализировать.Т.е. нужно выделить память для буфера, который будет принимать текст.
2) сделать проверку для Count. Он должен быть больше 0


 
redlord   (2006-05-24 18:57) [2]

lvitem.cchTextMax:=20;
lvitem.iSubItem:=1;
lvitem.pszText:="";

после всего етого она не инициализирована (
обьясните как ее инициализировать


 
GanibalLector ©   (2006-05-24 19:03) [3]


GetMem(Item.pszText, MaxLength);// где MaxLength например константа=$ff
  try
    Item.cchTextMax := MaxLength;
    Item.iSubItem := 0;
    repeat
      Dec(I);
      Item.iItem := I;
      SendMessage(ListView.Handle, LVM_GETITEMTEXT, I, Integer(@Item));
      MessageBox(0, Item.pszText, "Preved", MB_OK);
    until I = 0;
  finally
    FreeMem(Item.pszText)
  end


 
begin...end ©   (2006-05-24 21:20) [4]

> GanibalLector ©   (24.05.06 19:03) [3]

Я бы сделал немного по-другому:

var
 P: PChar;
begin
 GetMem(P, ...);
 try
   Item.pszText := P;
   ...
 finally
   FreeMem(P)
 end
end


Цитата из MSDN:

If the LVIF_TEXT flag is set in the mask member of the LVITEM structure, the pszText member must point to a valid buffer and the cchTextMax member must be set to the number of characters in that buffer. Applications should not assume that the text will necessarily be placed in the specified buffer. The control may instead change the pszText member of the structure to point to the new text, rather than place it in the buffer.

Т.е. ListView вместо заполнения выделенного нами буфера может изменить значение поля pszText, поместив в него адрес какого-либо своего готового буфера с текстом. И тогда код из [3] даст, во-первых, утечку памяти (потому что указатель на выделенный нами буфер мы потеряли), а во-вторых -- ошибку при попытке освобождения памяти по адресу Item.pszText (потому что мы её не выделяли).

И хотя это примечание из MSDN относится к сообщению LVM_GETITEM, вероятно, следует обращать на него внимание и в случае LVM_GETITEMTEXT.



Страницы: 1 вся ветка

Текущий архив: 2006.07.02;
Скачать: CL | DM;

Наверх




Память: 0.48 MB
Время: 0.033 c
1-1148481266
harddrive_tester
2006-05-24 18:34
2006.07.02
Тестирование жёсткого диска


15-1149584884
Kerk
2006-06-06 13:08
2006.07.02
Бойцовский клуб


2-1149817570
Василий
2006-06-09 05:46
2006.07.02
Работа печати через "Begindoc...EndDoc"


1-1148280772
deltav1
2006-05-22 10:52
2006.07.02
TThread


2-1149835201
Tempora
2006-06-09 10:40
2006.07.02
Как перезапустить службу?