GAS 複数シートでのアクティブセル

前提・実現したいこと

GASを使って、以下の機能のスプレッドシートを作成しています。

シートAで、従業員(列A)ごとに出勤(列B)、退勤時間(列C)を入力すると、勤務時間がガントチャートのように塗りつぶされる(列D~)表を作成し、
シートBで、従業員名(列A)、予定の開始時間(列B)、終了時間(列C)を入力すると、(従業員名の順番は、シートAと異なる)
シートAの該当の従業員のチャートが別の色で塗りつぶされる。

発生している問題・エラーメッセージ

勤務時間の塗りつぶしと予定時間の塗りつぶしは、別の関数で設定(timeChartとyoyakuTime)、
どちらも値の変更をトリガーにしています。

入力されたセルをアクティブセルとして取得しているのですが、
シートA、Bで別々にアクティブセルを定義しているにも関わらず、
２つのの関数が同時に発動します。

それぞれ、シートA編集時はtimeChart、
シートB編集時はyoyakuTimeのみ発動するようにしたいのですが、
どこが間違っているのでしょうか？

初めての質問のため、わかりづらいかもしれませんが、
ご回答お願いします。

該当のソースコード

//タイムチャート勤務時間
function timeChart() {
  var ash = SpreadsheetApp.getActiveSpreadsheet()
  var time =  ash.getSheetByName("A")
  
  var clear =time.getRange("A1").getBackground();
  var timetable = [];
  for(var c=0;c<12;c++){
  timetable[c] = time.getRange(1,4+c).getValue()
  }
  

 var actCell = time.getActiveCell();
  
 time.getRange(actCell.getRow(),4,1,12).setBackground(clear)
  
 var inTime = time.getRange(actCell.getRow(),2).getValue()
 var outTime = time.getRange(actCell.getRow(),3).getValue()
 

  var i = 0;
  var o = 0;
  
  for(var t=0;t<12;t++){
   
    if (inTime == timetable[t]){
      i = t;
    time.getRange(actCell.getRow(),4+i).setBackground("#0000ff");
      
   }
    
   else if(inTime == ""){
    time.getRange(actCell.getRow(),4).setBackground(clear);
   }
    if(inTime != ""){ 
    if(outTime == timetable[t]){
      o = t;
       time.getRange(actCell.getRow(),5+i,1,o-i-1).setBackground("#0000ff");
      
    }
 }
  }
}

//予約時間反映
  function yoyakuTime(){
    var ash = SpreadsheetApp.getActiveSpreadsheet()
    var yoyaku = ash.getSheetByName("B") 
    var time =  ash.getSheetByName("A")
    var ActCell = yoyaku.getActiveCell();
    var YstaffName = yoyaku.getRange(ActCell.getRow(),7).getValue()
    var startTime = yoyaku.getRange(ActCell.getRow(),8).getValue()
    var finishTime = yoyaku.getRange(ActCell.getRow(),9).getValue()
    
    var i=0;
    var o=0;
    var timetable = [];
     for(var c=0;c<12;c++){
         timetable[c] = time.getRange(1,4+c).getValue()
     } 
  
 for(var s=0;s<5;s++){
    //タイムチャートのスタッフ名を取得(列A)
 var TstaffName = []
  TstaffName[s] = time.getRange(2+s,1).getValue()
  
  if(YstaffName == TstaffName[s]){
       
for(var t=0;t<12;t++){
  if (startTime == timetable[t]){
      i = t;
      
        time.getRange(2+s,4+i).setBackground("#FF0000");
    }
   
      if(finishTime == timetable[t]){
      o = t;
       time.getRange(2+s,5+i,1,o-i-1).setBackground("#FF0000");
      
    }
  }
 }
}
       
}

行動規範の内容に同意します

回答1件

ベストアンサー

シートA、Bで別々にアクティブセルを定義しているにも関わらず、２つのの関数が同時に発動

→シートA編集時はtimeChart、シートB編集時はyoyakuTimeのみ発動するようにしたい

上記なら、シンプルトリガーの**onEdit(e)**より、編集されたシートを特定し、それぞれへ分岐するのがスマートかと思います。
※コードはザッと確認しただけですので、問題があるとかないとかは考慮しておりません。

javascript
1function onEdit(e){
2  var ShtName = e.source.getActiveSheet().getName();
3  if     (ShtName == 'A'){timeChart();}
4  else if(ShtName == 'B'){yoyakuTime();}
5}