A1~C3または1A~3Cまでの値を判定したい

Question

ゲームの座標判定でA1C3または1A3Cまでを座標判定したいのですが、下記の画像のようにうまく判定できていません。
また、このif文だと座標が増えた場合に書く量が膨大になってしまいます。
質問したいことは、
・なぜ下記のコードで判定できていないのか
・座標が増えた場合の今よりも簡単なif判定のアドバイス
をお願いします。

#include <stdio.h>
#include <string.h>

#define MAXDATA 255
#define DATA 4  //自分の環境ではAなどが2バイト、1などが1バイトのため、最後に¥nを含めた
　　　　　　　　　 // 4バイトにしています。（念のため、DATA 10などにもしてみましたがうまくいきませんでした）

int main(void) {
  char xy[DATA] = { };
  char flag = 0;
  char xy_last = 0;

  while (flag == 0) {
    printf("A1または1Aと入力してください\n");

    fgets(xy, MAXDATA, stdin);

    // 配列の最後を¥0にする(表示させる時の改行を防ぐ)
    xy_last = strlen(xy) - 1;
    xy[xy_last] = '\0';

    if (strlen(xy) <= DATA) {
      // (1~3, A~C)または、(A~C, 1~3)までで入力判定させたい
      if (((xy == "A1" || xy == "A2" == xy == "A3") || (xy == "1A" || xy == "2A" == xy == "3A") || (xy == "B1" || xy == "B2" == xy == "B3") || (xy == "1B" || xy == "2B" == xy == "3B") || (xy == "C1" || xy == "C2" == xy == "C3") || (xy == "1C" || xy == "2C" == xy == "3C"))) {
        printf("正しい入力です。\n");
        flag++;
      }
    }
  }

  return 0;
}

```

Comments

[kazuma-s]

> //自分の環境ではAなどが2バイト、1などが1バイトのため、

文字コードが Shift-JIS で、「Ａ］は全角文字で入力しているんですか？
また、「1」は、全角の「１」ではなく、半角の「1」ですか？

[gyoruo]

すみません、UTF-8なのでAも1バイトでした。1も半角なので1バイトです。

Answer 1

#include <stdio.h>  // puts, fgets, sscanf

#define MAXDATA 255

int main(void)
{
	char xy[MAXDATA], c;
	do {
		puts("A1または1Aと入力してください");
		fgets(xy, MAXDATA, stdin);
	} while ((sscanf(xy, "%*1[A-C]%*1[1-3]%c", &c) != 1 || c != '\n')
	      && (sscanf(xy, "%*1[1-3]%*1[A-C]%c", &c) != 1 || c != '\n'));
    xy[2] = '\0';
	puts("正しい入力です。");
}

見たことのない関数がありますか？

Comments

[gyoruo]

わざわざ書き直してくれたんですね！
ありがとうございます！
sscanfは見たことないですが概要などを調べて理解してみます！

Answer 2

xy == "A1" は、xy配列の先頭要素へのポインターと、(どこかに配置されている)"A1"という配列の先頭要素へのポインターが一致しているか、という判定です。
ご希望の処理になっているでしょうか？

Comments

[gyoruo]

ご回答ありがとうございます！
意味は理解できましたが、そのような処理では実現できないので他の方法を考えてみます。

Answer 3

・なぜ下記のコードで判定できていないのか

y_waiwaiさんの回答の通りです。

・座標が増えた場合の今よりも簡単なif判定のアドバイス

座標が16進文字列なら数値に変換すれば比較演算子で範囲チェックできますが、
飛び飛びの座標の場合ハッシュを作成してハッシュにあるか否かを判定した方が
可読性が高いと思います。
ハッシュ実装は以下をご参考に
お気楽Ｃ言語プログラミング超入門

Comments

[gyoruo]

そんな方法もあったんですね！
見てみます！

Answer 4

#include <stdio.h>
#include <string.h>
#include <stdbool.h>
#include <ctype.h>

#define DATA 4

int main(void) {
  char xy[DATA];
  bool invalid = true;

  while ( invalid ) {
    printf("A1または1Aと入力してください\n");

    fgets(xy, DATA, stdin);

    // 配列の最後を¥0にする(表示させる時の改行を防ぐ)
    xy[strlen(xy) - 1] = '\0';
    if ( strlen(xy) == 2 ) {
      char a = xy[0];
      char n = xy[1];
      if ( !isalpha(a) ) { char c = a; a = n; n = c; }
      // a が A,B,C のいずれか かつ n が 1,2,3 のいずれか ならば正しい入力
      if ( strchr("ABC",a) != NULL && strchr("123",n) != NULL ) {
        printf("正しい入力です。\n");
        invalid = false;
      }
    }
  }

  return 0;
}

Comments

[gyoruo]

ご回答ありがとうございます！
見たことない関数ばかりなので、1度調べてみますね！

[episteme]

標準関数は使わにゃ損ですから♪

Answer 5

C言語では文字列の直接の比較はできません
strcmp関数を使いましょう

　 xy == "2B" == xy == "3B"

また、こういう比較もできません

Comments

[gyoruo]

ご回答ありがとうございます！
strcmpを用いてできたのですが、defineや配列で"A1"などを定義すると、座標の数だけ定義しなければならないのですが、仕方のないことなのでしょうか？

[y_waiwai]

どういう座標の名付けになってるかわかってないですが、
Aを１００、Bを１１０とかにして、数値で座標を表すようにすれば、演算と比較で対応できます

[gyoruo]

ご回答ありがとうございます。
自分もそうしたかったのですが、仕様でA1と2次元配列を組み合わせないといけないらしく数値で座標指定ができないんです。。。