在本地函数中分配并初始化指针

Question

我想分配内存并将其填充到指针，这是函数参数之一，但我认为我没有得到一些重要的东西，请帮助我。

所以，如果我这样做，一切都会正常：

void alloc(char **p, int n)
{
*p = new char[n];
}

int main() {
char * ptr = NULL;
int n = 10;
alloc(&ptr, n);
for(int i = 0; i<(n - 1); i++)
    ptr[i] = '1';
ptr[n - 1] = '\0';

printf("%s", ptr);
return 0;
}

现在我想将分配的内存也初始化到

void alloc(char **p, int n)
{
*p = new char[n];
for(int i = 0; i<(n - 1); i++)
    *p[i] = '1';
*p[n - 1] = '\0';
} 

int main() {
char * ptr = NULL;
int n = 10;
alloc(&ptr, n); 
printf("%s", ptr);
return 0;
}

程序崩溃函数中。我不明白为什么。请问有人可以解释一下吗？

Answer 1

请尝试使用 (*p)[i] 和 (*p)[n - 1]。优先级规则导致 *p[i] 被计算为 *(p[i])。

Answer 2

试试这个：

((*p)[i]) = '1';

您的操作员评估顺序有问题。

Answer 3

可能是因为这个：

*p[i]

被解释为好像你写了这个：

*(p[i])

而不是好像你写了这个，这可能就是你的意思：

(*p)[i]

Answer 4

这应该可以解决问题，

void
alloc( char*& p, int n )
{
    p = new char[n];
    std::fill_n( p, n - 1, '1' );
    p[n - 1] = '\0';
}

如果您坚持使用指针作为参数，请更改 p 的 all
在函数中 (*p) （并且不要忘记检查空指针
如果你通过了，就做一些合理的事情。

但是，考虑到所有因素，您最好使用
std::string——这个函数甚至不是必需的，因为你
可以写成 std::string( n, '1' )。

Answer 5

试试这个

#include <stdio.h>   // for NULL

void alloc(char **p, int n)
{
    *p = new char[n];
    for(int i = 0; i<(n - 1); i++)
        *((*p) + i) = '1';
    *((*p) + n - 1) = '\0';
}

int main() {
    char * ptr = NULL;
    int n = 10;
    alloc(&ptr, n);
    printf("%s", ptr);

    // never forget delete pointers
    delete [] ptr;
    return 0;

}

Answer 6

这是运营商优先级问题。在你的 alloc 函数中尝试一下：

for(int i = 0; i<(n - 1); i++)
    (*p)[i] = '1';
(*p)[n - 1] = '\0';