140文字以上のツイートを送信したとき、エラーが表示されない問題を解決 refs #23797

git-svn-id: http://svn.sourceforge.jp/svnroot/nt-manager/NishioTweetManager/trunk@123 d8c9ecd3-d47d-4367-8645-de82c00e513f

diff --git a/src/twitter/gui/action/TweetMainAction.java b/src/twitter/gui/action/TweetMainAction.java
index dad3314..56fa40c 100644 (file)
--- a/src/twitter/gui/action/TweetMainAction.java
+++ b/src/twitter/gui/action/TweetMainAction.java
@@ -1321,16 +1321,29 @@ public class TweetMainAction {
         * tweetBoxPaneに書かれた文字をつぶやく
         */
        public void actionTweet() {
-               if (this.replyStatus != null) {
-                       tweetManager.replyTweet(tweetBoxPane.getText(), this.replyStatus
-                                       .getId());
-               } else {
-                       tweetManager.tweet(tweetBoxPane.getText());
+               boolean isTweet = false;
+               try {
+                       if (this.replyStatus != null) {
+                               tweetManager.replyTweet(tweetBoxPane.getText(), this.replyStatus
+                                               .getId());
+                       } else {
+                               tweetManager.tweet(tweetBoxPane.getText());
+                       }
+                       isTweet = true;
+               }catch(Exception e) {
+                       e.printStackTrace();
+               }
+               
+               if( isTweet ) {
+                       // ツイートした旨を表示
+                       this.information("メッセージをつぶやきました. 発言:" + tweetBoxPane.getText());
+                       tweetBoxPane.setText(""); // テキストをクリア
+               }else {
+                       this.information("つぶやきに失敗しました");
+                       JOptionPane.showMessageDialog(null, "つぶやきに失敗しました。文字数がオーバーしているか、ツイッターに接続ができませんでした。",
+                                       "Tweet Error",
+                                       JOptionPane.ERROR_MESSAGE);
                }
-               // ツイートした旨を表示
-               this.information("メッセージをつぶやきました. 発言:" + tweetBoxPane.getText());
-
-               tweetBoxPane.setText(""); // テキストをクリア
 
        }
 

diff --git a/src/twitter/manage/TweetManager.java b/src/twitter/manage/TweetManager.java
index df1592c..c98c44e 100644 (file)
--- a/src/twitter/manage/TweetManager.java
+++ b/src/twitter/manage/TweetManager.java
@@ -409,11 +409,11 @@ public class TweetManager {
                        // tweetIDを保存しておく
                        this.sinceMentionID = statuses.get(0).getId();
                        // 一番新しいtweetを一番したに持ってくる
-//                     for (Status status : statuses) {
-//                             tweetList.add(0, status);
-//                     }
-                       Collections.reverse( statuses );
-                       tweetList.addAll( statuses );
+                       // for (Status status : statuses) {
+                       // tweetList.add(0, status);
+                       // }
+                       Collections.reverse(statuses);
+                       tweetList.addAll(statuses);
                }
                try {
                        saveProperties();
@@ -545,11 +545,11 @@ public class TweetManager {
                        sinceMentionID = statuses.get(0).getId();
 
                        // 一番新しいtweetを一番したに持ってくる
-                       /*for (Status status : statuses) {
-                               tweetList.add(0, status);
-                       }*/
-                       Collections.reverse( statuses );
-                       tweetList.addAll( statuses );
+                       /*
+                        * for (Status status : statuses) { tweetList.add(0, status); }
+                        */
+                       Collections.reverse(statuses);
+                       tweetList.addAll(statuses);
                }
                try {
                        saveProperties();
@@ -593,18 +593,18 @@ public class TweetManager {
                        sinceTweetID = statuses.get(0).getId();
 
                        // 一番新しいtweetを一番したに持ってくる
-                       /*for (Status status : statuses) {
-                               tweetList.add(0, status);
-                       }*/
-                       Collections.reverse( statuses );
-                       tweetList.addAll( statuses );
+                       /*
+                        * for (Status status : statuses) { tweetList.add(0, status); }
+                        */
+                       Collections.reverse(statuses);
+                       tweetList.addAll(statuses);
                }
                // ログ保存
                try {
-                       /*for (Status s : tweetList) {
-                               logManager.add(s);
-                       }*/
-                       logManager.add( tweetList );
+                       /*
+                        * for (Status s : tweetList) { logManager.add(s); }
+                        */
+                       logManager.add(tweetList);
                } catch (IOException e) {
                        e.printStackTrace();
                }
@@ -635,18 +635,18 @@ public class TweetManager {
                        sinceTweetID = statuses.get(0).getId();
 
                        // 一番新しいtweetを一番したに持ってくる
-                       //for (Status status : statuses) {
-                       //      tweetList.add(0, status);
-                       //}
-                       Collections.reverse( statuses );
-                       tweetList.addAll( statuses );
+                       // for (Status status : statuses) {
+                       // tweetList.add(0, status);
+                       // }
+                       Collections.reverse(statuses);
+                       tweetList.addAll(statuses);
                }
                // ログ保存
                try {
-//                     for (Status s : tweetList) {
-//                             logManager.add(s);
-//                     }
-                       logManager.add( tweetList );
+                       // for (Status s : tweetList) {
+                       // logManager.add(s);
+                       // }
+                       logManager.add(tweetList);
                } catch (IOException e) {
                        e.printStackTrace();
                }
@@ -1056,7 +1056,8 @@ public class TweetManager {
                Configuration conf = this.getTwitterConfiguration();
                twitter = new TwitterFactory(conf).getInstance();
                // access token取得
-               AccessToken oAuthAccessToken = twitter.getOAuthAccessToken(username, password);
+               AccessToken oAuthAccessToken = twitter.getOAuthAccessToken(username,
+                               password);
                this.accessToken = oAuthAccessToken.getToken();
                this.secretToken = oAuthAccessToken.getTokenSecret();
 
@@ -1072,15 +1073,11 @@ public class TweetManager {
         * メッセージをつぶやく
         * 
         * @param message
+        * @throws TwitterException
         */
-       public void tweet(String message) {
+       public void tweet(String message) throws TwitterException {
                twitter4j.Status status;
-               try {
-                       status = twitter.updateStatus(message);
-               } catch (TwitterException e) {
-                       // TODO: エラー処理
-                       e.printStackTrace();
-               }
+               status = twitter.updateStatus(message);
        }
 
        /**
@@ -1088,18 +1085,14 @@ public class TweetManager {
         * 
         * @param message
         * @param replyToStatusID
+        * @throws TwitterException 
         */
-       public void replyTweet(String message, long replyToStatusID) {
+       public void replyTweet(String message, long replyToStatusID) throws TwitterException {
                twitter4j.Status status;
-               try {
-                       // status = twitter.updateStatus(message, replyToStatusID);
-                       StatusUpdate updateMsg = new StatusUpdate(message);
-                       updateMsg.setInReplyToStatusId(replyToStatusID);
-                       status = twitter.updateStatus( updateMsg );
-               } catch (TwitterException e) {
-                       // TODO: エラー処理
-                       e.printStackTrace();
-               }
+               // status = twitter.updateStatus(message, replyToStatusID);
+               StatusUpdate updateMsg = new StatusUpdate(message);
+               updateMsg.setInReplyToStatusId(replyToStatusID);
+               status = twitter.updateStatus(updateMsg);
        }
 
        /**