HELP with chars!!!!

hi,
please can ne one try and help me with this very annoying problem i have created a simple chat program BUT every time the message sent has less chars than last message sent there are chars left over from previous message :mad:

Code:

---

#include <windows.h>
#include <winsock.h>
#include <commctrl.h>
#include "resource.h"
#include <stdio.h>
#include <string>

#define WM_SCK    WM_USER+1

HBITMAP g_hbmBall = NULL;

char aa[1000];
char bb[1000];
SOCKET s;

char rBuffer[1000];

LPHOSTENT lpHostEntry;

struct sockaddr_in a;

HWND hStatus,hButton,hEdit,hwnd,hConsole;

int error = 0;
int d;
char ip[500];


char szNewBuffer[1024];
std::string szBuffer2;    //or rename it if you want

void OutputTextToScreen(char *p)
{
    szBuffer2.append(p);
    SetWindowText((HWND)hConsole, szBuffer2.c_str());
}


/* Declare Windows procedure */
LRESULT CALLBACK WindowProcedure(HWND, UINT, WPARAM, LPARAM);
/* Make the class name into a global variable */
char szClassName[ ] = "WindowsApp";
int WINAPI WinMain(HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPSTR lpszArgument, int nFunsterStil)
{
    InitCommonControls();

    HWND hwnd;              /* This is the handle for our window */
    MSG messages;            /* Here messages to the application are saved */
    WNDCLASSEX wincl;        /* Data structure for the windowclass */

    /* The Window structure */
    wincl.hInstance = hThisInstance;
    wincl.lpszClassName = szClassName;
    wincl.lpfnWndProc = WindowProcedure;      /* This function is called by windows */
    wincl.style = CS_DBLCLKS;                /* Catch double-clicks */
    wincl.cbSize = sizeof(WNDCLASSEX);

    /* Use default icon and mouse-pointer */
    wincl.hIcon = LoadIcon(GetModuleHandle(NULL), MAKEINTRESOURCE(IDI_FRUIT));
    wincl.hIconSm = LoadIcon(GetModuleHandle(NULL), MAKEINTRESOURCE(IDI_FRUIT));
    wincl.hCursor = LoadCursor(NULL, IDC_ARROW);
    wincl.lpszMenuName = NULL; /* No menu */
    wincl.cbClsExtra = 0;                      /* No extra bytes after the window class */
    wincl.cbWndExtra = 0;                      /* structure or the window instance */
    /* Use light-gray as the background of the window */
    wincl.hbrBackground = (HBRUSH) GetStockObject(LTGRAY_BRUSH);

    /* Register the window class, if fail quit the program */
    if(!RegisterClassEx(&wincl)) return 0;

    /* The class is registered, let's create the program*/
    hwnd = CreateWindowEx(
          0,                  /* Extended possibilites for variation */
          szClassName,        /* Classname */
          "Fruit MaFiA 2oo3 Chat v1.00 (Client)",        /* Title Text */
          WS_POPUP | WS_SYSMENU | WS_CAPTION, /* default window */
          CW_USEDEFAULT,      /* Windows decides the position */
          CW_USEDEFAULT,      /* where the window ends up on the screen */
          600,                /* The programs width */
          495,                /* and height in pixels */
          HWND_DESKTOP,        /* The window is a child-window to desktop */
          NULL,                /* No menu */
          hThisInstance,      /* Program Instance handler */
          NULL                /* No Window Creation data */
          );

    /* Make the window visible on the screen */
    ShowWindow(hwnd, nFunsterStil);
    /* Run the message loop. It will run until GetMessage( ) returns 0 */
    while(GetMessage(&messages, NULL, 0, 0))
    {
          /* Translate virtual-key messages into character messages */
          TranslateMessage(&messages);
          /* Send message to WindowProcedure */
          DispatchMessage(&messages);
    }

    /* The program return-value is 0 - The value that PostQuitMessage( ) gave */
    return messages.wParam;
}

