Foro de elhacker.net

Para matar mi aburrimiento tuve que hacer esto y ahora ya me tuvo encerrado esto... ayudenme a corregirlo... donde esta el error?? !!

#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
 
main(){
       char romano[]={};
       printf("Ingrese Numero Romando: ");
       scanf("%s",&romano);
       fflush(stdin);
 
       char rom[]={' ','I','V','X','L','C','D','M'};
       int valor[]={0,1,5,10,50,100,1000};
 
       convertidor(romano,rom,valor);
}
void convertidor(char rom1[],char rom2[],int v[]){
     int suma=0,anterior=0;
     char letra=' ';
     int i,j;
     for(i=0;i<rom1.length;i++){
                                letra=rom1[i];
                                for(j=0;j<rom2.length;j++){
                                                           if(letra==rom2[j]){
                                                                              suma=suma+v[j];
                                                                              if(anterior<valor[j]){
                                                                                                    suma=suma-anterior*2;
                                                                                                    anterior=valor[j];
                                                                              }
                                                                              else{
                                                                                   anterior=valor[j];
                                                                              }
                                                           }
                                }
     }
     printf("Numero: %d",suma);
}
 

Ese código no compila y tiene algunos errores que me cuesta entender que sean por accidente... No será para que te hagamos la tarea no?  :silbar:

no nada que ver... solo entiendo que dice incompatibilidad de las variables del metodo convertidor, a parte parece que toy manejando mal los arreglos char... comenzando dsd alli los estoy manipulando bien?.... pienso q radica mi error alli...

A ver, explicanos detalladamente como funciona el programa y que errores te da el compilador.

Funciona así:

Usuario ingresa un número romano y para que pueda convertir mas sencillo de romano a decimal hice un arreglo char que solo alberga romanos y otro con numeros decimales.

Pasando ya todo al metodo convertidor con variables que serán fundamentales para la conversion el char del cual ingresa el usuario un numero romano, el otro arreglo de romanos que hice mas un arreglo de enteros, cree tres variables, suma para el cual se sumaran de forma acumulativa por medio del ciclo for que cree, una variable char letra que servirá para comparar la posición de cada letra con los 2 arreglos que cree en un principio digamos XVII
primero va X, entonces recorrera primero mediante el chat buscar X y luego su posición sera igual para el metodo de enteros que hice y me asigna a un valor entero para lo cual despues se acumulará con el siguiente que sera V y sucesivamente--.-
10 + 5....
En el caso espeial si tuviera CXC.... cree la condicion
if(anterior<v[j]) asi restará en lugar de sumar con el valor que almacenará con el valor anterior...

ERROR: está en que dice conflicting types for "convertidor"....

vamos

Vamos, has hecho tareas más dificiles no te hagas de rogar viejo XDDDD  :laugh: :laugh: :xD :xD ;-) ;-) :silbar: :silbar:

Mira te corregi el error de la longitud del arreglo, no puede usarse "length()" en un arreglo de caracteres, para ello, esta el "strlen()" que hace lo mismo, devuelve un numero entero con la longitud en un arreglo, puede usarse en un char, char[] y en string, funciona perfectamente.

Pero dejame decirte que hay algunas cosas te falta corregir, como la asignacion de la letra, y que pasa si es digamos un numero romano con una letra que antecede a una letra de un valor mayor, ahi no se suma sino que se resta, como cuando tienes "IV" o "IX", me entiendes

sigue trabajando, estamos para ayudarte, pero si quieres aprender a hacerlo bien, tienes que sufrir y llorar sangre, amchacon ha dejado guias que realmente te dejan dolores de cabeza XDDDDDD pero funcionan, no se que le pasa el día de hoy, sera por el lunes  :silbar: :silbar:

#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
 
