¿Que hay de mal en estas funciones?

[Doommarine_Maxi]

Hola, pues estoy armando unas funciones para manejar mas cómodo las cadenas en C.
Y estoy teniendo un problema con este código.


Funciones de cadenas

Código C:

/*arrays.h*/
#include <stdio.h>
 
int LongCad(char* cad)
{
    int x=0;
    while(cad[x] != '\0')x++;
    return x;
}
int GetCadPos(char* txt, char* cad, int startpos)
{
    int CadLong = LongCad(cad); 
    int TxtLong = LongCad(txt);
    char* tmp = (char*)calloc(CadLong, sizeof(char));
    int i;
    int x;
    for (i=startpos; i <= (TxtLong-CadLong); i++)
    {
        for (x=0; x < CadLong; x++)
            tmp[x] = txt[i+x];
        if (isSameCad(tmp, cad) == 1)
            return i;
    }
    return -1;
}
int isSameCad(char* cad1, char* cad2)
{
    int Long1 = LongCad(cad1);
    int Long2 = LongCad(cad2);
    if (Long1 != Long2)
        return 0;
    int x;
    for (x=0; x < Long2; x++)
    {
        if (cad1[x] != cad2[x])
            return 0;
    }
    return 1;
}
char* ReplaceBetween(char* cadena, char* inicio, char* fin, char* remplazar)
{
    int Pos_I = GetCadPos(cadena, inicio, 0)+(LongCad(inicio)-1);
    int Pos_F = GetCadPos(cadena, fin, Pos_I);
    int rLong = LongCad(remplazar)-1;
    int ExtraSize = rLong - (Pos_F-Pos_I);
    int CadenaLong = LongCad(cadena);
    char* NewCad = (char*)calloc((CadenaLong+ExtraSize), sizeof(char));
    
    int x;
    for (x=0; x <= Pos_I; x++)
        NewCad[x] = cadena[x];
    int y=x;
    int i;
    int z;
    for (i=0; i <= rLong; i++)
        NewCad[y++] = remplazar[i];
    for (z=Pos_F; z <= CadenaLong; z++)
        NewCad[y++] = cadena[z];
 
    return NewCad;
}
char* ConstToCad(const char* cad)
{
    return (char *)(cad);
}
char* GetCadBetween(char* cadena, char* inicio, char* fin)
{
    int Pos_I = GetCadPos(cadena, inicio, 0)+(LongCad(inicio)-1);
    int Pos_F = GetCadPos(cadena, fin, Pos_I);
    char* NewCad = (char*)calloc((Pos_F-Pos_I)+1, sizeof(char));
   // printf("%i\n", Pos_I);
    int i;
    int x=0;
    for (i =Pos_I; i < Pos_F; i++)
        NewCad[x++] = cadena[i];
    
    return NewCad;
}
 

Main.c

Código C:

#include <stdio.h> 
#include "arrays.h"
#ifdef WINDOWS
    #include <direct.h>
    #define GetCurrentDir _getcwd
#else
    #include <unistd.h>
    #define GetCurrentDir getcwd
#endif
 
void S_pause()
{
     char c;
     do
     {
          c = getchar();
     } while (c != '\n');     
}
 
int main()
{
    char* cad2;
    cad2 = ReplaceBetween(ConstToCad("Stealzps<Doom>Hola</Doom>"), ConstToCad("<Doom>"), ConstToCad("</Doom>"), "Soy el Doom 4");
    printf("Prueba: %s\n", cad2); //Esto funciona bien
    char* cad3;
    cad3 = GetCadBetween(ConstToCad("Stealzps<Doom>Hola</Doom>"), ConstToCad("<Doom>"), ConstToCad("</Doom>")); 
    printf("Pueba2: %s", cad3);  //Esto no
    S_pause();
    return 0;
}
 

Salida:
Stealzps<Doom>Soy el Doom 4</Doom>
y luego el programa se crashea o me devuelve
lzps<Doom>Soy el Doom 4

Definitivamente esta mal la funcion GetCadPos., pero no se en que.
Si pudieran ayudarme se los agradecería mucho :D

[Doommarine_Maxi]

Lo arregle agregandole +1 a los calloc de las funciones.

Código C:

//CadLong+1
int GetCadPos(char* txt, char* cad, int startpos)
{
    ...
    ...
    char* tmp = (char*)calloc(CadLong+1, sizeof(char));
    ...
    ...
}
 
