0

Resulta que debo crear un código donde presente un menú con 4 funciones del , que serían strcat, strcmp, strcpy y sctcmd, donde se debe ingresar la opción a elegir e ingresar cualquier carácter, pero me presenta un error, y quisiera que me ayudaran.

    #include <stdio.h>    
#include <string.h>
int concat (char a,char b)
{
strcat (a,b);
return 0;
}
int comparar (char a,char b)
{
strcmp (a,b);
return 0;
}
int copiar (char a,char b)
{
strcpy (a,b);
return 0;
}
int tama (char a)
{
strlen (a);
return 0;
}
int main (void)
{
int opcion,caracterp,caracters,resultado;
    printf("\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n");
    printf("operaciones de funciones de cadenas de caracteres\n");
    printf("|------------------------------------------------------\n");
    printf("| Elije una opcion                                     \n");
    printf("| 1.- Concatenar                                       \n");
    printf("| 2.- Comparar                                         \n");
    printf("| 3.- Copiar                                           \n");
    printf("| 4.- Tamaño                                           \n");
    printf("|------------------------------------------------------\n");
    scanf("%i",&opcion);
    switch (opcion)
    {
        case 1:
                    printf("- Concatenar -\n");
                    printf("Introduce el primer caracter");
                    scanf("%c",caracterp);
                    printf("introduce el segundo caracter");
                    scanf("%c",caracters);
                    resultado = concat(caracterp, caracters);
                    printf("Su resultado es", resultado);
                    break;
        case 2:
                    printf("- Comparar -\n");
                    printf("Introduce el primer caracter");
                    scanf("%c",caracterp);
                    printf("introduce el segundo caracter");
                    scanf("%c",caracters);
                    resultado = comparar(caracterp, caracters);
                    printf("Su resultado es", resultado);
                    break;
        case 3:
                    printf("- Copiar -\n");
                    printf("Introduce el primer caracter");
                    scanf("%c",caracterp);
                    printf("introduce el segundo caracter");
                    scanf("%c",caracters);
                    resultado = copiar(caracterp, caracters);
                    printf("Su resultado es", resultado);
                    break;
        case 4:
                    printf("- Tamaño -\n");
                    printf("Introduce el primer caracter");
                    scanf("%c",caracterp);
                    printf("introduce el segundo caracter");
                    scanf("%c",caracters);
                    resultado = tama(caracterp);
                    printf("Su resultado es", resultado);
                    break;
        default:
                     printf("Esa no es una opcion valida\n");
                     printf("%i no esta en el menu\n", opcion);
                     break;
                 }
return 0;

}

1
  • 1
    Lo que tienes mal es lo que dice las advertencias. Como no puedo ver las advertencias no puedo saber qué tienes mal. Commented el 6 jul. 2017 a las 6:59

2 respuestas 2

0

soy un poco nuevo en esto pero creo saber dónde está el error, y es que le pasas los valores por referencia pero la función no retorna lo que quieres que retorne, es decir al llamar a

int comparar (char a,char b)
{
strcmp (a,b);
return 0;
}
Se reciben los valores y strcmp compara los caracteres y luego la función regresa 0, mientras que, para obtener el resultado correcto según lo que puedo ver en el program debería regresar el valor de esa comparación entonces debería quedar así:

int comparar (char a,char b)
{
res = strcmp (a,b);
return res;
}

Y eso estaría correcto pues una función de tipo entero regresa un valor de tipo entero que no es precisamente 0, sino el resultado de strcmp(a,b) (que es 0 cuando no hay diferencia entre los caracteres).

Ese sería el problema #1 que logro identificar. El #2 es el siguiente:

Todas las funciones las tienes como tipo int y algunas funciones que utilizas no retornan un entero:

  • strcat me parece que retorna tipo string.
  • strcmp retorna tipo integer.
  • strcpy me parece retorna tipo string.
  • strlen retorna tipo integer.

De igual forma, si cambias los valores de la función también lo haces con los de printf(%c)

Completo C++

