Как преобразовать римские числа в арабские и как применить?

Question

[ASA33]

всем привет. я еще учу js вот здесь я решил сложение чисел но с римскими немогу как решить или можете порешить эту задачку.

Comments

[Somewhere Intech]

норм решение))))

[ASA33]

Johnny Lowhunter, дальше не смог как решить?

Answer 1

[Somewhere Intech]

const digits = {Z:2000, M:1000,CM:900,D:500,CD:400,C:100,XC:90,L:50,XL:40,X:10,IX:9,V:5,IV:4,I:1};

function roman2arabic(str){
	if (!/^[IVXLCDMZ]+$/i.test(str)) throw new Error('Incorrect roman number format: ' + str)
	return str.toUpperCase().split('').reduce(function(r,v,i,arr){
		const [ a, b, c ] = [ digits[arr[i]], digits[arr[i+1]], digits[arr[i+2]]];
		if (b && c && a <= b && b < c)
			throw new Error('Incorrect roman number format: ' + str);
		return b > a ? r - a : r + a;
	}, 0)
}

function arabic2roman(num){
	if (!/^\-?\d+$/.test(num+'')) throw new Error('Can`t convert that arabic numeric to roman: ' + num)
	if (num < 1) return '';
	let result = '';
	for (let key in digits)
		while ( num >= digits[key] ) {
			result += key;
			num -= digits[key];
		}
	return result;
}

function calculator(string){
	let badChars = [];
	string = string.replace(/[^IVXLCDMZ\d+\-*\/]/gi, chr => {
		if (chr !== ' ') badChars.push(chr);
		return '';
	});
	if (badChars.length > 0) 
		throw Error('Символы не допустимы: ' + badChars.join(' '));
	let isRoman = /^[IVXLCDMZ]+$/i,
		vars = string.split(/[+\-*\/]/g),
		action = string.match(/[+\-*\/]/)[0];
	if (vars.length !== 2) 
		throw Error("Должно быть лишь два операнда");
	let r = vars.reduce((s,v)=> s + isRoman.test(v),0);
	if (r === 1)
		throw Error("Оба числа должны быть либо римскими, либо арабскими, исправьте выражение: " + string);
	else if (r === 2)
		vars = vars.map(v=>roman2arabic(v));
	else if (vars.reduce((s,v) => s + /^\d+$/.test(v)) < 2)
		throw Error("Приведенные операнды не допустимы, проверьте выражение: " + string);
	if (vars.some(v => v < 1 || v > 10))
			throw Error("Допустимо значение операндов лишь от 1 до 10 включительно")
	let result = Math.floor(eval(vars.join(action)))
	return r === 0 ? result.toString() : arabic2roman(result)
}

module.exports = calculator;

Comments

[ASA33]

function convert(roman) {
  var values = {I: 1, V: 5, X: 10, L: 50, C: 100, D: 500, M: 1000};
  var digits = Object.keys(values);

  roman = roman.toUpperCase();
  var res = 0;

  for (var q=0; q<roman.length; ++q) {
    if (digits.indexOf(roman[q]) < digits.indexOf(roman[q+1])) {
      res -= values[roman[q]];
    } else {
      res += values[roman[q]];
    }
  }

  return res;
}

а что скажете на это?

[Somewhere Intech]

ASA33, конвертирует только одно число, а Вам же всё выражение нужно было
вот выжимка ещё

function decode(inputString){
  const c = { I: 1, V: 5, X: 10, L: 50, C: 100, D: 500, M: 1000, Z: 2000 }
  return inputString.toUpperCase().replace(/[^IVXLCDMZ()+\-*\.\/\d]/g, '').replace(/[IVXLCDMZ]{1,}/g, e => e.split('').reduce((r,v,i,arr) => c[arr[i+1]] && +c[arr[i+1]] > c[v] ? r - c[v] : r+c[v], 0))
}

[ASA33]

Johnny Lowhunter, ах да спасибо

[Somewhere Intech]

ASA33, минимизировал ещё за спасибо)

[ASA33]

а что скажете на это как решить

[Somewhere Intech]

ASA33,

// в главной функции проверить входные данные регуляркой?
function entrypoint(str){ 
      if (!str.test(/[IVXLCDMZ\.\d]+.*[+\-*\/].*[IVXLCDMZ\.\d]+/)) throw new Error('Invalid input data, check expression for errors: ' + str )
}

Правда там римские цифры тоже нужно проверять на корректность..

[ASA33]

Johnny Lowhunter, бро сможешь отправить полное решение я еще новичок

[Somewhere Intech]

ASA33,

function calculator(string){
	if (!/[IVXLCDMZ\.,\d]+.*[+\-*\/].*[IVXLCDMZ\.,\d]+/.test(string)) 
		throw new Error('Invalid input data, check expression for errors: ' + string );
	string.replace(/[^IVXLCDMZ\.,\d+\-*\/()]/gi, function(badChar){
		if (badChar === ' ') return ''
		else throw new Error('Illegal symbol: ' + badChar);
	});
	const expr = string.replace(/[IVXLCDMZ\.,\d]+/gi, function(num){
		if ( /^\d+([\.,]\d+)?$/.test(num)) 
			return num.replace(/,/g,'.');
		else if (/^[IVXLCDMZ]+$/i.test(num))
			return roman2arabic(num)
		else throw new Error('Unknown numeric: ' + num)
	});
	try { 
		return eval(expr)
	} catch (e) {
		throw new Error('Illegal expression: ' + expr)
	}
	
}

function roman2arabic(str){
	const digits = { I: 1, V: 5, X: 10, L: 50, C: 100, D: 500, M: 1000, Z: 2000 };
	if (!/^[IVXLCDMZ]+$/i.test(str))
		throw new Error('Illegal roman numeric: ' + str);
	return str.toUpperCase().split('').reduce(function(r,v,i,arr){
		const [ a, b, c ] = [ digits[arr[i]], digits[arr[i+1]] || undefined, digits[arr[i+2]] || undefined ];
		if (b && c && a <= b && b < c)
			throw new Error('Illegal roman numeric: ' + str);
		return b > a ? r - a : r + a;
	}, 0);
}

module.exports = calculator;

проверяй

[ASA33]

Johnny Lowhunter, спасибо благодарен

[Somewhere Intech]

ASA33, upd: подправил чуть логику проверки ошибок..

[ASA33]

ок

[ASA33]

не сработал

[ASA33]

[Somewhere Intech]

ASA33, поправил решение - в ответе

[1__professor__3]

Somewhere Intech, братан это ты автотест написал или сам код такого калькулятора ?

[Somewhere Intech]

1__professor__3, это проверки входных данных всего лишь, их должны писать не тестеры