Me sorprende por qué se agrega un valor booleano al valor actual en mi funcionalidad de reducción.
var romanToInt = function (s) { const DICT = { I: 1, V: 5, X: 10, L: 50, C: 100, D: 500, M: 1000, } let reversedArr = s.split('').reverse() reversedArr return reversedArr.reduce((sum, cur, i, arr) => { let value = DICT[cur] // check if preceeding numeral subtracts if ((arr[i] += arr[i - 1] === 'VI') && i != 0) { sum -= value } else { // Set sum as the first sum += value } return sum }, 0) } console.log(romanToInt('III')) ¿Por qué esta expresión (curr += arr[i - 1] === 'VI') se evalúa como falso, verdadero, verdadero?
El valor de curr después de tres iteraciones es Ifalse, Ifalse, Ifalse . ¿Cómo está pasando esto?
Todo lo que quiero hacer es verificar si el valor actual y el valor anterior son iguales a la cadena 'VI'.
x += y es equivalente a x = x + y , y toda la expresión de asignación se evalúa como el valor del lado derecho: x + y . Si x + y es verdadera, toda la expresión resultante será verdadera.
Por la misma razón, su
if ((arr[i] += arr[i - 1] === 'VI') && i != 0) {no está haciendo lo que crees que es.
Parece que no necesita asignar nada al índice de matriz aquí, solo compare allí, y si se cumple la condición, cambie la sum (sin cambiar nada más dentro del ciclo).
var romanToInt = function (s) { const DICT = { I: 1, V: 5, X: 10, L: 50, C: 100, D: 500, M: 1000, } const reversedArr = s.split('').reverse(); return reversedArr.reduce((sum, cur, i, arr) => { let value = DICT[cur] // check if preceeding numeral subtracts if ((arr[i] + arr[i - 1] === 'VI') && i != 0) { sum -= value } else { // Set sum as the first sum += value } return sum }, 0) } console.log(romanToInt('III'))Pero todavía tiene mucho camino por recorrer para arreglar el resto de su algoritmo. Este es un buen enfoque.
versión más corta ;)
const romanToInt = (function() { const DICT = { I: 1, V: 5, X: 10, L: 50, C: 100, D: 500, M: 1000 } , Narr = [ 'IV','IX', 'XL', 'XC', 'CD', 'CM' ] ; return (str) => [...str].reverse().reduce((sum, curr, i, {[i-1]:prev}) => sum + (Narr.includes(`${curr}${prev}`) ? -DICT[curr] : DICT[curr]) , 0); })() console.log('III', romanToInt('III')) console.log('IV', romanToInt('IV')) console.log('VII', romanToInt('VII')) console.log('IX', romanToInt('IX'))