Почему мой код неисправен при попытке добавить фильтр в файл bmp, который был введен? - PullRequest
Новая
       78

Почему мой код неисправен при попытке добавить фильтр в файл bmp, который был введен?

0 голосов
/ 16 апреля 2019

Я пытаюсь добавить данный фильтр в файл bmp, но он вызывает ошибку при первом добавлении массива фильтра и tempRed (я нашел это с помощью gdb). Я думал, что это потому, что он пытался получить доступ к памяти из массива [-1], но так как мы начинаем с 1, я не думаю, что это будет так. Любая помощь очень ценится. Заранее спасибо! 

int edgeDect(struct HEADER *Header, struct INFOHEADER *InfoHeader, struct PIXEL **Data){
    int i = 0, j = 0;
    char filter[3][3] =
    {{0, -1, 0},
     {-1, 4, -1},
     {0, -1, 0}};
    char tempRed, tempGreen, tempBlue;

    for(i = 1; i < InfoHeader->Height - 1; i++){
            for(j = 1; j < InfoHeader->Width - 1; j++){
                    printf("The height is %d\n",i);
                    printf("The width is %d\n", j);

                    tempRed = 0;
                    tempGreen = 0;
                    tempBlue = 0;
                    //***Where seg fault occurs
                    tempRed += (filter[0][0] * Data[i-1][j-1].Red);
                    tempGreen += (filter[0][0] * Data[i-1][j-1].Green);
                    tempBlue += (filter[0][0] * Data[i-1][j-1].Blue);

                    tempRed += (filter[0][1] * Data[i-1][j].Red);
                    tempGreen += (filter[0][1] * Data[i-1][j].Green);
                    tempBlue += (filter[0][1] * Data[i-1][j].Blue);

                    tempRed += (filter[0][2] * Data[i-1][j+1].Red);
                    tempGreen += (filter[0][2] * Data[i-1][j+1].Green);
                    tempBlue += (filter[0][2] * Data[i-1][j+1].Blue);

                    tempRed += (filter[1][0] * Data[i][j-1].Red);
                    tempGreen += (filter[1][0] * Data[i][j-1].Green);
                    tempBlue += (filter[1][0] * Data[i][j-1].Blue);

                    tempRed += (filter[1][1] * Data[i][j].Red);
                    tempGreen += (filter[1][1] * Data[i][j].Green);
                    tempBlue += (filter[1][1] * Data[i][j].Blue);

                    tempRed += (filter[1][2] * Data[i][j+1].Red);
                    tempGreen += (filter[1][2] * Data[i][j+1].Green);
                    tempBlue += (filter[1][2] * Data[i][j+1].Blue);

                    tempRed += (filter[2][0] * Data[i+1][j-1].Red);
                    tempGreen += (filter[2][0] * Data[i+1][j-1].Green);
                    tempBlue += (filter[2][0] * Data [i+1][j-1].Blue);

                    tempRed += (filter[2][1] * Data[i+1][j].Red);
                    tempGreen += (filter[2][1] * Data[i+1][j].Green);
                    tempBlue += (filter[2][1] * Data[i+1][j].Blue);

                    tempRed += (filter[2][2] * Data[i+1][j+1].Red);
                    tempGreen += (filter[2][2] * Data[i+1][j+1].Green);
                    tempBlue += (filter[2][2] * Data[i+1][j+1].Blue);

                    Data[i][j].Red = tempRed;
                    Data[i][j].Green = tempGreen;
                    Data[i][j].Blue = tempBlue;
            }
    }
return 0;
}

Main: 

int main(int argc, char **argv){
    struct PIXEL *ColorData;
    struct INFOHEADER InfoHeader;
    unsigned char red, green, blue;
    struct HEADER Header;

    //Checks to make sure that there are enough command line arguments
    if(argc != 6){
            printf("Not enough input arguments in the command line.\n");
            exit(1);
    }

    printf("Inputing the information into the header structs.\n");
    //Inputs the header information into the header structs
    InputHeaders(argv[1], &InfoHeader, &Header);

    printf("Inputing the RGB componets into the struct.\n");
    //Inputs the colors from the picture into is RGB comnponets
    inputColors(argv[1], &InfoHeader, &ColorData, &Header);

    //Gets the integer value for the colors from the command line
    red = atoi(argv[3]);
    green = atoi(argv[4]);
    blue = atoi(argv[5]);

    printf("Adding filter to input image\n");
    //Edge detctor: FInd the edge of the picture
    edgeDect(&Header, &InfoHeader, &ColorData);
    printf("Saving the filter into outputfilename(edge).bmp.\n");
    //Saves the edge data to the correct output file
    saveEdge(argv[2], &InfoHeader, &Header, &ColorData);

    printf("Color changing function adding the command line numbers to the current RGB values.\n");
    //Changes the color of each RGB pixel with the inputed command line values
    colorChange(&ColorData, red, green, blue, &InfoHeader);
    printf("Saving the new bmp file with the color change\n");
    //Saves te shade file to the correct output file
    saveShade(argv[2], &InfoHeader, &Header, &ColorData);


return 0;
}

Сохранение значений RGB: 

int inputColors(char *filename, struct INFOHEADER *InfoHeader, struct PIXEL **Data, struct HEADER *Header){
    int i = 0, j = 0;
    FILE *inputFile;

    //Opens up the file so that it can be read from
    if((inputFile = fopen(filename, "rb")) == NULL){
            printf("Unable to open .bmp file\n");
            exit(1);
    }

    //Mallocing enough space for the 2D structures of pixels (colors)
    Data = (struct PIXEL **)malloc(InfoHeader->Height * sizeof(struct PIXEL *));
    for(i = 0; i < InfoHeader->Height; i++){
            Data[i] = (struct PIXEL *)malloc(InfoHeader->Width * sizeof(struct PIXEL));
    }

    //This goes until after we are down with the header
    fseek(inputFile, Header->Offset, SEEK_SET);

    //Inputing the data into the malloced struct
    i = 0, j = 0;
    for(i = 0; i < InfoHeader->Height; ++i){
            for(j = 0; j < InfoHeader->Width; ++j){
            //      printf("Width is %d\n", i);
            //      printf("Height is %d\n", j);
                    Data[i][j].Red = getc(inputFile);
            //      printf("The Red componet is %X\n", Data[i][j].Red);
                    Data[i][j].Green = getc(inputFile);
            //      printf("The green componet is %X\n", Data[i][j].Green);
                    Data[i][j].Blue = getc(inputFile);
            //      printf("The blue componet is %X\n", Data[i][j].Blue);
            }
    }

    fclose(inputFile);
return 0;
}

1 Ответ

0 голосов
/ 16 апреля 2019

Строка в inputColors, где вы назначаете Data, установит значение Data в этой функции. Его значение не будет передано вызывающей стороне (main), оставляя ColorData неинициализированным, когда вы попытаетесь использовать его позже.

Чтобы передать 2D-массив, созданный вами в inputColors, вам нужно передать данные как struct PIXEL ***Data, назначить их с *Data = malloc и изменить ColorData в main на struct PIXEL **ColorData. Это может упростить использование вновь выделенной памяти в inputColors, назначив эту память локальной переменной (назовите ее Data и измените параметр на struct PIXEL ***pData), и используйте имя параметра только один раз, когда вы сохранить его (*pData = Data;).

В качестве альтернативы, не передавайте Data в качестве параметра вообще. Передайте его обратно вызывающей стороне в операторе return.

...