//(CadenaLong+ExtraSize)+1
char* ReplaceBetween(char* cadena, char* inicio, char* fin, char* remplazar)
{
    ...
    ...
    char* NewCad = (char*)calloc((CadenaLong+ExtraSize)+1, sizeof(char));
    ...
    ...
}
//(Pos_F-Pos_I)+1
char* GetCadBetween(char* cadena, char* inicio, char* fin)
{
   ...
   ...
    char* NewCad = (char*)calloc((Pos_F-Pos_I)+1, sizeof(char));
   ...
   ...
}
 

Todavía tengo poca experiencia manejando arrays :S

[desgraciado]

¿Es necesario que programes en C?, porque no convierte ese programa en C++ y utilizas la STL (Standard Template Library) la cual te facilita el manejo de cadenas, arreglos dinámicos, etc y la necesidad de estar utilizado punteros así evitas los famosos “Memory leak”.

http://www.zator.com/Cpp/E5.htm

Y por el rendimiento que se pierde usando C++ +-5% es casi nada comparado a lo que vas a ganar en productividad.

[ipraetoriux]

Desgraciado:

Pues no estoy muy de acuerdo contigo, C ha sido la base de numerosos lenguajes de programacion, sirve como plataforma de formacion educativa, y al dia de hoy, sigue siendo un lenguaje MUY usado especialmente dentro del mundo Linux y Juegos...lo que pasa que como esta un poco de moda en la web habar de OOP, clases, objetos, etc, se cree que C++ ha sepultado a C, pues no...van en paralelo. Obviamente dependo mucho en que estemos trabajando o a donde estamos orientados...

[ecfisa]

Hola.

Estoy de acuerdo con ipraetoriux en la vigencia de C, basta echar una ojeada a TIOBE ( http://www.tiobe.com/index.php/content/paperinfo/tpci/index.html ) para confirmarlo. Pero también es cierto que el manejo de cadenas se ha simplificado enormemente en C++.

Por cierto, algunas funciones se pueden escribir de forma más compacta y eficiente utilizando punteros:

Código C:

int length_str(char *str) {
char *t = str;
  while(*t) t++;
  return t-str;
}
 
int compare_str(char *a, char *b) {
  for(; *a == *b; a++,b++)
    if (*a == '\0') return 0;
   return *a - *b;
}
 

Saludos.

[Doommarine_Maxi]

Iniciado por ecfisa

Hola.

Estoy de acuerdo con ipraetoriux en la vigencia de C, basta echar una ojeada a TIOBE ( http://www.tiobe.com/index.php/conte...pci/index.html ) para confirmarlo. Pero también es cierto que el manejo de cadenas se ha simplificado enormemente en C++.

Por cierto, algunas funciones se pueden escribir de forma más compacta y eficiente utilizando punteros:

Código C:

int length_str(char *str) {
char *t = str;
  while(*t) t++;
  return t-str;
}
 
int compare_str(char *a, char *b) {
  for(; *a == *b; a++,b++)
    if (*a == '\0') return 0;
   return *a - *b;
}
 

Saludos.

Muchas gracias por tu respuesta :D.
La razón por la que uso C, es por que leí que tiene buen rendimiento, ademas es como digamos casi la base de toda la programación moderna (La base supongo que es assembler), me gusta usarlo.
Otra duda que es un poco tonta.
¿Es verdad que C tiene un poco mas de rendimiento en cuanto a velocidad que C++?

[ecfisa]

Hola Doommarine_Maxi.

No creo que pueda darse una respuesta absoluta al rendimiento C vs C++. Depende de factores como por ejemplo el código y compilador que se usen. He leido por allí que C es un poco más veloz, pero la diferencia no era significativa.

Lo más importante a la hora de elegir un lenguaje es a que taréa se lo va a aplicar, por ejemplo; nada más rápido que un programa generado en ensamblador. Pero nadie con buén sentido desarrollaría un sistema de gestión en ese lenguaje, por que estaríamos pagando la ganancia en velocidad x1000 en complejidad y tiempo de desarrollo. También es improbable que alguién que programe interfaces para el S.O. se decida por utilizar COBOL.

Tanto C como C++ tiene sus defensores y detractores (como tantos otros lenguajes) pero en líneas generales creo que no hay buenos o malos lenguajes. Hay buenos o malos programadores o tal vez buenas o malas elecciones del lenguaje a aplicar.

me gusta usarlo

Ese es uno de los más valederos de los motivos; sentirse cómodo, afín con el lenguaje es muy importante.
Por otro lado si manejas C, sólo tendrás que aprender lo referente a la orientación a objetos de C++ y algunos puntos más para usarlo. Ya que luego, las diferencias entre ambos no son abismales.

Saludos.