1

我正在尝试将 javascript 中的函数重写为 php,但它不起作用。也许这里有人可以告诉我哪里出错了?先感谢您。

Javascript ...(工作功能)

var match;
var chords = ['C','C#','D','D#','E','F','F#','G','G#','A','A#','B','C','Db','D','Eb','E','F','Gb','G','Ab','A','Bb','B','C'];
var chords2 = ['C','Db','D','Eb','E','F','Gb','G','Ab','A','Bb','B','C','C#','D','D#','E','F','F#','G','G#','A','A#','C'];
var chordRegex = /(?:C#|D#|F#|G#|A#|Db|Eb|Gb|Ab|Bb|C|D|E|F|G|A|B)/g;
var rec = /\{(.*?)\}/g; // gets all info between { and }

function transposeUp(){
    var html = $('.yui-editor-editable').contents().find('body'); //gets the contents
    var matches = $(html).html().match(rec);/////Gets all matches within the html
    var text = $(html).html(); // html of the html var
    for (var i = 0; i < matches.length; i++) { /////foreach match do
        ///// initializes variables /////
        var currentChord = String(matches[i]); //// sets current chord 
        currentChord = currentChord.substring(currentChord.indexOf("{") + 1, currentChord.indexOf("}")); //sets current chord cont.
        var output = "";
        var parts = currentChord.split(chordRegex); // splits currentChord into parts in the chordRegex
        var index = 0;
        /////////////////////////////////
        while (match = chordRegex.exec(currentChord)) { // while the match is equal to the currentChord do
            var chordIndex = chords2.indexOf(match[0]); // find the position of the matched chord within the chords2 array
            output += parts[index++] + chords[chordIndex + 1]; // build the output
        }
        output += parts[index];
        text = text.replace(matches[i], '{'+output+'}'); //replace the text with the new chords
    }
    $(html).html(text); // set the html value as the new text
}

PHP...

function transposeUp($songchart){
$chords1 = array('C','C#','D','D#','E','F','F#','G','G#','A','A#','B','C','Db','D','Eb','E','F','Gb','G','Ab','A','Bb','B','C');
$chords2 = array('C','Db','D','Eb','E','F','Gb','G','Ab','A','Bb','B','C','C#','D','D#','E','F','F#','G','G#','A','A#','C');
$chordRegex = "/(?:C#|D#|F#|G#|A#|Db|Eb|Gb|Ab|Bb|C|D|E|F|G|A|B)/";

  $matches = preg_match_all('#{.*(.*)}#U', $songchart, $chords);
  $chord = $chords[1];
  foreach($chord as $key => $value){ 
    $output = '';
    $parts = preg_split($chordRegex, $value);
    $index = 0;
    while(preg_match($chordRegex, $value, $note)){
        $chordIndex = strpos($chords2, $note[0]);
        $output .= $parts[$index++] . $chords1[$chordIndex + 1];
    }
    $output .= $parts[$index];
    $songchart = preg_replace($value, $output, $songchart);
  }
return($songchart);
}
4

1 回答 1

1
$matches = preg_match_all('#{.*(.*)}#U', $songchart, $chords);

应该:

$matches = preg_match_all('#\{(.*)\}#U', $songchart, $chords);

在您的正则表达式中,第一个.*消耗了大括号内的所有内容,所以第二个什么都没有。

我不明白while循环。$value永远不会改变,那么循环将如何终止?

$chordIndex = strpos($chords2, $note[0]);

应该:

$chordIndex = array_search($chords2, $note[0]);

因为strpos用于搜索字符串,而不是数组。

$songchart = preg_replace($value, $output, $songchart);

应该:

$songchart = str_replace($value, $output, $songchart);

因为$value不是正则表达式。

我不确定这是做什么的,但我怀疑它可以通过使用来简化preg_replace_callback,或者甚至可以通过使用str_replace数组来简化搜索和替换参数。

于 2012-09-28T03:38:51.460 回答