GASでgetSheetByIDまがいの事をしていますが不細工です。

Question

GASで、既存のスプレッドシートのタグをコピーしたいのですが、
(開いているスプレッドシートとは別)
GASに『getSheetByID』というものが無いので、下記コードのように
『getSheetByName』を書いておりますものの、非常に不格好です。

何か、スマートな書き方はありますでしょうか？

      var inputday = Utilities.formatDate(new Date(), "JST", "yyyyMMddHHmmss");

      var newest_Link = sheet.getRange(row, column);
      var formula = newest_Link.getFormula();

// =HYPERLINK("https://docs.google.com/spreadsheets/d/シートID/edit?pli=1#gid=タグID,"表示名")
// として、セルに書かれている

      var sheet_ID = formula.substring(formula.indexOf("/d/") + 3,formula.indexOf("edit") - 1);
      var tag_ID = formula.substring(formula.indexOf("gid=") + 4,formula.indexOf(",") - 1);

      var ss_newest = SpreadsheetApp.openById(sheet_ID);

      var sheets = ss_newest.getSheets();
      var spreadsheetID = ss_newest.getId();

      for(var i=0; i<sheets.length; i++) {

        var sheetId = sheets[i].getSheetId();
        var sheetName = sheets[i].getSheetName();

        if ( tag_ID == sheetId ) {
           var tag_Name = sheetName;
        }
        
      }

      var ss_sheet_newest = ss_newest.getSheetByName(tag_Name);
      var ss_sheet_copy_newest = ss_sheet_newest.copyTo(ss_newest);
      ss_sheet_copy_newest.setName(inputday);

Answer 1

一例ですが、

formula からの ID取得部分
→　正規表現と match を使う

tag_ID から シート取得の部分
→　sheets に対して mapと findIndex を使い、IDが一致するシートのindexを取得し sheets[index]

という方法があります。（今コード書けなくて申し訳ないですが）

あと、スプレッドシートID部分をsheet_IDってのは違和感があります。

https://docs.google.com/spreadsheets/d/(ここはss_IDとかにする)/edit#gid=(ここをsheet_ID)

とした方が良いかと。

Comments

[ghon]

ありがとうございます。
名称につきましては申し訳ありません。スプレッドシート初心者ですので私の中でも統一出来ておりません。
ご容赦ください。

Answer 2

  var inputDay = Utilities.formatDate(new Date(), "JST", "yyyyMMddHHmmss");
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getActiveSheet();
  var row = 1;
  var column = 1;
  var link = sheet.getRange(row, column).getRichTextValue().getLinkUrl();

  try{
    var gid = link.match(/#gid=(\d+)/)[1];
    var ss2 = SpreadsheetApp.openByUrl(link);
    var sheets = ss2.getSheets();
    var originalSheet;

    for(sheet of sheets){
      if(sheet.getSheetId() == gid){
        originalSheet = sheet;
        break;
      }
    }

    var newSheet = originalSheet.copyTo(ss2);
    newSheet.setName(inputDay);

  }catch(error){
    console.log("urlがスプレッドシートのものではないです。")
  }

変更点
・urlの取得方法～getRichTextValue().getLinkUrl()
・gidの取得方法～正規表現
・openByUrlで開く
・gidと一致したシートをコピーする

名称の注意点
・シートID　→　スプレッドシートID
・タグID　→　シートのID(gid)

その他
・jsの変数はキャメルケースで統一したほうがいいかと思います。例：inputDay

Comments

[ghon]

ありがとうございます。
『openByUrl』、戴かせていただきます。