将 pbm 文件转换为 IplImage

Question

显然，这段代码在最后一行 cvCvtColor() 处给了我错误，有什么问题？这个功能有什么可以改进的吗？

+(IplImage*) LoadPbmAsIplImage: (NSString*) fileName{
    NSString *filePath = [[NSBundle mainBundle] pathForResource:fileName ofType:@"pbm"];
    NSData *data = [NSData dataWithContentsOfFile:filePath];
    NSString *content = [[NSString alloc] initWithData:data encoding:NSUTF8StringEncoding];
    NSArray *lines = [content componentsSeparatedByString:@"\n"];
    NSString *secondLine = [lines objectAtIndex:1]; //because first line (index 0) contains meta data
    int width = [secondLine length];
    int height = [lines count]-1;
    Mat *mat = new Mat(width, height, CV_8UC4);
    MatIterator_<uint> it = mat->begin<uint>();
    for (int i=0; i<[lines count]; i++){
        for (int j=0; j<[[lines objectAtIndex:i] length]; j++){
            int pixelValue = 0;
            if ([[lines objectAtIndex:i] characterAtIndex:j] == '1'){
                pixelValue = 255;
            }
            *it = pixelValue;
        }
    }
    IplImage iplImage = *mat;
    IplImage* rv = cvCreateImage(cvSize(iplImage.width, iplImage.height), IPL_DEPTH_8U, 3);
    cvCvtColor(mat, rv, CV_RGBA2BGR);
    return rv;
}

Answer 1

你将 C++ 接口 (cv::Mat) 与 C 接口 (IplImage) 混合在一起，为什么要这样做？

Mat *mat = new Mat(width, height, CV_8UC4);

这是内存泄漏，您永远不会删除该映射，也几乎没有理由使用 new 创建一个 Mat - 只需 Mat mat(width, height, CV_8UC4) 就可以了，您不需要删除它。

cvCvtColor(mat, rv, CV_RGBA2BGR);

cvCvtColor 需要两个IplImage*s 作为参数，你给它一个cv::Mat* 和一个IplImage*，这是行不通的。将第一个mat 替换为&iplImage，或者始终使用c++ 接口。