c segfault当修改数组值时

Question

我在将DNA [i]更改为u时会得到一个segfault，我仍然不明白为什么。

另外，我正在用strCMP比较t的位置的值，但是据我了解，这是字符串文字的内容，我可以将其与dna [i] =='t'进行比较。那对吗？谢谢。

#include <string.h>

char *dna_to_rna(char *dna) {
    int size = (int)( sizeof(dna) / sizeof(dna[0]));
    char *retour[size];
    strcpy(retour, dna);
    for (int i = 0; i < size; i++) {
        if (dna[i] == 'T') {
            dna[i] = 'U';
        }
    }
    return (char *) retour;
}

int main() {
    char *dna[] = {
            "TTTT",
            "GCAT",
            "GACCGCCGCC"
    };

    char *actual, *expected;
    size_t n;
    for (n = 0; n < 3; ++n) {
        actual = dna_to_rna(*(dna + n));
    }
    return 0;
}

Answer 1

您将传递到函数dna_to_rna指向字符串字面的指针

actual = dna_to_rna(*(dna + n));

，然后在函数中您试图更改字符串字面的

    if (dna[i] == 'T') {
        dna[i] = 'U';
    }

任何尝试更改字符串字面的尝试中的字符字面结果。

同样，在此声明中使用sizeof运算符的表达式也

int size = (int)( sizeof(dna) / sizeof(dna[0]));

没有意义。它评估了类型char *的指针的大小。

相反，您应该使用标准字符串函数strlen。

而且该声明

char *retour[size];

至少是不正确的，您需要一个字符阵列而不是一系列指针。

 char retour[size];

该功能将返回一个带有自动存储持续时间的数组的指针，该数组退出功能返回函数返回无效指针后将不会活着

char *dna_to_rna(char *dna) {
    //...
    char retour[size];
    //...
    return (char *) retour;
}

。

您应该在函数中动态分配一个字符数组，其中长度strlen（dna） + 1并更改并返回此数组。

看来您的意思是如下

#include <string.h>
#include <stdlib.h>

char * dna_to_rna( const char *dna ) 
{
    size_t n = strlen( dna );
    char *retour = malloc( n + 1 );

    if ( retour != NULL )
    {
        strcpy( retour, dna );

        for ( size_t i = 0; i < n; i++ ) 
        {
           if ( retour[i] == 'T' ) retour[i] = 'U';
        }
    }

    return retour;
}