main(){
       char romano[]={};
       printf("Ingrese Numero Romando: ");
       scanf("%s",&romano);
       fflush(stdin);
 
       char rom[]={' ','I','V','X','L','C','D','M'};
       int valor[]={0,1,5,10,50,100,1000};
 
       convertidor(romano,rom,valor);
}
void convertidor(char rom1[],char rom2[],int valor[]){
     int suma=0,anterior=0;
 
     // aqui hay un error, letra es un char
     char letra=' ';
     int i,j;
 
     printf("longitud char romano rom1: %d \n", strlen(rom1));
     printf("longitud char romano rom2: %d \n\n", strlen(rom2));
 
     //strlen(<cadena>): Devuelve la longitud de la cadena sin tomar en cuenta
     //el caracter de final de cadena.
 
     for(i=0; i<strlen(rom1) ;i++){
        letra=rom1; //aqui tratas de asignar letra como un arreglo de caracteres
 
        for(j=0; j<strlen(rom2); j++){
 
           if(letra==rom2[j]){
              suma=suma+valor[j];
 
              if(anterior<valor[j]){
                suma=suma-anterior*2;
                anterior=valor[j];
              }
              else{
                anterior=valor[j];
              }
 
           }
        }
     }
     printf("Numero: %d", suma);
}
 

jajajajaja me mataste de risa con tu comentario ;-) ;-) ;-) ;-) xD!!!! bueno toy iniciando en C pa que lo sepas ya que he estado muy pegado a tecnologia java y desarrollo de programacion movil jajaja y eso era lo que necesitaba del char con strlen jejeje no sabia pa serte sincero... bueno gracias con esto es mas que suficiente...  ;-)

#include <stdio.h>
#include "Herramientas.h"
 
int main()
{
 InitPistola(MUNICION_INFINITA);
 
 char acierto = DispararPistola(dato000);
 
 while (!acierto)
  {
    acierto = DispararPistola(dato000);
  } 
 
  puts("Muahahaha ^^");
 
  return 0;
}

Mis disculpas, esque hay un cierto "personaje" que hacia eso a menudo (ponía código inconexo para que al final le hiciéramos la tarea).


 :silbar: :silbar: :silbar: :silbar: :silbar:

 :silbar: :silbar: :silbar: :silbar: :silbar:

por ultima vez no espero que me lo hagan ¬¬ !!!!

Una pequeña variante operativa, sin el cero que no tiene sentido en los números romanos:

#include <stdio.h>
#include <string.h>
 
int main(){
    char romano [100]={0};
    printf("Ingrese Numero Romano: ");
    scanf ("%s",romano);
    printf ("Romano: %s\n",romano);
    char rom[8]={'I','V','X','L','C','D','M'};
    int valor[7]={1,5,10,50,100,500,1000};
    convertidor(romano, rom, valor);
    return 0;
}
void convertidor ( char romano [], char rom [9],int valor [8])
{
    int suma=0,i,j;
    for (i=0;i<strlen(romano);i++)
        for (j=0;j<9;j++){
            if (romano[i]==rom[j]){
                suma+=valor [j];
                break;
            }
            else
                continue;
        }
    printf ("Numero: %d",suma);
}
 

