javascriptの一致するテキスト

Question

簡単な質問、私はダブルアスタリスクから太字に変換したいmarkdown HTMLコンテンツがあります。

私は取得していますエラーは次のとおりです。 `キャッチされない例外TypeErrorは：ここ

「ヌルの '置き換え' メソッドを呼び出すことはできませんですjsfiddle：ここhttp://jsfiddle.net/fz5ZT/9/

のHTML：

<div class="comments">comment 1** is good**</div> 
<div class="comments">comment 2**is bad**</div> 

がここにありますJS：

function markdown(markdownable){ 

    var boldMatch = markdownable.match(/\*\*[A-Za-z0-9]+\*\*/gim), 
    boldReplace = boldMatch.replace(/\*\*[A-z0-9]+\*\*/gim, '<span style="font- weight:bold;color:blue;">'+boldMatch+'</span>'),     
    markdownable = markdownable.replace(boldMatch, boldReplace),  
    markdownable = markdownable.replace(/\*\*/gi, ""); 

    return markdownable; 
} 

$('.comments').each(function(){ 
    var markdownable=$(this).html(), comments=markdown(markdownable); 
}); 

もし私がgreを助けることができるのであればatly、

おかげで、 ティム

更新おかげですべてをそれに感謝！ http://jsfiddle.net/fz5ZT/30/

Answer 1

markdownable = markdownable.replace(/\*\*(.+?)\*\*/gm, '<strong>$1</strong>'); 

しかし、代わりに車輪の再発明で中途半端な、善意、運命ツー失敗の試みを実行するのは、なぜだけでなくuse an existing JavaScript Markdown library？

編集：ここでは、「オープン」または「クローズ」の前にした後に空白権利がないように要求する（マークダウンのような）より堅牢な正規表現です：

var bold = /\*\*(\S(.*?\S)?)\*\*/gm; 
markdownable = markdownable.replace(bold, '<strong>$1</strong>'); 

Answer 2

あなたの最初の正規表現は、文字列の空白を無視しています。許可された文字セットにスペースを追加する必要があります。[ a-z0-9]; iのためにA-Zは必要ありません。

また、matchは配列を返します。返される文字列にアクセスするには、最初に一致するものをboldMatch[0]にする必要があります。

Answer 3

boldMatchには、一致するものがない場合、つまり一致するものがあることがわかるまで、.replace（）を呼び出す必要はありません。

より安全なコンピューティング：

var boldMatch = markdownable.match(/\*\*[A-Za-z0-9]+\*\*/gim); 
if (boldMatch) { 
    var boldReplace = boldMatch.replace(/\*\*[A-z0-9]+\*\*/gim, '<span style="font- weight:bold;color:blue;">'+boldMatch+'</span>'); 
} 

など

更新：

var markdownable=$(this).html(), comments=markdown(markdownable); 

宣言：

このコード行は、それが困難で何が起こっているか追跡することができます1行にvaを持つ2つの変数rは一般的に眉をひそめます。ベター：

var markdownable=$(this).html(); 
if (markdownable) { 
    comments=markdown(markdownable); 
} 

Answer 4

は、あなたが取ることができるかもしれませ下記の溶液を見て：

Find text string using jQuery?

私はあなたが非常によく似た何かを行う必要があると考えている：

$('*:contains("I am a simple string")').each(function(){ 
if($(this).children().length < 1) 
     $(this).html( 
      $(this).text().replace(
       /"I am a simple string"/ 
       ,'<span containsStringImLookingFor="true">"I am a simple string" </span>' 
      ) 
     ) 
}); 

要素を太字にするには、置換が行われたらaddClass（）を使用する必要があります。

おかげ

Answer 5

function markdown(markdownable) { 

    var boldMatch = markdownable.match(/[\*]{2}(.+)?[\*]{2}/gim); 
    if (boldMatch && (boldMatch = boldMatch[0])) { 
     var boldReplace = boldMatch.replace(/[\*]{2}(.+)+?[\*]{2}/gim, '<span style="font-weight:bold;color:blue;">' + boldMatch + '</span>'); 
     markdownable = markdownable.replace(boldMatch, boldReplace); 
     markdownable = markdownable.replace(/\*\*/gi, ""); 
    } 
    return markdownable; 
} 

$('.comments').each(function() { 

    var markdownable = $(this).html(), 
     comments = markdown(markdownable); 

    console.log(comments); 
});​ 

これが最善の解決策で抜いていない...しかし、それはあなたの試みのための「修正」です。うまくいけば、あなたはどこに間違っていたかについて何かを学ぶことができます