yunyun_engineer
@yunyun_engineer (yunyun)

Are you sure you want to delete the question?

Leaving a resolved question undeleted may help others!

iOS上でjavascriptが一部だけNaNの値を返します

Q&A

Closed

解決したいこと

PCのブラウザ(Chrome、Safari)では正しく残り日数が表示されるのですが、iPhone12Pro上(Chrome、Safari、Brave)のブラウザではトップページの表示はされるのですが、詳細画面で正しく表示されません。(下記画像参照)
Android端末等、他の環境では確認しておりません。

何が原因でしょうか?

発生している問題・エラー

IMG_4780.PNG
IMG_4779.PNG

該当するソースコード

正しく表示される

window.addEventListener('load', () => {
  const arrjsonInput = document.getElementById("arr_json");
  for (let i = 0; i < arrjsonInput.value ; i++) {
    const expirationInput = document.getElementById("expiration-date"+i);
    const today = new Date();
    const expirationDate = new Date(expirationInput.innerHTML);
    const daysLeft = document.getElementById("days-left"+i);
    daysLeft.innerHTML = (Math.round( (expirationDate.getTime() - today.getTime()) / ( 1000 * 60 * 60 * 24 )));
    if (daysLeft.innerHTML <= 0) {
     const deadLine = document.getElementById("dead-line"+i);
     deadLine.setAttribute("style", "color:#7945F5;text-decoration:underline #7945F5;");
     daysLeft.setAttribute("style", "color:#7945F5;text-decoration:underline #7945F5;");
    };
  };
});

NaNと表示される↓

window.addEventListener('load', () => {
  const expirationInput = document.getElementById("expiration_date");
  const today = new Date();
  const expirationDate = new Date(expirationInput.innerHTML);
  const daysLeft = document.getElementById("days_left");
  daysLeft.innerHTML = (Math.ceil( (expirationDate.getTime() - today.getTime()) / ( 1000 * 60 * 60 * 24 )));
 if (daysLeft.innerHTML <= 0) {
     const deadLine = document.getElementById("dead_line");
     deadLine.setAttribute("style", "color:#7945F5;text-decoration:underline #7945F5;");
     daysLeft.setAttribute("style", "color:#7945F5;text-decoration:underline #7945F5;");
    };
});

show.html.erb


 <% if user_signed_in? && current_user.id == @item.user_id %>
        <div class="item-manage">
          <%= link_to "編集", edit_item_path(@item), class: :item__btn %>
          <%= link_to "削除", item_path(@item), method: :delete, class: :item__btn %>
        </div>
      <% end %>
      <div class="item_block">
        <div class="item_card">
            <div class="image-content">
              <%= link_to image_tag(@item.image.variant(resize: '200x200'), class: :card__img ) %>
            </div>

            <div class="daysleft-detail">
                <span id="dead_line">残り<span id="days_left"></span>日</span>
            </div>

            <div class="item-detail">
              <p class="detail__title">名前</p>
              <p class="detail__message">
                <%= @item.name %>
              </p>
            </div>
            <div class="item-detail">
              <p class="detail__title">数量</p>
              <p class="detail__message">
                <%= @item.quantity %>
              </p>
            </div>
            <div class="item-detail">
              <p class="detail__title">購入日</p>
              <p id="purchase_date">
                <%= @item.purchase_date %>
              </p>
            </div>
            <div class="item-detail">
              <p class="detail__title">賞味期限</p>
              <p id="expiration_date">
                <%= @item.expiration_date %>
              </p>
            </div>

            <% if @item.memo.present? %>
            <div class="item-detail">
              <p class="detail__title">メモ</p>
              <p class="detail__message">
                <%= @item.memo %>
              </p>
            </div>
            <% end %>
            <div class="nickname-content">
              <%= "by #{@item.user.nickname}" %>
            </div>

自分で試したこと

問題は-かなと思いましたが、replaceの使い方もよくわかりませんでした。

var dt = new Date(at.replace(/([\+|\-])([0-9]{2})([0-9]{2})$/,"$1$2:$3"));
now=new Date(d.replace(/-/g,"/"));
0

3Answer

PCのブラウザで見るとNaNにならないが、iOSのブラウザで見るとNaNになる、ということでしょうか?

ブラウザというのはウェブページを表示するものなので、PCやスマートフォン(Android、iOS)といった端末に関わらずブラウザです。
特定の環境によるものは、端末の種類(PC、スマートフォン、タブレット等)やブラウザの種類(Chrome、Safari、Edge等)について記載があると良いと思います。

あと、HTMLの部分も情報が欲しいところです。

