UNIX域套接字read()始终获取错误的值

时间:2013-12-11 18:51:33

标签: c++ sockets

我必须使用UNIX域套接字编写一个"Sea Battle"游戏作为赋值。服务器和客户端在同一台机器上,只是为了了解IPC。

每艘船都有一号。

我遇到了一个我无法解决的问题,我希望你有个主意。

首先我知道read(),write(),select(),...有一个返回值,我检查它们。但是我要告诉他们代码更易读。而且我还留下了其他不必要的代码。

服务器和客户端几乎完全相同,除了连接(这不是问题)并且它们处于相反的状态,使用turn == 0和turn!= 0进行检查。我再说一次。

所以我的问题:在else中,由于if中的控制台输入,read函数是非阻塞的。我尝试用select来解决这个问题,但我总是得错输入。在第一步中,它总是读取(0,0)。在敌人转弯时,它使用上次输入的数据。

int main(int argc, const char * argv[])
{
    if(argv[1]==NULL){
        std::cout << "no socket path" << std::endl;
        return 1;
    }

    Playfield field(5);

    fd_set rfds;
    FD_ZERO(&rfds);


    int turn = 0;
    int sock;
    struct sockaddr_un address;
    int cordinate[2];
    int status[1];
    int gameInfo[1];

    if((sock = socket(AF_UNIX, SOCK_STREAM,0)) < 0){
        std::cerr << "error creating socket" << std::endl;
    }

    address.sun_family = AF_UNIX;
    strcpy(address.sun_path, argv[1]);

    std::cout << "trying to connect..." << std::endl;

    if(connect(sock,(struct sockaddr*) &address, sizeof(address)) == 0){
        std::cout << "conected" << std::endl;

    int inputHeight;
    char inputLengthChar;
    bool noWinnerFound=true;
    FD_SET(sock, &rfds);

    while(noWinnerFound){

        read(sock, gameInfo, 1);
        if(gameInfo[0]<=0){
            std::cout << std::endl;
            std::cout << "you have won. congratulation!" << std::endl;
            noWinnerFound=false;
            break;
        }
        gameInfo[0] = field.getRemainingShips();
        write(sock, gameInfo, 1);

        if(field.getRemainingShips()<=0){
            std::cout << std::endl;
            std::cout << "you have lost." << std::endl;
            noWinnerFound=false;
            break;
        }

        if(turn%2!=0){
            std::cout << "your turn: " << std::endl;
            std::cout << std::endl;
            field.print();
            std::cout << "enter which field you want to attack" << std::endl;
            std::cin >> inputHeight >> inputLengthChar;
            if(inputHeight>field.getHeight() || inputHeight<0 ||((int)(inputLengthChar-65))>=field.getLength() || (int)inputLengthChar-65<0){
                std::cin.setstate(std::cin.failbit);
            }
            else if(field.getEnemyField(inputHeight-1,((int)(inputLengthChar-65))!=field.EMPTY_FIELD)){
                std::cout << "you already attacked that field" << std::endl;
                std::cin.setstate(std::cin.failbit);
            }
            while(std::cin.fail()){
                std::cout << "Please enter: number character" << std::endl;
                std::cin.clear();
                std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
                std::cin >> inputHeight >> inputLengthChar;
                if(inputHeight>field.getHeight() || ((int)inputLengthChar)-65>=field.getLength()){
                    std::cin.setstate(std::cin.failbit);
                }
                else if(field.getEnemyField(inputHeight-1,((int)(inputLengthChar-65))!=field.EMPTY_FIELD)){
                    std::cout << "you already attacked that field" << std::endl;
                    std::cin.setstate(std::cin.failbit);
                }
            }

            cordinate[0] = inputHeight-1;
            cordinate[1] = ((int)(inputLengthChar-65));
            if(write(sock, cordinate, 2) == -1)
                std::cerr << "error writing" << std::endl;
            //select(sock2, &rfds, NULL, NULL, &stop);
            int n = select(sock+1, &rfds, NULL, NULL, NULL);
            if(n == -1){
                std::cerr << "error select" << std::endl;
            } else{
                if(FD_ISSET(sock, &rfds)){
                    if(read(sock, status, 1) == -1)
                        std::cerr << "error reading" << std::endl;
                    if(status[0]==field.SHIP){
                        field.setEnemyField(cordinate[0], cordinate[1], field.SHIP);
                        std::cout << std::endl;
                        std::cout << "you hit a ship" << std::endl;
                    }

                    else{
                        field.setEnemyField(cordinate[0], cordinate[1], field.ATTACKED);
                    }
                }
            }

            field.print();
            ++turn;

        }

        else{
            std::cout << std::endl;
            std::cout << "enemy turn please wait" << std::endl;
            int n = select(sock+1, &rfds, NULL, NULL, NULL);
            if(n == -1){
                std::cerr << "error select" << std::endl;
            } else{
                if(FD_ISSET(sock, &rfds)){
                    if(read(sock, cordinate, 2) <= 0)
                        std::cerr << "error reading" << std::endl;
                    status[0] = field.getField(cordinate[0], cordinate[1]);
                    write(sock, status, 1);
                    if(status[0]==field.SHIP){
                        field.decreaseRemainingShips();
                    }
                    else if(status[0]==field.EMPTY_FIELD){
                        field.setField(cordinate[0], cordinate[1], field.ATTACKED);
                    }
                }
            }
            ++turn;
        }
    }
}
else{
    std::cout << "no server found" << std::endl;
    exit(1);
}

close(sock);

return 0;

}

这是一个例子:

让我们假设船只设置如下:( X代表船舶,O代表不是船舶的攻击)

   server:         client:
  A B C D E       A B C D E
1 X             1 X
2 X             2   X
3 X             3     X
4 X             4       X
5 X             5         X

游戏开始你看到一块白板代表敌人的板子和服务器攻击场(1,B)输出是:

   server:         client:
  A B C D E       A B C D E
1   X           1 
2               2   
3               3     
4               4       
5               5         

这显然是错误的但它在敌人转弯中使用了字段(1,A)的值它使用来自服务器输入字段的值。让我们说客户攻击(1,A)会撞到一艘船,但是自上次攻击没有撞到一艘船后,输出就是:

   server:         client:
  A B C D E       A B C D E
1   X           1 O
2               2   
3               3     
4               4       
5               5    

我希望我能说清楚。如果有人可以提供帮助,我会很感激。

1 个答案:

答案 0 :(得分:0)

您正在使用部分未初始化的变量。 gameInfo是一个int(通常是四个字节),但是只能读取1个字节。此外,由于您实际上将其用作其他地方的int,因此会出现字节序问题。

你很幸运,因为你很可能在一个小端平台上,但是你仍然不能通过读取一个值而使gameInfo为负,如果之前没有,反过来,可以'如果它之前是负面的话,那就是积极的。