Подматрица вне границ (OpenCV)

Question

Я пытаюсь разделить изображение на сетку и сохранить отдельные части. На данный момент я прокручиваю номер детали и получаю суб-изображение, которое затем сохраняю.

Может кто-нибудь объяснить, как правильно получить суб-изображения? Я слежу за аналогичными сообщениями в stackoverflow, но мой код продолжает игнорировать утверждение, которое проверяет границы суб-изображения и оригинала.

int unitWidth = image.rows/n; 
int unitHeight = image.cols/n; 
for(int i=0; i<n; i++) { 
    //Take the next tile in the nxn grid. Unit is the width and height of 
    //each tile. i%n and i/n are just fancy ways of a double x,y for loop 
    Mat subImage = image(Rect((i % n) * unitWidth, (i/n) * unitHeight, unitWidth,unitHeight)); 

    ostringstream oss; 
    oss << i << "_" << n << ".jpg"; 
    string name = oss.str(); 
    imwrite(name, subImage); 
} 

p.s. первый субимаж не нарушает программу, но второй делает (для сетки 2x2, так что конечная часть). Я сокращаю суб-изображение на 10, но это все еще сломало машину.

Answer 1

Ниже приведен ваш код, так что он разбивает изображение на nxn плитки.

Во-первых, вы исчисляете unitWidth и unitHeight, и это является причиной ошибки утверждения. Оно должно быть:

int unitWidth = image.cols/n; // you had image.rows/n; 
int unitHeight = image.rows/n; // " " image.cols/n; 

Кроме того, если вы хотите NxN черепицу, вы должны цикла N^2 раза, а не только п раз. Самый простой способ сделать это состоит в том, чтобы иметь только две петли, одну внутри другой, одну для n строк для строк, а другую - n раз для столбцов.

for(int i = 0; i < n; i++) { //i is row index 
    // inner loop added so that more than one row of tiles written 
    for(int j = 0; j < n; j++) { // j is col index 
     //Take the next tile in the nxn grid. Unit is the width and height of 
     //each tile. i%n and i/n are just fancy ways of a double x,y for loop 

     // Mat subImage = image(Rect((i % n) * unitWidth, (i/n) * unitHeight, unitWidth, unitHeight)); 
     // I didn't understand the above line, as ((i % n)==i and (i/n)==0. 
     // 
     Mat subImage = image(Rect(j * unitWidth, i * unitHeight, unitWidth, unitHeight)); 

     ostringstream oss; 
     oss << i << "_" << j << ".jpg"; 
     string name = oss.str(); 
     imwrite(name, subImage); 
    } 
} 

Самый простой способ отладки кода, как это сделать Прямоугольник отдельный объект, так что вы можете распечатать его х, у, ширина, высота и проверить их против утверждения сообщения OpenCV. Вы компилируете свой код в режиме отладки?

 cv::Rect roi(j * unitWidth, i * unitHeight, unitWidth, unitHeight); 
     cout << "(i, j) = (" << i << ", " << j << ")" << endl; 
     cout << "(i %n) = " << i%n << endl; 
     cout << "(i/n) = " << i/n << endl; 
     cout << "roi.x = " << roi.x << endl; 
     cout << "roi.y = " << roi.y << endl; 
     cout << "roi.width = " << roi.width << endl; 
     cout << "roi.height = " << roi.height << endl; 
     Mat subImage = image(roi); 

Answer 2

for(int i = 0; i < n-unitHeight; i++) { 
    for(int j = 0; j < n-unitWidth; j++) { 
     ........... 
     ........... 
     ........... 
    } 
}