Skip to main content
N
nekkoneko
Inspiring
June 8, 2022
Answered

【JavaScript】ダイアログを表示させ、存在する全てのレイヤーをチェックボックスの選択肢にしたい

  • June 8, 2022
  • 2 replies
  • 1509 views

お世話になっております。

 

var layer = app.activeDocument.layers;
var layerArray = [] ;

for(i=0;i<layer.length;i++){
    layerArray.push(layer[i]);
}

layerArray = layerArray;


var dialog = new Window ('dialog','レイヤー',undefined);
var num = dialog.add('checkbox',undefined,layerArray[0]);

dialog.center();
dialog.show();

 

全てのレイヤーを取得するのと、ダイアログやチェックボックスを表示させることは出来たのですが、ファイルごとにレイヤーの数も名前も違うので決まったチェックボックスの名前をつけても意味はないですよね。

 

取得したレイヤーの数だけチェックボックス生成を繰り返すんだと思うんですが、どうやって調べたらいいのかも分かりませんでした(汗)というかそもそもこれって可能なのでしょうか？

 

よろしくお願いいたします。

This topic has been closed for replies.
Correct answer ajabon grinsmith

Ten Aさんと別進行で添削しますが、
ボタンのとこ

for(j = 0; j < checkboxes.length; j++) {
    (function (j) {
        dialog.okBtn.onClick = function () {
            if(checkboxes[j].checked = true){
                alert(chkNames[j]);
            }
        };
    })(j);
}

 だとボタンクリックイベントを何回も上書き定義してしまうことになるので、

変数 j のループで最後に定義したもの（一番最後のチェックボックス）だけ処理対象となる、

という理屈です。

 

OKボタンを押したらチェックのついたレイヤー名を順々にアラートする、だと

dialog.okBtn.onClick = function () {
    for(j = 0; j < checkboxes.length; j++) {
       if(checkboxes[j].checked = true){
           alert(chkNames[j]);
       }
    };
}

 で妥当かと思います。

 

処理がこんがらがらないよう、自身でわかりやすくコメントを書いて整理していくといいです。

これは今回のように人にコードを見せる時にも大変有用なのでぜひやってください。

2 replies

Ten A
Community Expert
Ten ACommunity Expert
Community Expert
June 9, 2022

ミニマルな構成だとこうですね。

layerオブジェクトはnameプロパティを参照しないとだめです。また、layers自体がlayerコレクションですから元々配列です。

 

var lyrs = app.activeDocument.layers;
var ck = [];
var w = new Window("palette");
for (var i=lyrs.length-1;i>=0;i--) ck[i] = w.add('checkbox', undefined, lyrs[i].name);
w.show();

 

ボタンを付けてませんが、submitボタンを配置したとして、クリックしたときのonClickイベントでck配列を参照して各要素のvalueを見れはチェックボックスのステートは確認できます。

N
nekkonekoAuthor
Inspiring
June 9, 2022

配列を配列にしてたんですね(汗)

 

教えていただいたのを利用しながら作ってみます。

ありがとうございました。

Ten A
Community Expert
Ten ACommunity Expert
Community Expert
June 8, 2022

automatic layoutで行ける範囲ならforループでチェックボックスを生成してOKボタンクリック時に各チェックボックスの状態を調べれば良いかな？

N
nekkonekoAuthor
Inspiring
June 9, 2022

ありがとうございます。

 

やってみたんですが、何を選択しても生成された一番最後のチェックボックスだけ結果に表示されてしまいました・・・。

 

var layer = app.activeDocument.layers;
var layerArray = [] ;
var checkboxes = [] ;
var chkNames = [] ;

for(i=0;i<layer.length;i++){
    layerArray.push(layer[i]);
}

layerArray = layerArray;


var dialog = new Window ('dialog','レイヤー',undefined);


for(n=0;n<layerArray.length;n++){
    chkNames[n] = layerArray[n];
    checkboxes[n] = dialog.add('checkbox',undefined,layerArray[n]);
}

dialog.okBtn = dialog.add('button', undefined, 'OK');

for(j = 0; j < checkboxes.length; j++) {
    (function (j) {
        dialog.okBtn.onClick = function () {
            if(checkboxes[j].checked = true){
                alert(chkNames[j]);
            }
        };
    })(j);
}

dialog.center();
dialog.show();

 

 

ajabon grinsmith
Community Expert
ajabon grinsmithCommunity ExpertCorrect answer
Community Expert
June 9, 2022

Ten Aさんと別進行で添削しますが、
ボタンのとこ

for(j = 0; j < checkboxes.length; j++) {
    (function (j) {
        dialog.okBtn.onClick = function () {
            if(checkboxes[j].checked = true){
                alert(chkNames[j]);
            }
        };
    })(j);
}

 だとボタンクリックイベントを何回も上書き定義してしまうことになるので、

変数 j のループで最後に定義したもの（一番最後のチェックボックス）だけ処理対象となる、

という理屈です。

 

OKボタンを押したらチェックのついたレイヤー名を順々にアラートする、だと

dialog.okBtn.onClick = function () {
    for(j = 0; j < checkboxes.length; j++) {
       if(checkboxes[j].checked = true){
           alert(chkNames[j]);
       }
    };
}

 で妥当かと思います。

 

処理がこんがらがらないよう、自身でわかりやすくコメントを書いて整理していくといいです。

これは今回のように人にコードを見せる時にも大変有用なのでぜひやってください。

Terms & ConditionsAccessibility statement
Using the community Terms of use Privacy Cookie preferences Do not sell or share my personal information ad choicesAdChoices