Không biết sai chỗ nào

#1

Quái, không biết sai chỗ nào


  function loadScript2(url){
      var chk=false;
      var i=0;
      var scripts = document.getElementsByTagName("script");
      for(var i=0;i<scripts.length;i++){
        //console.log(scripts[i].src);
        //console.log("*"+scripts[i].src+"*"+"  *"+url+"*");
        if( new String (url) === new String (scripts[i].src) ){
          chk=true;
          exit;
        }else{
          i++;
        }
      }
      alert(JSON.stringify( {i:i,chk:chk}  ));
  }

anh chị em nào giúp với!!!

#2

Trước khi hỏi bạn sai ở chổ nào, mình sẽ hỏi vài thứ liên quan trong source code, có thể bạn chẳng hiểu bạn đang viết gì:
1/ Tại sao lại dùng new String mà không phải là phép check null thông thường?
2/ exit để làm gì? Và nó có ý nghĩa gì?
3/ i++ cuối vòng lặp để làm gì?

2 Likes
#3

Bonus:
4/ Đến 1 câu sai chỗ nào hay có chỗ nào không như mong muốn cũng không nói
Một câu hỏi không thể tệ hơn phản ánh quá nhiều thứ, haizzzz

PS: mình nhớ hình như bạn này newbie hơi lâu rồi, ngôn ngữ/fw nào cũng mó tới rồi thì phải nhưng chưa cái nào ổn định cả, kiến thức nền hơi có vấn đề

1 Like
#4

À cái này là mình không hiểu sao nó chạy tọn luôn vòng for mặc dù cái src nó trùng với một cái src của danh sách script

à à lộn rồi phải bỏ cái i++ hì.

#5

tọn có nghĩa là gì và bạn trả lời được các câu hỏi của mình không?

1 Like
#6

À nhầm cái i++ rồi.



  function loadScript2(src){
      var chk=false;
      var scripts = document.getElementsByTagName("script");
      for(var i=0;i<scripts.length;i++){
        //if( new String (src) === new String (scripts[i].src) ){
         if(src == scripts[i].src)
          //chk=true;
          //exit;
           return;//Cái script này đã tồn tại không cần load lại.
        }
      }
      if(chk==false){//
           //viết cái load cái script ở đây
      }

  }

Cám ơn nhé.

#7

Sửa tiếp lại luôn


  function loadScript2(src){
      var scripts = document.getElementsByTagName("script");
      for(var i=0;i<scripts.length;i++){
         if(src == scripts[i].src)
           return;//Cái script này đã tồn tại không cần load lại.
        }
      }
      //viết cái load cái script ở đây
      var script = document.createElement("script")
      script.type = "text/javascript";
      script.src = src;
      document.getElementsByTagName("head")[0].appendChild(script);
  }

Bi giờ chắc đẹp trai rồi
cái ơn cái i++

#8

if(document.querySelector("script[src='${src}']")) return;

3 Likes
#9

Xài Cái này thì khỏi cần cái vòng for kia à?

83% thành viên diễn đàn không hỏi bài tập, còn bạn thì sao?