指向指针功能参数的指针

时间:2017-08-21 21:31:57

标签: c++ function pointers

该功能应该只是读取一个矩阵。 输入第一个字符后为什么会冻结?

out_1<- sequence(rle(out_2)$lengths)
cbind(out_1, out_2)
      out_1 out_2
 [1,]     1    -1
 [2,]     2    -1
 [3,]     1     1
 [4,]     1    -1
 [5,]     2    -1
 [6,]     3    -1
 [7,]     4    -1
 [8,]     5    -1
 [9,]     6    -1

3 个答案:

答案 0 :(得分:3)

这是未定义的行为:您的数组是一个包含指向char的100个指针的数组。但是你从来没有初始化它们。因此,当你解决p[i]它会得到一个未初始化的指针,它可以指向任何地方,当你用p [i] [j]取消引用它时,你可能会冻结或遭受任何其他未定义行为的症状。

如果你想学习使用指针和数组:

解决方案1:将您的数组定义为char a[100][100];

解决方案2:在as()的外部循环中,开始使用p[i] = new char[m];分配字符

如果你想学习现代C ++:

解决方案3:忘记内存分配和释放,改为使用向量。向量是完全动态的,因此不再有最多100行:

void as(vector<vector<char>> &p, int n, int m)
{
    p.resize(n); 
    int i, j;
    for (i = 0; i < n; i++) {
        p[i].resize(m); 
        for (j = 0; j < m; j++)
        {
            cout << "p[" << i << "][" << j << "]=";
            cin >> p[i][j];
        }
    }
}

int main()
{
    vector<vector<char>>a;
    as(a, 3, 3);
    return 0;
}

如果您想尝试Is it possible to update a kubernetes service 'External IP' while watching for the service?

解决方案4:你想要现代的C ++,但是你想要使用a[]中的元素,因为它们是一个字符串,以便于输出和操作,只需使用与上述代码相同,但将vector<vector<char>>替换为vector<string>

在这里你可以online...略微简化的代码。

答案 1 :(得分:0)

我在mallocs reallocs和指针上有你的简单伪数组。也许这对你很有意思:

typedef struct arr_str_t{
    size_t rows, columns;
    char **table;
}dynamicStringTable_t;

int CreateStringTable(dynamicStringTable_t **ptr, int rows, int columns)
{
    int result = 0;
    *ptr = (dynamicStringTable_t *)malloc(sizeof(dynamicStringTable_t));
    if (ptr == NULL) return - 1;
    (*ptr)->rows = rows;
    (*ptr)->columns = columns;
    (*ptr) -> table = (char *)malloc(rows * columns * sizeof(char *));
    if (*ptr == NULL)
    {
        free(*ptr);
        return -1;
    }
    for (int i = 0; i < rows * columns; i++) (*ptr)->table[i] = NULL;
    return 0;
}

char *getString(dynamicStringTable_t *ptr, int x, int y)
{
    char *result = (ptr == NULL || x >= ptr->columns || y >= ptr->rows || !x || !y) ? NULL : "";
    if (result != NULL)
    {
        result = ptr->table[x + y * ptr->rows];
    }
    return result;
}

int putString(dynamicStringTable_t *ptr, int x, int y, const char *str)
{
    int result = (ptr == NULL || x >= ptr->columns || y >= ptr->rows || str == NULL || !x || !y) * -1;
    if (!result)
    {
        char *tmp = (char *)realloc(ptr->table[x + y * ptr->rows], (strlen(str) + 1) * sizeof(char));
        if (tmp == NULL) result = -2;
        else
        {
            ptr->table[x + y * ptr->rows] = tmp;
            strcpy(tmp, str);
        }
    }
    return result;
}

int removeString(dynamicStringTable_t *ptr, int x, int y)
{
    int result = (ptr == NULL || x >= ptr->columns || y >= ptr->rows || !x || !y) * -1;

    if (!result)
    {
        free(ptr->table[x + y * ptr->rows]);
        ptr->table[x + y * ptr->rows] = NULL;
    }

        return result;
}

int destroyStringTable(dynamicStringTable_t *ptr, int x, int y)
{
    int result = (ptr == NULL || x >= ptr->columns || y >= ptr->rows || !x || !y) * -1;

    if (!result)
    {
        if (ptr->table != NULL)
        {
            for (int i = ptr->rows * ptr->columns - 1; i >= 0; i--)
                free(ptr->table[i]);
            free(ptr->table);
        }
        free(ptr);
    }
    return result;
}

答案 2 :(得分:0)

您的代码中存在很大问题。你正面临一个UB:

char *a[100]; // an array of 100 pointer to a character
// it is not initialized yet
  • 上面没有元素被初始化(甚至没有分配)。

更正您的代码:

char *a[100];
// Allocating the array of 100 elements on the heap:
for(int i(0); i < 100; i++){
    a[i] = new char[100]; // let's say your array is n = m
}

as(a, 3, 3);

for(int i = 0; i < 3; i++){
    for(int j(0); j < 3; j++)
        cout << a[i][j] << ", ";
    cout << endl;
}

最后但并非最不重要的是当你完成动态数组时不要忘记释放内存:

for(int i = 0; i < 100; i++)
    delete[] a[i];