0

我对 C 很陌生。我试图编写一个非常基本的矩阵程序来练习。

矩阵的工作方式是使用给定数量的行和列创建它，然后它调用具有足够槽的单个一维数组（行*列槽......你明白了）。然后要访问一个槽，你matrix_getcell用单元格在矩阵上调用 a ，它会返回一个指向单元格的指针。

这是matrix.h：

#ifndef MATRIX_H
#define MATRIX_H

#include <stdlib.h>
#include <stdio.h>

typedef unsigned int uint;

typedef struct matrix matrix;
struct matrix {
    uint rows;
    uint cols;
    double *data;
};

matrix *matrix_new(uint rows, uint cols) {
    matrix *n = malloc(sizeof(matrix));
    if (n == NULL) exit(1);

    n->data = calloc(rows * cols, sizeof(*(n->data)));
    if (n->data == NULL) exit(1);

    n->rows = rows;
    n->cols = cols;

    return n;
}

void matrix_del(matrix *m) {
    if (m == NULL) return;

    free(m->data);
    free(m);
}

double *matrix_getcell(matrix *m, uint row, uint col) {
    if (row >= m->rows) {
        fprintf(stderr, "Invalid row: %d\n", row);
        exit(1);
    }

    if (col >= m->cols) {
        fprintf(stderr, "Invalid col: %d\n", col);
        exit(1);
    }

    uint pos = (m->rows * row) + col;

    return &(m->data[pos]);
}

#endif

这是main.c：

#include <stdio.h>

#include "matrix.h"

int main(int argc, char **argv) {
    matrix *m = matrix_new(3, 3);

            /* I know that a 3x3 will have 9 cells, so
             * fill them up with successive numbers
             */
    for (int i = 0; i < 9; i++) {
        m->data[i] = i;
    }

            /* Now, run through each cell, row by column
             * and print out the coords and the contents.
             */
    for (uint r = 0; r < 3; r++) {
        for (uint c = 0; c < 3; c++) {
            double *cur = matrix_getcell(m, r, c);
            printf("(%d, %d): %.3d\n", r, c, *cur);
        }
    }

    matrix_del(m);

    return 0;
}

我试图做的是将每个单独的单元格初始化为一个连续的数字，这样当我第二次循环它时，它有望输出：

(0, 0): 0
(0, 1): 1
(0, 2): 2
(1, 0): 3
(1, 1): 4
(1, 2): 5
(2, 0): 6
(2, 1): 7
(2, 2): 8

但相反，它输出

(0, 0): 0
(0, 1): 0
(0, 2): 0
(1, 0): 1
(1, 1): 1
(1, 2): 1
(2, 0): 2
(2, 1): 2
(2, 2): 2

我添加（然后删除）代码来测试是否matric_getcell返回了不正确的结果（似乎不是）。我已经更改了数据类型，我尝试过强制转换......我不知道还能尝试什么。

为什么似乎将每一列设置为相同的数字？

4

2 回答 2

1

matrix_getcell当您计算单元格的位置时，您的方法内部有一个错误。

double *matrix_getcell(matrix *m, uint row, uint col) {
    ...
    // Should be (m_cols * row) + col.
    uint pos = (m->rows * row) + col;

    return &(m->data[pos]);
}

打印双打时存在另一个错误。您应该使用%f而不是%d打印双打。

//                   v--- "%d" is the problem here
printf("(%d, %d): %.3d\n", r, c, *cur);
于 2013-09-01T05:12:45.980 回答
0

你需要

double **data;//将data其视为二维矩阵

在matrix_new使用以下分配矩阵

n->data = calloc(rows, sizeof(double*));
if (n->data == NULL) exit(1);    
for(int i = 0;i<rows;i++) {
    n->data[i] = calloc(cols, sizeof(double));
}

并为避免内存泄漏，请使用释放它

void matrix_del(matrix *m) {
  if (m == NULL) return;

  uint r=m->rows;
  uint c=m->cols;

  for(int i = 0; i <r ; i++)
    free(m->data[i]);

   free(m);
}
于 2013-09-01T05:10:29.967 回答