NDL（国立国会図書館）のAPIを導入してISBNから本の名前が表示されるスクリプトの組み方を教えてください。

Question

function NDLxml() {

//スプレッドシートを開く
var Spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
//フォームのシート
var Form = Spreadsheet.getSheetByName("シート1");
//一覧のシート
var Sheet = Spreadsheet.getSheetByName("シート2");
//最終行数を取得する
var LastRowForm = Form.getLastRow();
var LastRow = Sheet.getLastRow();
//フォームのデータをコピペする
var CopyISBN = Form.getRange(LastRowForm, 2).getValue();
Sheet.getRange(LastRow+1, 1).setValue(CopyISBN);
//ISBN取得
var ISBN = Sheet.getRange(LastRow+1, 1).getValue()
//NDLしばく
var BookURL = 'http://iss.ndl.go.jp/api/opensearch?isbn=' + ISBN;
//れすぽんす
var Response = UrlFetchApp.fetch(BookURL);
//XMLをぱーす
var xml = XmlService.parse(Response.getContentText());
//要素を取得
var Item = xml.getRootElement().getChildren("channel")[0].getChildren("item");
//なまえくうかん
var Dc = XmlService.getNamespace("dc", 'http://purl.org/dc/elements/1.1/');
var DcNDL = XmlService.getNamespace("dcndl", 'http://ndl.go.jp/dcndl/terms/');

//書籍名
var Title = Item[0].getchildText("title", Dc);
//しりーず
var Series = Item[0].getChildText("seriesTitle", DcNDL);
//おーさー
var Author = Item[0].getChildText("creator", Dc);
//出版社
var Publisher = Item[0].getChildText("publisher", Dc);
//出版年月日
var PubDate = Item[0].getChildText("pubDate");
//本のサイズ・ページ数
var Size = Item[0].getChildText("extent", Dc);
//巻数
var Volume = Item[0].getChildText("volume", DcNDL);
//版数
var Edition = Item[0].getChildText("edition", DcNDL);
//ここはすきにしてください
var Else = Form.getRange(LastRowForm, 3).getValues();
Sheet.getRange(LastRow+1, 10).setValue([[Else]]);

//書き込む
var GetSet = Sheet.getRange(LastRow+1, 2, 1, 8).setValues( [[Title, Series, Author, Publisher, PubDate, Size, Volume, Edition]]);
}

これを試してみたところ
TypeError: undefined のメソッド「get」を呼び出せません。（行 30、ファイル「コード」）
というメッセージが出ます。

A列にISBNを打ち込んだらB列に本の名前が出るようにしたいのですが、どの様にすればよいですか？
まだ何もわからない初心者なのでどうかアドバイスお願いします。

https://qiita.com/mnmtmym/items/a47e4552aeaea9ecb629
ここから引っ張ってきたものです。

正直なところ、スクリプトのスの字もわからない状況で丸投げされている状況で自分なりに理解しようとしているのですが、ちんぷんかんぷんになってしまいます。
非常に情けなく、申し訳ないお願いですが、素人でもわかりやすく教えて頂けませんでしょうか。
お願いします。

Comments

[CHERRY]

ソースコードや設定等は、コード部分を選択してツールバーの <code> ボタンを利用するなどの方法で見やすく記述していただけないでしょうか。　 [ソースコードを書きましょう]( https://teratail.com/help/question-tips#questionTips3-5-1 ) 参照

[fmy-]

https://qiita.com/mnmtmym/items/a47e4552aeaea9ecb629
ここから引っ張ってきたものです。

正直なところ、スクリプトのスの字もわからない状況で丸投げされている状況で自分なりに理解しようとしているのですが、ちんぷんかんぷんになってしまいます。
非常に情けなく、申し訳ないお願いですが、素人でもわかりやすく教えて頂けませんでしょうか。
お願いします。

Answer 1

要件も、どこから持って来たコードかも分からないので、漠然とした回答をします。

Formからの送信トリガで動作すると、Formから送信されたデータが最終行からうまくとれない(空白になってしまう)といった質問が過去散見されます。
(あいまいな記憶ですが、回答送信後、スプレッドシートへの反映が遅れるためデータを取ろうとしても取れないとか、1回の送信で複数回トリガが発動するとかいった質問がなされています)

こうした事象はそれなりに件数が発生しており、設定なのかGoogleの仕様なのかは知りませんが、実際に発生している確度が高いと捉えています。

従って、今回の場合、ISBNが空白となっている可能性はそれなりにあると思います。
(結果として、CHERRY様がおっしゃるAPIのレスポンスがエラーになっている、が起きていると思います)

対応として安直には、中断コードを入れることですが、これだとデータの取得ができないという点でやりたいことを叶えられないと思います。

var ISBN = Sheet.getRange(LastRow+1, 1).getValue()
if(!ISBN) { return; } //空なら中断

今回のコードに即時性は必要ないと思われるので、時間主導トリガにしてまとめて取得してはいかがでしょうか。

Answer 2

TypeError: undefined のメソッド「get」を呼び出せません。

ということは、エラーが出た行の変数の中身が不正なのだと思いますが、30行とすると... 変数 Item には、何が入っているのでしょうか？

たとえば、API がエラーを返すと DOM の中に item は存在しないと思いますので、変数 Item の中身は空になると思われますが、API でエラーを生じている等は無いでしょうか？
API のレスポンスを確認して正しいレスポンスが返ってきているかを確認してみてください。

Comments

[papinianus]

確認しましたが、ISBNとして妥当でない"test"のような文字列を渡すと、下記がレスポンスされます。

Itemノードはありませんので、こちらの回答のご指摘のとおり、Item[0]がundedfinedとなります。

<?xml version="1.0" encoding="UTF-8"?>
<rss xmlns:dcndl="http://ndl.go.jp/dcndl/terms/" xmlns:dcmitype="http://purl.org/dc/dcmitype/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:dcterms="http://purl.org/dc/terms/" xmlns:rdfs="http://www.w3.org/2000/01/rdf-schema#" xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#" xmlns:openSearch="http://a9.com/-/spec/opensearchrss/1.0/" version="2.0">
<channel>
<title>trial - 国立国会図書館サーチ OpenSearch</title>
<link>http://iss.ndl.go.jp/api/opensearch?isbn=trial</link>
<description>Search results for isbn=trial </description>
<language>ja</language>
<openSearch:totalResults>0</openSearch:totalResults>
<openSearch:startIndex>1</openSearch:startIndex>
<openSearch:itemsPerPage></openSearch:itemsPerPage>
</channel>
</rss>