1

我在这个程序上花了几个小时，并在网上花了几个小时搜索我的方法的替代方案，整个晚上都被崩溃和错误所困扰……

我有几件事想用这段代码实现。首先我会解释我的问题，然后我会发布代码，最后我会解释我对程序的需求。

程序只输出单个单词，而 concatenate 函数什么也不做。这似乎应该足够简单来修复......

我的第一个问题是我似乎无法让 concatenate 函数工作，我使用了不起作用的通用 strcat 函数，我在互联网上找到的另一个版本也没有（该函数在这里使用，它被称为“mystrcat” ）。我想让程序读取一个字符串并删除“分隔符”以创建一个由原始字符串中的每个单词组成的单个字符串。我正在尝试使用 strtok 和 strcat 函数。如果有更简单或更简单的方法，请我全神贯注。

另一个问题，不一定是问题，而是一个丑陋的烂摊子：main 后面的七行。我更喜欢按如下方式初始化我的变量： char variable[amt]; 但是我为 strtok 找到的代码使用了指针，而 strcat 函数的代码使用了指针。从长远来看，更好地理解字符串的指针和地址可能会帮助我。但是，我想以任何必要的方式摆脱其中的一些行。我不能有 6 行仅用于 2 个变量。当我有 10 个变量时，我不希望顶部有 30 行......</p> 

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

char *mystrcat(char *output, char *firstptr);

int main() {

char str[] = "now # is the time for all # good men to come to the # aid of their     country";
char delims[] = "# ";
char resultOrig[70];    //was [20];
char firstOrig[20];
//char *result = NULL, *first = NULL;
char result = resultOrig;    //was result = &resultOrig;
char first = firstOrig;    //was   first = &firstOrig;

first = strtok( str, delims );

while( first != NULL ) {
    mystrcat(resultOrig, firstOrig);
    printf( "%s ", first );
    printf("\n %s  this should be the concat\'d string so far\n", resultOrig);
    first = strtok( NULL, delims );

}
system("pause");
return 0;
}

char *mystrcat(char *resultptr, char *firstptr)
{
char *output = resultptr;

while (*output != '\0')
    output++;
while(*firstptr != '\0')
{
    *output = *firstptr;
    output++;
    firstptr++;
}
*output = '\0';

return output;
}

这现在只是一个测试程序，但我打算将它用于文件列表/数据库。我的文件有下划线、连字符、句号、括号和数字；我想将所有这些都设置为“分隔符”。我正计划通过一个循环，在那里我将删除一个分隔符（每个循环从 _ 更改为 - 到 . 等等）并创建一个字符串，我可能想用空格或句点替换分隔符。有些文件中已经有空格以及我想“分隔”的特殊字符。

我打算通过扫描文本文件来完成所有这些工作。在文件中，我也有一个这种格式的大小：“2,518,6452”。我希望我可以按字母顺序或大小、升序或降序对我的数据库进行排序。这只是一些附加信息，可能有助于了解我上面的具体问题。

下面我包含了一些关于这些名称如何出现的虚构示例。my_file(2009).ext second.File-group1.extls the.third.file-vol30.lmth

我将这篇文章的重点放在：关于如何让连接函数工作或替代 strcat 和/或 strtok 的问题。以及寻求帮助以整理不必要或冗余的代码。

我感谢所有的帮助，甚至感谢所有阅读我帖子的人。

太感谢了！

4

3 回答 3

3

strcat如果你使用first而不是firstOrig在你的循环中会工作。不需要mystrcat。可以简化为：

#include <stdio.h>
#include <string.h>

int main() {
    char str[] = "now # is the time for all # good men to come to the # aid of their     country";
    char delims[] = "# ";
    char result[100] = "";  /* Original size was too small */
    char* token;
    token = strtok(str, delims);
    while(token != NULL) {
        printf("token = '%s'\n", token);
        strcat(result, token);
        token = strtok(NULL, delims);
    }
    printf("%s\n", result);
    return 0;
}

输出：

token = 'now'
token = 'is'
token = 'the'
token = 'time'
token = 'for'
token = 'all'
token = 'good'
token = 'men'
token = 'to'
token = 'come'
token = 'to'
token = 'the'
token = 'aid'
token = 'of'
token = 'their'
token = 'country'
nowisthetimeforallgoodmentocometotheaidoftheircountry
于 2012-04-19T12:33:09.150 回答
0

您尚未初始化以下两个字符串：

char resultOrig[20];
char firstOrig[20];

并且您正在向它们附加字符。将它们更改为：

char resultOrig[20] = "";
char firstOrig[20] = "";

字符数组的名称也给出了它的起始地址。所以

result = &resultOrig;
first = &firstOrig;

应该：

result = resultOrig;
first = firstOrig;

改变

mystrcat(resultOrig, firstOrig);

至

mystrcat(resultOrig, first);

也使resultOrig足够大以容纳串联，例如：

char resultOrig[100] = "";
于 2012-04-19T11:54:52.747 回答
0

这里有几个问题：

  1. 来回缺少初始化（正如 codaddict 指出的那样）resultOrig。firstOrig

  2. first = &firstOrig不会做你想做的事。你稍后再做first = strtok(str, delims)，它first指向str. 它不会将数据读入firstOrig.

  3. 您分配小缓冲区（仅 20 个字节）并尝试用比这更多的内容填充它们。它会溢出堆栈。导致讨厌的错误。

于 2012-04-19T12:35:32.533 回答