GoogleMAPで吹き出しをつけたい

Question

GoogleMAP APIを使っている。表示してあるGoogleMAPのピンをクリックした時吹き出しが出るようにしたい。

var marker = [];
var infoWindow = [];
function initMap() {
                                for (var i = 0; i < data.length; i++) {
                                        markerLatLng = new google.maps.LatLng(data[i]);
                                        marker[i] = new google.maps.Marker({
                                         position:markerLatLng,
                                         map: map
                                       });

                                     var infoWindow;
                                        infoWindow = new google.maps.InfoWindow({
                                            content: '<div class="sample">test</div>'
                                        });


                                        marker[i].addListener('click', function() {
                                            infoWindow.open(map, marker);
                                        });
                                 }

}

とコードを書いた。ピンは目的の位置に描画されたが、ピンをクリックしても吹き出しが出なかった。Uncaught TypeError: b.get is not a function　とエラーが出た。

var marker = [];
var infoWindow = [];
function initMap() {

　　　　　　　for (var i = 0; i < data.length; i++) {
                                var markers = data.map(function(location) {
                                  return new google.maps.Marker({
                                    position: location,
                                    map:map,
                                    icon:pinImage

                                  });
                                });

　　　　　　　markerEvent(data[i]);

}

          function markerEvent(i) {
                                    marker[i].addListener('click', function() {
                                      infoWindow[i].open(map, marker[i]);
                                  });
                                }
}

とコードを書いた時は、open is defined とエラーが出た。

どう直せば良いのか？

参考URL:
http://www.googleappsscript.info/2018-03-25/map_show_map_marker.html
https://www.tam-tam.co.jp/tipsnote/javascript/post7755.html

Comments

[Lhankor_Mhy]

2番目のコードで、for文がなくなっていたり、infoWindow変数に代入がされていないのは、コピペミスですか？

[k0908]

いえ、コピペミスではないです。このコードを書いていました

[k0908]

書き方が何通りかあるのだと思います

[Lhankor_Mhy]

このとおりだとすると、i 変数はどのように定義されていますか？

[k0908]

すみません、やっと意味がわかりました。コードを修正いたしました。もしお分かりでしたらお願いします

[Lhankor_Mhy]

いえ、別に修正をしていただかなくてもよかったです。実際に動かしているコードとご質問のコードが違うと、回答しても話が通じなくなるので、確認をしたかっただけです。

Answer 1

ひとつ目の問題に、for文でループを組むなら、data.map()で配列を作る必要はない。このやり方だと、一種の二重ループ。
解決方法は、marker変数の処理は一番目のコードに戻すこと。

ふたつ目の問題は、infoWindow変数に代入がないこと。吹き出しを作るのはnew google.maps.InfoWindow()。
解決方法は、infoWindow変数の処理は一番目のコードに戻して、配列に収めること。

みっつ目は、 markerEvent()関数は、配列の添え字を引数に取るので、markerEvent(data[i]);という呼び出しはおそらく間違い。
解決方法は、修正前の方がおそらく合っているので、戻すこと。

以上。

Comments

[k0908]

ありがとうございます。解決方法は、infoWindow変数の処理は一番目のコードに戻して、配列に収めること。　の部分がよくわかりませんでした。
どのように配列に収めるのか例えばのコードを書いてもらえませんか？よろしくお願いいたします

[Lhankor_Mhy]

動くコードがほしい、ということでしたら、sousukeのご回答で十分かと。

[Lhankor_Mhy]

敬称忘れ失礼。

Answer 2

変数の使い回しでおかしくなるので関数分割してスコープ分けるのはいいと思うんですが
全体的に書き換え過ぎていると思います。
私も修正前のコードから関数外出しで直すほうがいいかなと思います。（提示のコードは未検証）

function initMap() {
    for (var i = 0; i < data.length; i++) {
        setMW(data[i]);
    }
}

function setMW(LatLng){
    var markerLatLng = new google.maps.LatLng(LatLng);
    var marker = new google.maps.Marker({
        position:markerLatLng,
        map: map
    });

    var infoWindow = new google.maps.InfoWindow({
        content: '<div class="sample">test</div>'
    });

    marker.addListener('click', function() {
        infoWindow.open(map, marker);
    });
}