回答編集履歴

コード等を追加

2019/03/04 16:29

投稿

jimbe

スコア12659

test CHANGED Viewed

@@ -43,3 +43,281 @@
 メインスレッドは常に使われております.
 OkHttp は, おそらく call.enqueue によってスレッドが使用されていると思います.
+---
+かなり豪快なコードになっていると思います.
+> //この１行を追加しました。
+> new MainActivity().CompAnswer(Looper.getMainLooper(),response.body().string());
+Activity は Android システムが管理していますので, 勝手に生成しても Activity としては機能いたしません. ですので findViewById が null を返しているのではないでしょうか.
+ 必要であれば okhttp3Client のパラメータとして受け取って使用してください.
+また逆に CompAnswer は Looper をパラメータとして受け取って Handler を生成していますが, CompAnswer は MainActivity のメソッドですので, MainActivity で先に生成済みの ```Handler handler = new Handler();``` の handler が使用できると思います.
+コード量と動作を考えますと, HttpClient クラスがあることによって動作イメージが崩れているのではないでしょうか. クラス分けは大事ですが, この場合 Http の処理は OkHttp が行っていますので, いっそ HttpClient クラスは無いほうが良いと思います.
+(ついでですが, 各ボタンのリスナを全て同じにした上でリスナ内でボタンにより分岐するのでは無く, 専用処理があるボタンには専用リスナを設定したほうが分かり易いと思います.)
+```java
+package com.example.hogehoge.calculater;
+import android.os.Bundle;
+import android.os.Handler;
+import android.support.v7.app.AppCompatActivity;
+import android.view.View;
+import android.widget.Button;
+import android.widget.TextView;
+import java.io.IOException;
+import okhttp3.*;
+public class MainActivity extends AppCompatActivity {
+  TextView formulaTextView;
+  Handler handler = new Handler();
+  @Override
+  protected void onCreate(Bundle savedInstanceState) {
+    super.onCreate(savedInstanceState);
+    setContentView(R.layout.activity_main);
+    formulaTextView = (TextView) findViewById(R.id.formula);
+    View.OnClickListener NumListener = new View.OnClickListener() {
+      @Override
+      public void onClick(View view) {
+        formulaTextView.append(((Button)view).getText());
+      }
+    };
+    findViewById(R.id.num1).setOnClickListener(NumListener);
+    findViewById(R.id.num2).setOnClickListener(NumListener);
+    findViewById(R.id.num3).setOnClickListener(NumListener);
+    findViewById(R.id.num4).setOnClickListener(NumListener);
+    findViewById(R.id.num5).setOnClickListener(NumListener);
+    findViewById(R.id.num6).setOnClickListener(NumListener);
+    findViewById(R.id.num7).setOnClickListener(NumListener);
+    findViewById(R.id.num8).setOnClickListener(NumListener);
+    findViewById(R.id.num9).setOnClickListener(NumListener);
+    findViewById(R.id.num0).setOnClickListener(NumListener);
+    findViewById(R.id.coron).setOnClickListener(NumListener);
+    findViewById(R.id.add).setOnClickListener(NumListener);
+    findViewById(R.id.substract).setOnClickListener(NumListener);
+    findViewById(R.id.divide).setOnClickListener(NumListener);
+    findViewById(R.id.multiply).setOnClickListener(NumListener);
+    findViewById(R.id.NEXT_INT).setOnClickListener(new View.OnClickListener() {
+      @Override
+      public void onClick(View view) {
+        formulaTextView.append(" ");
+      }
+    });
+    findViewById(R.id.Clear).setOnClickListener(new View.OnClickListener() {
+      @Override
+      public void onClick(View view) {
+        formulaTextView.setText("");
+      }
+    });
+    findViewById(R.id.equal).setOnClickListener(new View.OnClickListener() {
+      @Override
+      public void onClick(View view) {
+        okhttp3Client(formulaTextView.getText().toString());
+      }
+    });
+    findViewById(R.id.BACK).setOnClickListener(new View.OnClickListener() {
+      @Override
+      public void onClick(View view) {
+        Button button = (Button) view;
+        if (formulaTextView.length() > 0 && formulaTextView.getText() != null) {
+          String temp = formulaTextView.getText().toString();
+          temp = temp.substring(0, temp.length() - 1);
+          formulaTextView.setText(temp);
+        }
+      }
+    });
+  }
+  void okhttp3Client(final String formula) {
+    String url = "http://hogehoge.site";
+    OkHttpClient client = new OkHttpClient();
+    try {
+      Request request = new Request.Builder()
+              .url(url)
+              .post(RequestBody.create(MediaType.parse("text/plain;charset=utf-8"), formula))
+              .build();
+      Call call = client.newCall(request);
+      call.enqueue(new Callback() {
+        @Override
+        public void onFailure(Call call, IOException e) {
+          //エラー処理を書く
+        }
+        @Override
+        public void onResponse(Call call, Response response) throws IOException {
+          //この１行を追加しました。
+          compAnswer(response.body().string());
+        }
+      });
+    } catch(Exception e) {
+    }
+  }
+  //以下のメソッドを新しく追加しました。
+  private void compAnswer(final String Answer) {
+    handler.post(new Runnable() {
+      @Override
+      public void run() {
+        //このテキストビューまでは到達できました。
+        //実行しようとするとアプリ自体が落ちることは確認できました。
+        formulaTextView.setText(Answer);
+      }
+    });
+  }
+}
+```

記述間違い

2019/03/04 16:29

投稿

jimbe

スコア12659

test CHANGED Viewed

@@ -12,7 +12,7 @@
-この run() とは Handler で実行したい処理のことでしょうか. でしたら, Runnable を実装したクラスの中に書かなければなりません. そのクラスのインスタンスを Thread の post メソッドに渡す形になります.
+この run() とは Handler で実行したい処理のことでしょうか. でしたら, Runnable を実装したクラスの中に書かなければなりません. そのクラスのインスタンスを Handler の post メソッドに渡す形になります.
 サブスレッドで…というのは意味が分かりません.
@@ -26,7 +26,7 @@
-> そのrun()に到達すると、メインスレッドのHandlerに渡ってrun()の中に書いてあったことが実行される…？
+> そのrun()に到達すると、メインスレッドの Handler に渡ってrun()の中に書いてあったことが実行される…？