Improve company productivity with a Business Account.Sign Up

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 366
  • Last Modified:

String Concat problem, not working

I am trying to concat 9 edit controls. I don't see why this code is not working. Can anyone help me get this working?

    char szCombo1[2], szCombo2[2],szCombo3[2],
         szCombo4[2],szCombo5[2],szCombo6[2],
         szCombo7[2],szCombo8[2],szCombo9[2];
     
    GetDlgItemText(gl_hWnd, IDC_COMBO1, szCombo1, strlen(szCombo1));
    GetDlgItemText(gl_hWnd, IDC_COMBO2, szCombo2, strlen(szCombo2));
    GetDlgItemText(gl_hWnd, IDC_COMBO3, szCombo3, strlen(szCombo3));
    GetDlgItemText(gl_hWnd, IDC_COMBO4, szCombo4, strlen(szCombo4));
    GetDlgItemText(gl_hWnd, IDC_COMBO5, szCombo5, strlen(szCombo5));
    GetDlgItemText(gl_hWnd, IDC_COMBO6, szCombo6, strlen(szCombo6));
    GetDlgItemText(gl_hWnd, IDC_COMBO7, szCombo7, strlen(szCombo7));
    GetDlgItemText(gl_hWnd, IDC_COMBO8, szCombo8, strlen(szCombo8));
    GetDlgItemText(gl_hWnd, IDC_COMBO9, szCombo9, strlen(szCombo9));
              
    std::string strCombo1(szCombo1);
    std::string strCombo2(szCombo2);
    std::string strCombo3(szCombo3);
    std::string strCombo4(szCombo4);
    std::string strCombo5(szCombo5);
    std::string strCombo6(szCombo6);
    std::string strCombo7(szCombo7);
    std::string strCombo8(szCombo8);
    std::string strCombo9(szCombo9);
    std:string strUserCombo;
    strUserCombo = strCombo1+strCombo2+strCombo3+strCombo4+strCombo5+strCombo6+strCombo7+strCombo8+strCombo9;
    MessageBox(NULL,strUserCombo.c_str(),"DEBUG",NULL);

Open in new window


Thanks!
0
edvinson
Asked:
edvinson
  • 2
1 Solution
 
jkrCommented:
In the above, 'strlen()' will return 0 or an undefined value, that's why that won't work. You need to provide the actual size of the buffer as the last argument. Are you sure that should be two bytes?
0
 
edvinsonAuthor Commented:
I am not sure it should be two bytes, I am expecting the user to enter two digits, thats why i chose 2 bytes. should I only use 1? Or something different? What if I dont know how many characters they entered?
0
 
jkrCommented:
If you are expecting two digits, make that at least 3 bytes (the two digits plus the terminating NULL byte). However, a larger buffer won't hurt ;o)
Anyway, 'strlen()' is the wrong way, since it will report the lenght of the string stored in that buffer - and since there is none prior to your call, that's bound to fail.
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

Join & Write a Comment

Featured Post

Free Tool: Site Down Detector

Helpful to verify reports of your own downtime, or to double check a downed website you are trying to access.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now