My struct is global, isn't?

Code:

typedef struct {
	HINSTANCE hInstance;
	HWND hListView;
	char* szNombre;
	char* szPC;
} CLIENTESINFO, *LPCLIENTESINFO;

As global:

Code:

CLIENTESINFO ci;

First calling is from a child dialog proc.

Code:

char nom[128], pc[12];
GetDlgItemText(hDlg, IDC_EDIT1, nom, sizeof(nom)+1);
ci.szNombre = nom;
GetDlgItemText(hDlg, IDC_EDIT2, pc, sizeof(pc)+1);
ci.szPC =  pc;

Second call is from the main WinProc:

Code:

char t[12];
int total = ListView_GetItemCount(ci.hListView);
wsprintf(t, "%i", total+1);
InsertItem(ci.hListView, total, 0, t); // t is current position and it shows correctly
InsertItem(ci.hListView, total, 1, ci.szNombre); // empty string

InsertItem prototype:

Code:

void InsertItem(HWND hList, int iItemNo, int iSubNo, char *lpszText) { 
	LVITEM item = {0}; 
	item.mask = LVIF_TEXT; 
	item.pszText = lpszText; 
	item.iItem = iItemNo; 
	item.iSubItem = iSubNo; 
	if (iSubNo == 0) 
		ListView_InsertItem(hList, &item); 
	else 
		ListView_SetItem(hList, &item);
	return; 
}

Now my question is, why the listview always shows a empty string? Isn't the struct global? Thanks

Your struct is global, but nom[] and pc[] are locals, and szNombre and szPC point to those locals. Your should allocate memory for szNombre and szPC using winapi/new/malloc and copy those strings to that memory.

Yeah!

I see your point, that's why I change my struct to class

There is no significant difference between struct and class in C++, so you would have the same issue with each. Also, you don't need the typedef stuff for declaring structs in C++, use the same syntax as you do for classes (unless you share your struct with C code).