1

文字列のすべての文字をアルファベットの次の文字に変換しようとしています。たとえば、AはBになり、XはYになり、ZはAになります。

レターシフトが完了した後、すべての母音を大文字にしたいと思います。

function LetterChanges(str) {

   var c = str.split("");
    var vowels = ["a", "e", "i", "o", "u"]; 
   if (c == vowels) {
      vowels.toUpperCase();}
   if (c == "z") return "a";  
    return str.replace(/[a-z]/gi, function(s) {
      return String.fromCharCode(s.charCodeAt(c)+1);
      });

}
LetterChanges("cold buttz");

母音部分とztoa部分が機能していません。助けてください？

4

2 に答える 2

6

これが役立つかどうかを確認してください：

var str = 'cold buttz';

str = str.replace(/[a-z]/gi, function(char) {
  char = String.fromCharCode(char.charCodeAt(0)+1);
  if (char=='{' || char=='[') char = 'a';
  if (/[aeiuo]/.test(char)) char = char.toUpperCase();
  return char;
});

console.log(str); //= "dpmE cvUUA"

編集：あなたのコードは、私の最後の回答からの厄介なコピー/貼り付けのようなものだったことがわかります...何が問題なのかを簡単に説明します：

function LetterChanges(str) {

  var c = str.split(""); // array of letters from `str`
  var vowels = ["a", "e", "i", "o", "u"]; // array of vowels

  // `c` and `vowels` are two different objects
  // so this test will always be false
  if (c == vowels) {
    // `toUpperCase` is a method on strings, not arrays
    vowels.toUpperCase();
  }

  // You're comparing apples to oranges,
  // or an array to a string, this test will also be false
  // Then you return 'a'?? This was meant to be inside the `replace`
  if (c == "z") return "a";

  // OK, I see you recycled this from my other answer
  // but you copy/pasted wrong... Here you're basically saying:
  // "For each letter in the string do something and return something new"
  return str.replace(/[a-z]/gi, function(s) { // `s` is the letter
    // Here we find out the next letter but
    // `c` is an array and `charCodeAt` expects an index (number)
    return String.fromCharCode(s.charCodeAt(c)+1);

    // `.charCodeAt(0)` gives you the code for the first letter in a string
    // in this case there's only one.
  });
}
于 2013-07-04T06:28:40.770 に答える