MATLABのアップグレード問題

質問

MATLAB初心者です。
MATLAB6.1（おそらく）で開発されたコードがMATLAB7.01で動きません。
アドバイスを頂ければと思います。

1: out = {};
2: for i=1:length(files),
3: out = [out, helper([d '/' files{i}], pat(2:end))];
4: end

上記のコードの3行目で「Conversion to cell from char is not possible.」エラーが発生します。
MATLAB6.1から6.5のアップグレード問題で、空のセル配列の連結が原因のようです。
そこで下記のように修正しました。

1: out = {};
2: for i=1:length(files),
3: if i==1 out = helper([d '/' files{i}], pat(2:end));
4: else out = [out, helper([d '/' files{i}], pat(2:end))];
5: end
6: end

1回目のループの場合は連結せず、2回目以降は連結するように修正しています。
このように修正しても意図通り動いていません。
修正方法は間違っていますか。

回答No.1

Kules · ベストアンサー

間違っていると思います。
Conversion to cell from char is not possible
というのは、
「セル配列と文字列は連結できねーんだよ」
というエラーです。
outはセル配列ですが、 helper([d '/' files{i}], pat(2:end))が文字列を返しているため、
いわば[{},'ABC']みたいな配列を作る形になってしまっているのだと思います。
これってもともとできるんですかね？意外と出来るのかも…
もしこれが6.1から7.01に移行する時に出来なくなってしまったってことでしょうか？

おそらくループ数のサイズを持ったセル配列を作ろうとしているのだと思いますが、
変更後の動作を見ると、ループの前にout={};としたのが意味なくなってます（M-Lintが警告出しませんか？ifの中だから警告出さないのかな？）
現在の動作は
・outを空のセル配列にする
・i=1でoutにhelper([d '/' files{i}], pat(2:end))を代入する（この時点でhelper([d '/' files{i}], pat(2:end）)が文字列を返す関数なのであれば、outは文字列になります)
・iが2以上の時ouを連結する
という動作なので、おそらく最終的にはoutはただの長い文字列になります。

それならループに入る前にfiel数×1のセル配列を作っておいて、
ループごとにout{i}=helper([d '/' files{i}], pat(2:end));
にしたらいいと思われます。

参考になれば幸いです。

お礼

「Kules」様、ご返信ありがとうございます。
ご指摘の方法で解決できました。

ループに入る前にfiel数×1のセル配列を作っておいて、
ループごとにout{i}=helper([d '/' files{i}], pat(2:end));

※私が6.1で動作確認をしたわけではありませんが、動作実績はあったと聞いています。

大変助かりました。
ありがとうございました。