abhay

The author is trying to improve this code. Making some honest effort to improve is enough to push my rating up a notch. (It still needs a lot of improvement, but any journey is taken one step at a time.) I like that there are now more comments that describe what this does.

What can you do better? First, start thinking in terms of functions. Note that every tool that the MathWorks supplies are all functions. This is for a good reason. A function takes a set of inputs, processes them and returns only those variables that you need in the end.

Write help for your functions that clearly describes the inputs and outputs. What does it do? Think about the potential users of this code. Make it as easy as possible for them to use. Pretend that you have never seen your own functions. Would you be able to figure out how to use them?

Look at some sample functions from Matlab, or some from the file exchange that have gotten many good reviews. Look especially carefully at files that are denoted "Select". Consider how you might emulate them in your own help and your code.

  Having downloaded and read the file exchange submission, "Random deletion of matrix vectors", I wish to state that I fully concur with John D'Errico's evaluation of it. What John did not point out is that the script in 'hablossranddel.m', the only original part of the submission, involves a process so simple and so obvious that it does not, in my opinion, warrant being placed in the file exchange. It involves techniques matlab users learn in their very earliest studies of the matlab language. There is no need to overload the file exchange entry list with trivial material of this kind. It definitely deserves a "poor" rating in my estimation.

Roger Stafford