Saluditos!!. ....(http://st.forocoches.com/foro/images/smilies/ciao.gif)

XDDDDDDDDDD bien dicho amigo mio, como te quedo esa amchacon XDDDD  :laugh: :laugh: :laugh:

la de leo esta mejor, puesto que usa de una vez las variables que fueron llamadas a la función, es mucho mejor, aunque es confuso al principio XD

No veo donde puede estar la confusión, salvo porque no explico ni pijo lo que va haciendo el código, mea culpa.

En esencia se va pasando cada caracter de la cadena introducida, romano, a la función y en ella se va comparando con los caracteres/valores en romano, que conforman el array rom. Caso de encontrar una coincidencia retorno el equivalente en decimal de la variable valor. Así de sencillo.

Lo que no entendí fue el uso, a mi parecer innecesario, de la variable anterior.

No dejo de reconocer que el código que posteo 05K4R1N era muy deficiente, le faltaba el tipo int de la función main, el correspondiente return 0, el uso de .lengh, más cercano al C++ y alejado del C -creo que ahí confundió conceptos-, la presencia de la librería conio.h -para nada-, el uso de la función  indeseable fflush(stdin) y el resto de código muy disperso/descuidado.

Pero, en fin, parece que algo se lo había currado.

Tampoco entiendo el uso de una función para algo tan simple y  a la que sólo se le va a llamar una vez, para eso no hacen falta tantas alforjas. Mi criterio es reservar el uso de funciones para casos repetitivo, códigos muy complejos o que se va a trabajar entre varios y cada uno se va a encargar de una parte/función, pero en este caso ¡puafffff!, me sobra la función. Si la puse fue porque imagino será una tarea que le piden haga con una función.

Saluditos!. ... ..(http://st.forocoches.com/foro/images/smilies/ciao.gif)

Hey eso de la confusión es cuando se comienza a trabajar programación, siempre es "confuso" ver como trabaja C, solo era eso, siempre es dificil empezar, solo decia eso.  :silbar: :silbar:

dat000 lo que sucede es que recien toy comenzando con lenguaje C y si hice varias cosas innecesarias, vaya que me estan dando un jalon de orejas ustedes aunque si deberia agradecerles por eso =)... por otro lado les digo que no volverá a ocurrir

Hola 05K4R1N.

Otra opción:

#include <stdio.h>
#include <stdlib.h>
 
char *DecToRom(int num);
 
int main()
{
  int num;
  printf("Numero decimal:");
  scanf("%d",&num);
  while(getchar()!='\n');
  printf("%d en numeros romanos: %s\n",num,DecToRom(num));
  getchar();
  return 0;
}
 
char *DecToRom(int num)
{
  char *uni[] = {"","I","II","III","IV","V","VI","VII","VIII","IX"};
  char *dec[] = {"","X","XX","XXX","XL","L","LX","LXX","LXXX","XC"};
  char *cen[] = {"","C","CC","CCC","CD","D","DC","DCC","DCCC","CM"};
  char *udm[] = {"","M","MM","MMM"};
  char result[50];
 
  strcpy(result, udm[num/1000]);
  strcat(result, cen[num/100%10]);
  strcat(result, dec[num/10%10]);
  strcat(result, uni[num%10]);
  return (char*)result;
}
 

Saludos.

El problema con ese codigo es que si ingresas por ejemplo el cuatro (IV) el va a decir que es seis...

salu2 :)

Advertencia - mientras estabas escribiendo, una nueva respuesta fue publicada. Probablemente desees revisar tu mensaje.

te lo decia por esto q tu mismo dices q no lo entendias... Pero bueno si dices q esta bien no pasa nada  ;-)

Volviendo a lo q planteaba quien puso el hilo


mirar los numeros romanos :)
https://es.wikipedia.org/wiki/Numeraci%C3%B3n_romana

Solo señalaba alguna deficiencia obvia, pero gracias por ofrecerme tu ayuda

salu2 :)

Hola.

