Errors with chat client send() recv()

Code:

---

if (FD_ISSET(i, &read_fds))
                            { // we got one!!
                               
                               

                                if (i == STDIN_FILENO) /* Check keyboard */
                                    {                               
                                        /*fputs(identifier, stdout);*/
                                        fgets(msg, sizeof msg, stdin);
                                        /*send(sockfd, newLine, strlen(newLine), 0);
                                        send(sockfd, identifier, strlen(identifier), 0);*/
                                        int count = send(sockfd, msg, strlen(msg), 0);
                                        msg[count] = '\0';
                                    }
                                else if (i == sockfd)
                                {
                                        int count = recv(sockfd, buf, sizeof buf -1, 0);
                                        printf("%s", buf);
                                        buf[count] = '\0';
                                       
                                       
                                }
                        }

---

This code isn't working. It is an extract from my client.c file.

The first message works fine but subsequent messages, if shorter, will show the latter part of the previous message on the line below. I added a line to print out everytime recv was called and confirmed it was only once, so the problem is elsewhere

Anyone any suggestions?

Thanks

Looks like a classic case of "I've been staring at this screen WAY too long now" :p

Code:

---

                                        printf("%s", buf);
                                        buf[count] = '\0';

---

Do you see a problem here?

Im quite confused by your description.

Quote:

---

The first message works fine but subsequent messages,

---

Is the "first message" in your code above? Are you talking about send messages or receive messages thats the problem?

Quote:

---

I added a line to print out everytime recv was called and confirmed it was only once, so the problem is elsewhere

---

What? So is the problem in the above code or not?

Anyways, I would definitely change this

Code:

---

printf("%s", buf);
buf[count] = '\0';

---

to

Code:

---

buf[count] = '\0';
printf("%s", buf);

---

Quote:

---

Originally Posted by MK27

Looks like a classic case of "I've been staring at this screen WAY too long now" :p

Code:

---

                                        printf("%s", buf);
                                        buf[count] = '\0';

---

Do you see a problem here?

---

You have identified the error 100%. I was throwing a inabiliyToSeeTheCodeInFrontOfMe exception.

Thank you, once again.

I just had a quick 5 minute break to try and refresh.....not sure its worked but fingers crossed

Quote:

---

Originally Posted by nadroj

Im quite confused by your description.

---

Yes sorry my description was poor. But it's all sorted now. My brain is slowly shutting down :(

I appreciate your reply though.

It is only through this great forum that I am making more sense of C.