whats wrong with this code?

2019 1 11
3 个回答
更新时间：2019 1 11
4 次查看（30 天）

0 个投票

Hi I am getting the disp output four times instead of three which is correct and should be displayed after executing this code:
function where_is_solution_required(lit_kolonne)
form_uten_tiltak='SANDSTEIN';
for k=1:size(lit_kolonne,1)
my_lit=lit_kolonne(k);
if (strcmpi(my_lit,form_uten_tiltak))
continue
end
disp('Tiltaket for sonen settes i gang')
end
end

3 个评论
显示 1更早的评论 隐藏 1更早的评论

Adam
Adam 2019-1-11
How could we possibly know without knowing what the input is? Also please format your code using the 'CODE' section so it is more readable with indentations and without so many blank lines.
Muazma Ali
Muazma Ali 2019-1-11
hi,
the input is an array containing 4 cells, the third one named 'sandstein' and the other are named 'skifer'. Since the criteria is about 'sandstein' , occuring only once in the input array, I should only get the disp output three times not four, according to the code.
function where_is_solution_required(lit_kolonne)form_uten_tiltak='SANDSTEIN';
for k=1:size(lit_kolonne,1)
my_lit=lit_kolonne(k);
if (strcmpi(my_lit,form_uten_tiltak))
continue
end
disp('Tiltaket for sonen settes i gang')
end
end
Guillaume
Guillaume 2019-1-11
Your reasoning is correct. Therefore we can surmise that your input does not contain 4 cells, or that it contains 4 cells and that none of them contains sandstein. The only way to know for sure is if you share that input (as a mat file).
By the way, why write:
if (strcmpi(my_lit,form_uten_tiltak))
continue
end
disp('Tiltaket for sonen settes i gang')
when
if ~strcmp(my_lit, uten_tiltak)
disp('Tiltaket for sonen settes i gang');
end
is simpler and clearer.

请先登录，再进行评论。

请先登录，再回答此问题。

回答（3 个）

Luna
Luna 2019-1-11

0 个投票

Try this below:
function where_is_solution_required(lit_kolonne)
form_uten_tiltak = 'SANDSTEIN';
for k=1:numel(lit_kolonne)
my_lit=lit_kolonne{k};
if ~(strcmpi(my_lit,form_uten_tiltak))
disp('Tiltaket for sonen settes i gang')
end
end
end

4 个评论
显示 2更早的评论 隐藏 2更早的评论

Muazma Ali
Muazma Ali 2019-1-11
i will try this..but I used it because I wanted to learn using continue in my executions :)
I agree with that your suggetion seems to be easier
Luna
Luna 2019-1-11
编辑：Luna 2019-1-11
If you want to do it with continue, you should change the line lit_kolonne(k) with curly paranthesis {k} to reach the char array inside the cell, because your lit_kolonne is a cell array.
Currently, you are comparing a cell with a char with strcmp.
Also check your lit_kolonne if it is 4x1, or 1x4 cell array.
my_lit=lit_kolonne{k};
Please accept the answer if it helped you and you get the solution.
The code works with continue for below input that I have created.
lit_kolonne = {'a','b','SANDSTEIN','d'}';
Muazma Ali
Muazma Ali 2019-1-11
let me explain further..in my script I have a dataset and I set lit_kolonne equal to one of the columns of the dataset ;
lit_kolonne= petrophysics.litologi;
here petrophysics.litologi contains a character array.
Luna
Luna 2019-1-11
编辑：Luna 2019-1-11
char array, cell array, double array are different things.
Continue is working in my machine with that code. Are you sure that SANDSTEIN is all upper case? Maybe yours is not working because of case sensitivity. Which version of Matlab you are using?
Read about strcmpi and strcmp.

请先登录，再进行评论。

Muazma Ali
Muazma Ali 2019-1-11

0 个投票

i know the difference between strcmpi n that without i.
always use the one with i to be on the safe side.
matlab 2018b

5 个评论
显示 3更早的评论 隐藏 3更早的评论

Luna
Luna 2019-1-11
function where_is_solution_required2(lit_kolonne)
form_uten_tiltak='SANDSTEIN';
for k=1:size(lit_kolonne,1)
my_lit=lit_kolonne(k);
if (strcmpi(my_lit,form_uten_tiltak))
continue
end
disp('Tiltaket for sonen settes i gang')
end
end
I saved above function without changing anything from your code, created below input:
lit_kolonne = {'a','b','sandstein','d'}'; % 4x1 cell array
I called the function
where_is_solution_required2(lit_kolonne)
The result is below:
>> where_is_solution_required2(lit_kolonne)
Tiltaket for sonen settes i gang
Tiltaket for sonen settes i gang
Tiltaket for sonen settes i gang
It works for me. Again I'm telling show us what input you are using!
Guillaume
Guillaume 2019-1-11
As I commented to the question a while ago, although badly written the original code does what it is supposed to do. Therefore the problem is most likely with the input which we have yet to see.
always use the one with i to be on the safe side.
That is a strange statement. The safe side may be to be case sensitive. Always use the one that is relevant to your code.
Muazma Ali
Muazma Ali 2019-1-11
I have no problem using strcmpi/strcmp
back to your question abt the input..have mistakenly written it as an answer to the question but can write it again here..:
the input is a character array from my script;
lit= petrofysikk.litologi;
as you can see it was part of a dataset earlier.
and lit is then sent as an input to the function.
Image Analyst
Image Analyst 2019-1-11
Muazma, why are you refusing to attach your data in a .mat file with the paper clip icon like Guillaume requested?
I sense he's about to give up trying to help you and move on to easier questions.
Muazma Ali
Muazma Ali 2019-1-11
its ok I have already got some suggetions to the question that I am going to try tomorrow. :)
right now I dont have the possibility to do what you are saying since I havent my computer right in front of ..I also think that I have explained it in detail how i have done it. Anyway thanks so far.

请先登录，再进行评论。

Muazma Ali
Muazma Ali 2019-1-11

0 个投票

as I wrote earlier I think the input is a character array from my script..that I set equal to column from my dataset , in my script like this:
lit=petrofysikk.litologi
where petrofysikk.litologi is a character array within the dataset petrofysikk.
And lit is then sent as an input to the function.

类别

在 帮助中心 和 File Exchange 中查找有关 Loops and Conditional Statements 的更多信息

标签

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!

Translated by