comparar cadenas en C

[adalsuper]

muy buenas, estaba haciendo una funcion en C que me compara 2 cadenas entradas desde teclado.
el problema es que me dice siempre que son iguales
codigo:

Código C:

#include<stdio.h>
#define MAX 20
 
 
int comparaCad(char cad1[MAX],char cad2[MAX]);
 
 
int main(){
  char cad1[MAX],cad2[MAX];
  int result;
 
  printf("introduzca 1 cadena:\n");
  scanf("%s",cad1);
  printf("introduzca 1 cadena:\n");
  scanf("%s",cad2);
 
  result = comparaCad(cad1,cad2);
 
  if(result==0)
    printf("cadenas iguales\n");
  else{
    if(result<0)
      printf("cad1 es menor que cad2\n");
    else
      printf("cad1 es mayor que cad2\n");
  }  
 
  return 0;
}
 
int comparaCad(char cad1[MAX],char cad2[MAX]){
  int i,comp,j;
  char c1,c2;
 
  for(i=0,j=0; cad1[j]==cad2[i] && i<MAX && cad1[j]!='\0' && cad2[i]!='\0';++i,++j){
    c1 = cad1[j];
    c2 = cad2[i];
  }
  
  if(c1 == '\0' && c1!=c2)
    comp=-1;
  else{
    if(c2=='\0' && c2!=c1)
      comp=1;
    else{  
  if(c1 == c2)
    comp=0;
  else{
    if(c1 < c2)
      comp=-1;
    else
      comp=1;
  }
    }
  }
 
  return comp; 
}
 

[cesar_casla]

Hola! en el for tenes algunas condiciones de mas, i<MAX no es necesario.
Tenes q usar cad1[j]!='\0' o cad2[j]!='\0' no es necesario las dos juntas..
Pero el error del porque siempre te da que las cadenas son iguales es por usar c1 y c2. Cuando las cadenas son distintas la condicion del for es falso por lo que c1 y c2 siguen teniendo los mismos caracteres del ciclo anterior, por lo tanto siempre van a seguir siendo iguales. Para resolver esto tenes que usar directamente cad1[j] y cad2[i] en los if
Otra cosa q esta demas es usar dos subindices, con uno solo para los dos arreglos alcanza

Saludos

[W_NEMESIS]

prueba este codigo, es mas eficiente y mas corto

Código c++:

#include <cstdlib>
#include <iostream>
#include <string>
 
using namespace std;
 
#include "adjunto.h"
 
char fra[100],frad[100];
int x,r,v;
 
int main(int argc, char *argv[])
{   
    cout<<"primera frase: "<<endl;
    cin.getline(fra,100);
    strupr(fra);
    
    cout<<endl<<"frase a buscar dentro de la primera frase: "<<endl;
    cin.getline(frad,100);
    strupr(frad);
    int v=strlen(frad);
    x=comparar(fra,frad,x);
    system("cls");
    if (x>=0)
    {
    cout<<"La Palabra Fue Encontrada en las Posiciones: ";
    for (r=0;r<v;r++) cout<<x+(r+1)<<" ";
}
    else cout<<"La Palabra no Fue Encontrada"<<endl;
    cout<<endl<<endl;
    
    system("PAUSE");
    return EXIT_SUCCESS;
}
 

no te olvides de calificar y de pedir explicaion si hace falta