Множество циклов, множество вопросов

Question

Я написал этот код. Кажется, он работает корректно через первый набор вложенных циклов. Когда я помещаю отладчик выше финального цикла, он вернет мне правильную вещь (и если я хочу вернуть newArray, он вернется). Что я делаю не так?

Я хочу, чтобы он пропустил массив letterSplit, который я сделал из ввода, затем прокрутите массив значений и найдите соответствующее значение и нажмите соответствующее значение в newArray. Это работает с 1 буквой.

НО Я также хочу, чтобы он работал для нескольких букв, так что, если кто-то помещает в «кошку», он добавит их все в новый массив «total». Это то, что я пытался сделать с последним циклом цикла. Предложения? Идеи? Вы видите какое-то неуместное слово или символ?

var scrabble = function (letter) { 
var newLetter = letter.toLowerCase(); 
var letterSplit = newLetter.split(" "); 
var newArray = []; 
var stupidArray = []; 
var total = 0; 
var values = [["a", 1], ["b", 3], ["c", 3], ["d", 2], ["e", 1], ["f", 4], ["g", 2], ["h", 4], ["i", 1], ["j", 8], ["k", 5], ["l", 1], ["m", 3], ["n", 1], ["o", 1], 
       ["p", 3], ["q", 10], ["r", 1], ["s", 1], ["t", 1], ["u", 1], ["v", 4], ["w", 4], ["x", 8], ["y", 4], ["z", 10]]; 

    for (var i=0; i < letterSplit.length; i++) { 
     for (var i=0; i < values.length; i++) { 
      if (values[i][0] === letterSplit[0]) { 
      newArray.push(values[i][1]); 
      stupidArray += letterSplit.splice(0,1); 
      } 
     } 
    } 
    for (var i=0; i < newArray.length; i++) { 
     total += i; 
    } 

var result = total.toString(); 
return total; 
}; 

Answer 1

Ваш код исправлен

Javascript

var scrabble = function (letter) { 
    var letterSplit = letter.toLowerCase().split(''), 
     total = 0, 
     values = [ 
      ['a', 1], 
      ['b', 3], 
      ['c', 3], 
      ['d', 2], 
      ['e', 1], 
      ['f', 4], 
      ['g', 2], 
      ['h', 4], 
      ['i', 1], 
      ['j', 8], 
      ['k', 5], 
      ['l', 1], 
      ['m', 3], 
      ['n', 1], 
      ['o', 1], 
      ['p', 3], 
      ['q', 10], 
      ['r', 1], 
      ['s', 1], 
      ['t', 1], 
      ['u', 1], 
      ['v', 4], 
      ['w', 4], 
      ['x', 8], 
      ['y', 4], 
      ['z', 10] 
     ], 
     i, 
     j; 

    for (i = 0; i < letterSplit.length; i++) { 
     for (j = 0; j < values.length; j++) { 
      if (values[j][0] === letterSplit[i]) { 
       total += values[j][1]; 
       break; 
      } 
     } 
    } 

    return total; 
}; 

console.log(scrabble('cat')); 

На jsFiddle

Answer 2

Использование Object в качестве поиска для значения буквы, вероятно, будет проще/более разумным. Что-то вроде этого.

Javascript

var values = { 
    a: 1, 
    b: 3, 
    c: 3, 
    d: 2, 
    e: 1, 
    f: 4, 
    g: 2, 
    h: 4, 
    i: 1, 
    j: 8, 
    k: 5, 
    l: 1, 
    m: 3, 
    n: 1, 
    o: 1, 
    p: 3, 
    q: 10, 
    r: 1, 
    s: 1, 
    t: 1, 
    u: 1, 
    v: 4, 
    w: 4, 
    x: 8, 
    y: 4, 
    z: 10 
}; 

function scrabble(word) { 
    return word.toLowerCase().split('').reduce(function (acc, letter) { 
     if (values.hasOwnProperty(letter)) { 
      acc += values[letter]; 
     } 

     return acc; 
    }, 0); 
} 

console.log(scrabble('cat')); 

Выход

5 

На jsFiddle