Disculpas 05K4R1N, ahora caigo que entendí tu consulta al revés... (msg #14)   :-[

Podría ser:

#include<stdio.h>
#include<string.h>
#include<ctype.h>
 
#define NRO_ELEM(x) (sizeof(x) / sizeof(x[0]))
 
int RomToDec(char *nrom);
 
int main()
{
  char *roman = "MMMDLXXXIV";
  printf("%s = %d",roman,RomToDec(roman));
  getchar();
  return 0;
}
 
int RomToDec(char *nrom)
{
  char *NROROM[][9] = {
    {"M","MM","MMM"},
    {"C","CC","CCC","CD","D","DC","DCC","DCCC","CM"},
    {"X","XX","XXX","XL","L","LX","LXX","LXXX","XC"},
    {"I","II","III","IV","V","VI","VII","VIII","IX"}};
  int fil, col;
  int res =0, div = 1000;
  char *aux = (char*) nrom;
 
  for(fil=0; aux[fil]; fil++) aux[fil] = toupper(aux[fil]);
  for(fil=0; fil < NRO_ELEM(NROROM); fil++) {
    col = (fil == 0 ? 2 : 8);
    while ( col ) {
      if (strncmp(NROROM[fil][col], aux, strlen(NROROM[fil][col]))==0) {
        res += (col+1) * div;
        aux = &aux[strlen(NROROM[fil][col])];
      }
      col--;
    }
   div /= 10;
  }
  return (int)res;
}
 

Saludos.

Escribí un código distinto que no contempla los numeros IX, XL, etc, simplemente suma las letras, para algunos valores funciona, para otros no.

La idea no se como implementarla es:
1. Longitud del dato tecleado usando strlen().
2. Si es par leer primer y segundo caracter, revisar si hay alguna combinacion tipo IV, IX, XL, XC, CD, CM..., si verdadero restar digitos, de lo contrario sumar.
3. Si la longitud es impar se lee primer y segundo caracter, segundo y tercero, etc. Las condiciones son las mismas, buscar casos donde se tenga que restar y no sumar

Estoy pensando en algo como

//stringlenght = longitud de dato
 
if((stringlenght%2)==0) //Cuando la longitud es par
{
    for(i=0;i<(stringlenght-1);i+2)//lectura de primer caracter en par.
    {
        /*Aqui se me ocurre usar algo como isalpha()
        Si I entonce digito=1
        Si X ,digito= 10, etc
        se asigna el valor del caracter a una variable PrimerDigito
        */ 
        for(j=1;numero_romano[j]!=NULL;j+2) //lectura de segundo caracter en par.
        {
            /*Aca tambien algo similar
            si C = 100
            si D...
            tambien asignamos el valor del segundo caracter a SegundoDigito
            */
            if((PrimerDigito - SegundoDigito)<0)
            {
                //Si es negativo quiere decir que hay numeros tipo IV, IX, XL, etc
                //entonces aqui se asigna el resultado de los dos digitos en una nueva variable global
            }
        }
    }
}
else
{
    //Codigo para impar
}

Esto es el codigo que solamente suma:

int main()
{  
int i,romano=0,caracteres=0;
    char numero[10];
    printf("Escribe un numero romano:");
    gets(numero);
    for(i=0;i<10;i++)
    {
        switch(numero[i])
        {
            case 'M': romano+=1000;
                break;
            case 'D': romano+=500;
                break;
            case 'C': romano+=100;
                break;
            case 'L': romano+=50;
                break;
            case 'X': romano+=10;
                break;
            case 'V': romano+=5;
                break;
            case 'I': romano+=1;
                break;
        }
    }
    printf("\n%d",romano);
}

alguna idea?
saludos!!

Encontre una forma para los numeros cuya longitud es par pero al correr el código se queda parado en la linea 21, meti un printf y esta arrojando el valor pero no continua :(

El compilador me esta dando una advertencia:
Incompatible implicit declaration of built-in function 'strlen'[Enabled by default] (se me olvido agregar <string.h>)

#include <stdio.h>
#include <ctype.h>
#include <string.h>
#define M 1000
#define D 500
#define C 100
#define L 50
#define X 10
#define V 5
#define I 1
int main()
{
    int i,j,caracteres=0,arabigo[20]={0},resultado=0;
    char numero[20];
    printf("Escribe un numero romano:");
    gets(numero);
    caracteres = strlen(numero);
    for(i=0;i<caracteres;i++) //Convertir minusculas a mayusculas
    {
        numero[i] = toupper(numero[i]);
    }
printf("caracteres: %d\n",caracteres);
    for(i=0;i<caracteres;i++)
    {
        switch(numero[i])
        {
            case 'M':
                {
                    arabigo[i]= M;
                    break;
                }
            case 'D':
                {
                    arabigo[i]= D;
                    break;
                }
            case 'C':
                {
                    arabigo[i]= C;
                    break;
                }
            case 'L':
                {
                    arabigo[i]= L;
                    break;
                }
            case 'X':
                {
                    arabigo[i]= X;
                    break;
                }
            case 'V':
                {
                    arabigo[i]= V;
                    break;
                }
            case 'I':
                {
                    arabigo[i]= I;
                    break;
                }
           }
       }
        if(caracteres==1)
        {
            resultado+= arabigo[0];
        }
        else if(caracteres%2==0) //Cuando el numero que escribimos es par.
        {
            for(j=0;j<caracteres;j+2)
            {
                if(arabigo[j]-arabigo[j+1]<0)
                {
                    resultado+=arabigo[j+1]-arabigo[j];
                }
                else
                {
                    resultado+= arabigo[j]+arabigo[j+1];
                }
            }
        }
        printf("El numero romano %s en arabigo es: %d",numero[20],resultado);
}