프로그래밍

3206,

9/161

skwksk

이거이거 문제가 계속 나오는군요.ㅠㅠ

http://www.hackerschool.org/HS_Boards/zboard.php?id=QNA_programming&no=2169 [복사]

#include<winsock2.h>
#include<process.h>
#include<stdio.h>
#include<stdlib.h>
#include<windows.h>

#define BUF 100       //입력 데이터의 크기

HANDLE mutex;   //뮤텍스 선언
SOCKET sock_client2[5];     /*   쓰레드에서 접근할 데이터    */
int user_num=0;

void Error(char *msg);  //에러메세지 출력
DWORD WINAPI client_paly(void *arg);   //클라이언트 메세지받는 프로그램
void messagesend(char *message, int MSGlen);   //메세지 전달하기

int main(int argc, char **argv)
{
        WSADATA wsadata;
        SOCKET sock_serv, sock_client;
        SOCKADDR_IN addr_serv, addr_client;
        int addr_size,count=0;
        HANDLE thread;
        unsigned int dwThreadID;

        if(WSAStartup(MAKEWORD(2,2), &wsadata)!=0)
                Error("WSAStartup Error");

        mutex = CreateMutex(0,FALSE,0);  //뮤텍스 생성

        sock_serv = socket(PF_INET, SOCK_STREAM, 0);
        if(sock_serv == INVALID_SOCKET){
                WSACleanup();
                Error("SOCKET() Error");
        }

        memset(&addr_serv,0,sizeof(addr_serv));
        addr_serv.sin_family = AF_INET;
        addr_serv.sin_addr.s_addr = htonl(INADDR_ANY);
        addr_serv.sin_port = htons(atoi(argv[1]));

        if(bind(sock_serv, (SOCKADDR *) &addr_serv, sizeof(addr_serv))==SOCKET_ERROR){
                WSACleanup();
                Error("BIND Error");
        }

        if(listen(sock_serv,5)==SOCKET_ERROR){
                WSACleanup();
                Error("listen Error");
        }

        while(1){
                addr_size=sizeof(addr_client);
                sock_client=accept(sock_serv,(SOCKADDR *)&addr_client,&addr_size);
                if(sock_client==INVALID_SOCKET){
                        WSACleanup();
                        Error("sock_client Error");
                }

                WaitForSingleObject(mutex,INFINITE);  //뮤텍스 얻기
                sock_client2[user_num]=sock_client;
                user_num++;
                ReleaseMutex(mutex);   //뮤텍스 반환

                printf("새로 들어온 유저 ID : %d \n",sock_client);

                thread = (HANDLE)_beginthreadex(NULL, 0, (unsigned int(__stdcall *)(void *))client_paly,
                                      (void*)sock_client, 0, (unsigned int*)&dwThreadID);// 쓰레드 생성
                if(thread == 0){
                        WSACleanup();
                        Error("thread Error");
                }
        }

        WSACleanup();
        return 0;
}

DWORD WINAPI client_paly(void *arg)
{
        SOCKET sock_client = (SOCKET)arg;
        char buffer[BUF];
        int len,i;

        while((len=recv(sock_client, buffer, BUF, 0))!=0)
                messagesend(buffer,len);

        printf("접속 종료 유저 ID : %d \n",sock_client);
        WaitForSingleObject(mutex,INFINITE);
        for(i=0;i<user_num;i++){   //유저 연결 종료
                if(sock_client==sock_client2[i]){
                        for( ;i<user_num-1;i++)
                                sock_client2[i]=sock_client2[i+1];
                        break;
                }
        }
        user_num--;
        ReleaseMutex(mutex);
        closesocket(sock_client);
        return 0;
}

void messagesend(char *message, int MSGlen)
{
        int i;
        WaitForSingleObject(mutex,INFINITE);
        for(i=0;i<user_num;i++)
                send(sock_client2[i],message,MSGlen,0);
        ReleaseMutex(mutex);
}

void Error(char *msg)
{
        fputs(msg,stderr);
        fputc('\0',stderr);
        exit(1);
}

간단한 서버 프로그램 입니다. 메세지가 들어오면 접속되어 있는 클라이언트들에게 메세지를 뿌려주는 아주아주 간단한 채팅 서버입니다. 하지만... 이거원... 이런 간단한 프로그래밍도 쩔쩔 매고 있네요.ㅠㅠ 왜 메세지를 뿌려주지 못하는걸까요.. 클라이언트에서 접속해서 메세지를 보내면 돌려 받지를 못합니다. send()랑 클라이언트 소켓 할당하는부분등을 다시 봐도 이상한 곳이 없는거 같은데... 왜그럴까요.ㅠㅠ

Hit : 3181 Date : 2009/10/25 10:39


