引数の文字列を全て連結する関数について

質問

var config={
connect:{character:',',expand:true,operator:':'}
/*
character……何の文字列で連結するか
expand……展開するかどうか(true/false or number)　数字の場合は、展開する回数
operator……オブジェクト( {a:'a',b:'b'}など )の場合、キーと値を連結する文字列(key+operator+value)
*/
};
function connect(){
var connectString=config.connect.character,expand=config.connect.expand,operator=config.connect.operator,n=0;
var subCallee=function(arg){
var text=arg,i;
if(typeof arg=='object'){
var number=arg.length;
switch(expand){
case true:
text=subCallee(arg[0]);
foreach(arg,function(i,key,value){ //i……ループ回数-1 key……Object:キー,Array:i value……値
if(i){ //0回目はすでに再帰呼び出し済み
text+=connectString+subCallee(value); //再帰呼び出しで、展開
}
});
break;
case false:
case 0: //展開回数0(0)もしくは、展開しない(false)場合
text=arg[0];
foreach(arg,function(s){
if(s){
text+=connectString+this; //展開しないので、そのまま引数をつなげる。
}
});
break;
default:
if(typeof expand=='number'){ //expandがNumber型だった時
if(n>expand){ //nは最初に定義済み。展開回数が、指定された回数を越えた場合には、終了。
break;
}
foreach(arg,function(i,key,value){
i(i){
text+=connectString+subCallee(value);
}
});
n++; //何回展開したか、数える。
break;
}
return false;
}
}
return text;
};
return subCallee(arguments); //最初にargumentsを渡して、subCalleeを呼ぶ。
}

このように定義されていますが、Array型とObject型はtypeofしても、共にobjectが帰ってくるため、判別できません。
何か判別する方法はないでしょうか？
また、Object型の場合には、valueをkey+config.connect.operator+value(つまり、key:valueのような感じ)にしたいのですが、

text+=character+key+operator+subCallee(value);

とし、
var a=config.connect;
a.character='<>';
alert(connect({a:0,b:1}));

としましたが、alert表示されたのは、
undefined<>b:1
でした；

Object,Arrayの判定はしてないので、Arrayを入れてみたら↓
alert(connect(['a','b',['c',['d']]]));
↓
a<>1:b<>2:c<>1:d

となっています。
展開順に考えて、以下のようになるのが妥当かな、と思ったのですが；
まず、['a','b',[]] なので、
0:a<>1:b<>[]
で、次に[]の中の、['c',[]]が展開されるので、
<>2:c<>[]
最後に、['d']が展開されるので、
<>1:d
よって、
0:a<>1:b<>2:c<>1:d
かな、と。。

実際はa<>b<>c<>dでいいので、ここは気にしなくていいですｗ

Object,Arrayの判定方法と、Objectの場合、key:valueを取得する方法を、宜しくお願いします。

回答No.7

think49 · ベストアンサー

今更ですが…、JSON を利用するのではダメなんでしょうか？
https://developer.mozilla.org/En/Using_native_JSON

function connect () { return JSON.stringify(Array.prototype.slice.call(arguments)); }
connect(1, 2, 3, {foo: 'hoge1', piyo: 'hoge2'}); // [1,2,3,{"foo":"hoge1","piyo":"hoge2"}]

CSV の派生フォーマットを利用しているようですが、文字列とオブジェクトの境界があいまいな気がします。

connect("0:a", {0:a}); // 0:a<>0:a

JSON なら両者を区別できますし、パースするのも簡単です。

補足

ほとんどのブラウザで動作するのでしょうか？
主要ブラウザはしっかりとクロスブラウザ対策をしようと思っています。
IE,Firefox,NN,Opera,Chrome,Safariです。

色々と申し訳ありません。。
実は、他の質問に回答中のときに、間違えてsub.txtに保存してしまって、詳しい内容を消してしまいました。
sub.txtを更新しましたので、見てくださいm(_ _)m
http://kyukyoku.xrsp.net/cgi-bin/sample/sub.txt
ただ、毎回expandをswitchで回しているので、無駄な感じがします……
良い方法はないでしょうか？

回答No.6

Chaire

次のような関数があるとき、

