Eh aca una calculadora basica en C

[thedaket]

Código PHP:

  /*Calculadora Básica también decimales*/
#include<stdio.h>
#include<stdlib.h>
#define VAR 3
float ope[VAR];
void suma(void);
void resta(void);
void multiplicacion(void);
void divicion(void);
void impreciones(void);
void resultado(void);
     void suma(void) {
          ope[0]=ope[1]+ope[2];
          return;
          }
     void resta(void) {
          ope[0]=ope[1]-ope[2];
          return;
          }
     void multiplicacion(void) {
          ope[0]=ope[1]*ope[2];
          return;
          }
     void divicion(void) {
          ope[0]=ope[1]/ope[2];
          return;
          }
     void impreciones(void) {
          printf("Cual es tu primer valor: ");
          scanf("%f",&ope[1]);
          printf("Cual es tu segundo valor: ");
          scanf("%f",&ope[2]);
          return;
          }
     void resultado(void) {
          printf("Tu resultado es: %.2f\n\n\a",ope[0]);
          return;
          }
int main (void) {
     system("color 2f");
     int op;
     printf(".:CALCULADORA BASICA:.\n\n");
     do {
     printf("1=Suma\n2=Resta\n3=Multiplicacion\n4=Divicion\n5=Salir\n\nElige una opcion: ");
     scanf("%d",&op);
     switch(op) {
     case 1:
     impreciones();
     suma();
     resultado();
     break;
     case 2:
     impreciones();
     resta();
     resultado();
     break;
     case 3:
     impreciones();
     multiplicacion();
     resultado();
     break;
     case 4:
     impreciones();
     divicion();
     resultado();
     break;
     }
     if (op<=0) {
              printf("La opcion tiene que ser mayor igual a uno\n");
              }
     if (op>5) {
               printf("Opcion invalida intenta otra vez\n\n\a");
               }
     if (op!=5) {
                printf("/\\Nueva Operacion/\\\n\n");
                }
     } while(op!=5);
     printf("Hasta luego!!!\n\a");
     system("pause");
     return 0;
} 
   

Espero les agrade Si alguien encuentra errores por favor haga llegarme los soy nuevo en esto así que espero puedan dar criticas positivas gracias

[cesar_casla]

Hola! errores no hay, pero algunos te criticarian por usar system() ya que no es una funcion portable...
Cosas para mejorar:
si a las funciones impreciones() y resultado(), los llamas en todos los case ¿no es mejor escribirlas una sola ves? (es como sacar un denominador comun :P)
El switch tiene un default donde se entra si la condicion no coincida con ninguno de los case, ahi podrias poner los printf de los if

Te dejo el codigo un poco mejorado:

Código C:

/*Calculadora Básica también decimales*/
#include<stdio.h>
#include<stdlib.h>
#define VAR 3
float ope[VAR];
void suma(void);
void resta(void);
void multiplicacion(void);
void divicion(void);
void impreciones(void);
void resultado(void);
     void suma(void) {
          ope[0]=ope[1]+ope[2];
          return;
          }
     void resta(void) {
          ope[0]=ope[1]-ope[2];
          return;
          }
     void multiplicacion(void) {
          ope[0]=ope[1]*ope[2];
          return;
          }
     void divicion(void) {
          ope[0]=ope[1]/ope[2];
          return;
          }
     void impreciones(void) {
          printf("Cual es tu primer valor: ");
          scanf("%f",&ope[1]);
          printf("Cual es tu segundo valor: ");
          scanf("%f",&ope[2]);
          return;
          }
     void resultado(void) {
          printf("Tu resultado es: %.2f\n\n\a",ope[0]);
          return;
          }
int main (void) {
     system("color 2f");
     int op;
     int flag; /* este flag va indicar si se ingreso una opcion correcta */
     do {
     flag=1; /* suponemos q la opcion es correcta, flag toma el valor 1 */
     printf(".:CALCULADORA BASICA:.\n\n");
     impresiones();
     printf("1=Suma\n2=Resta\n3=Multiplicacion\n4=Divicion\n5=Salir\n\nElige una opcion: ");
     scanf("%d",&op);
     switch(op) {
     case 1:
     suma();
     break;
     case 2:
     resta();
     break;
     case 3:
     multiplicacion();
     break;
     case 4:
     divicion();
     break;
     case 5:
         printf("FIN DEL PROGRAMA\n\n");
         break;
     default:
         printf("Opcion no valida...\n");
         flag=0; /* se ingreso una opcion incorrecta */
         break;
     }
     if(flag==1) /* solo imprime el resultado si se ingreso una opcion valida */
           resultado();
     } while(op!=5);
     printf("Hasta luego!!!\n\a");
     system("pause");
     return 0;
}
 

Ojo! que haya mejorado tu codigo no significa que este mal el tuyo, de echo esta perfecto; solo queria mostrart como mejorarlo un poco

Otra cosa que podrias hacer es ingresar directamente los caracteres "+ - * /". Y poner eso en los case del switch, ejemplo:

Código C:

printf("Ingrese la operacion\n");
scanf("%c",&op);
switch(op) {
     case '+':
          ..........
          break;
     case '-':
          .........
         break;
     case '*':
          ..........
          break;
     case '/':
          .........
         break;
   }
 

Fijate el uso de las comillas simples

Saludos

[thedaket]

Muchas gracias, y pues estoy aprendiendo y me da gusto este tipo de criticas constructivas ñ.ñ y tomare muy en cuenta tus comentarios

[thedaket]

uhh me marca error al compilar en la linea de la función suma

Código PHP:

  void suma(void) { 
   

[sam90]

Que errores te marca y que codigo te da el fallo?