ブラックジャックのプログラムを作ったのですがAの判定を1と11でも判定するにはどうすれば良いでしょうか？

Question

コードpackage BJ;

import java.util.ArrayList;
import java.util.Random;
import java.util.Scanner;

public class BJ01 {


	static Scanner scan;

	public static void main(String[] args) {
		boolean flg = true;
		int playersum = 0;
		int cumsum = 0;

		scan =  new Scanner(System.in);

		ArrayList<String> playercard = new ArrayList<String>();
		ArrayList<String> comcard = new ArrayList<String>();

		for (int i = 0; i < 2; i++) {
			playercard.add(getcard("あなた"));
			comcard.add(getcard("ディーラー"));

		}
		System.out.println("");
		System.out.println("あなたの合計は" + cardsum(playercard) + "です。");
		System.out.println("ディーラーの合計は" + cardsum(comcard) + "です。");
		System.out.println("");

		while (true) {
			playersum = cardsum(playercard);
			System.out.println("現在の合計は" + playersum + "です。");
			System.out.println();
			if (playersum > 21) {
				break;
			}
			if(flg == false) {
				break;
			}

			flg = Key();
			if (flg == true) {
				playercard.add(getcard("あなた"));
			}
		}

		while(true) {
			if (cumsum <= 17 && playersum <= 21) {
				comcard.add(getcard("ディーラー"));
				cumsum = cardsum(comcard);
			}else {
				break;
			}
		}

		System.out.println("あなた「" + playersum + "」 ディーラー「" + cumsum +"」で");
		if (playersum > 21) {
			System.out.println("バースト！あなたの負けです。");
		}else if (cumsum > 21) {
			System.out.println("バースト！あなたの勝ちです。");
		}else if (playersum < cumsum) {
			System.out.println("あなたの負けです。");
		}else if (playersum > cumsum) {
			System.out.println("あなたの勝ちです。");
		}else {
			System.out.println("ドローです。");
		}
	}

	private static boolean Key() {

		while (true) {
			System.out.println("もう1枚カードを引きますか?(y/n) :");
			String yn = scan.next();

			if (yn.equals("y")) {
				return true;
			}else if (yn.equals("n")) {
				return false;
			}else {
				System.out.println("error: YまたはNを入力してください。");
			}
		}
	}

	private static String getcard(String turnplayer) {
		String[] cardlist = {"2","3","4","5","6","7","8","9","10","J","Q","K","A"};
		Random random = new Random();
		String card = cardlist[random.nextInt(13)];
		System.out.println(turnplayer + "に「" + card + "」が配られました。");

		return card;
	}
	private static int cardsum(ArrayList<String> cardlist) {
		int sum = 0;
		for(int i = 0; i < cardlist.size(); i++) {
			try {
				sum += Integer.parseInt(cardlist.get(i));

			}catch (NumberFormatException e) {
				if (cardlist.get(i).equals("J") || cardlist.get(i).equals("Q") || cardlist.get(i).equals("K"));

				sum += 10;
			}
		}

		for (int i = 0; i < cardlist.size(); i++) {

			if (cardlist.get(i).equals("A") && sum <= 10) {
				sum +=  11;

			}else if (cardlist.get(i).equals("A")){
				sum ++;
			}
		}
		return sum;
	}
}

エースのカードの処理の仕方が分からず困ってます！
エースのカードを1の場合と11の場合に処理したいのですが誰か教えて頂けたら助かります！
宜しく御願いします。
このコードの場合エースのカードは11で判定されてしまいます。
エースのカードを1と11でも判定処理できるようにしたいです。

Comments

[y_waiwai]

現行のコードではどういう動作になるんでしょうか。
質問を編集して追記しましょう

[ta.tatu]

追記しました。

[m.ts10806]

タイトルには要件を記載してください。
同じようにブラックジャックを作っていて何か別の問題が起きた時に同じタイトルで要件が伝わりますか？
なるべく同一人物による同一タイトルは避けるべきと思います。時間が経つと本人ですらタイトルから内容を思い出すことは出来ないでしょうし、同じ問題を抱えた誰かの参考になりません。

[ta.tatu]

分かりました！
これから気をつけます！

[m.ts10806]

今からでも変更されたほうが。

[ta.tatu]

変更しました。
指摘ありがとうございます。

Answer 1

  try {
    sum += Integer.parseInt(cardlist.get(i));
  }catch (NumberFormatException e) {
    if (cardlist.get(i).equals("J") || cardlist.get(i).equals("Q") || cardlist.get(i).equals("K"));
    sum += 10;
  }

catch の中の if 文が ";" で終わっているので, "A" の場合でも 10 足されてしまっています.

Comments

[ta.tatu]

ありがとうございます。
if文の”；”を無くせばいいのでしょうか？

[jimbe]

この if 文の目的は「 J,Q,K の時に sum に 10 を足す」ことでしょうから, ";" を消すか, sum += 10; を {} で囲む形にするかだと思います.

[ta.tatu]

有り難うございます！

Answer 2

Aを1とするか11とするかは、合計が決まったあとに決定するべきです。
一旦Aを1として計算し、合計が11以下でかつAがある場合に10を加算すればいいです。

Comments

[ta.tatu]

解答ありがとうございます！
参考にしてかんがえてみます！