我有一個函式 parse()，它基本上采用“X:0:Y;A:0:B;”形式的字串引數 并將其拆分為三個 GLOBAL 陣列 known_routers、Known_distances 和 known_next_hop。
當傳入的引數是不同的變數時，該函式可以正常作業，但如果相同的引數變數以不同的值傳遞，它只會替換全域陣列中的舊值。（我相信創建了一個參考，因此每次原始變數更改時，全域陣列中的值也會發生變化，因為它們只是指向原始引數）
例如，以下代碼按預期作業：
int main(int argc , char *argv[])
{
char receive_buf[10000]= "K:0:K;";
parse(receive_buf);
char receive_buf1[10000]= "C:0:C;";
parse(receive_buf1);
char receive_buf2[10000]= "D:0:D;";
parse(receive_buf2);
for(int i = 0; i < routers_added; i)
{
printf("-------------------------\n");
printf("%s\n",known_routers[i]);
printf("%d\n",known_distances[i]);
printf("%s\n",known_next_hop[i]);
printf("-------------------------\n");
}
return 0;
}
上面的代碼給出了預期值：
0 千
0℃
0 天
但下面的代碼將不起作用
int main(int argc , char *argv[])
{
char receive_buf[10000]= "K:0:K;";
char secondary_buf [10000];
memcpy(secondary_buf, receive_buf, 6);
parse(secondary_buf);
strcpy(receive_buf, "C:0:C;");
memcpy(secondary_buf, receive_buf, 6);
parse(secondary_buf);
strcpy(receive_buf, "K:0:K;");
memcpy(secondary_buf, receive_buf, 6);
parse(secondary_buf);
for(int i = 0; i < routers_added; i)
{
printf("-------------------------\n");
printf("%s\n",known_routers[i]);
printf("%d\n",known_distances[i]);
printf("%s\n",known_next_hop[i]);
printf("-------------------------\n");
}
return 0;
}
上面的代碼給出了一個輸出：
0 天
（所以基本上是給變數“receive_buf”的任何最后一個值）
在我的原始代碼中，我使用套接字 recv 函式不斷獲取新值以在此函式中傳遞引數。（但它基本上只是像上面的例子一樣改變了舊值）我試圖用于傳遞引數的代碼如下所示：
char receive_buf[10000];
while(1){
int read_size ;
if ((read_size=recv(sock, receive_buf, 10000-1, 0)) == -1) {
perror("recv");
exit(1);
}
else{
parse(receive_buf);
}
}
如果需要，函式決議如下所示。請幫忙
char* known_routers[20];
int known_distances[20];
char* known_next_hop[20];
int check_if_a_router_exists_in_the_system(char * router);
int routers_added=0;
void parse(char* packet){
printf("\nstarting to parse the following pocket: %s\n",packet);
int i = 0;
int l = 0;
char *p = strtok (packet, ";");
char *array[20] ;
char *array_2[20] ;
while (p != NULL)
{
array[i ] = p;
p = strtok (NULL, ";");
}
for (int j = 0; j < i; j){
char *po = strtok (array[j], ":");
while (po != NULL)
{
array_2[l ] = po;
po = strtok (NULL, ":");
}
}
int is_it_next_hop=0;
for(int k = 0; k < l; k ){
if(array_2[k][0]=='0'||array_2[k][0]=='1'||array_2[k][0]=='2'||array_2[k][0]=='3'||array_2[k][0]=='5'||array_2[k][0]=='6'||array_2[k][0]=='7'||array_2[k][0]=='8'||array_2[k][0]=='9'){
}
else{
if (is_it_next_hop){
is_it_next_hop=0;
}
else{
is_it_next_hop=1;
int is_it_in_the_system = check_if_a_router_exists_in_the_system(array_2[k]);
printf("checking is : %s is in the system: %d",array_2[k],is_it_in_the_system);
if(is_it_in_the_system==-1){ //if it isnt in the system
known_routers[routers_added]=array_2[k];
known_distances[routers_added]=atoi(array_2[k 1]) 1;
known_next_hop[routers_added]=array_2[0];
routers_added ;
printf("\n\n---------routers increased-------\n\n");
}
else{
printf("\n\n---------routers remain same-------\n\n");
if(known_distances[is_it_in_the_system]>atoi(array_2[k 1]) 1){
known_distances[is_it_in_the_system]=atoi(array_2[k 1]) 1;
known_next_hop[is_it_in_the_system]=array_2[0];
}
}
}
}
}
}
uj5u.com熱心網友回復：
首先，有點迂腐：C 中沒有參考之類的東西，參考是 C 的特性。C 有指標，可用于類似目的。
就您的程式而言，我認為問題在于：您的parse()函式使用指向您作為引數傳入的記憶體緩沖區的指標填充了幾個全域陣列。這一切都很好，但是在您的第二個示例中，您隨后用其他一些資料覆寫了該記憶體緩沖區：
parse(secondary_buf);
// at this point, your known_blah[] arrays contain pointers into (secondary_buf)
[...]
memcpy(secondary_buf, receive_buf, 6); // here you overwrite secondary_buf
...因此，您的指標現在指向與以前不同的資料也就不足為奇了——您覆寫了它們指向的資料。
至于如何避免該問題，您可以將不同的記憶體緩沖區傳遞給每個呼叫（就像您在第一個示例中所做的那樣），或者將 known_* 資料復制到單獨的位置（不會被覆寫）之前后續呼叫parse()，或通過重構程式來避免該問題，以便它在第二次parse()呼叫之前完全處理第一次呼叫的資料parse()（因此在下一次呼叫之后不需要訪問舊/覆寫的資料parse()）
uj5u.com熱心網友回復：
其中一條評論回答了我的問題。“strdup”是我需要的功能。它每次都創建一個全新的引數副本。
太感謝了。
uj5u.com熱心網友回復：
我真的不明白你想要實作什么，所以我只是正確地決議了輸入。我不會打擾第二個strtok()，因為每個塊中應該總是（？）三個欄位。所以我只是將它們分開。
確保使用錯誤的輸入來測驗代碼，例如“”、“;;”、“A:0”、“:::”等。
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <ctype.h>
void parse(char *packet)
{
printf("\nstarting to parse the following pocket: %s\n",packet);
// Parse a bunch of semicolon delimited fields, each of which is then
// delimited by a colon into 3 parts, e.g.:
// K:0:K;C:0:C;
// D:0:D;
char *triple = strtok( packet, ";" );
while ( triple != NULL )
{
// there is something to parse
char *field1 = triple;
char *field2 = strchr( triple, ':' );
char *field3 = strrchr( triple, ':' );
if ( field2 != NULL && field3 != NULL && field2 != field3 )
{
// we have all 3 parts
*field2 = '\0'; // trim the first field
field2 = 1;
*field3 = '\0'; // trim the second field
field3 = 1;
// Quickly validate the content of the fields by considering the first letter
if ( isalpha( field1[0] ) && isdigit( field2[0] ) && isalpha( field3[0] ) )
{
// TODO: DO SOMETHING USEFUL WITH field1, field2, field3
printf( "PARSED [%s], [%s], [%s]\n", field1, field2, field3 );
}
else
{
// Ignore, but nag
printf( "Incorrect field(s) [%s], [%s], [%s]\n", field1, field2, field3 );
}
}
else
{
// Ignore, but nag
printf( "Malformed triple [%s]\n", triple );
}
triple = strtok( NULL, ";" ); // find the next field
}
}
int main( int argc, const char **argv )
{
int i;
char *arg_copy;
for ( i=1; i<argc; i )
{
arg_copy = strdup( argv[i] );
if ( arg_copy != NULL )
{
parse( arg_copy );
free( arg_copy );
}
}
return 0;
}
轉載請註明出處，本文鏈接：https://www.uj5u.com/houduan/447339.html
上一篇：如何使用plutil獲取值？