在c中将字符串切换为512字节的块

时间:2015-04-27 22:29:29

标签: c c-strings

我试图编写一个函数,它接受一个字符串并将其切成512字节的块。它占用字符串中的前512个字节,并将其存储在chopped [0]中,然后将其存储在chopped [1] ...等中。当我在函数中打印出来时它似乎正在工作,但当我返回chopped [0]给我整个字符串而不仅仅是第512个。这里有什么想法吗?

char** chopString(){
    char string[]="1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200";

    int len=strlen(string)+1;
    int i,j,bytes,blocks;
    int blockSize=512;
    bytes=len*sizeof(char);
    blocks=(int)ceil((double)bytes/blockSize);  //determine number of blocks rounded up

    char* chunk=malloc(blockSize+1);
    char* newChunk=malloc(blockSize+1);
    char** chopped=malloc(sizeof(char*)*(blocks+1));  //blockSize+1 for  null character


    for(j=0; j<blocks; j++){
        len=strlen(string)+1;              //get length of string
        if(len<blockSize){                    //if the string can fit in one block
            bytes=len;                        //the number of bytes to write is the length
        }else{                                //if it doesn't fit in one block
            bytes=blockSize;                  //then bytes to write is one block
        }
        strcpy(chunk,string);        //copy newString into chunk
        newChunk=chunk;                 //keep pointer to begining of chunk

        for(i=0; i<bytes; i++){
            chunk++;
        }
        strcpy(string,chunk);        //set new string to remaining portion            
        *chunk='\0';                    //set end of chunk to null
        chopped[j]=newChunk;            //put chunk into array
        printf("Chopped[%d]: %s\n",j,chopped[j]);
        printf("length: %d",(int)strlen(chopped[j]));
    }
    chopped[j]=NULL;
    printf("\n"); 
    return chopped;
}

2 个答案:

答案 0 :(得分:3)

strcpy复制整个字符串。试试strncpy。请注意,您需要在字符串的末尾添加一个空终止符。

http://www.cplusplus.com/reference/cstring/strncpy/

答案 1 :(得分:3)

代码存在许多问题。这里有几个。

  1. 每次调用malloc都需要检查返回值(!= NULL)以确保操作成功

  2. 这一行:'newChunk = chunk;'覆盖malloc返回的指针,导致内存泄漏

  3. strcpy()不关注destination(1st)参数中的可用字节数。    所以需要将字节数限制为    第一个参数-1的长度    然后附加一个NUL字节    最简单的解决方法是使用strncpy()

  4. 设置char **的正确方法是:

    4a)分配数组

      char** myArray = malloc( count* sizeof(char*) )
      if (NULL == myArray)
         //handle error and exit
      else
         // following to make cleanup on error easy
         memset(myArray, 0x00, count*sizeof(char*) )
    

    4b)分配每个字符串

      for( int i=0; i<count; i++ )
          myArray[i] = malloc( blockSize+1 ) //+1 allows for termination byte
          if( NULL == myArray[i] )
              // cleanup and exit
    

    4c)清理将是

      for( int i=0; i<count; i++ )
          free(myArray[i])
      free(myArray)