Swapping last matrix columns

function [ newMatrixA, newMatrixB ] = SwapLastColumns( matrixA, matrixB )
% SwapLastColumns: Exchange the last columns of input matrices
% matrixA and matrixB and return new matrices.
%   Inputs: matrixA, matrixB - input matrices, must have same
%                              number of rows so columns can be swapped
%
%   Outputs: newMatrixA, newMatrixB - returned new matrices created
%                                     from input matrices with last
%                                     columns swapped 


    % Assign tempColumn with last column matrixA
    tempColumn = matrixA(:, end); % FIXME 
    
    % Assign newMatrixA with first columns of matrixA and last
    % column of matrixB
    newMatrixA = [matrixA(:,end-1), matrixB(:,end)]; % FIXME 
        
    % Assign newMatrixB with first columns of matrixB and last
    % column of matrixA
    newMatrixB = [matrixB(:,end-1), matrixA(:,end)]; % FIXME 
    
end

As you can see above, I've written code that I thought would correctly swap the last columns between the two matrices, but it gives me an error. For my test cases, when swapping a 3x2 and a 3x2 matrix, the code works. But, when swapping a 3x3 and a 3x2 matrix it won't work, and outputs a 3x2 answer instead of a 3x3. Not quite sure what I'm doing wrong. Any help is appreciated.

Answer 1

[James Tursa]

From your description, I don't see how your code works for anything other that inputs having two columns. I would guess that this

    newMatrixA = [matrixA(:,end-1), matrixB(:,end)]; % FIXME 
    newMatrixB = [matrixB(:,end-1), matrixA(:,end)]; % FIXME 

should be this to include all of the beginning columns

    newMatrixA = [matrixA(:,1:end-1), matrixB(:,end)];
    newMatrixB = [matrixB(:,1:end-1), matrixA(:,end)];

[Kiernan King]

That worked! Thanks so much

[Dyuman Joshi]

Accepted as James's answer solves the problem posted.