Borrar texto en la consola usando c

[hergue00]

Buenas, estoy escribiendo un programa en C con visual studio que te permite definir una contraseña.
En el programa, tu a medida que vas escribiendo caracteres te va imprimiendo *, el caso es que no se como hacer que al presionar la tecla de retroceso borre el anterior asterisco, no sé si me podréis ayudar, gracias de antemano

Código (c) [Seleccionar] Expandir


#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<malloc.h>
#include<conio.h>
#define length 16 //Longitud máxima de la contraseña

char* Password();

void main()
{
char *pass1,*pass2;
int cmp=1;
do
{
printf("Introduce una clave de %d caracteres: ", length);
pass1 = Password();
printf("Introduce de nuevo la clave de %d caracteres: ", length);
pass2 = Password();
cmp = strcmp(pass1, pass2);
if (cmp == 0)
printf("Clave introducida correctamente!!\n");
else
printf("Las claves no coinciden\n");
system("PAUSE");
if (cmp != 0)
system("CLS");
} while (cmp != 0);
}

char* Password()
{
char *pass,c = 0;
int i;
//Asignacion de memoria para los caracteres y '\0'
pass = (char*)malloc((length)+1);
if (pass == NULL)
{
printf("No hay memoria disponible\n");
system("PAUSE");
exit(1);
}

//Compobacion de que como mucho tiene los caracteres que hemos indicado. Cada vez que escribes un caracter, imprime *
do
{
i = 0;

while (c != 13) //Si no presionamos intro
{
if (i < 18) //Si la contraseña cabe en la cadena, se va introduciendo
{
c = _getch();
if (c != 8 && c != 13)
{
pass[i] = c;
printf("*");
}

if (c == 8) //Pulsa retroceso
{
c = NULL;
i = i - 2;
}
}
else //Si no entra siemplemente se imprimen los *, luego te mandara repetir contraseña
{
c = _getch();
if (c != 8&&c!=13)
{
printf("*");
}

if (c == 8)
{
c = NULL;
i = i - 2;
}
}
if (c == 13) //Pulsa intro
{
if (i < ((length)+1))
pass[i] = '\0';
else
{
pass[(length)+1] = '\0';
}
i--;
printf("\n");

}
i++;
}
if(i>length)
printf("Error de longitud, introducela de nuevo: ");
c = 0;
} while (i > length);

//Reasignacion de memoria para optimizar
pass = (char*)realloc(pass, i * sizeof(char) + 1);
return pass;
free(pass);
}

[hergue00]

Ya lo tengo!
Simplemente añadiendo un printf("\b \b"); (Con el espacio incluido) en la condición de retroceso pulsado sale.

[RayR]

Bueno, en realidad es "\b \b", pero supongo que sólo fue error de dedo. Veo algunos problemas con tu código, pero como no usaste etiquetas GeSHi, no sé si algunos sean sólo partes que se "comió" el foro.

Algunos detalles: ¿qué pasa si el usuario presiona y presiona y presiona retroceso muchas veces?

Luego tienes esto:

Código [Seleccionar] Expandir


return pass;
free(pass);
El return hace que la ejecución del programa regrese a main, por lo que no tiene sentido poner instrucciones después, ya que nunca se ejecutarán. Y de cualquier forma, ese free era incorrecto. Quien llama a tu función es quien debe liberar esa memoria. Si tú la liberaras dentro de la función Password, ya no es válida. Siendo un programa tan simple y pequeño, probablemente aún así funcionaría "correctamente" (por pura suerte) pero sería un error.