function forEach (o, ...) {
if (o instanceof Array) return ...;
if (o instanceof RegExp) return ...;
if (o instanceof Object) return ...;
}

// 最低 1 回、最大 3 回の instanceof 判定が常につきまとう
forEach (o, ...);

次のように表現し直すことができます。

Array.prototype.forEach = function () { return ...; };
RegExp.prototype.forEach = function () { return ...; };
Object.prototype.forEach = function () { return ...; };

// o の「型」に応じた forEach が実行され、何の判定もいらない
o.forEach (...);

ゆえに、そもそも論として『Object、Array の判定方法』なるものが必要なのか、という所から考え直してはいかがですか。

ここで問題があるとすれば、俗に「オブジェクト汚染」と呼ばれる組み込みオブジェクトの拡張の是非ですが、適当にやって下さい。Array.prototype.forEach()、Object.keys() などは ECMAscript 5 に含まれましたので標準として扱って良いでしょう。周りが行儀の良いスクリプトだけであるなら、forEach() の有無が Array を示す目印にもなります。
※実際には forEach() より reduce()、あるいは map() を使った方が早そうですけどね。

一瞬だけ Array.prototype.toString()、Object.prototype.toString() を乗っ取り、後で戻すのも一つの手ではあります。危険性は RegExp.$1 なんかと大差ないでしょうから。

補足

申し訳ないです。
Object.prototypeはIEだと無理なので、オブジェクト汚染は決してやらないように、心掛けていますm(_ _)m
以前はバンバン汚染していましたが、オブジェクト汚染というのを発見して、全ての関数を汚染しないように書き換えたばかりでした；；

回答No.5

think49

#2-4 です。

オブジェクトのプロパティ展開は以下のようにして実現できます。
http://jsbin.com/asago4/2/

# 面倒なので再帰処理はやってませんが、同じ仕組みで可能だと思います。

回答No.4

think49

#2-3 です。

> Object.prototype.toString.callがまずい、ということで、
Object 型以外のデータに Object.prototype.toString.call(arg) するのがまずいのであって Object 型に使用するのは問題ないです。

さておき、http://kyukyoku.xrsp.net/cgi-bin/download/sub.txt のやり方には少し問題があります。

function kindof (arg) {
　alert(arg instanceof Object); // true
　alert(arg instanceof Array); // true

　if (arg instanceof Array) {
　　return "array";
　} else if (arg instanceof Object) {
　　return "object";
　}
}
alert(kindof([])); // "array"

期待通り、"array" を返しますが、instanceof のテスト順が逆なら "object" を返します。
同じ理屈で（arg instanceof Array && arg instanceof RegExp) === true な arg が渡されたとき "array" を返しますが、"regexp" を返してもおかしくないですね。
また、"object" が返された時、「arg instanceof Object」なのか「native かつ [[Call]] を持たない Object 型」なのかを区別しづらいです。("function" も同じ)
false を返すケースも「host かつ [[Call]] を持たない Object 型」です。

ひとつのオブジェクトに対して [[Class]] はひとつしか存在しませんが、プロトタイプはプロトタイプチェーンでたどれる数だけ存在します。
Object.prototype.toString なら if-else 方式の検査で済みますが、isntanceof では問題があります。

instanceof を活用できるのは arg が Array.prototype.slice を持っているか検査する場合です。
Array を継承しているオブジェクトなら Array.prototype.slice も持っていると期待できます。(※slice をオーバーライドしている場合を除く)
[[Class]] は継承しませんので、Array を継承したオブジェクトを "Array" とみなしません。
質問者さんはどちらの動作を期待しているでしょうか？

if (getClass(arg) === 'Array') { // Object 型を判別して、[[Class]] を得る
　alert('arg is a [object Array]');
}
if (isObject(arg) && Object.prototype.toString.call(arg) === '[object Array]') { // 同上
　alert('arg is a [object Array]');
}
if (isObject(arg) && arg instanceof Array) { // Object 型を判別して、プロトタイプを検査する
　alert('arg is a [object Array]');
}

https://gist.github.com/887049
https://gist.github.com/862098

回答No.3

think49

#2 です。

