Problem with an if statement

Use the optional second argument to the input() function, 's', which tells input() to return exactly what the user entered, without evaluating it. You'll also need to compare orientation to 'v' rather than v or you'll just get the same error on the next line (and use strcmp/strcmpi in case they entered more than one character or an empty string).

orientation= input (' enter v or h', 's')
if strcmp(orientation,'v')
    % do v stuff
else
    % do h stuff
end

11 个评论
显示 9更早的评论隐藏 9更早的评论

Voss 2021-12-30

在 MATLAB Online 中打开

I would make the startingGrid input and the orientation input independent of each other. Something along the lines of this:

while true
    startingGrid = input(' Please choose the grid number you want your ship to start in : ');
    % validate startingGrid with your other code here
    if startingGrid >= 1 && startingGrid <= 10 % whatever your valid condition is
        break
    end
    disp('please try again');
end
while true
    orientation = input(' Please enter v if you want your ship to be placed vertically or h if you want it horizontally', 's');
    if ismember(orientation,{'v' 'h'})
        break
    end
    disp(' please choose v or h only and make sure to use small letters ');
end

then validate startingGrid and orientation together after that, i.e., check some condition(s) that depend(s) on both startingGrid and orientation, which is not possible to check until both have been input.

This way you only ever need one while loop for each input, rather than many.

You probably will want to wrap that whole thing in a while loop too, for the consistency check between startingGrid and orientation:

while true
    while true
        startingGrid = input(' Please choose the grid number you want your ship to start in : ');
        % validate startingGrid with your other code here
        if startingGrid >= 1 && startingGrid <= 10 % whatever your valid condition is
            break
        end
        disp('please try again');
    end
    
    while true
        orientation = input(' Please enter v if you want your ship to be placed vertically or h if you want it horizontally', 's');
        if ismember(orientation,{'v' 'h'})
            break
        end
        disp(' please choose v or h only and make sure to use small letters ');
    end
    
    % check consistency between startingGrid and orientation
    % if valid, break
    % else, issue statment (and the big while loop continues)
    
end

Tariq Hammoudeh 2021-12-30

Im sorry i realized i wasnt too clear, those 'specific numbers" are just grid numbers where if you choose v or h on them the ship will be placed out of bounds, so what my actual code is:

disp(' You will now place your corvette (1X2) ship')

startingGrid= input(' Please choose the grid number you want your ship to start in : ');

% orientation= input(' Please enter v if you want your ship to be placed vertically or h if you want it horizontally', 's');

if startingGrid == 1

while true

orientation = input(' Please enter v if you want your ship to be placed vertically or h if you want it horizontally', 's');

if ismember(orientation,{'v' 'h'})

break;

end

disp(' please choose v or h only and make sure to use small letters ');

end

if strcmp(orientation,'v')

...

else

...

end

elseif startingGrid == 2

.. repeat the same code above but do different v and h stuff..

elseif startingGrid == 3

.....

So what i need for example is when startingGrid == 6

if startingGrid == 6

while true

orientation = input(' Please enter v if you want your ship to be placed vertically or h if you want it horizontally', 's');

if ismember(orientation,{'v' 'h'})

break;

end

disp(' please choose v or h only and make sure to use small letters ');

end

if strcmp(orientation,'v')

..

else

display ('ship will be out of bounds please try again') and let the user try again

end.

because here when its h and starting grid 6 the ship will be out of bounds

Note: Im very well aware this isnt the most convenient way as ill have a over a 100 if statements, and ill have to go through each one individually, but its the only way i can think of.

Voss 2022-1-2

在 MATLAB Online 中打开

I would do it something like this. It doesn't check that ships don't overlap with each other, but it does check that each ship is on the board completely. I believe it has the structure you are looking for.

This code uses a 6-by-6 board, and assumes the definition of startingGrid is as follows: startingGrid = 1, 2, 3, 4, 5, 6 corresponds to the top row of the board, 7 is second row, first column, etc. (which is implied by your code). (Because this indexing is the opposite of how MATLAB does linear indexing in 2d matrices, it is convenient to build the transpose of the user-specified board and then transpose it at the end to get the final result.)

grid_size = 6;
board = zeros(grid_size);
ship_sizes = [2 3 5];
for i = 1:numel(ship_sizes)
    
    while true
        while true
            startingGrid = input(' Please choose the grid number you want your ship to start in : ');
            % validate startingGrid with your other code here
            if startingGrid >= 1 && startingGrid <= grid_size^2
                break
            end
            disp('please try again');
        end
        
        while true
            orientation = input(' Please enter v if you want your ship to be placed vertically or h if you want it horizontally', 's');
            if ismember(orientation,{'v' 'h'})
                break
            end
            disp(' please choose v or h only and make sure to use small letters ');
        end
        
        % check consistency between startingGrid and orientation
        n_max = grid_size-ship_sizes(i)+1;
        if strcmp(orientation,'h')
            if mod(startingGrid-1,grid_size)+1 > n_max
                disp('Your ship will be placed out of bounds (off the right side of the board)');
                continue
            end
            % if valid, place the ship ...
            board(startingGrid+(0:ship_sizes(i)-1)) = i;
        else
            if floor((startingGrid-1)/grid_size)+1 > n_max
                disp('Your ship will be placed out of bounds (off the bottom of the board)');
                continue
            end
            % if valid, place the ship ...
            board(startingGrid+grid_size*(0:ship_sizes(i)-1)) = i;
        end
        
        % ... and break
        break
        
    end
    
end
board = board.';
disp(board);

Tariq Hammoudeh 2022-1-2

编辑：Tariq Hammoudeh 2022-1-2

@Benjamin Again thank you i added this to my other code and it works fine, but i just want to ask what does the for loop do, I understood everything else but i just want to know what does this for loop do.