CSSとJavascriptによるスタイルの変更について

Question

前提・実現したいこと

Javascriptでバリデーションを行い、その結果(エラーがあるか否か)によって題目の色を変更しようとしています。

発生している問題・エラーメッセージ

コンソールによるとエラーは出ておらず、かつ引数となる変数の値も希望通りになっているのですが
色を変えるための処理は「エラーが有るときには行われるが無い時には行われない」という状況です。
尚、この関数は別の部分で作った処理をメソッドで一般化しているのですが、
元々作ったものは問題なく色が変わります。

該当のソースコード

<p class="controller hidecontents has" id="hoge" onclick="this.classList.toggle('hidecontents')">foobar</p>
    <div class="hidden_show" >
      <!--非表示ここから-->
  <div id="hoge_choice_valid"></div>
  <p>
  <input type="radio" name="hoge_choice" value="5" id="hoge_choice5" ><label for="hoge_choice5"></label><br>
<!--中略-->
  <input type="radio" name="hoge_choice" value="0" id="hoge_choice0" ><label for="hoge_choice0"></label><br>
  </p>

  <div id="hoge_written_valid"></div>
    <p>
  <textarea style="width:100%;" name="hoge_written" rows="8" ></textarea></p>
      <!--ここまで-->
</div><br>

p.has.errorshow {
    display:inline-block;
    padding: 15px;
    font-weight: bold;
    border: solid 2px black;
    cursor :pointer;
    background:#ff2800;
}
p.has.noerror {
    display:inline-block;
    padding: 15px;
    font-weight: bold;
    border: solid 2px black;
    cursor :pointer;
    background:#35a16b;
}

var $ID_hoge="hoge";
function errordetect_mini($message) {
  var $flag="N";
  if ($message.match(/<p style=/)) {
    $flag="T";
    }else if($message.match(/valid"></div>/)){
      $flag="F";
    }
    return $flag;
}
function errordetect_joint($bool1,$bool2) {
if ($bool1==="T"||$bool2==="T") {
      return "T";
    }else{
      return "F";
    }
}

function errordetect_generic($ID,$bool) {
  if ($bool==="T") {
    document.getElementById($ID).classList.add('errorshow');
    if (document.getElementById($ID).classList.contains('noerror')) {
      document.getElementById($ID).classList.remove('noerror');
    }else if($bool==="F"){
    document.getElementById($ID).classList.add('noerror');
    if (document.getElementById($ID).classList.contains('errorshow')) {
      document.getElementById($ID).classList.remove('errorshow');
    }
  }
}
}
function valid_hoge_choice($hoge_choice) {
  $error_hoge_choice="";
  if (!$hoge_choice&&$hoge_choice!=0) {
    $error_hoge_choice = "<div id=\"hoge_choice_valid\"><p style=\"color:#ff2800;\">"+"入力されていません。"+"</p></div>";
  }else if ((parseInt($hoge_choice)<0||parseInt($hoge_choice)>5)&&!isNumeric($hoge_choice)) {
        $error_hoge_choice = "<div id=\"hoge_choice_valid\"><p style=\"color:#ff2800;\">"+"不正なPOSTです。"+"</p></div>";
    }else{$error_hoge_choice="<div id=\"hoge_choice_valid\"></div>";}
  return $error_hoge_choice;
}

function valid_hoge_written($hoge_written) {
  $error_hoge_written="";
  if ($hoge_written==="") {
    $error_hoge_written = "<div id=\"hoge_written_valid\"><p style=\"color:#ff2800;\">"+"入力されていません。"+"</p></div>";
  }else if (chop($hoge_written)==="") {
        $error_hoge_written = "<div id=\"hoge_written_valid\"><p style=\"color:#ff2800;\">"+"不正なPOSTです。"+"</p></div>";
    }else{$error_hoge_written="<div id=\"hoge_written_valid\"></div>";}
  return $error_hoge_written;
}
$(function(){
       $("[name=hoge_choice]").change(function(){
       $hoge_choice=$("[name=hoge_choice]:checked").val();
        $('#hoge_choice_valid').html(valid_hoge_choice($hoge_choice));
       $error_hoge_choice_=valid_hoge_choice($hoge_choice);
       $error_hoge_written_=valid_hoge_written($hoge_written);
       var $bool1=errordetect_mini($error_hoge_choice_);
       var $bool2=errordetect_mini($error_hoge_written_);
       var $bool=errordetect_joint($bool1,$bool2);
       errordetect_generic($ID_hoge,$bool);
      });
       $("[name=hoge_written]").change(function(){
       $hoge_written=$("[name=hoge_written]").val();
        $('#hoge_written_valid').html(valid_hoge_written($hoge_written));
       $error_hoge_choice_=valid_hoge_choice($hoge_choice);
       $error_hoge_written_=valid_hoge_written($hoge_written);
       var $bool1=errordetect_mini($error_hoge_choice_);
       var $bool2=errordetect_mini($error_hoge_written_);
       var $bool=errordetect_joint($bool1,$bool2);
       errordetect_generic($ID_hoge,$bool);
      });
});

補足情報（FW/ツールのバージョンなど）

jQuery:3.2.0

Answer 1

関数１つずつ見ただけです。処理の流れは見ておりません。

function errordetect_generic($ID,$bool) {
  if ($bool==="T") {
    document.getElementById($ID).classList.add('errorshow');
    if (document.getElementById($ID).classList.contains('noerror')) {
      document.getElementById($ID).classList.remove('noerror');
    }else if($bool==="F"){
    document.getElementById($ID).classList.add('noerror');
    if (document.getElementById($ID).classList.contains('errorshow')) {
      document.getElementById($ID).classList.remove('errorshow');
    }
  }
}
}

とりあえず、このfunctionの中身が変に見えました。

かなり省略して書くと、

if ($bool==="T") {
  if($bool==="F"){
  }
}

こういう記述があることになっています。

通らないですよね。中のif文・・・