トップ Google スプレッドシートに関する質問 Google Apps Scriptで動作するTwitter Botを作成中、twitterとの連携認証のスクリプトどうしても動作しません。

編集履歴

回答編集履歴

対象行を修正

2021/11/15 00:33

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -356,88 +356,88 @@
 +  var queryStr = queryStr.slice(0, -1);
++  var response = service.fetch('https://api.twitter.com/1.1/search/tweets.json?' + +queryStr);
++  var result = JSON.parse(response)
++  return result.statuses
++}
+    // ② Twitterで検索する
+    var tweetList = findTweets(searchWord, lastTweetId);
+    // ③ 複数件ツイートを取得されるので for を使って1つずつツイートを取り出し いいね or RT をする
+    for (var j = 0, jl = tweetList.length; j < jl; j++ ) {
+      var tweet = tweetList[j];
+    // 重複処理をしないように最新のツイートIDを保存する
+    var titleRow = 1; // 『検索ワード』とか書いている部分の行数
+    var lastTweetIdCol = 3; // 『最終TweetId』の列までなので3列目まで
+    var updateCell = sheetData.getRange(i + 1 + titleRow, lastTweetIdCol, 1, 1); // i = 0 の時1行目なので+1してる
+    updateCell.setValue(lastTweetId);
+  }
+}
+// pickUpSearchWords()は、main()内にあっても問題なく動くが、他の関数から利用することを考えると外に出した方がよい。
++// 検索ワードをスプレッドシートから取得する
++function pickUpSearchWords () {
++  var titleRow = 1; // 『検索ワード』とか書いている部分の行数
++  var startRow = 1 + titleRow; // 1行目は『検索ワード』とか書いているので2行目から
++  var startCol = 1;
++  var endRow = sheetData.getLastRow() - titleRow; // 最後の行まで（2行目から始まっているので-1している）
++  // !!!!!!  変更  !!!!!!!
++  var endCol = 4; // 『実行回数』の列までなので4列目まで
++  // 一括で取得する
++  var cells = sheetData.getRange(startRow, startCol, endRow, endCol).getValues();
 +
++  // cellsの中身は
-+  var response = service.fetch('https://api.twitter.com/1.1/search/tweets.json?' + +queryStr);
++  // [ [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,  [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,....,]
-+  var result = JSON.parse(response)
++  // という形式になる
++
-+  return result.statuses
++  return cells;
 +}
-+
-+    // ② Twitterで検索する
-+    var tweetList = findTweets(searchWord, lastTweetId);
-+
-+    // ③ 複数件ツイートを取得されるので for を使って1つずつツイートを取り出し いいね or RT をする
-+    for (var j = 0, jl = tweetList.length; j < jl; j++ ) {
-+      var tweet = tweetList[j];
-+
-+    // 重複処理をしないように最新のツイートIDを保存する
-+    var titleRow = 1; // 『検索ワード』とか書いている部分の行数
-+    var lastTweetIdCol = 3; // 『最終TweetId』の列までなので3列目まで
-+    var updateCell = sheetData.getRange(i + 1 + titleRow, lastTweetIdCol, 1, 1); // i = 0 の時1行目なので+1してる
-+    updateCell.setValue(lastTweetId);
-+  }
-}
-// pickUpSearchWords()は、main()内にあっても問題なく動くが、他の関数から利用することを考えると外に出した方がよい。
-+// 検索ワードをスプレッドシートから取得する
-+function pickUpSearchWords () {
-+  var titleRow = 1; // 『検索ワード』とか書いている部分の行数
-+  var startRow = 1 + titleRow; // 1行目は『検索ワード』とか書いているので2行目から
-+  var startCol = 1;
-+  var endRow = sheetData.getLastRow() - titleRow; // 最後の行まで（2行目から始まっているので-1している）
-+  // !!!!!!  変更  !!!!!!!
-+  var endCol = 4; // 『実行回数』の列までなので4列目まで
-+  // 一括で取得する
-+  var cells = sheetData.getRange(startRow, startCol, endRow, endCol).getValues();
-+
-+  // cellsの中身は
-+  // [ [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,  [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,....,]
-+  // という形式になる
-+
-+  return cells;
-+}
 }
 ```

2021/11/15 00:33

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -288,6 +288,8 @@
 ```diff
