From: spark_xp <spark_xp@d8c9ecd3-d47d-4367-8645-de82c00e513f>
Date: Mon, 2 May 2011 08:56:13 +0000 (+0000)
Subject: 140文字以上のツイートを送信したとき、エラーが表示されない問題を解決 refs #23797
X-Git-Tag: v0.6~83
X-Git-Url: http://git.sourceforge.jp/view?a=commitdiff_plain;h=aedb4638dc7c841ece27e3664baa66c4065904ff;p=nt-manager%2Fnt-manager.git

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
--- 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
--- 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);
 	}
 
 	/**