質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.32%

  • JavaScript

    17569questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    9600questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

フィルタ検索機能を追加:2

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 236

 前提・実現したいこと

再度、お世話になります。
fc2カートにてショッピングページを運営しております。
先日、ここでご教授頂いた方法に基づいてチェックボックスによるフィルタ検索機能を追加しましたが、他のcheckboxの機能と競合してしまったらしく、
検索機能と他のcheckboxによる機能が両立できる解決策を探しております。

htmlは書き換えられますが、大元のショップのjavascriptは仕様で書き換えられないので、
別ファイルにして読み込ませようと考えております。
アドバイスいただけましたら、お願いいたします。

なお、/?word=AB+CD+{テキストボックスの語句}という結果になるのであれば、
checkboxにての機能追加にはこだわっておりません。

大元のショップのスクリプトファイルのコードですが、
リンクを貼れるようですので、掲載いたします。
こちら

実験中のショップのURLも掲載いたします。
こちら

他にも何か必要なものがありましたら、ご連絡くださいませ。

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

「https://xxx.com//?word=」の後に、「AB+CD+{テキストボックスの語句}」にするという機能自体はとても上手く機能しているのですが、
メールの送信ボタンやカートに入れるボタンなど、他のcheckbox機能を阻害している状況です。

エラーメッセージ

firefox:

TypeError: can't convert undefined to object kissdesigndoll.cart.fc2.com:147:9
map self-hosted:282:13 <匿名関数>
https://xxx.cart.fc2.com/:147:9

ページ内該当コード
147  var v=[].map.call(t.elements["w"],function(x){
148    return (x.type=="text" || x.type=="checkbox"&&x.checked)?x.value:"";
149  }).filter(function(x){return x!==""}).join(" ");

 該当のソースコード

詳細検索のhtmlとjavascript

実際に使用しているコードは長いため、同じ仕様の短いものを記述しております。
<form method="get" action="{search.url}">
    <span class="menusp">ああああ</span>
    <div class="menus1">
        <div>
            <a><label><input class="checkbox" name="w" value="AB" type="checkbox"><span>AB</span></label></a>
            <a><label><input class="checkbox" name="w" value="CD" type="checkbox"><span>CD</span></label></a>
        </div>
        <div>
            <a><label><input class="checkbox" name="w" value="EF" type="checkbox"><span>EF</span></label></a>
            <a><label><input class="checkbox" name="w" value="GH" type="checkbox"><span>GH</span></label></a>
        </div>
    </div>
    <div class="searchbox">
        <input name="w" value="" type="text">
        <input value="さがす" class="submit" type="submit">
    </div>
</form>
document.addEventListener('submit',function(e){
  var t=e.target;
  e.preventDefault();
  var f=document.querySelector('#f0');
  if(f) f.parentNode.removeChild(f);
  var v=[].map.call(t.elements["w"],function(x){
    return (x.type=="text" || x.type=="checkbox"&&x.checked)?x.value:"";
  }).filter(function(x){return x!==""}).join(" ");
  var f=document.createElement("form");
  f.setAttribute("id","f0");
  f.setAttribute("action",t.getAttribute("action"));
  f.setAttribute("method",t.getAttribute("method"));
  f.style.display="none";
  var word=document.createElement("input");
  word.type="text";
  word.name="word";
  word.value=v;
  f.appendChild(cword);
  document.querySelector('body').appendChild(f);
  f.submit();
});


例としてカートに入れるボタンのhtml(javascriptはこちらを参照ください。)

<dd class="submit">
    <input value="カートに入れる" class="btn_submit_cart" type="submit">
    <input name="Submit_cart" value="true" type="hidden">
    <input name="item_id" value="511" type="hidden">
</dd>

 試したこと

変数のバッティングかと思い、t,f,x,wordなどの変数を他の単語に変えてみたりしましたが、やはり同じ結果でした。

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

環境:PCブラウザのfirefox,chrome、スマホブラウザのchromeにて状況確認

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • mts10806

    2018/07/26 06:33

    JavaとJavaScriptは似て非なるものです。質問内の表記を調整してください。

    キャンセル

  • kissdesign_shop

    2018/07/27 01:13

    修正しました

    キャンセル

回答 1

checkベストアンサー

+1

JavaScriptの該当行通りのエラーだと思います。148行目を確認すると、

