Comportamiento extraño al colorear un ListView

[Ziara]

Buenas tardes!

Snf2 es un objeto único? O existe uno por cada Item en el ListView?

Es un objeto único.

Prueba a añadir en tu código DefaultDraw = false;

He probado a añadir esto en el código, y cuando ejecuto la aplicación, los paquetes que se van capturando no aparecen visibles, es decir, sé que captura paquetes ya que el scroll vertical va bajando, pero no los visualizo en la pantalla, parecen invisibles, que podría ser?

Gracias.

[[maeyanes]]

Hola...

Me imagino que lo que estás haciendo es como un registro de actividad de lo que recibe el objeto, no es así?

Entonces, lo que sucede es que el componente al redibujar siempre va a leer el último valor que tiene el objeto, es por eso que cuando haces un scroll ya no se ven como cuando se insertaron la primera vez...

Para esto podrías usar la propiedad Data del TListItem (el ejemplo te lo pongo en Delphi, pero deberías poder traducirlo a C sin problemas):

Código Delphi [-]
// Agregas un TListItem nuevo:
with ListView1.Items.Add do
  if snf2.Proto = 'TCP' then
    Data := Pointer(0)
  else
    if snf.Proto = 'UDP'
      Data := Pointer(1)
    else
      Data := Pointer(2);

// Ahora en el evento OnCustomDrawItem:
procedure TForm1.ListView1CustomDrawItem(Sender: TCustomListView; Item: TListItem;
  State: TCustomDrawState; DefaultDraw: Boolean);
begin
  case Integer(Item.Data) of
    0:
      Sender.Canvas.Font.Color := clGreen;
    1:
      Sender.Canvas.Font.Color := clBlue;
    2:
      Sender.Canvas.Font.Color := clPurple
  else
    Sender.Canvas.Font.Color := clRed
  end
end;

Ya con esto cada que hagas un scroll se deben dibujar bien los items en la lista...


Saludos...

[Ziara]

Buenas,

Me imagino que lo que estás haciendo es como un registro de actividad de lo que recibe el objeto, no es así?

mmm, creo que te entiendo y creo que si, en una clase llamada Sniffer hay una serie de métodos que hace la función de capturar paquetes, esto lo tengo en un Unit3 (por ejemplo) y es donde voy incluyendo los paquetes en el ListView.

Como el ListView lo tengo en Form1 en Unit1 pues traslado el valor que me interesa con snf->Proto() donde snf es un objeto de la clase Snifer, así paso el valor que me interesa (en este caso el protocolo) del Unit3 al Unit1 para usarlo dentro del método de ListViewCustomDrawItem como lo he indicado anteriormente.

Según el código:

// Agregas un TListItem nuevo:
with ListView1.Items.Add do
 if snf2.Proto = 'TCP' then
    Data := Pointer(0)
else
    if snf.Proto = 'UDP' 
      Data := Pointer(1)
    else
      Data := Pointer(2);

donde lo agrego? en Unit1 únicamente tengo el metodo
ListViewCustomDrawItem.

No lo tengo muy claro. Gracias

[[maeyanes]]

Hola...

Ese código lo vas a poner en donde agregas los items en el ListView... Busca en tu código donde haces ListView.Items.Add y ahí lo puedes adaptar o agregar...

Por ejemplo si tienes en tu código algo como:

TListItem AItem = ListView->Items->Add;
AItem->Caption = snf2->Proto();
/* Aquí agregas el código */
if (AItem->Caption = '"TCP")
{
  AItem->Data = Pointer(0);
}
else
  if (AItem->Caption = "UDP")
  {
    AItem->Data = Pointer(1);
  }
  else
  {
    AItem->Data = Pointer(2);
  }

Ahora, tiene años que no programo nada en C++, así que el casting de entero a puntero me imagino que no se hace de la forma en que lo puse...


Saludos...

[Ziara]

Buenas tardes!

Todavía no lo tengo muy claro, te voy a plantear mis dudas.

Ese código lo vas a poner en donde agregas los items en el ListView... Busca en tu código donde haces ListView.Items.Add y ahí lo puedes adaptar o agregar...

Bien, el ListView lo tengo en un Unit4, por ejemplo, que se ejecuta como un hilo donde me va mostrando, en una función, la información en el ListView. Ahora mismo lo tengo así:

void __fastcall Thread::Mostrarlineas()
{
 TListItem* item = Form1->ListView->Items->Add();
 item->Caption = cadena1;
 item->SubItems->Add(cadena2);
 item->SubItems->Add(cadena3);
}

Ahora agrego el código siguiente:

void __fastcall Thread::Mostrarlineas()
{
 TListItem* item = Form1->ListView->Items->Add();
 
//agrego otro objeto item
 TListItem* dataitem = Form1->ListView->Items->Add();
 dataitem->Caption = cadena2; //cadena2 indica el protocolo(UDP,TCP..)
 if (dataitem->Caption == "TCP")
 {
  dataitem->Data = Pointer(0);
 }
else if (dataitem->Caption == "UDP")
 {
  dataitem->Data = Pointer(1);
 }
else
 {
  dataitem->Data = Pointer(2);
 }

 item->Caption = cadena1;
 item->SubItems->Add(cadena2);
 item->SubItems->Add(cadena3);
}

No se si esto es correcto, que está haciendo el nuevo código introducido?

Gracias.

[[maeyanes]]

Hola...

Lo que tienes que hacer es:

void __fastcall Thread::Mostrarlineas()
{
  TListItem* item = Form1->ListView->Items->Add();
  item->Caption = cadena1;
  item->SubItems->Add(cadena2);
  item->SubItems->Add(cadena3);
  // Aquí el código:
  if (item->Caption == "TCP")
  {
    item-Data = 0;
  }
  else if (item->Caption == "UDP")
  {
    item->Data = 1;
  }
  else
  {
    item->Data = 2;
  }
}

Ahora, aquí estoy suponiendo que item->Caption tiene como valor el nombre del protocolo, si no es así, solo tienes que cambiar las comparaciones en los if...

También toma en cuenta que item->Data es una propiedad de tipo Pointer, esto al menos en Delphi, en C++ no se como se trate. También la asignación de los datos a item->Data puede que esté erronea, ya que C++ no es mi fuerte.

Una vez que logres esto, el códido en el evento OnCustomDraw del TListView puede ser una estructura switch..case sobre la propiedad item->Data...


Espero te sirva...


Saludos...

[Ziara]

Hola de nuevo,

entonces si mi columna es la tercera me quedaría así:

void __fastcall Thread::Mostrarlineas()
{
 TListItem *item = Form1->ListView->Items->Add();
 item->Caption = cadena1;
 item->Caption = Add(cadena2);
 item->Caption = Add(cadena3);
//codigo nuevo
if (item->SubItems->Strings[2] == "TCP")
{
  item->Data = 0;
}
else if (item->SubItems->Strings[2] == "UDP")
{
  item->Data = 1;
}
else
{
  item->Data = 2;
 }

}

Se usa strings[] para concretar la columna no? si entre corchetes escribo 2 me estoy refiriendo a la columna nº3?

Cuando compilo me saca, el builder me saca un error:

no puede convertir un int en un *void

ya que item->data devuelve un *void y estoy asignadole un int.

Había pensando crear una variable para recoger el contenido de item->Data
algo como:

item->Data = var

pero var tiene que ser un variable que no devuelva nada y no sé qué tipo de variable usar para declarar var.

también había pensado en convertir item->Data en un int:

int var;
int (item->Data) = var;

pero esto me da el siguiente error:

No se puede modificar un objeto constante.

Hay alguna alternativa?

Gracias.