C / C++ / MFC

m_payeeList.GetText(pItems[i], pStr[i]);
StrList.AddTail(pStr[i]);

CString text;
m_payeeList.GetText(pItems[i], text);
StrList.AddTail(text);

POSITION p;<br />
	CString s;<br />
	s.Format("%s",StrList.GetNext(p));	<br />
	AfxMessageBox(s,MB_OK);	 

POSITION p = StrList.GetHeadPosition();
CString s = StrList.GetNext(p); 
AfxMessageBox(s,MB_OK);

 FillReportList(StrList);<br />
<br />
FillReportList(CStringList list)<br />
{<br />
}<br />

FillReportList(const CStringList <big>&</big> list)
{
}

FillReportList(CStringList <big>&</big> list)
{
}

TheinstruTara wrote:

But I did initialised pStr:
char *pStr[10];

for (i = 0; i < iCount; i++)
{
    pStr[i] = new char[100];
    m_payeeList.GetText(pItems[i], pStr[i]);
}

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

int iCount = 0;
   int i;
   iCount = m_payeelist.GetSelCount();
   if (iCount <= 0) return ;
   int* pItems = new int [iCount];
   m_payeelist.GetSelItems(iCount, pItems);
   CStringList StrList;
   for (i=0;i<m_payeelist.GetSelCount();i++)
   {
   char *pStr=new char[10];
   m_payeelist.GetText (pItems[i],pStr);

   //AfxMessageBox (pStr);
       StrList.AddTail(pStr);
   delete pStr;
   }

hfry wrote:

Any reason why you can't use std::string?

hfry wrote:

You have a memory leak for unexName and you have delegated responsiblity to the caller to free partName.

// returns 0 on success
// returns -1 on failure
// returns >0 if buffer isn't big enough

int getpartname(char *filename, char *partname, size_t partnamesize)
{
  int dot = -1;
  int filenamelen = strlen(filename);
  int pos;
  for(pos = filenamelen; pos >= 0; pos--)
  {
    if(filename[pos] == '.')
    {
      dot = pos;
      break;
    }
  }

  if(dot == -1) return -1;
  if(partname == 0 || partnamesize < dot + 4 + 1)
  {
    if(partnamesize != 0) partname[0] = '\0';
    return dot + 4 + 1;
  }

  strncpy(partname, filename, dot);
  partname[dot] = '\0';
  strcat(partname, ".000");

  return 0;
}

// use an array on the stack that has more than enough buffer to store the partname
char partname[80];
getpartname("is730enu.exe.mtx", partname, sizeof(partname));

// or
char *partname;
int len = getpartname("is730enu.exe.mtx", 0, 0);
partname = (char*) malloc(len);
getpartname("is730enu.exe.mtx", partname, len);

// use partname
    :     :

free(partname); // free the memory allocated on the heap

Zac Howland wrote:

not sure how it will handle the double extensions...

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

<br />
strncpy(unexName,filename,strlen(filename)-4);<br />
unexName[strlen(filename)-4]='\0';<br />
strcpy(partName,unexName);<br />
strcat(partName,".000");<br />
printf("\n%s",partName);<br />

---

<marquee scrollamount="1" scrolldelay="1" direction="up" height="10" step="1">--[ ]--

[My Current Status]

I try to add something that's larger than that, why does it add it anyways without crashing