Can'；t使用c语言处理多组客户机的服务器在客户机之间平均分配工作

我已经编写了一个服务器来处理多线程客户端。唯一的问题是，即使我锁定了关键部分，它也无法在客户端之间平均分配工作

这是服务器

#include <stdio.h>
#include <sys/socket.h>
#include <string.h>
#include <stdlib.h>
#include <arpa/inet.h>
#include <unistd.h>
#include <pthread.h>
#include <time.h>
#include <math.h>

void *con(void *);
int r, x, convr, convx, flag;
double sqrtr,doubler;
pthread_mutex_t var;

int main(int argc,char *argv[]){
    srand(time(NULL));
    r=rand();

    r=(r%(21474836146-2000000000))+2000000000;
    r=340;
    doubler=(double)r;
    x=2;
    convr=htonl(r);
    convx=htonl(x);

    sqrtr=sqrt(doubler);
    flag='p';

    int soc, clientsoc, *new, asize;
    struct sockaddr_in server, client;

    soc=socket(AF_INET,SOCK_STREAM,0);
    if(soc==-1){
        perror("Error creating socket");
    }

    server.sin_family=AF_INET;
    server.sin_addr.s_addr=INADDR_ANY;
    server.sin_port=htons(7897);

    if(bind(soc,(struct sockaddr *)&server,sizeof(server))<0){
        perror("Error binding socket");
        return 1;
    }
    if(listen(soc,5)!=0){
        perror("Error listening");  
    }
    int start='n';
    printf("Are you ready to start?  Don't continue until you have connected all the"
             " clients you with to use to the server.  (press y for yes)\n");
    start=getchar();
    while(start!='y'){
        printf("Are you ready to start?  Don't continue until you have connected all the"
             " clients you with to use to the server.  (press y for yes)\n");
        start=getchar();
    }
    asize=sizeof(struct sockaddr_in);
    while((clientsoc=accept(soc,(struct sockaddr *)&client,(socklen_t*)&asize))){
        pthread_t client_thread;

        new=malloc(1);
        *new=clientsoc;
        if(pthread_create(&client_thread,NULL,con,(void*) new)<0){
            perror("Error creating thread");
            return 1;
        }
        pthread_join(client_thread,NULL);
    }
    if(clientsoc<0){
        perror("Error accepting");
        return 1;
    }
    return 0;
}

//the handeler for a connection
void *con(void *socket){
    int soc=*(int*)socket;
    int flagtemp, flagtempconv;
    //what it will do here
    int rs;
    while(1){
        pthread_mutex_lock(&var);
        printf("Sending %d\n",r);
        if(write(soc,&convr,sizeof(convr))<0){
            printf("Failed to send");
            exit(1);
        }
        printf("Sending %d\n\n",x);
        if(write(soc,&convx,sizeof(convx))<0){
            printf("Failed to send");
            exit(1);
        }
        if(rs=read(soc,&flagtemp,sizeof(flagtemp))>0){
            flagtempconv=ntohl(flagtemp);
            printf("Received %d\n",flagtempconv);
            if(flagtempconv=='n')
                flag=flagtempconv;
            if(flag=='n'){
                printf("The numer %d is divisible by %d and is not prime\n",r,x);
                break;
            }else if(x>=sqrtr){
                printf("The number %d is prime\n",r);
                break;
            }
            x++;
            convx=htonl(x);
        } else if(rs==0){
            printf("Disconnected from client");
        } else if(rs<0){
            printf("Read error");   
        }
        pthread_mutex_unlock(&var);
    }
    close(soc);
    exit(0);
}

#包括
#包括
#包括
#包括
#包括
#包括
#包括
#包括
#包括
无效*（无效*）；
int r，x，convr，convx，flag；
双sqrtr，倍频器；
pthread_mutex_t var；
int main（int argc，char*argv[]）{
srand（时间（空））；
r=rand（）；
r=（r%（21474836146-2000000000））+2000000000；
r=340；
倍频器=（双）倍频器；
x=2；
convr=htonl（r）；
convx=htonl（x）；
sqrtr=sqrt（倍频器）；
flag='p'；
int soc，clientsoc，*新，亚洲；
服务器、客户端中的结构sockaddr_；
soc=套接字（AF_INET，SOCK_STREAM，0）；
如果（soc==-1）{
perror（“创建套接字时出错”）；
}
server.sinu family=AF\u INET；
server.sin\u addr.s\u addr=INADDR\u ANY；
server.sinu port=htons（7897）；
if（bind（soc，（struct sockaddr*）&server，sizeof（server））首先：这不是一个关于C的问题。这是一个关于POSIX线程和BSD套接字等的问题；但这些都不是C语言的一部分。如果您要使用C，那么熟悉ISO 9899及其功能和不功能是一个非常好的主意

第二：请注意，即使使用这种类型的正确编写的程序，在分布式系统中调度工作也不是一个简单的问题。您可以强制执行各种简单的排序，例如循环，但多线程服务器（尤其是使用每会话线程模型）不太可能具有可预测的行为

关于代码

    if(pthread_create(&client_thread,NULL,con,(void*) new)<0){
        perror("Error creating thread");
        return 1;
    }
    pthread_join(client_thread,NULL);

clientsock
被定义为
int
。你真的运行在
sizeof（int）==1
的平台上吗？只有当
CHAR\u BIT
至少为16时，这才可能是真的，这是…不寻常的，我们应该说。很可能是你造成了未定义的行为，现在任何事情都可能发生

将硬编码值传递给
malloc
是件坏事。不要这样做。尝试：

new=malloc(sizeof *new);

（也可能是，你想把名字“New”看作一个标识符，以防你想把这个代码移到C++。虽然坦白地说，我的偏好是永远不要用C++实现C源；如果你想要C++，写C++）< <
我没有寻找其他问题。
永远不要访问
argv[0]之外的内容
不必首先检查
argc
以确保命令行参数实际存在。当命令行参数不存在时，向
stderr
输出一条显示调用程序正确方式的用法消息，然后立即退出程序。即
fprintf（stderr，“用法：%s\n”，argv[0]）；退出（EXIT_FAILURE）；
为了便于阅读和理解：1）通过一个空行分离代码块（for、if、else、while、do…while、switch、case、default）。2）通过2或3个空行分离函数（保持一致）3）遵循公理：每行只有一条语句，最多每个语句一个变量声明。4）变量名称应表示
用法
或
内容
（或两者都更好）。像
x
和
r
这样的名称，即使在当前上下文中关于这两行：
r=（r%（21474836146-2000000））+20000000000；r=340；
。代码试图做什么？代码正在计算
r
，该计算与
340
重叠。它不能同时是两个值这行：
while（（clientsoc=accept（soc，（struct sockaddr*）和client，（socklen_t*）&asize））{不正确，因为
accept（）`函数在出错时将返回一个
-1
，需要特别检查该错误条件，否则
-1
将满足
while（）
，并在成功调用
malloc（）后进入
while（）
循环体
，任何以后的退出/返回都应该在返回/退出之前将从
malloc（）
返回的指针传递到
free（）
    new=malloc(1);
    *new=clientsoc;

new=malloc(sizeof *new);