commio	그냥 접으세요 ㅡㅡ^ 존나 어이 없네요 가르쳐준지 하루가 밖이 안지났어요 ㅡㅡ^ 이렇게 이해력없고 같은걸로 실수할정도라면 그냥 맘편히 먹고 접으세요 희망이 안보임	2009/10/25
tseug	말씀이 지나치십니다.... 희망이 없다니요.......	2009/10/25
commio	똑같은 문제로 하루도 안지나서 답변을 하는걸... 어떻게 생각하십니까...?	2009/10/25
skwksk	음... 어제는 메인 쓰레드가 종료되서 같이 종료되는거 아니였나요?? 이거는 while로 계속돌게 되어있어서 종료되지 않는데...... 그리고 commio님은 한번 실수한걸 다시는 실수 하지 않나보네요... 저는 그렇지 못해서 반복학습을 해야되거든요.... 실수 하지 않으셔서 좋겠습니다. 부럽네요.ㅎㅎ	2009/10/25
commio	아.. - _-;; 잘못봤네여...흑...ㅈㅅ여.. 에러처리는 전부해보셨나요..? 중간중간 확인과 패킷스니핑은요? ... 전 누군가에게 질문하기전에 2~3일은 혼자 고민하고 해결하는 스타일이라... 한번 실수한건 잊을래야 잊을수가 없죠 ㅠㅠㅠㅠ 조금더 생각해보시고... 모든 에러처리, 중간중간의 데이터확인, 패킷스니핑.. 이정도는 해보시면 답이 나올듯한데..	2009/10/25
k1rha	commio 님이 답변을 너무 열심히 달아주시다보니 ㅋㅋ 이래저래 힘드신듯 --ㅋ	2009/10/26
commio	k1rha 님이 옹호해줘서...ㅋㅅㅋ... 말도 나쁘게 한것도 있고해서... 그대로 가져다가 테스트 해봤는데 아무런 문제가 없는데요..ㅠ_ㅠ.. 클라이언트쪽 문제 아닌가요 - _-?	2009/10/26
commio	그리고 전에도 애기한거 같지만 중복되는 헤더파일 선언과 쓰지않는 헤더파일 선언시 어떻게 될지 모르므로 왠만하면 피하시는게 좋습니다. ( 이런식으로 헤더파일을 선언할거면 #include <headall.h> 라는걸 만들어 두고 쓰겠죠. 위험하니깐 안하는거에요 프로그램에도 부하가 많이 걸리고 ) 또한 동기화 함수의 남발... 물론 동기화를 해야할 필요성은 분명히 있습니다. 하지만 동기화 방법에서 문제가 있는거죠, 바로 뮤텍스 사용입니다. 뮤텍스는 커널 동기화 모듈임니다. 어플리케이션이 커널에 진입하기에는 조금많이 힘이듬니다, 이렇게 힘이드는것을 사용할 이유가 없으니, 크리티컬섹션을 사용하세요. 무조건 상위의 함수를 쓴다고 좋은 프로그램이 아님니다, 필요에 따라서 적당한 함수는 쓰는것이 좋습니다.	2009/10/26
pr0sp3r	commio // 의욕이 넘쳐서 욕을 먹는 케이스네요 ^^; 조금더 멀리 넓게 보시고 열공하세요 모두가 다 내맘 같지는 않으니 해당 문제원인을 찾으려면 어디서 문제가 발생하는지 부터 알려줘야 예의가 아닌가 싶군요. 기본적인 sock 동작은 아래와 같습니다.(물론 리눅스입니다) 윈도우도 크게 틀리진 않습니다. 아래 각각 함수들이 어떤 역활을 하고 입력과 출력이 어떤 형태로 이루어지는지 안다면 어떤 부분에서 문제가 발생했는지 알기는 쉽겠지요. 이런식의 문제 해결 접근 방법은 디버깅시에도 분석 스킬을 향상 시켜줍니다. 그런것 배우려고 언어를 공부하는게 아닌가요? ^^; socket()->bind()->accept()->loop:start->listen()->recv()/send()->loop:end->close()	2009/10/29
pr0sp3r	나의 단점과 잘못을 지적해주는 사람의 말까지도 받아 들인다면 더 많은걸 얻을 수 있을 겁니다.	2009/10/29
commio	문제를 찾으려고 노력을 하지않았기 때문임니다, 적어도 자신이 알지 못하는 에러가 나왔다면 2일은 혼자 고민해봐야 되는거아닌가요..? 고작 하루조차되지않아 질문하는사람들을 보면 그저 답답하기만 합니다. 그정도의 노력도 들이지 않는다면 그건 그저 따라하는것일 뿐이고 다른 사람들의 그림자가 될뿐입니다. 그리고 저 코드는 테스트 결과 아무런 문제가 없었습니다 - -;; 결국 테스트조차 제대로 해보지않았고 디버깅이라는거 자체를 해보지않았다는거죠.. 이런 성의없음에 어떻게 대답하는게 옳을까요..?	2009/10/29
crownw	제가 한마디만 할께요 정말 오해마세요 제 경험을 직접 얘기해드리는거에요 저도 예전에 제가 풀어야할 소스를 몰라서 해커스쿨 여기나 다른사이트에 대신 수정좀 해달라고 힌트좀 달라고 올린적이 있었습니다 그러나 시간이 지나고 난뒤 그 소스들은 남이 해준것이기 때문에 다 잊어버렸죠 하지만 지금은 그렇지 않아요 제가 생각하고 스스로 끝까지 바탕이된 기초로 생각하다보면 정말로 제가 소스를 풀 수있게되었어요 또 그 소스는 바로 제 것이 되었구요 제가 하고 싶은 말은 결론 딱 한가지 입니다. 스스로 한번 해보세요 정말 생각하면 다됩니다. 물론 기초가 바탕이 되어있어야겠죠 정말 입니다.	2009/10/31
goaqjrjwlrp	vc++6.0에서 위 소스들 컴파일할떄 멀티쓰레드,w2_32.lib추가시킨다음 DWORD를 unsinged int로 바꾸어야 됨 이렇게 해서 컴파일에러나면 위소스가 틀린거임	2009/11/17