El realloc está de más. Es un ejemplo perfecto de optimización prematura. Intentar ahorrarte unos pocos bytes realmente no optimiza nada (quizás hace un par de décadas y si estuvieras programando para un microcontrolador con unos pocos KB de memoria, pero hoy...) y de hecho puede ser contraproducente. Por cuestiones de eficiencia, funciones como malloc y realloc pueden reservar más memoria que la que les pides. Puede ser, por ejemplo, en bloques múltiplos de cierto número. El caso es que, sobre todo sabiendo que estás reservando sólo unos pocos bytes, es probable que realloc considere más eficiente dejar que se "desperdicien" unos pocos bytes, y decida no tocar esa memoria, con lo cual, sólo estarías complicando más tu código, llamando a una función innecesariamente, y sin ahorrarte un sólo byte. Las optimizaciones hay que hacerlas cuando realmente tengamos la necesidad, hayamos identificado un problema, y hayamos sopesado pros y contras. Hacerlas por hacerlas suele traer problemas y muy poco (normalmente ningún) beneficio.

[hergue00]

Si, ha sido un error al escribir.

Lo del return me he dado cuenta luego, y el free(pass) en el código lo he quitado. ¿Dices que debería liberarla en el main() que es desde donde la llamo?

Respecto a lo de realloc, la verdad no lo sabía, lo había metido porque el código es parte de un trabajo para la universidad, que estamos aprendiendo a programar C, y estaba probando las funciones de asignación dinámica. Si me dices que si no me ahorro mucha memoria puede ser contraproducente, mejor lo quito.

Lo del retroceso... Cuando me lo has dicho, lo he probado y he borrado todo el mensaje de la consola me he reído la verdad jeje. Creo que lo he solucionado poniendo:

Código (c) [Seleccionar] Expandir


if(i==0&&c==8)
  continue;


Debajo justo del primer "c=_getch();" (Línea 53).

[RayR]

Respecto a lo de realloc, la verdad no lo sabía, lo había metido porque el código es parte de un trabajo para la universidad, que estamos aprendiendo a programar C, y estaba probando las funciones de asignación dinámica. Si me dices que si no me ahorro mucha memoria puede ser contraproducente, mejor lo quito.

Si es por practicar, lo puedes dejar. No va a fallar ni a darte problemas. Simplemente te decía que cuando hablamos de cosas tan simples, como unos pocos bytes, normalmente no vale la pena intentar optimizar. El realloc perfectamente puede decidir no cambiar nada, es decir, no "reducir" la memoria, y simplemente dejar todo como está. Es perfectamente válido y tu programa funcionará sin problemas, pero en ese caso estarías haciendo esa llamada de forma innecesaria (a eso me refería con contraproducente: estás trabajando de más, posiblemente sin beneficio).

Es simplemente un consejo general, que antes de optimizar hay que revisar bien lo que hacemos. Muchas veces una "optimización" hecha sin análisis, puede incluso hacer que un programa tenga un rendimiento peor al que tenía antes de la supuesta optimización. Pero te repito, como práctica no hay problema. Cuando tengas más experiencia ya sabrás cuándo vale la pena buscar optimizar y cuándo no.

[hergue00]

Vale, muchas gracias, me has ayudado bastante!

Una última cosa, como puedo hacer para que si el usuario presiona las teclas ins, supr... no las detecte como caracteres?

[RayR]

Cuando se presiona una tecla "especial", como ins, supr, las flechas, etc., getch lee y retorna dos valores, uno que indica precisamente el hecho de que se presionó una tecla especial, y el siguiente, que devuelve el código de esa tecla. En Turbo C++ el código era 0, pero creo que en Windows es distinto. Puedes agregar un printf que muestre el código leido, justo después de tu primer _getch:

printf("%d", c);

Al ejecutarlo, presiona la tecla que quieras probar, por ejemplo ins. y verás en pantalla los dos códigos, el primero, que es igual para todas las teclas especiales, y el segundo que es específico de ins, etc. Creo que sólo te interesa el primero, es decir, detectar tecla especial, y simplemente ignorarla (y la siguiente). En ese caso, pones justo después de tus _getch algo como:

Código [Seleccionar] Expandir

           while (c == <codigo de teclas especiales>) {
               // Quitar del buffer el código de la tecla específica
               _getch();

               // Volver a leer una tecla
               c = _getch();
           }


Sólo se entrará al bucle si se presiona tecla especial, y se sale cuando se presiona una tecla "normal", que te interese procesar.