今の段階で言えるのは、一行ずつ結果を出力して確認していく、でしょうか。
あと気になるのは、正しく表示されている方はMath.roundを使っていますが、NaNになる方はMath.ceilを使っている点です。

キャッシュが影響していることも考えられるので、キャッシュを消して確認するのも良いと思います。

1Like

Comments

  1. コメントありがとうございます!
    キャッシュ消去して試しましたが変わらずです。
    ceilをroundに置き換えましたが、こちらも駄目でした。
    ご指摘の件、本文に追加記載しました。ありがとうございます!
  2. 追加の部分を拝見しましたが、残念ながら、これというものは見つけられませんでした。

    私なら、まずどの部分で違いが出ているのかを探します。
    例えばこれらの値を出力してみます。

    - today.getTime()
    - expirationInput.innerHTML
    - expirationDate.getTime()

    また、直接テキストを入れてみて正しく動くか確認してみます。

    const expirationDate = new Date(expirationInput.innerHTML);

    const expirationDate = new Date('2021-05-24');
  3. やはりご指摘の通り
    expirationDateがInvalid Dateでうまくデータを取得できてませんでした。
    expirationInput.innerHTMLまでは取得できています。
    つまりnew Date 自体が機能してないのでどうしようもありません。
    何故か一覧表示の方はexpirationDateの値は取得できていました、、、
    ありがとうございました

推測ですが、おそらく expirationInput.innerHTML が日付として無効な形式なのだと思います。
サポートしている日付の形式はブラウザによって異なるので、規定されている形式以外のものはブラウザによってうまく動かないことがあると思います。

まずは、ブラウザで該当のページを開いて expirationInput の具体的な値がなにか確認してみてください。

あるいは、ソースに console.log("[" + expirationInput.innerHTML + "]"); のようにして内容を確認してみる方法あると思います。

例えば、PCのChromeとiPhoneのSafariで確認したところ以下のようになりました(前後のスペースの違い)。

PCのChrome
new Date('2021-05-05').getTime(); // OK (スペースなし)
new Date(' 2021-05-05').getTime(); // OK (前にスペース)
new Date('2021-05-05 ').getTime(); // OK (後ろにスペース)
iPhoneのSafari
new Date('2021-05-05').getTime(); // OK (スペースなし)
new Date(' 2021-05-05').getTime(); // OK (前にスペース)
new Date('2021-05-05 ').getTime(); // NaN (後ろにスペース)

なので、スペースの有無も含めて確認したほうが良いと思います。

1Like

Comments

  1. ご丁寧な回答ありがとうございます!感激です
     iPhoneでsafari検証ツールで一つ一つ見てみます!
  2. やはりご指摘の通り
    expirationDateがInvalid Dateでうまくデータを取得できてませんでした。
    expirationInput.innerHTMLまでは取得できています。
    つまりnew Date 自体が機能してないのでどうしようもありません。
    何故か一覧表示の方はexpirationDateの値は取得できていました、、、
    ありがとうございました
  3. 上でも書いたのですが、半角スペースが含まれていることはありませんか?
    もし含まれているなら html 側から削除するか、javascript 側で trim() すればよいと思います。

    const expirationDate = new Date(expirationInput.innerHTML.trim());
  4. const expirationDate = new Date(expirationInput.innerHTML.trim());

    これで解決できました。PC、スマホすべてで表示できました。
    本当に感動しました。ありがとうございます!
    大変勉強になりました。感謝してもし尽くせません!

expirationDateがInvalid Dateでうまくデータを取得できてませんでした。
expirationInput.innerHTMLまでは取得できています。
つまりnew Date 自体が機能してないのでどうしようもありません。

他の部分では機能していることからnew Data()自体に問題があるとは考えにくいですね。
となると値の方に問題があると思われるので、他の方の回答にもあるように、余分なスペースや改行コードなどが含まれている可能性が疑われます。
その場合、不要な文字をトリムするのが直接的な解決方法になります。

他には見せるデータと処理用のデータを別にする、という考えでデータ属性を使用します。
これによりHTMLの改行やインデント、日付のフォーマット変更に依存しない形になります。

あるいはJavaScriptにこだわらないのであれば、サーバサイド(Ruby?)で日付を計算して表示する、という方法もあります。

1Like

Comments

  1. コメントありがとうございます
    初学者なのですみません、そうなのですね

    色々な方法があるのですね

    まずはもう一度インデント等を調べてみます!
  2. .trim()
    ご指摘の通りトリムで解決できました。本当にありがとうございました。
    データ属性、Rubyでの日付計算等
    こちらもこ教示いただきありがとうございます!
    勉強してみます!

Your answer might help someone💌