Aufgabe:
Ein Programm in C++ coden, welches eingegebene Zahlen in Römische umrechnet. Das Programm soll so klein wie möglich sein und über die laufen Eingabeaufforderung laufen. Vorgegebener Code:

#include <iostream>
using namespace std;

int main()
{
  int eingabe;
  cin >> eingabe;

  return 0;
}

Es dürfen keine weiteren Bibliotheken eingebunden werden. Die Subtraktionsregel muss nicht beachtet werden (=>wikipedia).

Bsp:
76 = LXXVI
512 = DXII
566 = DLXVII
316 = CCCXVI
4444 = MMMMCCCCXXXXIIII

Bewertung:
Es wird nur die Größe des kompilierten Codes beachtet.

Hinweise:
1. Der Code wird unter Mac OS 10.5 kompiliert
2. Es werden per Zufallsgenerator 5 Zahlen erzeugt, mit denen der Rechner getestet wird.
3. Es gibt nichts zu Gewinnen.
4. Das sind keine Hausaufgaben. Ich habe vorher natürlich auch ein Programm geschrieben, welches 16 KB hat. Wenn ihr mir nicht glaubt poste ich es.

So und jetzt viel Spaß

Willst du meinen Quellcode sehen?

Immer her damit.

Glaubst du mir jetzt?
Ich möchte nur mal sehen, wie andere das Problem angehen und ob es möglich ist, den Quellcode kürzer zu halten...

Soll jetzt der Quellcode kürzer sein, oder dessen Kompilat?
Btw. ist es der Quellcode, der später wartbar und übersichtlich sein muss, nicht das Kompilat.

Aber dir scheint es hier weniger um einen Coding-Contest zu gehen als darum wer die eleganteste Lösung bietet, richtig?

Also eher eine Frage der Ästethik. Dein Code ist so schon in Ordnung, zwar sehr lang, aber er ist weder schwer zu verstehen noch unübersichtlich. Die 16KB kommen hauptsächlich dadurch zu Stande, dass du die C++ IOStreams benutzt (die verschlingen schon ganz schön was an Platz im Verhältnis zu deinem Code) und wahrscheinlich ist es auch noch ein Debug Build. Mit Mac OS kenn ich mich nicht so aus und ich weiß nicht welchen Compiler du benutzt hast, daher kann ich dir nicht genau sagen woran es liegt.

Postet einfach mal eure Codes oder Vorschläge

EDIT: Wie kann ich die Iostream rausnehmen?

Die andere Richtung wird hier verlangt. Ich bin bei 175 Zeichen

und das funktioniert???

Dein Code ist so schon in Ordnung, zwar sehr lang, aber er ist weder schwer zu verstehen noch unübersichtlich.

Ein sehr langer Code ist auch immer unübersichtlich.

int i = 0, wert = 1000;
while( eingabe > 0 )
    if ( eingabe >= wert )
    {
        cout << i["MDCLXVI"];
        eingabe -= wert;
    }
    else
        wert /= ++i % 2 ? 2 : 5;

Dürfte kaum zu erheblich kleinerem binary führen.

.filmor schrieb:

Die andere Richtung wird hier verlangt. Ich bin bei 175 Zeichen

und das funktioniert???

duerfte sogar einfacher sein...

ich bin derzeit bei 8054 Byte im Release-Mode (mit deinen Compilerflags sogar nur 7819 Byte)

#include <cstdio>

int main()
{
int eingabe;
scanf( "%u", &eingabe );

while ( eingabe >= 1000 )
{
putchar( 'M' );
eingabe -= 1000;
}
while ( eingabe >= 500 )
{
putchar( 'D' );
eingabe -= 500;
}
while ( eingabe >= 100 )
{
putchar( 'C' );
eingabe -= 100;
}
while ( eingabe >= 50 )
{
putchar( 'L' );
eingabe -= 50;
}
while ( eingabe >= 10 )
{
putchar( 'X' );
eingabe -= 10;
}
while ( eingabe >= 5 )
{
putchar( 'V' );
eingabe -= 5;
}
while ( eingabe >= 1 )
{
putchar( 'I' );
eingabe--;
}

putchar( '\n' );
}

geht sicher besser...

S.T.A.L.K.E.R. schrieb:

Dein Code ist so schon in Ordnung, zwar sehr lang, aber er ist weder schwer zu verstehen noch unübersichtlich.

Ein sehr langer Code ist auch immer unübersichtlich.

int i = 0, wert = 1000;
while( eingabe > 0 )
    if ( eingabe >= wert )
    {
        cout << i["MDCLXVI"];
        eingabe -= wert;
    }
    else
        wert /= ++i % 2 ? 2 : 5;

Dürfte kaum zu erheblich kleinerem binary führen.

kannst du das prog mal so posten, dass es funktioniert?

kannst du das prog mal so posten, dass es funktioniert?

Das Programmgerüst hast du doch vorgegeben.

cout << i["MDCLXVI"];

Komm schon, das hast du nur gemacht um ihn zu flachsen

camper schrieb:

cout << i["MDCLXVI"];

Komm schon, das hast du nur gemacht um ihn zu flachsen

ehrlich gesagt find ich den else block unübersichtlicher