jquery undefinedを表示させないようにしたい。

Question

前提・実現したいこと

jquery初心者です。

undefindの表記を表示させないようにしたいのですが、うまく非表示にすることが来ません。
どなたかアドバイスを頂けないでしょうか？

また、この書き方の場合、もっとこう書いたほうがいいなどあれば、ぜひよろしくおねがいします。

jqueryを使って投稿の印刷機能を作ろうとしています。
ボタン一つでポストのようなものを印刷できるようにしています。
印刷ボタンを押すとモーダルが出てきて、ラジオボタンでフレームを選択、選択したフレームにtextを配置し印刷するようにしています。

該当のソースコード

$(function() {
  $('.print').on('click', function () {
    var one      = $('table td')[1];
    var two      = $('table td')[3];
    var three    = $('table td')[5];
    var four     = $('table td')[7];
    var five     = $('table td')[9];
    var six      = $('table td')[11];
    var text1    = document.getElementById("content");
    var image    = document.getElementById('image');
    var htmldoc  = document.body.innerHTML;

    $('button').click(function () {
      var radio = $('input[name="radio"]:checked').val();
      if (radio === 'frameone') {
        frm = '<%= image_tag "00.png", style:"width: 100%;" %>';
      } else if (radio === 'frametwo') {
        frm = '<%= image_tag "01.png", style:"width: 100%;"%>';
      };

      var text2 = "<div class='tdtop'>" + one + "</div>" +
      "<div class='tdcell'>" + two + "</div>" +
      "<div class='tdcell'>" + three + "</div>" +
      "<div class='tdcell'>" + four + "</div>" +
      "<div class='tdcell'>" + five + "</div>" +
      "<div class='tdcell'>" + six + "</div>";

      if ( text2 == 'undefined' ){
        text2 == "";
      };

    window.document.body.innerHTML =
      "<div>"  + frm +
      text1.innerHTML + "</strong>" +
      "<div>"  + image.innerHTML + "</div>" +
      text2.innerHTML +
      "</div>";
    window.print();
    document.body.innerHTML = htmldoc;
    location.reload();
    });
  });
});

試したこと

ググってみて、if文で分岐して空の要素（””）を入れるなど試したところ、うまくいきませんでした。

補足情報

jquery　1.12.4
Rails 5.1.4
Ruby 2.4.2

Comments

[kei344]

回答が付いた質問の編集は慎重に行ってください。質問文のコードについて回答にて指摘があった場合は「追記」し、元のコードを編集する場合も「直したこと」がわかるようにしてください。

Answer 1

textはundefに限らずnullも除外したいですよね。
以下の書き方がスマートかなと。

text = text || "";

textをbooleanにしたときにtrueならそのままtextを使うし、falseなら ""を使うようになります。

Comments

[NagaMind]

非表示にできました！ありがとうございます！

Answer 2

text2って文字列じゃないですか？
文字を出力したいならinnerHTMLいらない気がします。

Comments

[NagaMind]

アドバイスありがとうございます！

Answer 3

下記部分、nameからtechniqまで、定義していない変数があるのが問題なのでは？

      var text2 = "<div class='tdtop'>" + name + "</div>" +
      "<div class='tdcell'>" + date + "</div>" +
      "<div class='tdcell'>" + type + "</div>" +
      "<div class='tdcell'>" + size + "</div>" +
      "<div class='tdcell'>" + place + "</div>" +
      "<div class='tdcell'>" + techniq + "</div>";

あと、上記でtext2 に代入していながら text と比較しているのも変です。（どちらにせよこれでは比較できませんが）

      if ( text == 'undefined' ){
        text == "";
      };

Comments

[NagaMind]

回答ありがとうございます！
codeの書き換えを間違えていたため、書き直しました！
ご指摘ありがとうございます。