我声明了以下变量
char min[6] = "";
char max[6] = "";
char hr[6] = "";
char div[6] = "";
char vol[6] = "";通过下面的代码,我得到了一些带有strtok的标记
if (pos)
{
ptr = strtok(buff, "Nodo_,=:V()");
i = 0;
while (ptr != NULL)
{
if (i == 0)
strcat(min, ptr);
if (i == 2)
strcat(tmax, ptr);
if (i == 4)
strcat(hr, ptr);
if (i == 6)
strcat(div, ptr);
if (i == 8)
strcat(vol, ptr);
ptr = strtok(NULL, "Nodo_,=:V()");
i++;
}
printf("Results: %s, %s, %s, %s, %s\n", min, max, hr, div, vol);结果是这样的
Results: 4, 23, 45, 67, 89我想创建一个if语句,它将执行以下操作(其中s是本地时间)
char *table1[10] = {0};
if(*min == "4")
{
table1[5]=min;
table1[6]=max;
table1[7]=hr;
table1[8]=div;
table1[9]=vol;
fprintf(ft,"%s %d %d %d %d %s %s %s %s %s",s,table1[0],table1[1],table1[2],table1[3],table1[5],table1[6],table1[7],table1[8],table1[9]);
}
else
{
table1[0]=min;
table1[1]=max;
table1[2]=hr;
table1[3]=div;
table1[4]=vol;
fprintf(ft,"%s %s %s %s %s %d %d %d %d %d",s,table1[0],table1[1],table1[2],table1[3],table1[5],table1[6],table1[7],table1[8],table1[9]);
}问题是我不知道如何编写if语句来与值4进行比较。
if(*min == "4")
if(*min == '4')
if(&min == "4")
etc.其次,你能给我一个更好的方法来写这个糟糕的fprintf吗?因为我想要这样的结果:
0 0 0 0 4 23 45 67 89或
5 23 45 67 89 0 0 0 0 发布于 2013-06-21 15:58:28
您的代码中几乎没有什么问题。
第一个。至少可以说,您正在以一种非常危险的方式比较字符串。它可以在一些奇怪的情况下工作。一般来说,这应该被认为是错误的。在c中,您应该使用适当的函数,如strcmp或类似函数。
第二名。除了格式字符串(%s或%d)中的预期类型之外,您的fprintf(...)看起来完全相同。但是,table1的类型不会改变!使用错误的说明符可能会导致奇怪的输出观察,导致很难找到内存错误。在我看来,你应该一直使用%s。因为它的类型都是char*。另外,为了简化打印,我猜您可以使用一个简单的for循环。
3.你的if可以切换到更合适的case,或者至少是if else,因为你永远不会在一次遍历中执行两个分支。但这并不重要,因为它不会改变整体逻辑。在某些情况下,这可能是性能关键。
编辑:
好的,您的if可以用两种方式编写。
char *table[10] = {0};这将创建一个指向10个字符串的指针表。您仍然需要进一步预留空间才能实际保存字符。此外,table1[5]=min;不会将字符串min放入/复制到table1的插槽5中。这与==和strcmp的错误是一样的。您可能应该阅读一篇关于在c中使用字符串的简短介绍/教程。
你需要更多类似的东西:
char *table[10] = {0}; //create a table with pointers to 10 "strings"
size_t MAX_STR_SIZE = 6; // I assumed this from your example code.
for(int i = 0; i < 10; ++i){
table[i] = malloc(MAX_STR_SIZE); //allocate space for string
//initialize
table[i][0] = '0'; //put '0' into as 1st char
table[i][1] = '\0'; //put null character at 2nd space
//table[i][0] = '\0'; // You can use this line or the one below this to initialize it
//table[i][0] = 0; // to a nullstring like you did in your 1st code snippet
}然后你可以去你的if
if(strcmp(min,"4") == 0)
{
strcpy(table[5], min); //This is how you copy null terminated strings in `c`.
strcpy(table[6], max);
strcpy(table[7], hr);
//...
} else {...}
fprintf(ft,"%s ", s);
for(int i = 0; i < 10; ++ 1) fprintf(ft,"%s ", table[i]);这不会重现你所有的代码,但应该会让你达到一个只是一个简单的下坡路的地步。
EDIT2:
我以为你想避免使用min,max等,就把它换成了tables。是我的错。当然,您可以使用table并重用这些指针,以避免像我之前建议的那样不必要地复制字符串。但是,table[i]必须始终包含指向正确字符串的正确指针。即不是空指针,也不是指向一些随机存储器垃圾的指针。我建议这样做:
char default_str[1] = ""; // or default_str[2] = "0" depending on your needs.
char *table[10] = {0}; //create a table with pointers to 10 "strings"
//...
if(*min == "4")
{
table[0]=table[1]=table[2]=table[3]=table[4]=default_str;
table[5]=min;
table[6]=max;
table[7]=hr;
table[8]=div;
table[9]=vol;
}
else
{
table[5]=table[6]=table[7]=table[8]=table[9]=default_str;
table[0]=min;
table[1]=max;
table[2]=hr;
table[3]=div;
table[4]=vol;
}
fprintf(ft,"%s ", s);
for(int i = 0; i < 10; ++ 1) fprintf(ft,"%s ", table[i]);这样,您就可以设置未使用的指针来引用default_str。它现在更接近于你的原始代码,并且使用更少的内存,因为table条目与原始变量“共享”内存。
您也可以使用char *table[10] = {default_str},然后跳过在if分支中重新初始化它,但我不确定c编译器是否允许这样做。
发布于 2013-06-21 15:56:59
因为C中的字符串实际上是一个字符数组,而且C中的所有数组变量都是指针,所以当您比较样式*min == "4"时,您实际上是在比较min的第一个位置中包含的任何内容与给定常量字符串"4"的任何位置。
为了执行字符串比较,您必须使用函数strcmp,该函数就是为此而设计的。
如果您希望以一种更干净的方式打印输出,那么可以将结果字符串保存在char **数组中,然后使用for循环打印它们。
发布于 2013-06-21 17:53:08
使用strcmp或strncmp,参数是char*,所以你必须设置if (strcmp(min,"4") ==0)
请注意,您必须包含string.h .h(我认为您这样做是为了使用strcat )。
PS:您的代码行,如table15=min;将不起作用,因为Table15是字符,而min是字符*
https://stackoverflow.com/questions/17230164
复制相似问题