[英]String concatenation in C
所以我得到了这个函数,如果没有空间,该函数可以在“ /” C前面放置一个空格。 并且它可以减少字符串,但是当我尝试将字符串连接在一起时,我收到错误,可能是内存违规。 请帮帮忙。
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
char* substr(const char *pstr, int start, int numchars) {
char* pnew = malloc(numchars + 1);
strncpy(pnew, pstr + start, numchars);
pnew[numchars] = '\0';
return pnew;
}
char* fixString(char str[]) {
char* position;
char* newString = "";
char* finalString;
int oldPosition = 0;
printf("Original str: %s\n", str);
printf("Original length: %d\n\n", strlen(str));
position = strchr(str, '/');
while (position != NULL) {
int charPosition = position - str;
printf("String position: %d->%d\n", oldPosition, charPosition);
newString = substr(str, oldPosition, charPosition - oldPosition);
oldPosition = charPosition;
if (charPosition > 0 && str[charPosition - 1] != ' ') {
printf("Previous char: %c\n", str[charPosition - 1]);
newString = strcat(newString, " ");
}
printf("String: |%s|\n", newString);
if (strlen(newString) > 0) {
finalString[0] = strcat(finalString, newString);
}
printf("------------\n");
position = strchr(position + 1, '/');
}
char* lastString = substr(str, oldPosition, strlen(str));
finalString = strcat(finalString, lastString);
printf("lastString: %s\n\n", lastString);
return finalString;
}
int main() {
char* testString = "/Filter /FlateDecode/Length 7108/Subtype /Type1C";
printf("%s", fixString(testString));
return 0;
}
您永远不会分配目标缓冲区。 finalString
变量未初始化为任何东西。
这也不是代码的唯一问题。 您似乎将char *
视为某种智能字符串类型,但它不过是指向内存位置的指针。 例如,这个:
newString = strcat(newString, " ");
不连接两个字符串并返回连接结果。 它在char缓冲区中追加一个空格,newString指向并返回相同的缓冲区。 分配给newString是无害的,但具有误导性。
// It is the callers responsibility to free the returned string.
char *fixString(char *str) {
int len;
char *s;
char *dest;
int after_space;
// First pass, figure out the size of the output.
len = 0;
after_space = 0;
for (s = str; *s; s++) {
len += 1 + (!after_space && *s == '/');
after_space = *s == ' ';
}
dest = malloc(len + 1);
s = dest;
after_space = 0;
while(*str) {
if (!after_space && *str == '/') *s++ = ' ';
after_space = (*s++ = *str++) == ' ';
}
return dest;
}
你需要为finalString
分配一些内存。
尝试这样做:
#define BUFFER_SIZE 1024
char* finalString = malloc(BUFFER_SIZE);
// ...
strncat(finalString, "something", 1024);
记得在你不再需要它时调用指针上的free()
(不是在你的函数中 - 在函数返回之后,在客户端代码中的某个地方,只要不需要结果)。
你永远不malloc
内存为您newString
和finalString
指针。 strcat
期望你给它的目标指针有足够的空间来存储字符串。
必要时,您可能还必须检查malloc()和realloc()之后的大小。 欢迎来到美丽的C语言内存管理世界:)
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.