将每个单词的首字母大写

Question

var name = "AlbERt EINstEiN";

function nameChanger(oldName) {
var finalName = oldName;
// Your code goes here!
finalName = oldName.toLowerCase();

finalName = finalName.replace(finalName.charAt(0), finalName.charAt(0).toUpperCase());

for(i = 0; i < finalName.length; i++) {
    if (finalName.charAt(i) === " ")
        finalName.replace(finalName.charAt(i+1), finalName.charAt(i+1).toUpperCase());
}


// Don't delete this line!
return finalName;
};

// Did your code work? The line below will tell you!
console.log(nameChanger(name));

我的代码原样返回“阿尔伯特·爱因斯坦”。我想知道我哪里出错了？ 如果我添加

console.log(finalName.charAt(i+1));

在 if 语句之后，并注释掉其余部分，它会打印“e”，因此它会像它应该的那样识别 charAt(i+1)...我只是无法让它将第二个单词的第一个字母大写。

Answer 1

您的代码示例存在两个问题。我会一一介绍。

字符串是不可变的

这并不像你想象的那样工作：

finalName.replace(finalName.charAt(i+1), finalName.charAt(i+1).toUpperCase());

您需要将其更改为：

finalName = finalName.replace(finalName.charAt(i+1), finalName.charAt(i+1).toUpperCase());

在 JavaScript 中，字符串是不可变的。这意味着一旦创建了字符串，就不能更改它。这听起来可能很奇怪，因为在您的代码中，您似乎正在使用 replace() 之类的方法在整个循环中更改字符串 finalName。

但实际上，您并没有真正改变它！ replace() 函数接受一个输入字符串，进行替换，然后生成一个新的输出字符串，因为它实际上不允许更改输入字符串（不变性）。所以，tl;dr，如果你没有通过将replace() 的输出分配给一个变量来捕获它，那么被替换的字符串就会丢失。

顺便说一句，将它分配回原始变量名是可以的，这就是为什么你可以这样做finalName = finalName.replace(...)。

---

替换是贪心的

您会遇到的另一个问题是，当您使用 replace() 时，您将替换字符串中的 所有 匹配字符，而不仅仅是您正在检查的位置的字符.这是因为replace() 是贪婪的 - 如果你告诉它用 'E' 替换 'e'，它会替换所有的！

本质上，您需要做的是：

1. 找到一个空格字符（你已经这样做了）
2. 抓取所有字符串，包括空格；字符串的这个“边”很好。
3. 将下一个字母转换为大写，但只转换那个字母。
4. 获取字符串的其余部分，过去您转换的字母。
5. 将所有三个部分放在一起（字符串开头、大写字母、字符串结尾）。

slice() 方法会做你想做的事：

if (finalName.charAt(i) === " ") {
    // Get ONLY the letter after the space
    var startLetter = finalName.slice(i+1, i+2);
    // Concatenate the string up to the letter + the letter uppercased + the rest of the string
    finalName = finalName.slice(0, i+1) + startLetter.toUpperCase() + finalName.slice(i+2);
}

---

另一个选项是正则表达式（regex），其他答案都提到了。这可能是一个更好的选择，因为它更清洁。但是，如果您是第一次学习编程，通过编写原始循环更容易理解这种手动字符串工作。以后你可以乱用有效的方法来做到这一点。

工作 jsfiddle：http://jsfiddle.net/9dLw1Lfx/

进一步阅读：

• Are JavaScript strings immutable? Do I need a "string builder" in JavaScript?
• slice() method

Answer 2

if (finalName.charAt(i) === " ")

不应该

if (finalName.charAt(i) == " ")

不 === 检查对象类型是否相等，这不应该是因为一个是字符，另一个是字符串。

Answer 3

如果您将 RegExp /pattern/flags 和 函数 传递给 str.replace 而不是使用子字符串，则可以大大简化此操作

function nameChanger(oldName) {
    var lowerCase = oldName.toLowerCase(),
        titleCase = lowerCase.replace(/\b./g, function ($0) {return $0.toUpperCase()});
    return titleCase;
};

在此示例中，我将更改应用于 word boundary \b 之后的任何字符 .，但您可能需要更具体的 /(^| )./g

Answer 4

这个问题的另一个好答案是使用 RegEx 来为你做这件事。

var re = /(\b[a-z](?!\s))/g;
var s = "fort collins, croton-on-hudson, harper's ferry, coeur d'alene, o'fallon"; 
s = s.replace(re, function(x){return x.toUpperCase();});
console.log(s); // "Fort Collins, Croton-On-Hudson, Harper's Ferry, Coeur D'Alene, O'Fallon"

所使用的正则表达式可能需要稍微改变一下，但这应该让您了解可以使用正则表达式做什么

Capitalize Letters with JavaScript

Answer 5

问题是双重的：

1) 您需要为 finalName.replace 返回一个值，因为该方法返回一个元素但不会改变它所基于的元素。

2) 您没有遍历字符串值，因此您只更改了第一个单词。你不想把每个单词都改成小写吗？

这段代码会更好地为您服务：

   var name = "AlbERt EINstEiN";

function nameChanger(oldName) {

    // Your code goes here!

    var finalName = [];
    oldName.toLowerCase().split(" ").forEach(function(word) {
        newWord = word.replace(word.charAt(0), word.charAt(0).toUpperCase());
        finalName.push(newWord);
    });


    // Don't delete this line!
    return finalName.join(" ");
};

    // Did your code work? The line below will tell you!
    console.log(nameChanger(name));