> 前回の質問で私が書いたコードも同じようなことをしています。(今見ると Object 型の判定が甘かったですね…。)
すみません。よく見たら「型」のチェックを全くやっていませんでした…。
型チェック付の isArray を下記URLにUPしておきました。
http://jsfiddle.net/qCF3B/3/

> foreach(arg,function(s){
細かいことですが、foreach は JavaScript にはないですね。
Array.forEach のような関数をユーザ定義しているのでしょうか。
掲示されたコードは動きませんでした。

回答No.2

think49

まず、型を判定したいのか、[[Class]] を判定したいのか、をはっきりさせてください。

ES5 に Array 型は存在しませんし、typeof arg === "object" だと Null 型で true を返してしまう上に一部の Object 型を取り逃がします。
https://gist.github.com/862085
https://gist.github.com/887049

配列は Object 型で [[Class]] === "Array" でインスタンス生成元(コンストラクタ) が Array です。

console.log(typeof []); // "object"
console.log(Object.prototype.toString.call([])); // "[object Array]"
console.log([] instanceof Array); // true

ES5 の Array.isArray(arg) では次のように判定しています。
http://es5.github.com/#x15.4.3.2

1. arg の型が Object 型でなければ false を返す
2. [[Class]] === "Array" ならば true を返す
3. false を返す

[[Class]] は Object 型だけに存在する内部プロパティなのでまず、型を判定してから [[Class]] を判定しています。
前回の質問で私が書いたコードも同じようなことをしています。(今見ると Object 型の判定が甘かったですね…。)
http://okwave.jp/qa/q6590566.html

Object.prototype.toString は [[Class]] を返す便利なメソッドですが、ひとつ注意点があります。
ES3 規定によれば、初めに [[Class]] プロパティを得ることになっていますが、そもそも Object 型でなければ [[Class]] がありませんので、String 型や Null 型を渡してどういう結果になるか予想できません。(最悪、TypeError を出力して処理がとまるかもしれません。)
http://www2u.biglobe.ne.jp/~oz-07ams/prog/ecma262r3/15-2_Object_Objects.html#section-15.2.4.2
ES5 ではこの規定は更新され、Undefined 型、Null 型はそれぞれ [object Undefined], [object Null] を返し、"[object " + class + "]" を返す仕様になっています。
http://es5.github.com/#x15.2.4.2
対象ブラウザを最新のブラウザに限定するなら別ですが、古いブラウザにも対応するなら ES5 で規定された動作は期待できません。

IE6 では Object.prototype.toString.call(null) === "[object Object]" です。ES5 の規定に従っていないことがわかります。(IE8 も同じだったと思います)
ES5 規定の Array.isArray と同じように「型」をチェックしてから [[Class]] をチェックする手法が無難だと思います。

お礼

ご丁寧に、ありがとうございました＾＾

補足

Object.prototype.toString.callがまずい、ということで、

http://yas-hummingbird.blogspot.com/2009/07/javascript.html
こちらを参考にして、getType関数を作ってみました。

判定は出来ましたが、指定された回数、展開をする、というのと、オブジェクトの場合には、キー+separator+値 で取得、というのがどうしてもできませんでした＞＜
関数群に関しては、以下のファイルにまとめておきました。
具体的な質問内容に関しても、そちらに記載しています。
http://kyukyoku.xrsp.net/cgi-bin/download/sub.txt
要望が多くて申し訳ないですが、御回答宜しくお願いします。

回答No.1

fujillin

よくわかってませんが…

＞Object,Arrayの判定方法と、Objectの場合、key:valueを取得する方法を～

判定方法には以下が利用できませんでしょうか？
　Object.prototype.toString.call(array) // [object Array]
　Object.prototype.toString.call(object) // [object Object]

Objectのkeyとvalueは、こんなのでは？
var obj = {
　 c : 1,
　b : 2,
　a : 3,
　hoge : function(){ alert("OK"); }
};
//obj = window;

var key, value, result = [];
for(key in obj){
　value = obj.key?obj.key:obj[key];
　result.push(key + " / " + value);
}
alert(result.join("\n"));

お礼

ありがとうございます。
参考になりました。
Object.prototype.toStringは、初めて聞きました＾＾