1

strcpy を使用して、一部の文字列を構造体の文字列にコピーしようとしています。私はコードを投稿しています：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <math.h>
typedef struct floating_point{
    char sign[2];
    char exponent[9];
    char fraction[24];
}FLOATING_POINT;
FLOATING_POINT stringToFloatingPoint(char* str){
    struct floating_point fp;
    char *integer_str,*fraction_str,*integer_bin,*fraction_bin,*sign;
    sign=(char*)malloc(2*sizeof(char));
    int i=1,integer=0,fraction=0,comma=0,remainder=0,size=0;
    double fraction_double;
    //str=(char*)malloc(200*sizeof(char));
    //str="108,53";
    char * pch=(char*)malloc((strlen(str)+1)*sizeof(char));
    pch=strchr(str,',');
    if(pch==NULL){
        pch=str+strlen(str);
    }
    comma=pch-str;
    integer_str=(char*)malloc((comma+1)*sizeof(char));
    strncpy(integer_str,str,comma);
    integer_str[comma]='\0',
    integer=atoi(integer_str);
    fraction_str=(char*)malloc((strlen(str)-comma+2)*sizeof(char));
    strncpy(fraction_str,str+comma+1,strlen(str)-1);
    fraction_str[strlen(str)-comma]='\0';
    fraction=atoi(fraction_str);
    printf("%d",fraction);
    printf("%s",fraction_str);
    if(integer<0){
        strcpy(sign,"1");
        integer=-1*integer;
    }
    else{
        strcpy(sign,"0");
    }
    size=(int)(log(integer)/log(2))+2;
    integer_bin=(char*)malloc(size*sizeof(char));
    while(integer>0){
        remainder=fmod(integer,2);
        integer=integer/2;
        *(integer_bin+size-i-1)=(char)(remainder+48);
        i++;
    }
    *(integer_bin+size-i-1)=(char)(integer+48);
    *(integer_bin+size-1)='\0';
    printf("%s\n",integer_bin);
    fraction_bin=(char*)malloc(24*sizeof(char));
    fraction_double=atof(fraction_str);
    fraction_double=fraction_double/(pow(10,strlen(fraction_str)));
    printf("frac= %f",fraction_double);
    i=0;
    while((i<23)&&fraction_double!=0){
        fraction_double=2*fraction_double;
        if(fraction_double<1){
            *(fraction_bin+i)='0';
        }
        else{
            fraction_double=fraction_double-1;
            *(fraction_bin+i)='1';
        }
        i++;
    }
    *(fraction_bin+i)='\0';
    printf("\n%s",integer_bin);
    printf("\n%s",fraction_bin);
    size=strlen(integer_bin);
    for(i=0;i<strlen(fraction_bin)-(size-1);i++){
        *(fraction_bin+strlen(fraction_bin)-1-i)=*(fraction_bin+strlen(fraction_bin)-size-i);
    }
    for(i=1;i<size;i++){
        *(fraction_bin+i-1)=*(integer_bin+i);
    }
    printf("\n%s",fraction_bin);
    free(integer_bin);
    integer_bin=(char*)malloc(9*sizeof(char));
    strcpy(integer_bin,"00000000");
    printf("integer_bin %s",integer_bin);
    integer=127+size-1;
    i=1;
    while(integer>0){
        remainder=fmod(integer,2);
        integer=integer/2;
        *(integer_bin+8-i)=(char)(remainder+48);
        i++;

    }

    *(integer_bin+8-i)=(char)(integer+48);
    *(integer_bin+8)='\0';
    printf("\n\n%s %s %s",sign,integer_bin,fraction_bin);
    strcpy(fp.exponent,integer_bin);
    strcpy(fp.sign,sign);
    strcpy(fp.fraction,fraction_bin);
    free(integer_bin);
    free(integer_str);
    free(fraction_bin);
    free(fraction_str);
    free(sign);
    free(pch);
}
int main()
{

    struct floating_point fp=stringToFloatingPoint("108,53");
    printf("\n\n%s %s %s",fp.sign,fp.exponent,fp.fraction);
}

strcpy 部分まではすべて正常に動作します。printf("\n\n%s %s %s",sign,integer_bin,fraction_bin); このprintfはうまく機能します。そして、私の文字列のサイズは構造体のものと同じです。malloc を使用して割り当てたメモリ量を確認できます。エラーは発生しませんが、メイン関数で値を出力すると正しくありません。問題は何ですか？

4

3 に答える 3

5

return関数にステートメントがありません。

コンパイラの警告にもっと注意を払うか、警告をオンにするか、より優れたコンパイラを入手する必要があります。

于 2011-04-23T17:01:49.617 に答える
2

関数stringToFloatingPointの戻り値を見逃しました...少なくともこれは問題です...

追加

あなたに知らせるgcc -std=c99 -pedantic -Wall ./boh.c -lmために、あなたのコンパイラのために実行するか、または同様のものは、言うでしょう：

./boh.c: In function ‘stringToFloatingPoint’:
./boh.c:105: warning: control reaches end of non-void function

Valgrindはこう言うでしょう：

==6480== HEAP SUMMARY:
==6480==     in use at exit: 7 bytes in 1 blocks
==6480==   total heap usage: 7 allocs, 7 frees, 59 bytes allocated
==6480== 
==6480== LEAK SUMMARY:
==6480==    definitely lost: 7 bytes in 1 blocks

ほとんどの場合、特に2文字程度の割り当てを回避できます...

さらに、それを実行すると問題が発生します（ダブルフリーまたは破損）...

おそらくリークはここにあります：

   char * pch=(char*)malloc((strlen(str)+1)*sizeof(char));
    pch=strchr(str,',');
    if(pch==NULL){
        pch=str+strlen(str);
    }

ここではメモリを割り当てています。次に、charを検索し、割り当てられたメモリへのポインタを破棄します...次にfree（pch）を実行しようとすると、問題が発生します...

于 2011-04-23T17:04:58.790 に答える
1

関数は、FLOATING_POINT構造体を返すように指定されています。

FLOATING_POINT stringToFloatingPoint（char * str）

しかし、関数のどこにもreturnステートメントはありません。

于 2011-04-23T17:05:29.430 に答える