質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.75%

  • Google Apps Script

    722questions

    Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

スプレッドシートで処理時間の短縮をしたいんですが

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 314

dream-pusher

score 3

スプレッドシートで特定の列の8行から最終行までチェックして、それぞれの処理をしています!

function timer() {
   var Timerss_id     = SpreadsheetApp.openById('XXXXX'),
       Timersheet     = Timerss_id.getSheetByName('name'),
       Timerrange     = Timersheet.getActiveRange(),
       Timerrow       = Timerrange.getRow(),
       Timerclock     = Timersheet.getRange("I1"),
       Timertimeid    = Timersheet.getRange("J1"),
       Timerlast      = Timersheet.getLastRow(),
       Timerday       = Utilities.formatDate(new Date(), "JST", "MM/dd"),
       Timertime      = Utilities.formatDate(new Date(), "JST", "H:mm"),
       Timeridday     = Utilities.formatDate(new Date(), "JST", "MMdd"),
       Timeridtime    = Utilities.formatDate(new Date(), "JST", "HHmm");

     Timerclock.setValue([Timerday] + ' ' +[Timertime]);
     Timertimeid.setValue([Timeridday]+[Timeridtime]);   

  for (var i = 8; i <= Timerlast ; i++) {

     Timerbase1     = Timersheet.getRange(i, 3)    
     Timerbased1    = Timerbase1.getValue()
     Timerbased2    = Timersheet.getRange(i,10)
     Timerid        = Timersheet.getRange(i, 11).getValues()

    if ( Timerbased1 == "時間" ){
      Timerbase1.setBackground('#FF0000')
    }else if ( Timerbased1 == ""){ 
      Timerbase1.setBackground(null)
    }else{
       Timerset1   = Utilities.formatDate(Timerbased1, "JST", "MMddHHmm")
       Timerbased2.setValue(Timerset1)                
      if (Timerid <= 0) {
        Timerbase1.setBackground("#FFFFFF")
          } else if (Timerid <= 15950000) {
        Timerbase1.setBackground("#FF0000")
          } else if (Timerid <= 15950070) {
        Timerbase1.setBackground('#f4a460')
          } else if (Timerid <= 15950100) {
        Timerbase1.setBackground('#FFFF00')
          } else if (Timerid <= 15952000) {
        Timerbase1.setBackground('#98fb98')
     } else{
        Timerbase1.setBackground('#FFFFFF')
          }
      } 
     }
  }

改良その①

function Ttimer() {
   var Ttss_id     = SpreadsheetApp.openById('xxxx'),
       Ttsheet     = Ttss_id.getSheetByName('name'),
       Ttrange     = Ttsheet.getActiveRange(),
       Tvalues     = Ttsheet.getDataRange().getValues(),   //追加コード      
       Ttrow       = Ttrange.getRow(),
       Ttclock     = Ttsheet.getRange("I1"),
       Tttimeid    = Ttsheet.getRange("J1"),
       Ttlast      = Ttsheet.getLastRow(),


       Ttday       = Utilities.formatDate(new Date(), "JST", "MM/dd"),
       Tttime      = Utilities.formatDate(new Date(), "JST", "H:mm"),
       Ttidday     = Utilities.formatDate(new Date(), "JST", "MMdd"),
       Ttidtime    = Utilities.formatDate(new Date(), "JST", "HHmm");

     Ttclock.setValue([Ttday] + ' ' +[Tttime]);
     Tttimeid.setValue([Ttidday]+[Ttidtime]);   

  for (var i = 8; i <= Ttlast ; i++) { 
         Ttbase1     = Tvalues[i-1][3]      //編集コード1  
      // Ttbased1    = Ttbase1.getValue()
       Ttbased2    = Tvalues[i-1][10]     //編集コード2
       Ttid        = Tvalues[i-1][11]        //編集コード3

    if ( Ttbase1 == "時間" ){
      Ttbase1.setBackground('#FF0000')
    }else if ( Ttbase1 == ""){ 
      Ttbase1.setBackground(null)
    }else{
       Ttset1   = Utilities.formatDate(Ttbase1, "JST", "MMddHHmm")
       Ttbased2.setValue(Ttset1)                
      if (Ttid <= 0) {
        Ttbase1.setBackground("#FFFFFF")
          } else if (Ttid <= 15950000) {
        Ttbase1.setBackground("#FF0000")
          } else if (Ttid <= 15950070) {
        Ttbase1.setBackground('#f4a460')
          } else if (Ttid <= 15950100) {
        Ttbase1.setBackground('#FFFF00')
          } else if (Ttid <= 15952000) {
        Ttbase1.setBackground('#98fb98')
     } else{
        Ttbase1.setBackground('#FFFFFF')
          }
      } 
     }
  }


動作はしているのですが、処理時間が掛かり過ぎているので、for文の方でgetValueを多用しているので、まずgetDataRange().getValues()全体の値を取得してその値を利用してその後の処理時間を短縮したいです。
改良その①ではsetBackgroundが動作しないので、原因を調べています。
皆様のお力をお貸し下さい。
後、お勧めの本やサイトがありましたら教えて下さい。
おねがいします!

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kei344

    2017/10/09 15:55

    「二次元配列の理解が不十分」とありますが、どのようなコードに変更しようとしていて、それにどういった問題があるのかをコードとともに追記されてはいかがでしょうか。

    キャンセル

  • dream-pusher

    2017/10/09 15:59

    ご指摘ありがとうございます。ただいま修正します

    キャンセル

回答 1

checkベストアンサー

+1

getValues で返ってくるのは二次元配列に値が入ったものなので、setBackground など Range クラスのメソッドは使えないと思います。

色指定を配列にして全体のRangeでまとめて指定するのが良いと思います。

【Class Range  |  Apps Script  |  Google Developers】
https://developers.google.com/apps-script/reference/spreadsheet/range#setbackgroundscolor

 var ss = SpreadsheetApp.getActiveSpreadsheet();
 var sheet = ss.getSheets()[0];

 var colors = [
   ["red", "white", "blue"],
   ["#FF0000", "#FFFFFF", "#0000FF"] // These are the hex equivalents
 ];

 var cell = sheet.getRange("B5:D6");
 cell.setBackgrounds(colors);

また、背景を変えるくらいならGASで書かなくても「条件付き書式」でカバーできるかもしれません。

【Google スプレッドシートで条件付き書式ルールを使用する - パソコン - ドキュメント エディタ ヘルプ】
https://support.google.com/docs/answer/78413?co=GENIE.Platform%3DDesktop&hl=ja

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/09 16:50

    回答ありがとうございます!
    教えて頂いたHPを参考にコード改良してみたいと思います。
    条件付き書式も試してみます!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.75%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Google Apps Script

    722questions

    Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。