PHP関数の返り値のエラー

Question

0から127までの10進数を受け渡すと、教科を返す関数を作成しましたが、返り値が正しくありません。

例えば120を受け渡すと本来”国語（小学校）算数（小学校） 国語（中学校） 数学（中学校）”で返ってくるべきところ、”国語（小学校）国語（中学校） 数学（中学校）”と一部飛ばされてしまいます。飛ばされる教科の法則性を見いだせていません。

//  subjectの10進数から指導教科（文字列）を返す
function check_subject($subject){
 $bi_subject = decbin($subject);
 $es_jamanese       = 0b1000000;
 $es_mathmatics     = 0b100000;
 $js_jamanese       = 0b10000;
 $js_mathmatics     = 0b1000;
 $js_science        = 0b100;
 $js_social_studies = 0b10;
 $js_english        = 0b1;
 if($bi_subject & $es_jamanese){
   $str_subject .= "国語（小学校）";
 }
 if($bi_subject & $es_mathmatics){
   $str_subject .= " 算数（小学校）";
 }
 if($bi_subject & $js_jamanese){
   $str_subject .= " 国語（中学校）";
 }
 if($bi_subject & $js_mathmatics){
   $str_subject .=" 数学（中学校）";
 }
 if($bi_subject & $js_science){
   $str_subject .=" 理科（中学校）";
 }
 if($bi_subject & $js_social_studies){
   $str_subject .=" 社会（中学校）";
 }
 if($bi_subject & $js_english){
   $str_subject .=" 英語（中学校）";
 }
 //$str_subject = trim($str_subject);
 if(empty($str_subject)){
   return NULL;
 }else{
   return $str_subject;
 }
}

Comments

[退会済みユーザー]

とりあえず128パターンぐらい自分でテストロジックを書きましょう

[m.ts10806]

それぞれの関数の呼び出し元と提示されていない関数の処理も提示してください。きちんとデバッグはしたんですよね？

[m.ts10806]

ちなみに「エラー」ではないですよね。自身がそのように組んでしまっているからに他なりません。

Answer 1

120をdecbinで"1111000"という文字列に変換してしまっていますが、これは&演算の時に10進数の1111000とみなされます。2進数であると言う情報がどこにも無いので、そうなります。
これは0b100001111001111011000なので、お書きの結果になります。

hichonさん指摘のように、decbinで文字列化したのが間違い。

Comments

[DaisukeKusakari]

詳細な解説をいただきありがとうございます。
後学になりました。

Answer 2

つ$bi_subject = decbin($subject); → $bi_subject = $subject;
ビット演算するのに文字列に変換するのは余計です。

Comments

[DaisukeKusakari]

ご教示いただきありがとうございます。