Saludos, porfavor si me pudieran ayudar con un problemita en este pequeno codigo
#include <stdio.h>
#include <stdlib.h>
int openbin(void);
int main(){
openbin();
return 0;
}
int openbin(void){
FILE *pfile;
char namebin[50];
printf("Name:\n");
fgets(namebin,sizeof(namebin), stdin);
if ((pfile = fopen(namebin, "rb"))== NULL)
{
printf("Couldn't open the file\n");
exit(1);
}
printf("The file was opened successfully\n");
return 0;
}
lo que pasa es que cuando introdusco el name del archivo a abrir este nunca es abierto(obvio, estando el archivo en la misma carpeta), no se en que pueda estar fallando porfavor alguien que me de luz sobre el tema, Gracias
¡Buenas!
fgets, te esta leyendo la pulsacion del intro. Eliminalo de la cadena y ya lo tienes:
http://www.cplusplus.com/reference/clibrary/cstdio/fgets/
Por cierto, si no vas a usar el fichero, cierralo antes de salir de la funcion.
¡Saludos!
he hecho lo que me dices pero sigue igual. Tambien he intentado con gets que no toma el intro pero tampoco
¿Seguro que poniendo esto no te va?
#include <stdio.h>
#include <stdlib.h>
int openbin(void);
int main(){
openbin();
return 0;
}
int openbin(void){
FILE *pfile;
char namebin[50];
printf("Name:\n");
gets(namebin);
if ((pfile = fopen(namebin, "rb"))== NULL)
{
printf("Couldn't open the file\n");
exit(1);
}
printf("The file was opened successfully\n");
return 0;
}
Si es eso lo que has puesto, cuando te pida el nombre del archivo en la consola, pon el nombre del fichero seguido de .txt (o la extensión que tenga).
estoy en linux, no se si eso importe
Como dije en el otro mensaje. ¿Has probado ya a poner en la consola el nombre del archivo seguido de .txt o la extensión que tenga tu archivo?
Si es así, no se me ocurre qué puede fallar.
Yo uso Windows, pero creo (aunque no estoy seguro) que en este caso es igual.
Hombre ya he intentado de todo pero no funca, lo de gets es en forma de prueba pues esta funcion es muy peligrosa
A mí con ese mismo código (el que puse yo, que era el tuyo pero con gets en vez de fgets), me he funcionado bien. Poniendo hola.txt (mi archivo se llama hola). Será entonces porque usas Linux y yo Windows...
Este programa funciona bien para esa tarea hasta ahora. Toda la información está en los comentarios.
Simplemente usa una función más, que se limita a buscar el carácter '\r' o '\n' y lo modifica directamente en la cadena original, cambiándolo por un carácter nulo '\0'. Ni siquiera se necesita malloc para ese pequeño ajuste de cadena.
#include <stdio.h> //printf, fopen, fgets, fclose
#include <stdlib.h> //exit
#include <string.h> //strlen
int openbin(void);
void striptrail(char *, size_t);
int main(void)
{
openbin();
return 0;
}
int openbin(void)
{
//Definimos el manejador de archivo y nuestro búfer de cadena:
///
FILE *pfile;
char namebin[50];
//Mostramos un mensaje pidiendo un nombre de archivo
//y leemos dicha entrada de usuario con fgets:
///
printf("Nombre de archivo: ");
fgets(namebin,sizeof(namebin), stdin);
//A continuación mostramos el tamaño de la cadena leída
//con fgets, y luego aplicamos la función para eliminar
//nuevas líneas/retornos de carro, para preparar la cadena
//y aprovechamos a mostrar la nueva longitud (si es que se
//ajustó):
///
printf("Longitud de cadena sin procesar: %d\n",strlen(namebin));
//Le pasamos directamente el nombre del búfer de cadena, porque
//este es ya en sí un puntero directo al búfer de cadena:
///
striptrail(namebin, sizeof(namebin));
printf("Longitud de cadena procesada con 'striptrail': %d\n\n",strlen(namebin));
//Tratamos de abrir el archivo. Si no se puede abrir, mostramos
//un error; si se abrió mostramos mensaje de éxito. Después
//cerramos el manejador de archivo y salimos al sistema.
///
if ((pfile = fopen(namebin, "rb"))== NULL)
{
printf("ERROR: No se pudo abrir el archivo.\n");
exit(1);
}
printf("Archivo abierto exitosamente.\n");
fclose(pfile);
return 0;
}
//Esta función simple está pensada para usarse con fgets.
//
//Lo que hace es tomar un puntero a una cadena, y el tamaño
//esperado de la cadena.
//
//Entonces busca un carácter de retorno de carro '\n' o nueva
//línea '\n', introducidos por fgets al presionar ENTER.
//
//NOTA: Si no se encuentra ninguno de esos caracteres, entonces
// el último carácter, al final del búfer de cadena de texto,
// se reemplaza incondicionalmente por un byte nulo '\0' para
// terminar la cadena ASCIIZ, por seguridad.
//
// Es por eso que esta función está diseñada para trabajar con
// fgets u otro código que guarde '\r' o '\n' al final de la
// cadena esperada.
//
///
void striptrail(char * tostrip, size_t sizze)
{
//Tratamos de aplicar optimización de uso de
//registros de CPU, para mayor velocidad para
//el contador del bucle:
///
register size_t x=0;
//Recorremos toda la cadena:
///
for(x=0;x<sizze;x++)
{
//Si el carácter que estamos leyendo actualmente es
//regreso de carro o nueva línea, sabemos que tenemos
//que cambiarlo por el byte nulo '\0' para indicar
//el fin de la cadena. Nos detenemos tan pronto como
//encontremos uno de esos caracteres, regresando de
//la función de inmediato, para evitar seguir escaneando
//la cadena si el objetivo ya está cumplido:
///
if(tostrip[x]=='\r' || tostrip[x]=='\n' || tostrip[x]=='\0')
{
tostrip[x]='\0';
return;
}
}
//Si x tiene el valor del tamaño esperado del búfer,
//reemplazamos el carácter en esa posición con un byte
//nulo para terminar la cadena. Esta línea se puede dejar
//o eliminar dependiendo de cómo funcione nuestro programa,
//y si estamos totalmente seguros de que no necesitamos
//esta "característica de seguridad" (pero vital en este programa):
///
tostrip[x]='\0';
}
muchas gracias por la ayuda, te haz pasado pero sigo teniendo el problema, siempre me va a "ERROR: no se pudo abrir.....". no se si estoy haciendo algo mal pero no los abre.
aunque sigo con el problema aprecio mucho la ayuda gracias
¿Has probado a incluir errno.h y a usar perror para ver exactamente que problema tienes? Cuando fopen te devuelve NULL no quiere decir necesariamente que no se pueda abrir el archivo porque no se encuentra. También puede ser un problema de permisos. Estoy en Linux y me va perfecto este código en el que sólo he cambiado el gets por el scanf, lo cual no tiene nada que ver (con el gets también me va bien):
int openbin(void) {
FILE *pfile;
char namebin[50];
printf("Name:\n");
scanf("%s",namebin);
if ((pfile = fopen(namebin, "rb"))== NULL) {
printf("Couldn't open the file\n");
exit(1);
}
printf("The file was opened successfully\n");
return 0;
}
Saludos
he intentado con perror y sale esto "fopen fallo por : No such file or directory" segun esto no lo encuentra pero el archivo esta en la misma capeta, tambien he intentado con la ruta pero tampoco
y lo de los permisos no creo ,tiene permiso de todo y ademas ejecuto el programa como sudo
¿?
A ver. ¿Estás seguro de que el archivo de texto está en el mismo directorio que tu programa o de que usas la ruta absoluta o relativa correcta? No ha lugar a errores con este código. Comprueba que estás introduciendo correctamente el nombre del archivo, respetando mayúsculas y minúsculas y con la extensión apropiada (en Linux es necesario que introduzcas el nombre del archivo exacto). Si acaso, introduce un printf para imprimir la cadena en que guardas el nombre del archivo y así comprobar que se guarda correctamente y que coincide exactamente con el nombre del archivo de texto.
En última instancia, dinos la ruta del ejecutable (con los nombre incluidos), la ruta del fichero de texto y la entrada de datos que efectúas cuando ejecutas el programa.
Y por cierto, no hace falta que ejecutes el programa con sudo. Usa sudo lo menos posible.
Saludos
Bueno ya he solucionado el problema aunque no tengo ni idea de porque se arreglo. lo unico que hice fue mover el ejecutable y el archivo a un nuevo directorio y funciono.
Muchas gracias a todos por las respuestas
Me acabo de dar cuenta que los archivos con nombre numerico no los abre, estaba intentando con un archivo llamda "311" y intente con un archivo con nombre "hola" y lo abrio perfectamente
¿Por ser un nombre numérico? Extraño. A mí con nombres numéricos me va perfecto. Entre las recomendaciones a la hora de dar nombre a un fichero en sistemas Unix no figura "No dar nombres numéricos" (de hecho, los directorios asociados a los procesos en /proc/ tienen como nombres el PID de los procesos, que son totalmente numéricos). Me parece raro. Bueno, ya has solucionado en parte tu problema, así que no sé si te apetece seguir investigando. ¿Has probado otro nombre numérico que no sea 311? Tal vez haya algún fichero en alguno de los directorios del PATH que se llame exactamente igual, aunque no creo que sea esto lo que pase, porque en ese caso lo que suele ocurrir es que se le dé prioridad a ese fichero. Para estar seguros, podrías introducir ./311 en vez de 311 a secas y ver si te sigue dando la matraca.
¡Saludos!