2015-04-05 219 views
1

我有一些代码从一个从套接字接收的字符串中获取所有字符(连接没问题,不包括连接到它的变量)。正如你所看到的,在预处理字符串log_res_str我分配给我的字符串数组logins,一切都很好。但是,当我打印出各自的登录名时,它只显示第二个登录名。有什么问题?只显示最后一个字符串的字符串数组

int logincount = 2; 
char login_buff[CHUNK_SIZE] = {0}; 
char *logins[100]; 
char log_res_str[CHUNK_SIZE] = {0}; 
for(int i = 0; i < logincount; i++) 
{ 
    int lenf = 0; 
    int received = 0; 
    memset(login_buff, 0, sizeof(login_buff)); 

    // the length of the UID stored in the first byte 
    if((received = recv(client_sock, login_buff, CHUNK_SIZE, 0)) < 0) 
     DIE("Receive Error"); 

    // getting the lenght of the login 
    lenf = login_buff[0] - '0'; 
    for(int count = 1; count < lenf; count++) 
     log_res_str[count-1] = login_buff[count]; 

    printf("log_res_str: %s\n", log_res_str); 
    logins[i] = log_res_str; 

    printf("login: %s\n", logins[i]); 

    // acknowledgment 
    if(send(client_sock, login_buff, sizeof(login_buff), 0) < 0) 
     DIE("Acknowledge Error"); 
} 

这就是我得到:

log_res_str: root 
log_res_str: _warmd 
login: _warmd 
login: _warmd 

这就是我想要的:

og_res_str: root 
log_res_str: _warmd 
login: root 
login: _warmd 
+0

'登录[I] = 1 og_res_str'存储一个指向'log_res_str'缓冲区的指针,而不是字符串本身。如果你想要一个字符串副本,你必须为拷贝分配内存,然后复制(使用'malloc' +'strcpy'或'strdup')。 – SleuthEye 2015-04-05 21:05:50

+0

@SleuthEye你可以把它给一个答案?工作,我想接受你的解决方案。 – jvitasek 2015-04-05 21:08:49

回答

0

的问题是,logins[i] = log_res_str存储指针log_res_str缓冲区,而不是字符串本身。

如果您想要一个字符串副本,您必须为副本分配内存,然后复制该字符串。如果字符串为空值终止的,你可以用任何malloc其次是strcpy做到这一点:

logins[i] = malloc(lenf); 
strcpy(logins, log_res_str); 

或者干脆strdup

logins[i] = strdup(log_res_str); 

如果另一方面在log_res_str输入字符串不为空封端的(或至少不能保证是),那么就应该使用memcpy用于复制:

logins[i] = malloc(lenf+1); // +1 if you want a null-terminated copy 
logins[i][lenf] = '\0';  // this line only if you allocated +1 above 
memcpy(logins, log_res_str, lenf); 
相关问题