スプレッドシートとGASで請求書自動作成

Question

前提・実現したいこと

　グーグルのスプレッドシートで、google apps script　を利用して、受注データを登録した請求先毎にシートに分けて請求書を出力しようとしています。
既に、完成させている方のWEBサイトを参考にそのまま利用しようとしていますが、シート出力の際に「同じ名前のシートが既に存在している」旨のエラーで先に進めません。

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

エラーメッセージ
「シート名「〇〇」はすでに存在しています。別の名前を入力してください」

### 該当のソースコード

google apps script

・下記の処理の際に、既に処理した取引先を処理しようとするためにエラーになります。
　同名の処理が発生するまでは、問題なく作動します。
・既出の取引先名がある場合、処理をスルーする方法を加えたい。
以下、エラーとなるソースコード部分

    //プログラムA-6-3｜シートを追加して、シート名を各取引先に変更
    var newsheet = template.copyTo(spreadsheet);//「請求書フォーマット」のシートをコピーする
    newsheet.setName(torihiki[i]);//コピーしたシートの名前を「torihiki[i]」にする
    


如何せん、プログラムについては全くのど素人で、これから勉強する段階の為、質問の仕方にも不備がありましたら、お手数ですがご指摘とご指導お願いいたします。

Answer 1

torihiki[i]が何なのか分からないので、代替案が提案できないですが、同じ名前のシートは作れないので、適当に違う名前になるよう適当な数値を足すとかするしかないかと。
(スクリプトを何度も実行することを考えると、固定の数字をたしても次に実行したとき重複するので、数字を増やすとか、現在時刻をたすとか、回避策が必要になりそうです)

---

↓ここを

newsheet.setName(torihiki[i]);//コピーしたシートの名前を「torihiki[i]」にする

↓こう直してください。

newsheet.setName(torihiki_list[i]);//コピーしたシートの名前を「torihiki_list[i]」にする

Comments

[BECosaka]

ありがとうございます。

・「「torihiki[i]が何なのか分からないので、」」の部分については、どのような内容を提示するとよいでしょうか？

・新しいシートを追加する際に、既に処理済みの取引先名が出てきた場合は、処理せずに次に移りたい、という質問の方がより正しかったです。

・製作者様のサイトには、プログラムが一式記載されていますが、リンク等を貼っていい物かどうか判断がつきませんでしたので、一部だけ記載しました。
　元情報を貼った方が圧倒的にわかりやすいとは思いますが、そういったことは大丈夫なのでしょうか？
（モラル的なものも含めて）

[papinianus]

> 既出の取引先名がある場合、処理をスルーする方法を加えたい。

書いておられましたね。失礼。エラーを解消したいのだと捉えてました。

「既に処理済みの取引先名が出てきた場合」がプログラム上どう判定できるかわからないので、何ともいえないです。概念的には if とかで判断してください、としか。

大丈夫かどうかは、著作権や利用規約の問題ですし、有料コンテンツの場合全員が見られるわけでもないので、こちらからは何とも言えないです。

[BECosaka]

該当製作者様のサイトについては、有料や会員制とかではなく、一般的な検索で辿り着いたサイトです。
アドレスを貼ってみますので、一見して頂けるとさいわいです。

https://fastclassinfo.com/entry/gas_seikyusho/

[papinianus]

データ依存なのでこの修正がが 100% 正しいのかわかりませんが、バグですね。
(デバッグをしちゃうと質問者さまじゃなくて、その人のお金もうけの手助けになるのでいやなんですが)

A-5 で重複を排除していて A-6 の for は重複排除した配列でループしているのに、エラーの箇所では重複排除前のやつをつかってます(しかもtorihiki[i]はここにしか出現してません)。
この直しでは完全に直らないかもしれないですが、少なくとも torihiki と torihiki_list は別モノなので、これを直さないと異常なデータが作られるはずです。

[BECosaka]

ありがとうございます。

・「正しい直し方」自体を把握できるか自信がありませんが、ご教授いただいたポイントを見直してみます。（自分で触らないと勉強にならないとも思いますので。）

（余談）
この短時間で中身を理解されたpapinianusさんすげー！（表現があれですが）と思いました。
初めての質問でしたが、丁寧に対応していただいたのが何よりうれしかったです。
ありがとうございます。

[papinianus]

↑さっきのやつに対応するよう回答を直していたのですが、ちょっと説明不足にもほどがある書き方だったので、直してほしいところを具体化しました。

[BECosaka]

！ご丁寧にありがとうございます。

先の不具合は、無事に通過しました！（別のバグも出ましたが、引き続き調べていきます。
自分が考えたのと答え合わせも出来て有難いです。
（手助け頂くのも、自分で書いたコードではないので、諸事情も納得できます。）

また縁があればぜひお力添えください！
ありがとうございました。