-1

私は簡単にJavascriptを読んで理解することができますが、私自身のコードを書くのは苦労します。このように、私は私の目的を果たすためにパッチを当てたGoogleマップAPIのいくつかのコードに出くわしましたが、あまりにも多くの繰り返しがあります。私のGoogle Map API JSコードを維持するのに助けが必要DRY

私は特に、(コードブロックの最後に向かって)各マーカーに.addListenerを別々に配置しなければならない部分について言及しています。私は配列がクリーンアップに役立つだろうと思っているが、私はそれが正確にどのように動作するかについてまだ失われている。どんな助けでも大歓迎です!あなたは何をすべき

<script> 
    function initMap() { 
    // Styles a map in night mode. 
    var map = new google.maps.Map(document.getElementById('cc-qline-map'), { 
     center: {lat: 50.1332, lng: -100.6700}, 
     zoom:4, 
     mapTypeControl: false, 
     scrollwheel: false, 

     styles: [ 
     {elementType: 'geometry', stylers: [{color: '#222222'}]}, 
     {elementType: 'labels.text.stroke', stylers: [{color: '#191919'}]}, 
     {elementType: 'labels.text.fill', stylers: [{color: '#d3d3d3'}]}, 
     { 
      featureType: 'administrative.locality', 
      elementType: 'labels.text.fill', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'poi', 
      elementType: 'labels.text.fill', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'poi.park', 
      elementType: 'geometry', 
      stylers: [{color: '#191919'}] 
     }, 
     { 
      featureType: 'poi.park', 
      elementType: 'labels.text.fill', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'road', 
      elementType: 'geometry', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'road', 
      elementType: 'geometry.stroke', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'road', 
      elementType: 'labels.text.fill', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'road.highway', 
      elementType: 'geometry', 
      stylers: [{color: '#191919'}] 
     }, 
     { 
      featureType: 'road.highway', 
      elementType: 'geometry.stroke', 
      stylers: [{color: '#191919'}] 
     }, 
     { 
      featureType: 'road.highway', 
      elementType: 'labels.text.fill', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'transit', 
      elementType: 'geometry', 
      stylers: [{color: '#191919'}] 
     }, 
     { 
      featureType: 'transit.station', 
      elementType: 'labels.text.fill', 
      stylers: [{color: '#191919'}] 
     }, 
     { 
      featureType: 'water', 
      elementType: 'geometry', 
      stylers: [{color: '#d3d3d3'}] 
     }, 
     { 
      featureType: 'water', 
      elementType: 'labels.text.fill', 
      stylers: [{color: '#222222'}] 
     }, 
     { 
      featureType: 'water', 
      elementType: 'labels.text.stroke', 
      stylers: [{color: '#d3d3d3'}] 
     } 
     ] 
    }); 

    var markerBc = new google.maps.Marker({ 
    position: {lat: 49.054201, lng: -122.383454}, 
    map: map, 
    title: '30310-30360 Old Yale Rd, Abbotsford, BC V4X 2N7' 
    }); 
    var markerBcText = '<div class="cc-map-text">'+ 
     '<h3>Q-Line Trucking</h3>'+ 
     '<h4>Abbotsford, British Columbia</h4>'+ 
     '<p>30310-30360 Old Yale Rd, Abbotsford, BC V4X 2N7</p>'+ 
    '</div>'; 
    markerBc.addListener('click', function() { 
     new google.maps.InfoWindow({ 
     content: markerBcText, 
     }).open(map, this); 
    }); 




    var markerAb = new google.maps.Marker({ 
    position: {lat: 53.540016, lng: -113.797466}, 
    map: map, 
    title: '#625-26409 TWP Road 525A, Acheson, AB T7X 5A6' 
    }); 
    var markerAbText = '<div class="cc-map-text">'+ 
     '<h3>Q-Line Trucking</h3>'+ 
     '<h4>Acheson, Alberta</h4>'+ 
     '<p>#625-26409 TWP Road 525A, Acheson, AB T7X 5A6</p>'+ 
    '</div>'; 
    markerAb.addListener('click', function() { 
     new google.maps.InfoWindow({ 
     content: markerAbText, 
     }).open(map, this); 
    }); 



    var markerSk = new google.maps.Marker({ 
    position: {lat: 52.172385, lng: -106.649831}, 
    map: map, 
    title: 'PO Box 110-B RR#4, Corman Industrial Park Saskatoon, SK S7K 3J7' 
    }); 
    var markerSkText = '<div class="cc-map-text">'+ 
     '<h3>Q-Line Trucking</h3>'+ 
     '<h4>Saskatoon, Saskatchewan</h4>'+ 
     '<p>PO Box 110-B RR#4, Corman Industrial Park Saskatoon, SK S7K 3J7</p>'+ 
    '</div>'; 
    markerSk.addListener('click', function() { 
     new google.maps.InfoWindow({ 
     content: markerSkText, 
     }).open(map, this); 
    }); 



    var markerMb = new google.maps.Marker({ 
    position: {lat: 49.177328, lng: -97.936206}, 
    map: map, 
    title: 'PO Box 1870 #6 3rd St S, Winkler, MB R6W 4B7' 
    }); 
    var markerMbText = '<div class="cc-map-text">'+ 
     '<h3>Q-Line Trucking</h3>'+ 
     '<h4>Winkler, Manitoba</h4>'+ 
     '<p>PO Box 1870 #6 3rd St S, Winkler, MB R6W 4B7</p>'+ 
    '</div>'; 
    markerMb.addListener('click', function() { 
     new google.maps.InfoWindow({ 
     content: markerMbText, 
     }).open(map, this); 
    }); 


    var markerIa = new google.maps.Marker({ 
    position: {lat: 42.464988, lng: -92.339417}, 
    map: map, 
    title: '1410 Columbus Dr, Waterloo, IA 50702' 
    }); 
    var markerIaText = '<div class="cc-map-text">'+ 
     '<h3>Q-Line Trucking</h3>'+ 
     '<h4>Waterloo, Iowa</h4>'+ 
     '<p>1410 Columbus Dr, Waterloo, IA 50702</p>'+ 
    '</div>'; 
    markerIa.addListener('click', function() { 
     new google.maps.InfoWindow({ 
     content: markerIaText, 
     }).open(map, this); 
    }); 



    var markerOn = new google.maps.Marker({ 
    position: {lat: 43.576325, lng: -79.812785}, 
    map: map, 
    title: '7105 Auburn Road, Milton, ON L9T7V9' 
    }); 
    var markerOnText = '<div class="cc-map-text">'+ 
     '<h3>Q-Line Trucking</h3>'+ 
     '<h4>Milton, Ontario</h4>'+ 
     '<p>7105 Auburn Road, Milton, ON L9T7V9</p>'+ 
    '</div>'; 
    markerOn.addListener('click', function() { 
     new google.maps.InfoWindow({ 
     content: markerOnText, 
     }).open(map, this); 
    }); 


    } 

