Konwersja z systemu rzymskiego na dziesiętny

Question

#include <iostream>
using namespace std;

int zmiana (char x)
{
	if (char==I) return 1;
	if (char==V) return 5;
	if (char==X) return 10;
	if (char==L) return 50;
	if (char==C) return 100;
	if (char==D) return 500;
	if (char==M) return 1000;
	

int main() {
	string tekst;
	int suma(0);
	cin>>tekst;
	for (int i=0; i<tekst.size(); i++)
	{
		if (zmiana(tekst[i])<zmiana(tab[i+1])) suma-=zmiana(tekst[i]);
		else suma+=1;
	}
	 
cout<<suma;

	return 0;
}

napisałam kod, który w założeniu ma zmieniać liczby z systemu rzymskiego na dziesiętny. Co musiałabym w nim zmienić żeby działał dobrze?

 

 

poprawiłam tak:

#include <iostream>
using namespace std;
 
int zmiana (char x)
{
    if (x=='I') return 1;
    if (x=='V') return 5;
    if (x=='X') return 10;
    if (x=='L') return 50;
    if (x=='C') return 100;
    if (x=='D') return 500;
    if (x=='M') return 1000;
}  
 
int main() {
    string tekst;
    int suma(0);
    cin>>tekst;
    for (int i=0; i<tekst.size(); i++)
    {
        if (zmiana(tekst[i])<zmiana(zmiana[i+1])) suma-=zmiana(tekst[i]);
        else suma+=zmiana(tekst[i]);
    }
 
cout<<suma;
 
    return 0;
}

ale jeszcze coś jest nie tak. Czy można w ogóle zrobić tak jak ja postawiłam warunek w ifie?

Comments

Domknij również klamrę od funkcji zmiana.

Answer 1

Przede wszystkim w warunkach powinieneś porównywać sam argument a nie jego typ i nie == A tylko =='A' bo nie ma czegoś takiego jak zmienna A (u ciebie), zaś znak 'A' już istnieje

czyli reasumując np:

if(x=='A') return 2;

reszty kodu nie sprawdzałem.

Answer 2

podając funkcji zmiana wynik tej samej funkcji pewnie podajesz jej jakąś strasznie wielką liczbę która nie jest poprawnym znakiemsystemu rzymskiego więc zamień "zmiana(zmiana(tekst[i]))" na "zmiana(tekst[i])"

Answer 3

#include <iostream>
#include <string>
using namespace std;
  
int zmiana (char x)
{
    if (x=='I') return 1;
    if (x=='V') return 5;
    if (x=='X') return 10;
    if (x=='L') return 50;
    if (x=='C') return 100;
    if (x=='D') return 500;
    if (x=='M') return 1000;
}  

int konwersja(string liczbaRzymska) 
{
	int wynik = zmiana(liczbaRzymska[0]);
	int aktualnyZnak;
	int poprzedniZnak;

	if(liczbaRzymska.length() > 1)
	{
		for (int i = 1; i < liczbaRzymska.length(); i++)
		{
			aktualnyZnak = zmiana(liczbaRzymska[i]);
			poprzedniZnak = zmiana(liczbaRzymska[i - 1]);
			if (poprzedniZnak == aktualnyZnak) 
				wynik += aktualnyZnak;
			else if (poprzedniZnak > aktualnyZnak) 
				wynik += aktualnyZnak;
			else 
				if (wynik < aktualnyZnak)
					wynik = aktualnyZnak - wynik;
				else
					wynik = (wynik - poprzedniZnak) + (aktualnyZnak - poprzedniZnak);
		}
	}

	return wynik;
}
  
int main() {
    string tekst;
    int suma(0);
    cin>>tekst;
    suma = konwersja(tekst);
  
    cout<<suma;
  
    return 0;
}

Trochę mnie poniosło i postanowiłem rozwiązać do końca to zadanie. Sprawdziłem jak zachowuje się program jeśli poda się przykładowe liczby rzymskie z wikipedii i na tej podstawie postawiłem w funkcji konwersja takie a nie inne warunki (szczególnie ciekawe są te w zagnieżdżonym else):
https://pl.wikipedia.org/wiki/Rzymski_system_zapisywania_liczb