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

編集履歴

回答編集履歴

対象行を修正

2021/11/15 00:33

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -177,25 +177,25 @@
 +  }
 +  // &が余計に付いているので削除しておく
 +  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で検索する
+    // ② Twitterで検索する
-+    var tweetList = findTweets(searchWord, lastTweetId);
+    var tweetList = findTweets(searchWord, lastTweetId);
-+
-+    // ③ 複数件ツイートを取得されるので for を使って1つずつツイートを取り出し いいね or RT をする
+    // ③ 複数件ツイートを取得されるので for を使って1つずつツイートを取り出し いいね or RT をする
-+    for (var j = 0, jl = tweetList.length; j < jl; j++ ) {
+    for (var j = 0, jl = tweetList.length; j < jl; j++ ) {
-+      var tweet = tweetList[j];
+      var tweet = tweetList[j];
-+
-+    // 重複処理をしないように最新のツイートIDを保存する
+    // 重複処理をしないように最新のツイートIDを保存する
-+    var titleRow = 1; // 『検索ワード』とか書いている部分の行数
+    var titleRow = 1; // 『検索ワード』とか書いている部分の行数
-+    var lastTweetIdCol = 3; // 『最終TweetId』の列までなので3列目まで
+    var lastTweetIdCol = 3; // 『最終TweetId』の列までなので3列目まで
-+    var updateCell = sheetData.getRange(i + 1 + titleRow, lastTweetIdCol, 1, 1); // i = 0 の時1行目なので+1してる
+    var updateCell = sheetData.getRange(i + 1 + titleRow, lastTweetIdCol, 1, 1); // i = 0 の時1行目なので+1してる
-+    updateCell.setValue(lastTweetId);
+    updateCell.setValue(lastTweetId);
-+  }
+  }
 }
 // pickUpSearchWords()は、main()内にあっても問題なく動くが、他の関数から利用することを考えると外に出した方がよい。

2021/11/15 00:33

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -143,6 +143,7 @@
 上記の回答コードではこの部分を修正し、関数をforループの外（厳密には、さらにその外であるmain()の外）に出しています。
 また、pickUpSearchWords()関数もmain()の外に出しています。
 ```diff
+※質問文の一番最初のバージョンに記載されているコード
 function main () {
   // ① 検索ワードをスプレッドシートから取得する
   var searchWords = pickUpSearchWords();

2021/11/14 02:44

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -138,7 +138,7 @@
 # 追記(コメントより)
-元の質問文では、下記色付き行で強調している部分が、forループの中に関数が定義されている状態になっています。
+元の質問文では、下記色付き行で強調している部分のうち、findTweets()関数が、forループの中に定義されている状態になっています。
 （単純なコピペミスでしょうか？）
 上記の回答コードではこの部分を修正し、関数をforループの外（厳密には、さらにその外であるmain()の外）に出しています。
 また、pickUpSearchWords()関数もmain()の外に出しています。
@@ -156,6 +156,7 @@
     var lastTweetId = searchWords[i][2];
+// findTweets()は、forループの中に定義されてしまっているため、外に出した方がよい。
 +// ツイートを検索する
 +// 【参考】 https://developer.twitter.com/en/docs/tweets/search/api-reference/get-search-+tweets
 +function findTweets (searchWord, lastTweetId) {
@@ -196,25 +197,25 @@
 +  }
 }
+// pickUpSearchWords()は、main()内にあっても問題なく動くが、他の関数から利用することを考えると外に出した方がよい。
++// 検索ワードをスプレッドシートから取得する
++function pickUpSearchWords () {
++  var titleRow = 1; // 『検索ワード』とか書いている部分の行数
++  var startRow = 1 + titleRow; // 1行目は『検索ワード』とか書いているので2行目から
++  var startCol = 1;
++  var endRow = sheetData.getLastRow() - titleRow; // 最後の行まで（2行目から始まっているので-1している）
-// 検索ワードをスプレッドシートから取得する
++  // !!!!!!  変更  !!!!!!!
-function pickUpSearchWords () {
-  var titleRow = 1; // 『検索ワード』とか書いている部分の行数
++  var endCol = 4; // 『実行回数』の列までなので4列目まで
-  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();
++  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

answer CHANGED Viewed

@@ -140,7 +140,7 @@
 元の質問文では、下記色付き行で強調している部分が、forループの中に関数が定義されている状態になっています。
 （単純なコピペミスでしょうか？）
-上記の回答コードではこの部分を修正し、関数をforループの外に出しています。
+上記の回答コードではこの部分を修正し、関数をforループの外（厳密には、さらにその外であるmain()の外）に出しています。
 また、pickUpSearchWords()関数もmain()の外に出しています。
 ```diff
 function main () {

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

2021/11/14 02:37

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -134,4 +134,87 @@
   return result.statuses
 }
+```
+# 追記(コメントより)
+元の質問文では、下記色付き行で強調している部分が、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

answer CHANGED Viewed

@@ -1,21 +1,28 @@
+・**認証が完了しているのであれば、
-・認証が正常にできているのであれば、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()関数以下で行おうとしていることは単に、
 「シート1の特定セルから値（ツイートID）を取得して、同じ値を同じ場所に書き込む」（検索結果がなければ何もしない）という処理なので
 実行しても何も動いていないように見えるだけであると考えられます。

2021/11/14 01:56

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -22,7 +22,7 @@
 （for文の中に関数が定義されている点が不可解ですが・・・）
 ---
-・下記に例として、シート1に指定した検索ワードの検索結果を表示するコードを記載します。
+・下記に例として、シート1に指定した検索ワードの検索結果をログ表示するコードを記載します。
 authorize()、authCallback()は変更していません。
 main()以下を変更しています。
 for文の中に関数が定義されている部分等も修正しています。

2021/11/14 00:02

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

@@ -23,7 +23,7 @@
 ---
 ・下記に例として、シート1に指定した検索ワードの検索結果を表示するコードを記載します。
-authorize()、anthCallback()は変更していません。
+authorize()、authCallback()は変更していません。
 main()以下を変更しています。
 for文の中に関数が定義されている部分等も修正しています。

2021/11/14 00:00

投稿

退会済みユーザー

スコア0

answer CHANGED Viewed

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