+※質問文の一番最初のバージョンに記載されているコード
 function main () {
   // ① 検索ワードをスプレッドシートから取得する

2021/11/14 02:44

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -278,7 +278,7 @@
-元の質問文では、下記色付き行で強調している部分が、forループの中に関数が定義されている状態になっています。
+元の質問文では、下記色付き行で強調している部分のうち、findTweets()関数が、forループの中に定義されている状態になっています。
 （単純なコピペミスでしょうか？）
@@ -314,6 +314,8 @@
+// findTweets()は、forループの中に定義されてしまっているため、外に出した方がよい。
 +// ツイートを検索する
 +// 【参考】 https://developer.twitter.com/en/docs/tweets/search/api-reference/get-search-+tweets
@@ -394,45 +396,45 @@
+// pickUpSearchWords()は、main()内にあっても問題なく動くが、他の関数から利用することを考えると外に出した方がよい。
-// 検索ワードをスプレッドシートから取得する
++// 検索ワードをスプレッドシートから取得する
-function pickUpSearchWords () {
++function pickUpSearchWords () {
-  var titleRow = 1; // 『検索ワード』とか書いている部分の行数
++  var titleRow = 1; // 『検索ワード』とか書いている部分の行数
-  var startRow = 1 + titleRow; // 1行目は『検索ワード』とか書いているので2行目から
++  var startRow = 1 + titleRow; // 1行目は『検索ワード』とか書いているので2行目から
-  var startCol = 1;
++  var startCol = 1;
-  var endRow = sheetData.getLastRow() - titleRow; // 最後の行まで（2行目から始まっているので-1している）
++  var endRow = sheetData.getLastRow() - titleRow; // 最後の行まで（2行目から始まっているので-1している）
-  // !!!!!!  変更  !!!!!!!
++  // !!!!!!  変更  !!!!!!!
-  var endCol = 4; // 『実行回数』の列までなので4列目まで
++  var endCol = 4; // 『実行回数』の列までなので4列目まで
-  // 一括で取得する
++  // 一括で取得する
-  var cells = sheetData.getRange(startRow, startCol, endRow, endCol).getValues();
++  var cells = sheetData.getRange(startRow, startCol, endRow, endCol).getValues();
++
-  // cellsの中身は
++  // cellsの中身は
-  // [ [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,  [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,....,]
++  // [ [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,  [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,....,]
-  // という形式になる
++  // という形式になる
++
-  return cells;
++  return cells;
-}
++}
 }

2021/11/14 02:43

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -282,7 +282,7 @@
 （単純なコピペミスでしょうか？）
-上記の回答コードではこの部分を修正し、関数をforループの外に出しています。
+上記の回答コードではこの部分を修正し、関数をforループの外（厳密には、さらにその外であるmain()の外）に出しています。
 また、pickUpSearchWords()関数もmain()の外に出しています。

コメントでの質問を受けて追記