/* This function is called by the Windows function DispatchMessage( ) */
LRESULT CALLBACK WindowProcedure (HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    switch (message)                  /* handle the messages */
    {
        case WM_DESTROY:
        {
                DeleteObject(g_hbmBall);
                PostQuitMessage (0);      /* send a WM_QUIT to the message queue */
        }
        break;
        case WM_CREATE:
        {

                hStatus = CreateWindowEx(0, STATUSCLASSNAME, NULL,WS_CHILD | WS_VISIBLE | SBARS_SIZEGRIP, 0, 0, 0, 0,hwnd, (HMENU)IDC_MAIN_STATUS, GetModuleHandle(NULL), NULL);
                if(hStatus == NULL)
                        MessageBox(hwnd, "Could not create Status Bar.", "Error", MB_OK | MB_ICONERROR);

                hButton = CreateWindowEx(0, "BUTTON", "Connect",WS_BORDER | WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON,130, 50, 90, 20, hwnd, (HMENU)IDC_MAIN_CONNECT, GetModuleHandle(NULL), NULL);
                if(hButton == NULL)
                        MessageBox(hwnd, "Could not create a Connect Button.", "Error", MB_OK | MB_ICONERROR);

                hButton = CreateWindowEx(0, "BUTTON", "Send",WS_BORDER | WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON,230, 50, 90, 20, hwnd, (HMENU)IDC_MAIN_SEND, GetModuleHandle(NULL), NULL);
                if(hButton == NULL)
                        MessageBox(hwnd, "Could not create a Connect Button.", "Error", MB_OK | MB_ICONERROR);

                hEdit = CreateWindowEx(WS_EX_CLIENTEDGE, "EDIT", "127.0.0.1",WS_CHILD | WS_VISIBLE | ES_AUTOHSCROLL,20, 50, 100, 20, hwnd, (HMENU)IDC_MAIN_IP, GetModuleHandle(NULL), NULL);
                if(hEdit == NULL)
                        MessageBox(hwnd, "Could not create edit box.", "Error", MB_OK | MB_ICONERROR);

                hConsole = CreateWindowEx(WS_EX_CLIENTEDGE, "EDIT", "",WS_CHILD | WS_VISIBLE | ES_AUTOHSCROLL | ES_AUTOVSCROLL | ES_MULTILINE | WS_VSCROLL  ,20, 120, 430, 210, hwnd, (HMENU)IDC_MAIN_CONSOLE, GetModuleHandle(NULL), NULL);
                if(hConsole == NULL)
                        MessageBox(hwnd, "Could not create edit box.", "Error", MB_OK | MB_ICONERROR);

                hEdit = CreateWindowEx(WS_EX_CLIENTEDGE, "EDIT", "",WS_CHILD | WS_VISIBLE | ES_AUTOHSCROLL | ES_AUTOVSCROLL | ES_MULTILINE | WS_VSCROLL  ,20, 340, 430, 90, hwnd, (HMENU)IDC_MAIN_TEXTAREA, GetModuleHandle(NULL), NULL);
                if(hEdit == NULL)
                        MessageBox(hwnd, "Could not create edit box.", "Error", MB_OK | MB_ICONERROR);

                hEdit = CreateWindowEx(WS_EX_CLIENTEDGE, "EDIT", "",WS_CHILD | WS_VISIBLE | ES_AUTOHSCROLL | ES_AUTOVSCROLL | ES_MULTILINE | WS_VSCROLL  ,460, 120, 120, 310, hwnd, (HMENU)IDC_MAIN_CLIENTS, GetModuleHandle(NULL), NULL);
                if(hEdit == NULL)
                        MessageBox(hwnd, "Could not create edit box.", "Error", MB_OK | MB_ICONERROR);


                int statwidths[] = {200, -1};

                SendMessage(hStatus, SB_SETPARTS, sizeof(statwidths)/sizeof(int), (LPARAM)statwidths);
                SendMessage(hStatus, SB_SETTEXT, 0, (LPARAM)"Ready...");

                g_hbmBall = LoadBitmap(GetModuleHandle(NULL), MAKEINTRESOURCE(IDB_BACKGROUND));
                if(g_hbmBall == NULL)
                MessageBox(hwnd, "Could not load IDB_BACKGROUND!", "Error", MB_OK | MB_ICONEXCLAMATION);
        }
        break;
        case WM_PAINT:
        {
                BITMAP bm;
                PAINTSTRUCT ps;

                HDC hdc = BeginPaint(hwnd, &ps);

                HDC hdcMem = CreateCompatibleDC(hdc);
                HBITMAP hbmOld = (HBITMAP)SelectObject(hdcMem, g_hbmBall);

                GetObject(g_hbmBall, sizeof(bm), &bm);

                BitBlt(hdc, 0, 0, bm.bmWidth, bm.bmHeight, hdcMem, 0, 0, SRCCOPY);

                SelectObject(hdcMem, hbmOld);
                DeleteDC(hdcMem);

                EndPaint(hwnd, &ps);
        }
        break;
        case WM_COMMAND:
        {
          switch( wParam )
          {
            case IDC_MAIN_CONNECT :
            {

            error = 0;

            int len = GetWindowTextLength(GetDlgItem(hwnd, IDC_MAIN_IP));
            if(len > 0)
            {
            int i;
            char* buf;

            buf = (char*)GlobalAlloc(GPTR, len + 1);
            GetDlgItemText(hwnd, IDC_MAIN_IP, buf, len + 1);
            strcpy(ip,buf);
            GlobalFree((HANDLE)buf);
            }else{
            MessageBox(0,"Please Enter A Address","Error",0);
            error = 1;
            }
            if(error != 1){
                    WSADATA ws;
                    d = WSAStartup(0x101,&ws);
                    if(d != 0){
                              SendMessage(hStatus, SB_SETTEXT, 0, (LPARAM)"WSA...Failed");
                              error = 1;
                    }else{
                              SendMessage(hStatus, SB_SETTEXT, 0, (LPARAM)"WSA...OK");
                    }
            }
            if(error != 1){
                    s = socket(AF_INET,SOCK_STREAM,0);
                    if(s == -1){
                              SendMessage(hStatus, SB_SETTEXT, 0, (LPARAM)"Socket...Failed");
                              error = 1;
                    }else{
                              SendMessage(hStatus, SB_SETTEXT, 0, (LPARAM)"Socket...OK");
                    }
            }
            if(error != 1){

                    a.sin_family = AF_INET;
                    a.sin_port = htons(6000);
                    a.sin_addr.s_addr = inet_addr(ip);

                    WSAAsyncSelect(s, hwnd, WM_SCK, FD_READ | FD_CONNECT | FD_WRITE | FD_CLOSE);

                    d = connect(s, (struct sockaddr *)&a,sizeof(a));
                    if(d != 1){
                              SendMessage(hStatus, SB_SETTEXT, 0, (LPARAM)"Connect...OK");
                              error = 1;
                    }else{
                              SendMessage(hStatus, SB_SETTEXT, 0, (LPARAM)"Connect...Failed");
                    }
            }

            }
            break;
            case IDC_MAIN_SEND :
            {
            int len = GetWindowTextLength(GetDlgItem(hwnd, IDC_MAIN_TEXTAREA));
            if(len > 0)
            {
            int i;
            char* buf;

            buf = (char*)GlobalAlloc(GPTR, len + 1);
            GetDlgItemText(hwnd, IDC_MAIN_TEXTAREA, buf, len + 1);
                sprintf(bb,"\0");
                sprintf(bb,"Server Sending...\"%s\"\r\n",buf);
                OutputTextToScreen(bb);
                        d = send(s,buf,strlen(buf),0);
            }else{
            MessageBox(0,"Please Enter A Message","Error",0);
            error = 1;
            }
            }
            break;
          }
        }
        break;
        case WM_SCK:
        {
        switch(WSAGETSELECTEVENT(lParam))
        {
        case FD_READ:
        {
          recv(s, rBuffer, sizeof(rBuffer), 0);
          sprintf(bb,"Server Receiving...\"%s\"\r\n",rBuffer);
          OutputTextToScreen(bb);
        }
        break;
        }
        }
        break;
        default:                      /* for messages that we don't deal with */
            return DefWindowProc (hwnd, message, wParam, lParam);
    }

    return 0;
}

---

thx in advance

this is just a guess (i didn't actually read all the code..) but if you have a buffer that you store input in (i assume that's what rBuffer is?) then i think you'll need to clear it between inputs. just have a function such as:

Code:

---

void clearBuff(char buffer[], int size)
{
  for(int i = 0; i < size; ++i)
      buffer[i] = '\0';
}

---

and call that before you get input. i may be wrong, since i didn't actually check all the code :rolleyes:

btw, what OS is that...?

thx alot mad hatter im using xp but i doesnt matter no thx alot :D :D :D :D :D :D :D :D