C程序复制字符串而不使用具有足够内存的strcpy()

时间:2017-10-25 05:10:17

标签: c strcpy

我正在尝试输出:

Comparing results of concat and strcat ...
strcmp("Plain old stringTroy", "Plain old stringTroy") says: 0

如果两个字符串参数相同,则strcmp返回0。如果结果为0,则concat的行为与库函数strcat完全相同。

这是我对concat方法的看法。

#define MAXSIZE 32       
void concat(char dest[], char src[])                                        
{                                                                           
    int i=length(src);                                                          
    int j=0;                                                                    
    for(j; j<src[j] !='\0'; j++) {                                                                           
      dest[i+j] = src[j];                                                       
    }                                                                           
    dest[i+j] = '\0';                                                        
} 

长度法是:

 int length(char str[])                                                      
 {                                                                           
      // Add code here to return the length of the                              
      // string str without using the strlen function                           
      // Do not count the null character '\0'                                   
      // in computing the length of the string                                  
      int len=0;                                                                
      int i;                                                                    
      for(i=0;i<str[i];i++) {                                                    
          len++;                                                                  
      }                                                                         
      return len;                                                               
 }  

这是我的主要

int main()                                                                  
{                                                                           
      // Variable declarations for all parts        
      char str2[] = "Troy";                                                                                                
      char str4[] = "Plain old string";                                         
      char str6[MAXSIZE];   
   // Part 6                                                                 
      printf("\n----- Part 6 -----\n");                                         
      // Make a copy of the destination string first, to be reused later        
      strcpy(str6, str4);                                                       
      concat(str4, str2);                                                       
      strcat(str6, str2);                                                       
      printf("Comparing results of concat and strcat ...\n");                   
      printf("strcmp(\"%s\", \"%s\") says: %d\n", 
             str4, str6, strcmp(str4, str6)
            );   

      return 0;                                                                 
}

这是我运行时的输出:

----- Part 6 -----
Comparing results of concat and strcat ...
strcmp("PlaiTroy", "Plain old stringTroy") says: -1

第一个字符串与第二个字符串不同,这就是我得到-1的原因。我的问题在于我的concat方法,但我似乎无法理解为什么它不能很好地执行。是因为空间吗?是0和&#39; \ 0&#39;执行得不好?

2 个答案:

答案 0 :(得分:2)

您的代码中存在多个问题:

  • length函数中的循环测试不正确:而不是i < str[i],它应该是:

     for (i = 0; str[i] != '\0'; i++)
    
  • concat函数中的同一问题。将循环更改为:

     for (j = 0; src[j] != '\0'; j++) {
    
  • 同样在concat函数中,i应该是dst的长度,而不是src的长度。对于此变量,您可以使用len代替i

  • 函数str4中的数组main最后没有任何可用空间可供concat附加任何内容。以这种方式用更大的尺寸定义它:

    char str4[MAXSIZE] = "Plain old string";      
    

以下是更正后的版本:

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

#define MAXSIZE 32

void concat(char dest[], char src[]) {
    int len = length(dest);
    int j;
    for (j = 0; src[j] != '\0'; j++) {
        dest[len + j] = src[j];
    }
    dest[len + j] = '\0';
}

int length(char str[]) {
    int len = 0;
    int i;
    for (i = 0; i < str[i]; i++) {
        len++;
    }
    return len;
}

int main(void) {
    // Variable declarations for all parts
    char str2[MAXSIZE] = "Troy";
    char str4[MAXSIZE] = "Plain old string";
    char str6[MAXSIZE];
    // Part 6
    printf("\n----- Part 6 -----\n");
    // Make a copy of the destination string first, to be reused later
    strcpy(str6, str4);
    concat(str4, str2);
    strcat(str6, str2);
    printf("Comparing results of concat and strcat ...\n");
    printf("strcmp(\"%s\", \"%s\") says: %d\n",
           str4, str6, strcmp(str4, str6));
    return 0;
}

答案 1 :(得分:1)

这两个功能都有几个问题:

<input type="text" name="T2" id="d1" size="5" maxlength="2" min=0 max=10 step="2" Onkeypress="return isNumberKey(this.value)">

concat

这里的退出条件是什么?,for(j; j<src[j] !='\0'; j++) { 就足够了。

src[j] != '\0'

在这里添加偏移量为dest[i+j] = src[j]; 的数据,但我的长度为i,而不是src

因此纠正的功能可能是:

dst

<强> void concat(char dest[], char src[]) { /* descriptive variable name */ int len_dst = length(dst); int j=0; /* clear exit condition */ for(; src[j] != '\0'; j++) { dest[len_dst+j] = src[j]; } dest[len_dst+j] = '\0'; }

length

同样的说法,退出条件是什么? for(i=0;i<str[i];i++) { 就足够了

因此纠正的功能可能是:

src[i] != '\0'

<强> int length(char str[]) { int len=0; int i; /* clear exit condition */ for ( i=0; str[i] != '\0'; i++) { len++; } return len; }

main函数中的警告:

main

您没有足够的空间存储结果。写下类似的东西:

char str4[] = "Plain old string";
concat(str4, str2);  /* <- erases what is after str4 */