Строки с++

[eleonora_zv]

Задание: первести арабские числа в римские.
Исходный работающий код

#include<stdlib.h>
#include<stdio.h>
#include<conio.h>
#include<string.h>
#include<iostream>

using namespace std;

void main()
{
	int arabian;
	int p=13;
	cout<<"Enter the arabian number: ";
	cin>>arabian;
	int arab_mas[13]={1,4,5,9,10,40,50,90,100,400,500,900,1000};
	char *roman_mas[13]={"I","IV","V","IX","X","XL","L","XC","C","CD","D","CM","M"};
	char roman[]={0};
	while ((p--)!='\0')
		while (arabian>=arab_mas[p])
	{
		arabian=arabian-arab_mas[p];
		strcat(roman,roman_mas[p]);
	}
	cout<<"The equel roman number: "<<roman;
	getch();
}

Но нужно переписать из main в отдельную функцию, не работает:

#include<stdlib.h>
#include<stdio.h>
#include<conio.h>
#include<string.h>
#include<iostream>

using namespace std;
char stroka (char roman);

void main()
{
	int arabian;
	int p=13;
	cout<<"Enter the arabian number: ";
	cin>>arabian;
	char roman[]={0};
	stroka(roman);	
	getch();
}

char stroka (char roman)
{
	int arabian;
	int p=13;
	int arab_mas[13]={1,4,5,9,10,40,50,90,100,400,500,900,1000};
	char *roman_mas[13]={"I","IV","V","IX","X","XL","L","XC","C","CD","D","CM","M"};
	roman=0;
	while ((p--)!='\0')
		while (arabian>=arab_mas[p])
	{
		arabian=arabian-arab_mas[p];
		strcat(roman,roman_mas[p]);
	}
	cout<<"The equel roman number: "<<roman;
}

Помогите!

[BDA]

Не хочется Вас расстраивать, но исходный код (без функций) работает чудом и может "сломаться" в любой момент.

Сообщение от www.cplusplus.com

char * strcat ( char * destination, const char * source );
Concatenate strings
Appends a copy of the source string to the destination string. The terminating null character in destination is overwritten by the first character of source, and a null-character is included at the end of the new string formed by the concatenation of both in destination.

destination and source shall not overlap.

Parameters:
destination
Pointer to the destination array, which should contain a C string, and be large enough to contain the concatenated resulting string.
source
C string to be appended. This should not overlap destination.

Если коротко и по-русски, то destination должен быть большего размера, чтобы уместить исходную и дописанную строки.

[eleonora_zv]

Что мне исправить?

[Stilet]

char roman[255]={0};

И передавать в функцию как указатель на строку, а не просто как символ.
Если ты не в курсе как работать со строками в Си то стоит почитать об этом.
Если ты в принципе не в курсе что с этим делать, то закажи у фрилансеров программу за денюжку.

P.S. Я бы вообще эту программу переписал полностью. Даже с таким решением она врядли сработает как положено.

[BDA]

Может выбросить работу со строками?

#include <iostream>
#include <сstdio>

using namespace std;

const int p = 13;
const int arab_mas[p]={1,4,5,9,10,40,50,90,100,400,500,900,1000};
const char *roman_mas[p]={"I","IV","V","IX","X","XL","L","XC","C","CD","D","CM","M"};

void
toroman(int arabian)
{
    cout << "The equel roman number: ";
    for (int i = p - 1; i >= 0; --i) {
        while (arabian >= arab_mas[i]) {
            arabian -= arab_mas[i];
            cout << roman_mas[i];
        }
    }
}

int
main()
{
    int arabian;
    cout << "Enter the arabian number: ";
    cin >> arabian;
    toroman(arabian);
    getch();
}

[Bugrimov]

BDA интересный код, очень полезно следить за вашей перепиской.
Только поясните пожалуйста почему подключаются и iostream и cstdio. Разве iostream не достаточно для работы кода... ?

[BDA]

Bugrimov, честно говоря, подключал библиотеки "от балды". Для getch, по идее, нужна conio.h. Кстати, если уж менять код, то еще бы "развернул" массивы, так как ++ может быть быстрее -- (вроде ).

#include <iostream>
#include <conio.h>

using namespace std;

const int p = 13;
const int arab_mas[p]={1000, 900, 500, 400, 100, 90, 50, 40, 10, 9, 5, 4, 1};
const char *roman_mas[p]={"M", "CM", "D", "CD", "C", "XC", "L", "XL", "X", "IX", "V", "IV", "I"};

void
toroman(int arabian)
{
    cout << "The equel roman number: ";
    for (int i = 0; i < p; ++i) {
        while (arabian >= arab_mas[i]) {
            arabian -= arab_mas[i];
            cout << roman_mas[i];
        }
    }
}

int
main()
{
    int arabian;
    cout << "Enter the arabian number: ";
    cin >> arabian;
    toroman(arabian);
    getch();
}

[Bugrimov]

Честно говоря не знал, что инкремент (++) быстрее декремента (--)..
Лично я предпочитаю system("pause"); ну и конечно дописать #include <cstdlib>