return (x.type=="text" || x.type=="checkbox"&&x.checked)?x.value:"";


となっています。読み解くと、フォーム属性がチェックボックスであり、かつチェックされているかという判定なので、現状だと全部ここで処理されてしまうことになります。

なので、解決できるかもしれない方法としたら

  1. 別のCheckBox要素と同一Formタグ内に記述しない。
  2. そのスクリプト処理を行う前に、該当部分の処理を行い、そこで処理を中断するようにする

ソースの構造は詳しくわかっていないので確信はとれませんが、方法とすると、現在問題が発生しているスクリプトを読み込む前に別のスクリプトを読み込ませ、そこで処理を中断するのが一番適切な方法でしょうか。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/04 00:04 編集

    ご回答、ありがとうございます。
    他のCheckBox要素とは同一Formタグ内ではないですが、仰られるとおりここのフォーム内のみでスクリプトが処理されるような形になれば干渉しないとは思うのですが、どう記述していいのか困っております。

    キャンセル

  • 2018/08/06 13:29

    それなら、こういう手はどうでしょうか?

    まず、選択チェックボックスがあるファイルにスクリプトタグを設け、javascript内にonClickイベントと定義関数を記述、定義関数内で、新たにFormオブジェクトを作り、そこへsubmitしてみてはどうでしょうか?

    自分もちょっと検証テストしてみます。

    キャンセル

  • 2018/08/06 14:30

    例ですが、Formタグを使わずにチェックボックスの値を回収する書き方。このあとで任意のFormオブジェクトをdocument.createElement('form')で作ったり、hiddenにしてデータを投げたりする手もあるかな…と。
    <!doctype>
    <html lang="ja">
    <head>
    <script src="http://code.jquery.com/jquery-1.11.1.min.js"></script>
    </head>
    <body>
    <form >
    <label>東京</label><input type="checkbox" class="chk" name="chk" value="tokyo">
    <label>名古屋</label><input type="checkbox" class="chk" name="chk" value="nagoya">
    <label>大阪</label><input type="checkbox" class="chk" name="chk" value="osaka">
    <input type="button" name="bt" value="検索" onclick="selValue()">


    </form>

    <script>
    var selValue = function(){
    var chkflags = [];
    var chkcls = document.getElementsByClassName('chk');
    var len = chkcls.length;
    for(var i = 0 ; i < len; i++ ){
    if(chkcls[i].checked){
    chkflags[i] = true;
    }else{
    chkflags[i] = false;
    }
    }
    }
    </script>
    </body>
    </html>

    キャンセル

  • 2018/08/07 01:05

    再度のコメントありがとうございます。
    試してみましたが、どうもうまくいかないみたいで申し訳ないです。
    実験中の物ではありますが、ショップのリンクも記載いたしました。
    よろしければ見てみてください。

    今、使用している画像スライダーのように、特定のclassにのみ実行する形で記述できないか検討しております。

    キャンセル

  • 2018/08/07 09:55

    見たところ、form枠はしっかり分けられていますね。
    ただ、Formタグのアクションプロパティがこうなっています。

    action="/?tplid=570343&preview=66b3106462ff5ab63c1

    パラメータのみ書かれていて、肝心の送信先ファイル名の記述がないようですが…?

    action="/●●●?tplid=570343&preview=66b3106462ff5ab63c1

    こういう記述になるのが正しいはずだとは思います。カートの部分はcart_in?xxxxx…ってなっていますので。

    キャンセル

  • 2018/08/07 10:22

    あと、これってhtmlとJavaScriptのみのべた書きじゃないですよね?何かECサイトの機能借りたり、WordPressみたいなものを使ったりしていませんか?Actionの記述が何かそれっぽいので…

    キャンセル

  • 2018/08/09 13:07

    仰られるとおり、借り物です。
    お店ですので、支払い関係を考えると自前で組むのはちょっと危なかったので。
    こちらでもご意見を参考にいろいろ試したのですが、どうにも大元のスクリプトが干渉しますので、何か別の視点で方法を考えたいと思います。

    キャンセル

  • 2018/08/09 13:08

    お付き合いいただき、本当に感謝しております。
    いろいろとありがとうございました。

    キャンセル

同じタグがついた質問を見る

  • JavaScript

    17569questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • HTML

    9600questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。