Make a better code to check the neighbours of a cell in a matrix and replace a value
4 次查看(过去 30 天)
显示 更早的评论
Hello, I want to ask for help to make my code better, or maybe simpler. My idea is to check the 8 neighbors of my cell isla(i,j)==2 and if one of them has the value 0, insert in that location the value 2, but only in the first one that the code finds.
My code is:
for i=3:len+2
for j=3:len+2
if isla(i,j)==2
if isla(i,j+1)==0 %espacio a la derecha vacio
isla(i,j+1)=2;
elseif isla(i,j-1)==0 %espacio a la izquierda vacio
isla(i,j-1)=2;
elseif isla(i+1,j)==0 %espacio de abajo vacio
isla(i+1,j)=2;
elseif isla(i-1,j)==0 %espacio de arriba vacio
isla(i-1,j)=2;
elseif isla(i-1,j-1)==0
isla(i-1,j-1)=2;
elseif isla(i-1,j+1)==0
isla(i-1,j+1)=2;
elseif isla(i+1,j+1)==0
isla(i+1,j+1)=2;
elseif isla(i+1,j-1)==0
isla(i+1,j-1)=2;
end
end
end
end
end
Hope you can see the problem is that the code isn't simple. I wish I could use fewer lines to don't have to modify the entire code every time when I have to change the conditions. I hope someone can help me. I apologize if I misspelled a word, English isn't my native language. Greetings
0 个评论
采纳的回答
Image Analyst
2013-5-19
编辑:Image Analyst
2013-5-19
Do you have the Image Processing Toolbox? And are all the numbers guaranteed to be 0 or positive?
If you want any neighbor of a 2 that is a 0 to be set to 2, then don't use elseif - have separate if statements.
if isla(i,j+1)==0 %espacio a la derecha vacio
isla(i,j+1)=2;
end
if isla(i,j-1)==0 %espacio a la izquierda vacio
isla(i,j-1)=2;
end
if isla(i+1,j)==0 %espacio de abajo vacio
isla(i+1,j)=2;
end
if isla(i-1,j)==0 %espacio de arriba vacio
isla(i-1,j)=2;
end
if isla(i-1,j-1)==0
isla(i-1,j-1)=2;
end
if isla(i-1,j+1)==0
isla(i-1,j+1)=2;
end
if isla(i+1,j+1)==0
isla(i+1,j+1)=2;
end
if isla(i+1,j-1)==0
isla(i+1,j-1)=2;
end
2 个评论
Image Analyst
2013-5-19
编辑:Image Analyst
2013-5-19
Then put the order of the "if"s in the order that you want, and inside each one put "continue" so that once it gets inside the first "if" it will go to the bottom of the loop and skip all the other ifs. For example:
if isla(i-1,j-1)==0
isla(i-1,j-1)=2;
continue;
end
Do that in all the ifs.
更多回答(1 个)
Ahmet Cecen
2013-5-19
If I understand correctly from your comment at the previous answer here is a pseudo-code that can do this in a somewhat simpler manner:
ok=0
While ok=0
i= a random number within the bounds of your index (use matlab help to find the function rand)
j= another random number
if isla(i,j)==0 && ( isla(i+1,j)==2 isla(i-1,j)==2 isla(i,j+1)==2 isla(i,j-1)==2 ) %Put as many neighbors within the OR logic as you want.
isla(i,j)=2;
ok=1;
endif
end
1 个评论
Ahmet Cecen
2013-5-19
Hmm the ORs didnt show up on the text.
It is supposed to be:
if isla(i,j)==0 AND ( isla(i+1,j)==2 OR isla(i-1,j)==2 OR isla(i,j+1)==2 OR isla(i,j-1)==2 )
另请参阅
类别
在 Help Center 和 File Exchange 中查找有关 Creating and Concatenating Matrices 的更多信息
Community Treasure Hunt
Find the treasures in MATLAB Central and discover how the community can help you!
Start Hunting!