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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Google Apps Script

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

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

1回答

2043閲覧

不要な行を削除したい

mozuq

総合スコア15

Google Apps Script

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

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2018/07/06 07:20

前提・実現したいこと

スプレッド―シート中で、特定の行を削除したいです。
大まかな流れは以下のとおりです。
なお、I列には必ず「未完」か「完了」の文字が入っています。

  1. I列に「未完」の文字があれば、D列にある日時と現在日時を比較し、現在日時より前であれば、I列に「完了」を挿入する。

  2. I列に「完了」の文字があれば、D列にある日時と現在日時より30日前の日時と比較し、30日前よりも前であれば、その行を削除する。

1番目は問題なくできるのですが、2番目の行削除ができません。
どのようにすれば2番目が実行されるか、ご教示いただけないでしょうか。

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

deleteRowが見つかりません

該当のソースコード

function(){ var Today = new Date(); var YMD = new Date(Today.getFullYear(),Today.getMonth(),Toay.getDate(),0,0,0);//秒数は0にする var LastMonth = new Date(Today.getFullYear(),Today.getMonth(),Today.getDate() -30,0,0,0);//30日前 var mySheet = SpreadsheetApp.getActiveSheet(); var dat = mySheet.getDataRange().getValues(); for(var i=1;i<dat.length;i++){ if(dat[i][8] == "未完"){ var TestDate = dat[i][3] if(TestDate < YMD){ dat[i][8] = '完了' mySheet.getRange(1,1,i,9).setValues(dat); } }else if(dat[i][8] == "完了"){ var TestDate = dat[i][3] if(TestDate < LastMonth){ mySheet.deleteRow(dat[i]) } } }

試したこと

Loggerでdat[i]を出力したところ、行ではなく、その行の値全部が出てきたので、ここが間違っているというのはわかるのですが、どう直したらいいのかわかりません…。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

まずはリファレンスを読むようにしましょう。
https://developers.google.com/apps-script/reference/spreadsheet/sheet#deleteRow(Integer)

deleteRowは引数に行番号を取ります。
よって

js

1mySheet.deleteRow(i + 1);

が正解です。
(配列の行番号とシートの行番号はずれている。)

ただし、deleteRowすると、それ以降の行番号がgetDataRange()の際とずれてしまうので
下から上に向かって処理する方がよさそうです。

##追記
こんな感じじゃないですかね?(試してないけど)

js

1 for(var i = dat.length - 1; i >= 0; i--) { 2 if(dat[i][8] == "未完") { 3 var TestDate = dat[i][3]; 4 if(TestDate < YMD){ 5 mySheet.getRange(i + 1, 9).setValue("完了"); // 毎回APIを呼び出すことになるが…。 6 } 7 } 8 else if(dat[i][8] == "完了"){ 9 var TestDate = dat[i][3]; 10 if(TestDate < LastMonth){ 11 mySheet.deleteRow(i + 1); 12 } 13 } 14 }

投稿2018/07/06 07:34

編集2018/08/08 07:35
macaron_xxx

総合スコア3191

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

mozuq

2018/07/17 08:43

お返事が遅くなり申し訳ありません。 ご教示いただきました方法で、行削除をすることができました。ありがとうございました。 ただ、おっしゃるとおり、行ズレが生じるため、削除は最終行から上に向かってする必要がありました。 ここで、もしお手数でなければ再度ご教示いただきたいのですが、 getLastRowで最終行を取得し、そこから上に向かって削除していく方法がわかりません…。 上記のソースコードに最終行取得を足し、for文を以下のように書きかえてみましたが、うまくいきませんでした。 var lastrow = mySheet.getLastRow(); for(var i=lastrow;i<0;i--){ 何かとんちんかんなことをしているのだろうと思いますが、ネットや本で調べてもたどりつくことができません…。 大変お手数をおかけしますが、再度ご教示いただけると助かります。 よろしくお願いいたします。
macaron_xxx

2018/07/17 08:48

for(var i=lastrow;i>0;i--){ ですね。 不等号の向きが逆です。
mozuq

2018/08/08 05:53

またまたお返事が遅れて申し訳ありません。 ご提示いただきました for(var i=lastrow;i>0;i--){ でやってみましたが、配列と行列の関係でうまくいきませんでした。 for(var i=lastrow-1;i>-1;i--){ とすると、配列と行列は一致するのですが、  mySheet.getRange(1,1,i,9).setValues(dat); の部分で値をセットできませんでした。 間違っている内容はわかるのですが、いまいちどのようにすればいいのかわかりません。 何かヒントをいただけないでしょうか…。
mozuq

2018/08/09 05:00

ありがとうございました。 思っていたとおりのことができました! コメントに書いてくださっていた、毎回APIを呼び出す件につきましては、filterを使うとうまくいくのかな?と調べ中です。 色々と教えてくださって、本当にありがとうございました。
macaron_xxx

2018/08/09 07:50

配列に「未完」「完了」をunshiftとかで入れていって、最後にまとめてsetValuesでいれるのがいいのかな~という感じがしますが、そこまで考えるのがめんどうだった手抜きですw
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問