Передача IplImage * в функцию, оригинал не обновляется

У меня возникли проблемы с передачей параметров IplImage* в functions. Это для школьного проекта, и, к сожалению, я не должен редактировать файлы, вызывающие мои функции фильтра. В основном изображение вводится как параметр командной строки, затем создается его копия и передается в правильную функцию фильтра, как таковая:

IplImage * floating = cvCreateImage (cvSize (img->width, img->height), IPL_DEPTH_32F, 3);
cvConvertScale (img, floating, 1/255., 0);

IplImage * filtered;

switch (filter.ImageFormat)
{
    case YUV24:
        filtered = cvCreateImage (cvSize (floating->width, floating->height), IPL_DEPTH_32F, 3);
        cvCvtColor (floating, filtered, CV_BGR2YCrCb);
        break;
    case BGR24:
        filtered = cvCloneImage (floating);
        break;
    case Gray:
        filtered = cvCreateImage (cvSize (floating->width, floating->height), IPL_DEPTH_32F, 1);
        cvCvtColor (floating, filtered, CV_BGR2GRAY);
        break;
}

cvNamedWindow ("original", CV_WINDOW_AUTOSIZE);
cvShowImage ("original", img);

filter.ImageFilter (filtered, p);

if (filter.ImageFormat == YUV24)
{
      cvCvtColor (filtered, filtered, CV_YCrCb2BGR);
}

cvNamedWindow (filterName.c_str(), CV_WINDOW_AUTOSIZE);
cvShowImage (filterName.c_str(), filtered);

А вот код одного из моих фильтров:

void median (IplImage * image, int k){
    cout << "image address: " << &image << endl;
    Mat matImage(image);        //convert IplImage to Mat for work
    vector<Mat> bgr_channels;   //vector to hold each channel
    int i,j,m,n;                //row/column indeces
    int kernelSize = (2*k+1)*(2*k+1);
    vector<float> vals(kernelSize); //kernelSized vector to hold all values of image
                //within kernel centered at a given pixel
    int vecIndex = 0;   //then sorted to get the median
    int chanIndex = 0;  //index used to for each channel

    //add padding to account for border issues with convolution
    copyMakeBorder( matImage, matImage, k, k, k, k, BORDER_REPLICATE);
    //split channes to do work on individual channels
    split(matImage, bgr_channels);

    for(chanIndex=0; chanIndex < matImage.channels(); chanIndex++){
        //outer loop for scanning entire image
        for(i=k; i<matImage.rows-k; i++)/*image row index*/{
            for(j=k; j<matImage.cols-k; j++)/*image column index*/{
                //inner loop for scanning image only in kernel boundaries
                vecIndex = 0;   //reset vecIndex at start of each kernel loop
                for(m=i-k; m<(i+k+1); m++)/*kernel row index*/{
                    for(n=j-k; n<(j+k+1); n++)/*kernel column index*/{                      
                        vals[vecIndex++] = bgr_channels[chanIndex].at<float>(m,n);
                    }
                }
                insertionSort(vals, 0, vals.size()-1);  //insertion sort from CSCI362 text, see references  
                bgr_channels[chanIndex].at<float>(i,j) = vals[vals.size()/2]; //new value chosen from middle element                                                                //of sorted vector
            }
        }
    }

    merge(bgr_channels, matImage);      //merge channels together
    imshow("Median: Mat", matImage);    //left this in becuase the original doesn't seem to get modified    
                                        //when converting the Mat back to an IplImage
    image = cvCloneImage(&(IplImage)matImage);  //convert backto IplImage for DesktopMain
}

Проблема в том, что отфильтрованное изображение, показанное в основном, не отражает фактическое изображение. Он показывает только исходное изображение. Когда я вывожу изображение matImage в стиле Mat в свою функцию фильтра, оно показывает отфильтрованное изображение. Сразу после того, как я конвертирую обратно в IplImage и устанавливаю входной параметр IplImage* равным преобразованной отфильтрованной версии. Но изменения не отражают изображение, отображаемое в основной функции.

Это усложняет создание некоторых моих других фильтров, таких как Gaussian и Sobel, потому что эти фильтры сами вызывают другие функции, прежде чем выполнять свои манипуляции, и я не получаю обратно отредактированные данные. Есть ли что-то, что мне не хватает в том, как редактировать переданную переменную IplImage*?

Спасибо заранее за любую помощь!


person jrpharis    schedule 10.02.2013    source источник


Ответы (3)


Вместо этого:

image = cvCloneImage(&(IplImage)matImage); 

использовать это:

cvCopy(&(IplImage)matImage,image);
person Vaughn Cato    schedule 10.02.2013
comment
Да, это отлично сработало. Сначала были некоторые проблемы, потому что я дополнил границу matImage, но удалил ее и отлично работал. Из любопытства и необходимости понять, в чем разница между двумя вышеперечисленными. Оба работают с образом IplImage*. Если я делаю cvShowImage(image, image) сразу после вызова cvCloneImage, я вижу отфильтрованную версию. Почему эти изменения изображения не влияют на изображение из основного, как это делает cvCopy? Огромное спасибо за помощь! - person jrpharis; 10.02.2013
comment
@JoshPharis: вы передаете аргумент по значению, т. е. копия указателя передается функции. Это означает, что вы можете изменить то, на что указывает указатель, и это изменение будет видно за пределами функции, но если вы присвоите аргументу совершенно новое значение, вы только измените копию, переданную в функция. Все аргументы в C передаются по значению, а в C++ вы передаете по ссылке, когда используете что-то вроде; void foo(SomeType& arg); - person Ed S.; 11.02.2013

Ваша функция никогда не редактирует переданное ей изображение.

Эта строка предположительно делает копию изображения.

Mat matImage(image);        //convert IplImage to Mat for work

Затем ваш код изменяет эту копию matImage.

И в конце функции эта локальная копия уничтожается, а IplImage, на который указывает image, никогда не изменялась.

Редактировать: ответ Вона Като конкретно показывает, как вы можете решить эту проблему.

person Drew Dormann    schedule 10.02.2013

Чтобы перейти по 3 каналам, вы должны использовать итератор:

Mat out;
*********open out********
Mat_<Vec3b>::iterator it = out.begin<Vec3b>();
MatConstIterator_<Vec3b> it_end = out.end<Vec3b>();

for(; it != it_end; ++it)
{
    //your 3 channels
    (*it)[0] = ...;
    (*it)[1] = ....;
    (*it)[2] = ......;
}
person Thomas Specte    schedule 11.02.2013