ソース文字列へのポインターを受け入れ、宛先文字列へのポインターを返す文字列関数があります。この関数は現在機能していますが、malloc、realloc、および free を再評価するベスト プラクティスに従っていないのではないかと心配しています。
私の関数の違いは、宛先文字列の長さがソース文字列と同じではないため、関数内で realloc() を呼び出す必要があることです。ドキュメントを見ればわかるのですが…
http://www.cplusplus.com/reference/cstdlib/realloc/
再割り当て後にメモリアドレスが変更される可能性があることに注意してください。これは、C プログラマーが他の関数の場合のように「参照渡し」できないことを意味し、新しいポインターを返す必要があります。
したがって、私の関数のプロトタイプは次のとおりです。
//decode a uri encoded string
char *net_uri_to_text(char *);
関数を実行した後にポインターを解放する必要があるため、私はそれをやっている方法が好きではありません:
char * chr_output = net_uri_to_text("testing123%5a%5b%5cabc");
printf("%s\n", chr_output); //testing123Z[\abc
free(chr_output);
つまり、malloc() と realloc() は関数内で呼び出され、free() は関数外で呼び出されます。
私は高級言語 (perl、plpgsql、bash) のバックグラウンドを持っているので、私の本能はそのようなものを適切にカプセル化することですが、それは C でのベストプラクティスではないかもしれません.
質問: 私のやり方はベスト プラクティスですか、それとも従うべきより良い方法はありますか?
完全な例
未使用の argc および argv 引数に対して 2 つの警告を表示してコンパイルおよび実行します。これら 2 つの警告は無視しても問題ありません。
example.c:
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
char *net_uri_to_text(char *);
int main(int argc, char ** argv) {
char * chr_input = "testing123%5a%5b%5cabc";
char * chr_output = net_uri_to_text(chr_input);
printf("%s\n", chr_output);
free(chr_output);
return 0;
}
//decodes uri-encoded string
//send pointer to source string
//return pointer to destination string
//WARNING!! YOU MUST USE free(chr_result) AFTER YOU'RE DONE WITH IT OR YOU WILL GET A MEMORY LEAK!
char *net_uri_to_text(char * chr_input) {
//define variables
int int_length = strlen(chr_input);
int int_new_length = int_length;
char * chr_output = malloc(int_length);
char * chr_output_working = chr_output;
char * chr_input_working = chr_input;
int int_output_working = 0;
unsigned int uint_hex_working;
//while not a null byte
while(*chr_input_working != '\0') {
//if %
if (*chr_input_working == *"%") {
//then put correct char in
sscanf(chr_input_working + 1, "%02x", &uint_hex_working);
*chr_output_working = (char)uint_hex_working;
//printf("special char:%c, %c, %d<\n", *chr_output_working, (char)uint_hex_working, uint_hex_working);
//realloc
chr_input_working++;
chr_input_working++;
int_new_length -= 2;
chr_output = realloc(chr_output, int_new_length);
//output working must be the new pointer plys how many chars we've done
chr_output_working = chr_output + int_output_working;
} else {
//put char in
*chr_output_working = *chr_input_working;
}
//increment pointers and number of chars in output working
chr_input_working++;
chr_output_working++;
int_output_working++;
}
//last null byte
*chr_output_working = '\0';
return chr_output;
}