jsonに文字列値が含まれていないか確認してください。

Question

文字列がjsonかどうかを確認できるようにしようとしています。私がやっているやり方は、jsonの値を配列に入れて、値が入っているかどうかを調べることです。問題は、それが1つずつチェックすることです。これにより、ユーザはシフトを行っていないと言います。その後、次のループで、彼らは働いていると言います。

private void CheckIfAlreadyWorking(String result) throws JSONException { 
    //removed code to condense 
    //result is a json of the days that the user is already working 
    if (datematcher.find()) { 
     String date = datematcher.group(1); //Date of the shift the user is already working 
     JSONArray jsonArray = new JSONArray(result); 
     String[] yourshifts = new String[jsonArray.length()]; 
     boolean end = false; 
     for (int i = 0; i < jsonArray.length(); i++) { 
       JSONObject obj = jsonArray.getJSONObject(i); 
       yourshifts[i] = obj.getString("date"); 
      if (yourshifts[i].contains(date)) { 
       //Already working that day 
       Toast.makeText(getApplicationContext(), "Error: You are already working this day", Toast.LENGTH_SHORT).show(); 
      } else if(end == false){ 
       AddShift(); 
       end = true; 
      } 
     } 

    } 
    else { 
     //Error 
    } 
} 

Answer 1

試してみてください。 ループの外で今boolean isfound = false;

for (int i = 0; i < jsonArray.length(); i++) { 
       JSONObject obj = jsonArray.getJSONObject(i); 
       yourshifts[i] = obj.getString("date"); 
      if (yourshifts[i].contains(date)) { 
       isfound = true; 
       break; 

      } 
     } 

してください：

if(isFound ==true) 
{ 
    Toast.makeText(getApplicationContext(), "Error: You are already working this day", Toast.LENGTH_SHORT).show(); 
} 
else 
{ 
    AddShift(); 
} 

Answer 2

あなたは単純にブレークを使用できます。あなたの作ったトーストの直後にあなたのコードののステートメント。だから、

は場合、彼らはあなたのためのループを終了しますトーストにbreak文を作った後、その持ち運びにくい上で作業している場合は、あなたjsonArrayこのよう

に残っているオブジェクトを通過しません。

if (yourshifts[i].contains(date)) { 
    //Already working that day 
    Toast.makeText(getApplicationContext(), "Error: You are already working this day", Toast.LENGTH_SHORT).show(); 
    break; 
} 

このよう