指针字符串

Question

复制到正确的位置，但在达​​到计数后不会停止。我认为我的代码应该如下工作

char har *orig, int start, int count, char *final);

int main(void)
{
    const char source[] = "one two three";
    char result[] = "123456789012345678";

    printf("%s\n",GetSubstring(source, 4, 3, result));

    return 0;
}

char r *orig, int start, int count, char *final)
{    
    char *temp = (char *)orig;

    final = temp;


   }

    for ( ; *temp && (count > 0) ; count--)
    {
    rn final;
}

Answer 1

1. 第一个for 循环不检查temp 数组是否存在（它如何在不以某种方式询问内存管理器的情况下检查分配的内存是否存在？！）。 temp 只是一个指针。您正在检查的是 orig 字符串在第一个 start 字节内没有零。没关系，也许'这就是你所说的“存在”。

2. 您的意图是从orig 复制到final，但您将final 重置为orig。 这就是您的错误所在。您必须删除该行才能解决问题。

3. 您不需要创建temp 指针，您可以使用orig 指针。你可以随意修改它——记住，函数参数实际上是局部变量。 C 中的函数参数是按值传递的，您通过将指针（即值！）传递给数据来实现按引用传递。

我应该补充一点，这个函数的前提有些被打破了。它“有效”，但这不是人们合理预期的。值得注意的是：

1. 没有迹象表明源字符串短于start。

2. 没有迹象表明源字符串短于start + count。

   也许这些都可以，但在这些条件可能是错误的情况下，函数的用户应该可以得到它的指示。调用者会知道什么是预期的，什么不是，所以调用者可以确定它只要你向调用者提供一些反馈。

3. 您返回的位置是输出末尾之后的位置 - 超过了零终止符。那不是很方便。如果要使用返回值连接后续字符串，则必须先将其减一。

下面是固定代码，变量命名合理。

char *GetSub(const char *src, int start, int count, char *dst)
{
    for ( ; *src && (start > 0) ; start--)
    {
        src++; /* Note: *src++ works too, but is pointless */
    }

    for ( ; *src && (count > 0) ; count--)
    {
        *dst++ = *src++;
    }

    *dst++ = 0;
    return dst; /* Notice: This returns a pointer to the end of the
                memory block you just wrote. Is this intentional? */
}

Answer 2

您所写的内容存在几个问题。我们来列举一下：

1. char *temp = (char *)orig; - 您将 const char *（您承诺不修改）分配给 char *（您违反了该承诺）。做错事了。

2. final = temp。不，这根本不会使 original final（调用者持有的副本）发生变化。它一无所获。它会将final 的您的（函数）副本更改为指向temp 所指向的同一位置。

3. *temp++; - 如果你不打算使用它，那么取消引用它是没有意义的。当然，增加它是正确的 [参见下面 KubaOber 的评论主题]。

4. final++ = *temp++; - 这只是令人困惑。

5. *final++ = 0; return final; - 您将地址 final 的值设置为“0”。然后你增加它（指向空间中的某个地方，可能指向一个黑洞）。然后返回该指针。这也是错误的。

您真正应该做的是以方便的方式包装strncpy。

但如果你坚持自己编写，你可能希望你的函数像这样简单：

char *GetSub(const char *orig, int start, int count, char *final)
{    
  int i;

  for (i = 0; i < count; i++)
    {
      final[i] = orig[i+start];

      if (final[i] == '\0')
        break;
    }
  final[i] = '\0';

  return final; /* Yes, we just return what we got.  */
}

Answer 3

问题出在以下行：

final = temp;

删除它，问题应该解决了。

Answer 4

char *a="abcdefgh";

我希望将字符串“cde”复制到另一个中。

我得到的索引是 3（你的起点）。

char *temp=malloc(3*sizeof(char))
strncpy(temp,a+3,3);

这是你需要的吗？

Answer 5

更改您的GetSub函数：

char *GetSub(const char *orig, int start, int count, char *final)
{
    char *temp = (char *)orig;

    // with original final = temp and final++ you loose final valid pointer
    char *final2 = final;


    for ( ; *temp && (start > 0) ; )
    {
        start--;

        // you don't need to dereference temp
        temp++;
    }

    for ( ; *temp && (count > 0) ; count--)
    {
        *final2++ = *temp++;
    }

    *final2 = 0;

    // return a valid pointer
    return final;
}

Answer 6

你的代码有一些错误：

char *GetSub(const char *orig, int start, int count, char *final)
{
    char *temp = (char *)orig;

    //final = temp; /* Why this? */

    for ( ; *temp && (start > 0) ; )
    {
        start--;
        temp++; /* Instead of *temp++ */
    }

    for ( ; *temp && (count > 0) ; count--)
    {
        *final++ = *temp++;
    }

    *(final+count) = '\0';
    return final;
}

希望对您有所帮助。