2021/11/14 02:37

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -271,3 +271,169 @@
 ```
+# 追記(コメントより)
+元の質問文では、下記色付き行で強調している部分が、forループの中に関数が定義されている状態になっています。
+（単純なコピペミスでしょうか？）
+上記の回答コードではこの部分を修正し、関数をforループの外に出しています。
+また、pickUpSearchWords()関数もmain()の外に出しています。
+```diff
+function main () {
+  // ① 検索ワードをスプレッドシートから取得する
+  var searchWords = pickUpSearchWords();
+  // searchWordsの中身は
+  // [ [ '投稿内容', 'いいね or RT', '最終TweetId'] ,  [ '投稿内容', 'いいね or RT', '最終TweetId'] ,  [ '投稿内容', 'いいね or RT', '最終TweetId'] ,....,]
+  // という形式になっているので1つずつ見ていく
+  for (var i = 0, il = searchWords.length; i < il; i++ ) {
+    var searchWord = searchWords[i][0];
+    var type = searchWords[i][1];
+    var lastTweetId = searchWords[i][2];
++// ツイートを検索する
++// 【参考】 https://developer.twitter.com/en/docs/tweets/search/api-reference/get-search-+tweets
++function findTweets (searchWord, lastTweetId) {
++  var service  = twitter.getService();
++  var query = {
++    q: searchWord, // 検索ワード
++    lang: 'ja', // 日本語検索
++    locale: 'ja', // 日本限定で検索
++    result_type: 'recent', // 直近のツイートを検索
++    since_id: lastTweetId // これ以前のツイートは見ない
++  }
++  // 検索の内容を queryStr にまとめていく
++  var queryStr = '';
++  for (var key in query) {
++    // URLに日本語や記号を付けると上手く検索できないことがあるので#も変換する encodeURIComponent をする
++    queryStr += key + '=' + encodeURIComponent(query[key]) + '&'
++  }
++  // &が余計に付いているので削除しておく
++  var queryStr = queryStr.slice(0, -1);
++
++  var response = service.fetch('https://api.twitter.com/1.1/search/tweets.json?' + +queryStr);
++  var result = JSON.parse(response)
++  return result.statuses
++}
++
++    // ② Twitterで検索する
++    var tweetList = findTweets(searchWord, lastTweetId);
++
++    // ③ 複数件ツイートを取得されるので for を使って1つずつツイートを取り出し いいね or RT をする
++    for (var j = 0, jl = tweetList.length; j < jl; j++ ) {
++      var tweet = tweetList[j];
++
++    // 重複処理をしないように最新のツイートIDを保存する
++    var titleRow = 1; // 『検索ワード』とか書いている部分の行数
++    var lastTweetIdCol = 3; // 『最終TweetId』の列までなので3列目まで
++    var updateCell = sheetData.getRange(i + 1 + titleRow, lastTweetIdCol, 1, 1); // i = 0 の時1行目なので+1してる
++    updateCell.setValue(lastTweetId);
++  }
+}
+// 検索ワードをスプレッドシートから取得する
+function pickUpSearchWords () {
+  var titleRow = 1; // 『検索ワード』とか書いている部分の行数
+  var startRow = 1 + titleRow; // 1行目は『検索ワード』とか書いているので2行目から
+  var startCol = 1;
+  var endRow = sheetData.getLastRow() - titleRow; // 最後の行まで（2行目から始まっているので-1している）
+  // !!!!!!  変更  !!!!!!!
+  var endCol = 4; // 『実行回数』の列までなので4列目まで
+  // 一括で取得する
+  var cells = sheetData.getRange(startRow, startCol, endRow, endCol).getValues();
+  // cellsの中身は
+  // [ [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,  [ '投稿内容', 'いいね or RT', '最終TweetId', '実行回数'] ,....,]
+  // という形式になる
+  return cells;
+}
+}
+```

修正

2021/11/14 02:35

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -1,38 +1,52 @@
+・**認証が完了しているのであれば、
-・認証が正常にできているのであれば、authorize()やauthCallback()には何の問題もありません。
+　authorize()やauthCallback()には何の問題もありません。**
+```text
-（「認証が正常にできている」とは
+ここでの「認証が完了している」とは
-　authorize()関数を手動実行
+ 　スクリプトエディタ上で authorize() 関数を手動実行
-=>表示されるリンクをブラウザに貼り付けて移動。
+=> 表示されたリンクをブラウザのURL入力欄に貼り付けて移動。
-=>Twitterの連携画面が表示されるので連携承認ボタンを押す
+=> Twitterの連携画面が表示されるので連携承認ボタンを押す。
-=>左上に「SUCCESS!!」と表示される。
+=> ページが切り変わって左上に「SUCCESS!!」と表示される。
 までが完了している、という意味です）
+```
-・authCallback()関数は、認証手続中に内部的に呼ばれるものであり、直接実行するものではありません。
+・**authCallback()関数は**、認証手続中に内部的に呼ばれるものであり、**直接実行するものではありません**。
-（authCallback()を直接実行するとエラーが出るのは仕様です）
+（authCallback()を直接実行するとエラーが出るのは**仕様**です）
+また、**reset()**を実行すると、authorize()で設定したデータが**消えてしまいます**。
+したがって、上記の手順でauthorize()を実行～「SUCCESS!!」表示までが正常に表示された後は、
+reset()やauthCallback()を実行する必要はありません。
 ・認証が完了している状態で「『authCallback』の実行を飛ばして、『main』について、実行すると下記のように実行完了しますが、動作はしません。」とのことですが、
-動作していないように見えるだけで、質問文記載のコード自体は動いていると思われます。
+**動作していないように見えるだけ**で、質問文記載のコード自体は動いていると思われます。
 質問文記載のコードのmain()関数以下で行おうとしていることは単に、

2021/11/14 01:56

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -46,7 +46,7 @@
 ---
-・下記に例として、シート1に指定した検索ワードの検索結果を表示するコードを記載します。
+・下記に例として、シート1に指定した検索ワードの検索結果をログ表示するコードを記載します。
 authorize()、authCallback()は変更していません。

2021/11/14 00:02

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -48,7 +48,7 @@
 ・下記に例として、シート1に指定した検索ワードの検索結果を表示するコードを記載します。
-authorize()、anthCallback()は変更していません。
+authorize()、authCallback()は変更していません。
 main()以下を変更しています。

2021/11/14 00:00

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -104,6 +104,12 @@
+// セルを取得
+var sheetData = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("シート1");  // 「シート1」はシート名
 function main() {
   // ① 検索ワードをスプレッドシートから取得する