Funcion impresa

Iniciado por iAmChapin_, 31 Agosto 2015, 02:27 AM

0 Miembros y 1 Visitante están viendo este tema.

iAmChapin_

Buenas! Vengo a molestar nuevamente. Tengo este código y funciona a la perfección, pero me di cuenta de un error en la funcion, al principio era tipo int y returnaba el valor de la variable a, pero al hacer eso me imprimia el numero ingresado, decidi convertirlo a tipo void que no devuelve nada y me salta un error molesto en la linea 82: Illegal structure operation. Pensé que era porque estaba todo junto pero no, aun separandolo por lineas diferentes, siempre me indica el error en el esas lineas, y aun cambiando los ; por << siempre salta uno que otro error. Alguien podria darme una solucion sin que la funcion no devuelva nada? Les dejo el codigo para que puedan ver que pueden cambiar

Código (cpp) [Seleccionar]
#include <iostream.h>
#include <conio.h>
#include <windows.h>

//FUNCION TRANSFORMAR FECHA
void fechaTransformada(int a){

//DETERMINAMOS EL MES INGRESADO CON SWITCH
 switch(a){

     case 1:
     textcolor(15); cprintf("Enero");
     break;

     case 2:
     textcolor(15); cprintf("Febrero");
     break;

     case 3:
     textcolor(15); cprintf("Marzo");
     break;

     case 4:
     textcolor(15); cprintf("Abril");
     break;

     case 5:
     textcolor(15); cprintf("Mayo");
     break;

     case 6:
     textcolor(15); cprintf("Junio");
     break;

     case 7:
     textcolor(15); cprintf("Julio");
     break;

     case 8:
     textcolor(15); cprintf("Agosto");
     break;

     case 9:
     textcolor(15); cprintf("Septiembre");
     break;

     case 10:
     textcolor(15); cprintf("Octubre");
     break;

     case 11:
     textcolor(15); cprintf("Noviembre");
     break;

     case 12:
     textcolor(15); cprintf("Diciembre");
     break;

 }

}

int dia;
int mes;
int cuatro;

void main(){

  gotoxy(6,4); cprintf("Transformo tu fecha de este tipo: 4/07/2011 asi: 4 de Julio del 2011");
  textcolor(15);
  gotoxy(29,5); cprintf("Ingresa una fecha:");
  textcolor(15);
  gotoxy(33,7); cprintf("/");
  textcolor(15);
  gotoxy(38,7); cprintf("/");
  gotoxy(30,7); cin >> dia;
  gotoxy(35,7); cin >> mes;
  gotoxy(40,7); cin >> cuatro;
  textcolor(15);
  gotoxy(26,9); cprintf("La fecha transformada es:");
  textcolor(15);
  gotoxy(27,11); cout << dia; cprintf(" de "); cout << fechaTransformada(mes); cprintf (" del "); cout << cuatro;

getch();

}

Stakewinner00

#1
Varias cosas.
1. No existe la lib iostream.h, es "iostream" (sin el .h) ya que las librerías estándar de C++ no acaban en .h
2. la función main debería devolver un int (que usualmente indica si el programa finalizo exitosamente o no)
3. cout << fechaTransformada(mes) cout no puede imprimir por pantalla una función que no devuelve dato, justamente por eso. Si la función devuelve nada como va a imprimir algo?. En todo caso podrías llamar a la función tal cual. "fechaTranformada(mes)"
4. No tendrían que haber variables globales

Con estos cambios el código final vendría a ser algo como

Código (cpp) [Seleccionar]
   #include <iostream>
   #include <conio.h>
   #include <windows.h>

    using namespace std;
   
   //FUNCION TRANSFORMAR FECHA
   void fechaTransformada(int a){
   
   //DETERMINAMOS EL MES INGRESADO CON SWITCH
     switch(a){
   
         case 1:  textcolor(15); cprintf("Enero");
         break;
   
         case 2:
         textcolor(15); cprintf("Febrero");
         break;
   
         case 3:
         textcolor(15); cprintf("Marzo");
         break;
   
         case 4:
         textcolor(15); cprintf("Abril");
         break;
   
         case 5:
         textcolor(15); cprintf("Mayo");
         break;
   
         case 6:
         textcolor(15); cprintf("Junio");
         break;
   
         case 7:
         textcolor(15); cprintf("Julio");
         break;
   
         case 8:
         textcolor(15); cprintf("Agosto");
         break;
   
         case 9:
         textcolor(15); cprintf("Septiembre");
         break;
   
         case 10:
         textcolor(15); cprintf("Octubre");
         break;
   
         case 11:
         textcolor(15); cprintf("Noviembre");
         break;
   
         case 12:
         textcolor(15); cprintf("Diciembre");
         break;
   
     }
   
   }
   
   
   int main(){

   int dia;
   int mes;
   int cuatro;
   
      gotoxy(6,4); cprintf("Transformo tu fecha de este tipo: 4/07/2011 asi: 4 de Julio del 2011");
      textcolor(15);
      gotoxy(29,5); cprintf("Ingresa una fecha:");
      textcolor(15);
      gotoxy(33,7); cprintf("/");
      textcolor(15);
      gotoxy(38,7); cprintf("/");
      gotoxy(30,7); cin >> dia;
      gotoxy(35,7); cin >> mes;
      gotoxy(40,7); cin >> cuatro;
      textcolor(15);
      gotoxy(26,9); cprintf("La fecha transformada es:");
      textcolor(15);
      gotoxy(27,11); cout << dia; cprintf(" de "); fechaTransformada(mes); cprintf (" del "); cout << cuatro;
   
   getch();
   }



iAmChapin_

#2
Se me olvidó aclarar que trabajo con un compilador viejo, Borland 5.0.2 ya que de momento ese utilizo para los ejercicios de universidad, por lo que si ves cosas obsoletas no les hagas caso (void main, iostream.h). Y excelente, salí de la duda con eso que me explicaste, muchísimas gracias por la ayuda y muchas gracias por la solución amigo!  ;-) ;-) ;-)

Stakewinner00

Cita de: iAmChapin_ en 31 Agosto 2015, 02:49 AM
Se me olvidó aclarar que trabajo con un compilador viejo, Borland 5.0.2 ya que de momento ese utilizo para los ejercicios de universidad, por lo que si ves cosas obsoletas no les hagas caso (void main, iostream.h). Y excelente, salí de la duda con eso que me explicaste, muchísimas gracias por la ayuda y muchas gracias por la solución amigo!  ;-) ;-) ;-)
No es que sean obsoletas, es que están mal.

en C antes de que se estandarizara muchos usaban void main(), pero en C++ esta especificado que la función main tiene que devolver un int. Tienes un buen post que lo explica https://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html