避免在C代码中使用strcat。最干净，也是最重要的，最安全的方法是使用snprintf:

char buf[256];
snprintf(buf, sizeof(buf), "%s%s%s%s", str1, str2, str3, str4);

一些评论者提出了一个问题，即参数的数量可能与格式字符串不匹配，代码仍然会被编译，但如果出现这种情况，大多数编译器已经发出警告。

int main()
{
    char input[100];
    gets(input);

    char str[101];
    strcpy(str, " ");
    strcat(str, input);

    char *p = str;

    while(*p) {
       if(*p == ' ' && isalpha(*(p+1)) != 0)
           printf("%c",*(p+1));
       p++;
    }

    return 0;
}

strcat()的第一个参数需要能够为连接的字符串保留足够的空间。因此，分配一个缓冲区，使其具有足够的空间来接收结果。

char bigEnough[64] = "";

strcat(bigEnough, "TEXT");
strcat(bigEnough, foo);

/* and so on */

Strcat()将第二个参数与第一个参数连接起来，并将结果存储在第一个参数中，返回的char*就是第一个参数，只是为了方便。

您不会得到一个连接第一个和第二个参数的新分配字符串，我猜这是您根据代码所期望的。

试试类似的方法:

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

int main(int argc, const char * argv[])
{
  // Insert code here...
  char firstname[100], secondname[100];
  printf("Enter First Name: ");
  fgets(firstname, 100, stdin);
  printf("Enter Second Name: ");
  fgets(secondname,100,stdin);
  firstname[strlen(firstname)-1]= '\0';
  printf("fullname is %s %s", firstname, secondname);

  return 0;
}

不要忘记初始化输出缓冲区。strcat的第一个参数必须是一个空结束的字符串，并为结果字符串分配足够的额外空间:

char out[1024] = ""; // must be initialized
strcat( out, null_terminated_string ); 
// null_terminated_string has less than 1023 chars

尝试修改字符串字面量是未定义的行为，这是类似于:

strcat ("Hello, ", name);

会尝试去做。它将尝试将名称字符串附加到字符串字面量“Hello，”的末尾，这没有很好地定义。

试试这个。它达到了你想要达到的效果:

char message[1000];
strcpy (message, "TEXT ");
strcat (message, var);

这将创建一个允许修改的缓冲区，然后将字符串字面量和其他文本复制到该缓冲区。只是要小心缓冲区溢出。如果您控制输入数据(或事先检查)，可以使用像我这样的固定长度的缓冲区。

否则，您应该使用缓解策略，例如从堆中分配足够的内存以确保可以处理它。换句话说，就像:

const static char TEXT[] = "TEXT ";

// Make *sure* you have enough space.

char *message = malloc (sizeof(TEXT) + strlen(var) + 1);
if (message == NULL)
     handleOutOfMemoryIntelligently();
strcpy (message, TEXT);
strcat (message, var);

// Need to free message at some point after you're done with it.