+2

こんにちはケニー、ようこそ。コードレビューが必要な場合は、[専用サイト](https://codereview.stackexchange.com)を投稿するのが良いかもしれないと思います。ただし、コード内の何かが機能しない場合は、いくつかの詳細を追加できますか? – damienc

+1

@damiencアドバイスをいただきありがとうございます!間違いなく私はそれ以上のウェブサイトが必要だよLOL – kenny

答えて

0

あなたのマーカーを作成するために必要なすべてのデータの配列を持っています。それ以上のループ。何かのように:それは意味を成していた場合でも、あなたが<h3><h4><p>のために必要な各部分に、さらにtextを打破することができ

var markerData = [ 
    { 
     lat: 49.054201, 
     lng: -122.383454, 
     title: '30310-30360 Old Yale Rd, Abbotsford, BC V4X 2N7', 
     text: '<h3>Q-Line Trucking</h3>'+'<h4>Abbotsford, British Columbia</h4>'+'<p>30310-30360 Old Yale Rd, Abbotsford, BC V4X 2N7</p>' 
    }, 
    { 
     lat: 53.540016, 
     lng: -113.797466, 
     title: '#625-26409 TWP Road 525A, Acheson, AB T7X 5A6', 
     text: '<h3>Q-Line Trucking</h3>' + '<h4>Acheson, Alberta</h4>'+'<p>#625-26409 TWP Road 525A, Acheson, AB T7X 5A6</p>' 
    }, 
    // ... etc 
]; 


for (var i = 0; i < markerData.length; i++) { 
    createMarker(markerData[i]);  
} 

function createMarker(data) { 
    var marker = new google.maps.Marker({ 
    position: {lat: data.lat, lng: data.lng}, 
    map: map, 
    title: data.title 
    }); 
    var markerText = '<div class="cc-map-text">' + data.text + '</div>'; 
    marker.addListener('click', function() { 
     new google.maps.InfoWindow({ 
     content: markerText, 
     }).open(map, this); 
    }); 
} 

+0

私はこれをテストしていないが、これはまさに私が探しているようだ。ありがとう! – kenny

関連する問題