Besides efficiency (which is often less important than you think, but not always), it's wrong. Consider

@oldarray = ('one', 'between one and two', 'two');
[download]