9

所以目前我有一个返回静态数组的函数，为了提高效率，有没有办法让它返回一个动态数组？

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

int *charpos(char *str, char ch)
{
    int *bff, bc, ec, i, strln;
    static int ret[255];
    bc = 0;
    ec = 0;

    for(i = 0; str[i] != '\0'; i++)
        ;

    strln = i;
    for(i = 0; i <= strln; i++)
    {
        if(str[i] == ch)
            ec++;
    }

    bff = malloc(sizeof(int)*ec);
    if(sizeof(bff) > sizeof(ret))
    {
        free(bff);
        return 0;
    }

    for(i = 0; i <= 255; i++) ret[i] = '\0';
    for(i = 0; i <= strln; i++)
    {
        if(str[i] == ch)
        {
            ret[bc] = i;
            bc++;
        }
    }

    free(bff);
    return ret;
}
4

2 回答 2

6

函数不能返回数组，句号。您当然可以使用指针或指向调用者已分配的内存块的指针。所以，在你的情况下......

int *ret = malloc(255 * sizeof int);  // caller must deallocate!

但是，这确实会改变代码的语义。函数的调用者现在负责调用free()返回的指针。如果他们不这样做，您将泄漏内存，因此这会增加一些以前不存在的复杂性。我更喜欢这样的东西：

void charpos(int *p, size_t size, const char *str, char ch) {
    // initialize the memory 
    memset(p, 0, size * sizeof int);
    
    // your other code here...

    size_t len = strlen(str);
    // fill the caller's memory
    for(i = 0; i < len; ++i)
    {
        if(str[i] == ch)
            p[bc++] = i;
    }
}

您正在返回一个指向int静态分配数组的第一个元素的指针。

于 2012-07-31T22:39:43.633 回答
0

实际上，您可以使用 static int 分配更多空间，而不必担心动态。这是我解决它的方法：

//indefinite reads, actually only up to 20
int * readDataAD7142(int addr, int numRegs){
  static int data[20];
  int i = 0;

  //Do something with this data array. Something that requires less then 20 elements

  return data;  
}

这是调用它的代码

 int *p;
 int i;
 p = readDataAD7142(0x0000, 6);

 for(i=0; i<6; i++){
  Serial.println(p[i], HEX);
 }

如果你有更多的内存和很少的时间（你也需要有点懒惰），那就完美而简单。

于 2017-01-30T16:47:04.440 回答