0

そのため、このDynamic Breadcrumbsの記述をいじっていたところ、ディレクトリ名にスペースが含まれていると、実際に表示されるブレッドクラムに%20が追加されるという問題が発生しました。これはdecodeURI()関数を使用して削除されますか、それともより良い方法がありますか?

これがjsです:

 var crumbsep = " • ";
 var precrumb = "<span class=\"crumb\">";
 var postcrumb = "</span>";
 var sectionsep = "/";
 var rootpath = "/"; // Use "/" for root of domain.
 var rootname = "Home";

 var ucfirst = 1; // if set to 1, makes "directory" default to "Directory"

 var objurl = new Object;

 // Grab the page's url and break it up into directory pieces
 var pageurl = (new String(document.location));
 var protocol = pageurl.substring(0, pageurl.indexOf("//") + 2);
 pageurl = pageurl.replace(protocol, ""); // remove protocol from pageurl
 var rooturl = pageurl.substring(0, pageurl.indexOf(rootpath) + rootpath.length); // find rooturl
 if (rooturl.charAt(rooturl.length - 1) == "/") //remove trailing slash
 { 
 rooturl = rooturl.substring(0, rooturl.length - 1); 
 }
 pageurl = pageurl.replace(rooturl, ""); // remove rooturl from pageurl
 if (pageurl.charAt(0) == '/') // remove beginning slash
 { 
 pageurl = pageurl.substring(1, pageurl.length); 
 }

 var page_ar = pageurl.split(sectionsep);
 var currenturl = protocol + rooturl;
 var allbread = precrumb + "<a href=\"" + currenturl + "\">" + rootname + "</a>" + postcrumb; // start with root

 for (i=0; i < page_ar.length-1; i++)
 { 
 var displayname = "";
   currenturl += "/" + page_ar[i];
   if (objurl[page_ar[i]])
   { 
   displayname = objurl[page_ar[i]]; 
   }
   else
   { 
   if (ucfirst == 1)
  { 
  displayname = page_ar[i].charAt(0).toUpperCase() + page_ar[i].substring(1); 
  }
  else
  { 
  displayname = page_ar[i];
  }
   }
   if ( i < page_ar.length -2 )
  { 
  allbread += precrumb + crumbsep +  "<a href=\"" + currenturl + "\">" + displayname + "</a>" + postcrumb; 
  }
   else
    { 
    allbread += crumbsep +  displayname; 
    }
 }
 document.write(allbread);

decodeURI()を使用する場合、正確にはどこに行きますか?また、もっと無関係に、ディレクトリ内の実際のページを最後のディレクトリではなく最後のアイテムとしてブレッドクラムに含めるオプションを上記のコードに追加できますか?それほど重要ではありませんが、私も質問したいと思いました。ご入力いただきありがとうございます。

4

2 に答える 2

2

はい、decodeURIトリックを行います。displayname = decodeURI(displayname);次の行の直前に行を追加できifますif ( i < page_ar.length -2 )

...
displayname = decodeURI(displayname);
if ( i < page_ar.length -2 )
...

displayname生のHTML文字列に直接埋め込まれるため、currenturl最初に特別なHTML文字をエスケープする必要があることに注意してください。そうしないと、 XSS攻撃を受ける可能性があります(yoursite.com/validのようなサイトへのリンクを投稿する悪意のある個人を想像してください)。 / page /%3Cscript%3Ealert%28%22Oh%20no%2C%20not%20XSS%21%22%29%3C%2Fscript%3E)。これを行う最も簡単な方法の1つは、jQueryが必要ですが、この回答でカバーされています。

現在のページをブレッドクラムに含めたい場合は、ループを変更して、次のpage_ar.length代わりに0からに変更するだけで十分だと思いpage_ar.length - 1ます。

...
for (i=0; i < page_ar.length; i++)
...
于 2010-11-17T03:05:23.213 に答える
1

decodeURIComponent()これではなく、を使用する必要がありますdecodeURI()。何をしようとしているのかを理解するのは少し難しいですが、現在のURIの「ディレクトリ」の配列をデコードして提供する簡単なコードを次に示します。

var dirs = location.pathname.split('/');
for (var i=0,len=dirs.length;i<len;++i){
  dirs[i] = decodeURIComponent(dirs[i]);
}
于 2010-11-17T03:19:01.277 に答える