#include <stdio.h>    
#include <string>
#include <conio.h> //para getch() (que se cierre hasta apretar alguna tecla)
#include <iostream> // para cin y cout
std::string concat(char* a, char* b)
{
    return strcat(a, b);
}
int comparar(char* a, char* b)
{
    return strcmp(a, b);
}
std::string copiar(char* a, char* b)
{
    return strcpy(a, b);
}
int tama(char* a)
{
    return strlen(a);
}
int main(void)
{
    int opcion, resultado_entero;
    char caracterp[10] = { 0 }, caracters[10] = {0};
    std::string resultado_string;
    printf("\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n");
    printf("operaciones de funciones de cadenas de caracteres\n");
    printf("|------------------------------------------------------\n");
    printf("| Elije una opción                                     \n");
    printf("| 1.- Concatenar                                       \n");
    printf("| 2.- Comparar                                         \n");
    printf("| 3.- Copiar                                           \n");
    printf("| 4.- Tamaño                                           \n");
    printf("|------------------------------------------------------\n");
    scanf("%i", &opcion);
    switch (opcion) {
    case 1:
        printf("- Concatenar -\n");
        printf("Introduce el primer caracter");
        std::cin >> caracterp;
        std::cout << "\nIntroduce el segundo caracter: ";
        std::cin >> caracters;
        resultado_string = concat((char*)caracterp, (char*)caracters);
        std::cout << "\nResultado: " << resultado_string;
        getch();
        break;
    case 2:
        printf("- Comparar -\n");
        printf("Introduce el primer caracter");
        std::cin >> caracterp;
        std::cout << "\nIntroduce el segundo caracter: ";
        std::cin >> caracters;
        resultado_entero = comparar((char*)caracterp, (char*)caracters);
        std::cout << "\nResultado: " << resultado_entero;
        getch();
        break;
    case 3:
        printf("- Copiar -\n");
        printf("Introduce el primer caracter");
        std::cin >> caracterp;
        std::cout << "\nIntroduce el segundo caracter: ";
        std::cin >> caracters;
        resultado_string = copiar((char*)caracterp, (char*)caracters);
        std::cout << "\nResultado: " << resultado_string;
        getch();
        break;
    case 4:
        printf("- Tamaño -\n");
        printf("Introduce el caracter:");
        std::cin >> caracterp;
        resultado_entero = tama((char*)caracterp);
        std::cout << "\nResultado: " << resultado_entero;
        getch();
        break;
    default:
        printf("Esa no es una opcion valida\n");
        printf("%i no esta en el menu\n", opcion);
        break;
    }
return 0;
}
17
  • Entendi un poco el primero problema, pero el segundo no, intento cambiar el codigo pero me sigue dando el mismo error de advertencia: passing argument 1 of "strcat" makes pointer from integer without a cast Commented el 6 jul. 2017 a las 2:25
  • Con respecto a que si se que puedo ponerlo directamente, pero me lo pidieron llamarlo en una función. Commented el 6 jul. 2017 a las 2:28
  • Sí,por cierto, se me olvidó, no declares resultado como entero, sino hasta que sepas que es entero pues como mencioné antes, las funciones devuelven diferente tipo. Y el error es que strcat trabaja con punteros, con char* así que es probable sea ese el error.
    – ben
    Commented el 6 jul. 2017 a las 2:46
  • Para que te podamos ayudar mejor, por favor específica más cuál es el problema pues puede haber una infinidad de problemas
    – ben
    Commented el 6 jul. 2017 a las 2:47
  • Ya declaré los char* char* concat (char* as,char* bs) { return strcat (as,bs); } pero me sigue diciendo lo mismo + assigment makes integer from pointer without a cast... La verdad me siento muy mal y la profesora que nos enseña es muy... como decirlo, desgraciada y disculpe la palabra, recien nos mandó a investigar que es un puntero y ya nos manda a meter eso en un ejercicio, y no entiendo lo que es nada. Commented el 6 jul. 2017 a las 2:49
0
#include <stdio.h>    
#include <string.h>

char concat ()
{
    char prim[100];
    char seg[100];
    printf ("Ingrese dos cadenas de caracteres a concatenar\n\n");
    scanf ("%s",prim);
    scanf ("%s",seg);
    strcat(prim,seg);
    printf ("\n %s \n",prim);
    return 0;   
}
char comp ()
{
    char pr[100];
    char sr[100];
    printf ("\n Ingrese dos cadenas de caracteres a comparar\n\n");
    scanf ("%s",pr);
    scanf ("%s",sr);
    int cond;
    cond= strcmp(pr,sr);
    if (cond<0||cond==1){
        printf ("\n Ambas Cadenas son diferentes");
    }
    else 
    if (cond==0){
        printf ("\n Ambas Cadenas son Iguales");
            }
    return 0;
}

char cantidad ()
{
    char can[100];
    int aux;
    printf ("\n Escriba una Palabra \n\n");
    scanf ("%s",can);
    aux=strlen(can);
    printf ("\n La cantidad de caracteres que posee son: %d",aux);
    return 0;
}

char copiar ()
{
    char cad1[30]="Usted es de :";
    char cad2[30];
    char cad3[30];
    printf (" \n En que ciudad vives? ");
    scanf ("%s",cad3);
    strcpy (cad2,cad1);
    strcpy (cad2+20,cad3);
    printf ("\n\n%s",cad2);
    return 0;       
}

int main(){
   int opc;
      printf("\n Elige una de las siguientes opciones\n");
      printf("\n 1.- Concatenar");
      printf("\n 2.- Comparar");
      printf("\n 3.- Cantidad de caracteres");
      printf("\n 4.- Copiar\n\n");
      scanf("%d",&opc);
      switch(opc){
         case 1: concat(); break;
         case 2: comp(); break;
         case 3: cantidad(); break;
         case 4: copiar(); break;
         default: printf(" \n Haz elegido una opción incorrecta\n\n");
      }
   return 0;
}

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.