Foro de elhacker.net

Hola, muy buen día, ¿cómo están? -
Les comento, termine de estudias con el manual de COMO PROGRAMAR C C++ Y JAVA DEITEL&DEITEL-4° Edición-Español y para refrescar conceptos estoy leyendo El lenguaje de programación C - Kernighan, Ritchie, y piden que haga el siguiente programa. -


La primera duda que tengo es ¿pondrían el código en una variable char o en un archivo?
Desde ya muchas gracias, saludos. -

Sería ideal cargar el código a una cadena. Así pues, el código puede estar inicialmente en un archivo si quieres: luego, lo cargas y listo.
Trabajar directamente con el archivo sería más lento y algo más complejo.
Y por memoria no habrá problema. Un código dificilmente va a ocupar más de 1 MB.

Hola.
Decirme si lo que pienso hacer es una forma correcta, cuento con getchar() el total de caracteres que tiene el archivo y luego con este dato reservo memoria dinámica para cargarlo en precisamente en memoria ¿es una buena alternativa?. -
Saludos.

Para saber el tamaño del archivo, posicionas el cursor al final (con fseek y SEEK_END, por ejemplo), y luego obtienes la posición con ftell.

	if( ptrArchivo != NULL ){
		fseek(ptrArchivo,0,SEEK_END);
		tamanio = ftell(ptrArchivo); 
		fclose( ptrArchivo );	
	}else{
		printf( "\n ERROR - el archivo no se pudo abrir..." );
		exit(EXIT_FAILURE);
	}

Ahora me quiero ver con el resto. - :huh: :huh: :huh:
Saludos.

Hola, muy buen Domingo.
Tengo un problema con la función cargarArchivoMemoria (línea 36)no sé en que estoy fallando pero no me vuelca el contenido del fichero en la memoria, me podrían decir donde estoy fallando, leí, busque, practique pero debo recurrir a Uds. porque no lo logro. - 

El programa:

#include <stdio.h>
#include <stdlib.h>
 
long int tamanioArchivo( FILE *ptrArchivo, long int tamanio );
char *cargarArchivoMemoria( FILE *ptrArchivo, long int tamanio, char *archivoMem ); 
void corrector( long int tamanio, char *archivoMem );
 
int main( int argc, char **argv ){
	FILE *ptrArchivo = NULL;
	long int tamanio = 0;
	char *archivoMem = NULL;
 
	ptrArchivo = fopen("corregir.c", "r+t");
 
	if( ptrArchivo != NULL ){
		tamanio = tamanioArchivo( ptrArchivo, tamanio );		
		archivoMem = cargarArchivoMemoria( ptrArchivo, tamanio, archivoMem );
		corrector( tamanio, archivoMem );
		free( archivoMem );
		fclose( ptrArchivo );	
	}else{
		printf( "\n ERROR - el archivo no se pudo abrir..." );
		exit(EXIT_FAILURE);
	}
 
	return EXIT_SUCCESS;
}
 
long int tamanioArchivo( FILE *ptrArchivo, long int tamanio ){
	fseek(ptrArchivo,0,SEEK_END);
	tamanio = ftell(ptrArchivo); 
 
	return tamanio;
}
 
char *cargarArchivoMemoria( FILE *ptrArchivo, long int tamanio, char *archivoMem ){
	int i=0;
	archivoMem = (char *)calloc( tamanio, sizeof(char) );
 
	if( archivoMem != NULL ){
		while( !feof(ptrArchivo) ){
			*(archivoMem+i) = fgetc( ptrArchivo );
			i++;
		}
	}else{
		printf( "\n ERROR - Fallo la asignación de memoria..." );
		exit(EXIT_FAILURE);
	}
 
	return archivoMem;
} 
 
void corrector( long int tamanio, char *archivoMem ){
	int i;
	for( i=0; i<tamanio; i++ ){
		printf("%d", *(archivoMem+i) );
	}
}

El archivo a copiar:

#include <stdio.h>
 
int main( void ){
 
	printf( "\n Hola mundo\n\n" );
 
	return 0;
 
}

Gracias y un abrazo. -

Debes colocar el indicador de posición al comienzo del fichero nuevamente

long int tamanioArchivo( FILE *ptrArchivo, long int tamanio )
{
    fseek(ptrArchivo,0,SEEK_END);
    tamanio = ftell(ptrArchivo);
    rewind(ptrArchivo); //volver al inicio
    return tamanio;
}

Hola, geeke, muchas gracias por ocuparte. -
Me voy a dar un fuerte tirón de orejas, a esta altura no me debería ocurrir estos errores tan de primer grado. -

Saludos.

Tu función cargarArchivoMemoria no está bien diseñada.  En ella pasas como argumento archivoMem que resulta ser una declaración de una variable local de la función. Si retornas el dato a través del retorno de la función deberías quitar ese argumento de la llamada y transformarlo en declaración de variable local.
Si lo quieres como argumento de llamada deberías pasarlo por referencia (puntero a puntero) y modificarlo bien dentro de la función. Entonces puedes devolver el puntero a la cadena como valor de retorno u otra cosa para indicar éxito o fallo de la función.

Hola, MAFUS.
Si amigo muchas gracias por la aclaración, cuando tengas un ratito de tiempo revisarla, por favor, para saber si ahora está correcto, dejo todo el código para que se fijen en la función en la línea 55, parece que da resultados correctos, lo hice de esta manera para no utilizar mil if o un switch. - 
En cuanto a las barras invertida antes de la " y \ es porque me daba error y parece que no las tiene en cuenta como un elemento mas. -

#include <stdio.h>
#include <stdlib.h>
 
long int tamanioArchivo( FILE *ptrArchivo, long int tamanio );
char *cargarArchivoMemoria( FILE *ptrArchivo, long int tamanio ); 
void corrector( long int tamanio, char *archivoMem );
 
int main( int argc, char **argv ){
	FILE *ptrArchivo = NULL;
	long int tamanio = 0;
	char *archivoMem = NULL;
 
	ptrArchivo = fopen("corregir.c", "r+t");
 
	if( ptrArchivo != NULL ){
		tamanio = tamanioArchivo( ptrArchivo, tamanio );		
		archivoMem = cargarArchivoMemoria( ptrArchivo, tamanio );
		corrector( tamanio, archivoMem );
		free( archivoMem );
		fclose( ptrArchivo );	
	}else{
		printf( "\n ERROR - el archivo no se pudo abrir..." );
		exit(EXIT_FAILURE);
	}
 
	return EXIT_SUCCESS;
}
 
long int tamanioArchivo( FILE *ptrArchivo, long int tamanio ){
	fseek(ptrArchivo,0,SEEK_END);
	tamanio = ftell(ptrArchivo); 
	rewind(ptrArchivo);
 
	return tamanio;
}
 
char *cargarArchivoMemoria( FILE *ptrArchivo, long int tamanio ){
	int i=0;
	char *archivoMem = NULL;
	archivoMem = (char *)calloc( tamanio, sizeof(char) );
 
	if( archivoMem != NULL ){
		while( !feof(ptrArchivo) ){
			*(archivoMem+i) = fgetc( ptrArchivo );
			i++;
		}
	}else{
		printf( "\n ERROR - Fallo la asignación de memoria..." );
		exit(EXIT_FAILURE);
	}
 
	return archivoMem;
} 
 
void corrector( long int tamanio, char *archivoMem ){
	int i,j, operUno[16] = {0}, linea=1;
	char c,	*operDos = "\"#.(){}[]<>;,=/\\";
 
 
	for( i=0; i<tamanio; i++ ){
		c = *(archivoMem+i);		
		if( c != '\n' ){
			for( j=0; j<16; j++ ){
				if( c == *(operDos+j) ){
					operUno[j] += 1; 
				}
			}		
		}else{
			linea++;
		}
	}		
 
	for( j=0; j<16; j++ ){
		printf("\n %c = %d", *(operDos+j) , operUno[j] );
	}
}

Saludos.

Vale , ahora dos cosas más, tres:
calloc es un poco lenta puew además de adquirir memoria la pone a 0. Como llenas toda esta memoria con el contenido del archivo no hace falta ponerla a 0, por tanto puedes usar malloc.

Ya qu adquieres memoria con malloc deberías liberarla, es una buena práctica,  despues de usarla. En main debes usar free para liberar archivoMem.

De igual forma tienes abierto un archivo en ptrArchivo. Debes liberarlo cuando termines de usarlo con fclose.

Lo que me sugieres que haga no está contemplado en las lineas 19 y 20.

Tienes razón  :rolleyes: No lo vi.

Tengo casi todo preparado para hacer lo más importante que es corregir errores de sintaxis pero ete aquí que no se me cae una misera idea, preguntarle si hay algo escrito como un tutorial sobre el tema ni lo pienso porque busque bastante en internet y nada, salvo que aya buscado equivocadamente. -
Les dejo una pregunta para tratar de que me den un empujoncito para comenzar, si la primera línea es:

include <stdio.h>

Como harían para encontrar el error. -

Saludos.

Es algo complicado.
C no trabaja con comprobaciones sin más. Diría que es más como un autómata finito.
Cuando dudes, fíjate en las salidas que generaría tu compilador:

a < b >

Da error: expected '=', ',', ';', 'asm' or '__attribute__' before '<' token

Ha encontrado un nombre que no es una palabra reservada, y espera una de esas cadenas después.

Hombre, no es por ponerte trabas, pero hacer un analizador de C para todos los casos es complicado. Tendrás que tener un montón de palabras y construcciones en cuenta y estudiar a fondo las estructuras que posee.
Una cosa es verificar los comentarios, llaves, corchetes, parantesis y comillas, y otra ya es analizar cada palabra xd

En primer lugar en el enunciado del ejercicio no se menciona detectar este tipo de errores, te aconsejo resolver primero lo propuesto luego intentar ir mas alla. Te puedo dar una idea de como lo puedes encarar, lo que puedes hacer es usar contadores para cada caso establecerlos a cero. Luego por ejemplo si encuentras una llave de apertura incrementar el contador correspondiente y decrementarlo si encuentras una llave de cierre, posteriormente verificar el contador

Saludos

Hola a todos. -
Gracias Iván por tu participación, siempre con buen aporte, geeke tienes toda la razón me estaba yendo por las ramas, me diste un impulso importante ya que estaba por dejarlo por un tiempo seguir con el libro y retomarlo luego ya que continuar con esto me va a dar mucho conocimiento, bueno a pensar y practicar, a una última cosa, debería tener mucha mala suerte si encuentro un código entre comillas con un solo paréntesis y si ocurre habrá que darse idea, tal vez cuando encuentre comillas dobles debería saltar hasta encontrar la siguiente comillas. -

printf( "\n Hola (mundo\n\n" );

Saludos

Las comillas las puedes tratar como cualquier paréntesis o llave. Las únicas diferencias, es que se abre y se cierra con el mismo símbolo, y la más importante: cuando haya una comilla, ignorarás los paréntesis, llaves y corchetes que haya hasta que se cierren las comillas de nuevo.

Ten en cuenta también, los caracteres de escape. Por ejemplo:

"abc\"sdf"

O un pelín más complejo:

"abc\\"
"abc\\\"sdf"

Hola a todos. -
Continuando con mi proyecto me encuentro con la necesidad de guardar la línea donde se encuentra por Ej. una apertura de llave para luego si encuentro un cierre de la misma poner el contador a cero en el correspondiente elemento del vector contador(geeke), pero si no encuentro el cierre de la misma es un error y debo mostrar el mensaje y una referencia en que línea se encuentra el mismo, para ello tengo que utilizar memoria dinámica ya que si bien conozco la cantidad de columnas no así la cantidad de filas (renglones), porque puede ocurrir que haya 3 aperturas de llaves y luego se encontrarían los cierre de las mismas (o no) y de no ser así necesito las referencias de las líneas donde se encuentran. -
Hasta aquí me parece que estoy bien rumbado pero la cuestión es que no sé cómo hacer una tabla con memoria dinámica, para un arreglo lo hice de esta manera(código a continuación) y todo funciona pero para un matriz no me doy idea y no pude encontrar ejemplos en internet, encontré algunos pero es sabiendo de antemano las filas y las columnas. -

   (http://i66.tinypic.com/vx22dw.png)

void ingreso( void ){
	size_t ingresos, contador = 0, n, ok, ch;
	int *numeros = NULL, *mas_numeros = NULL;
 
	do{
		do{
			printf( "\n Ingrese un numero entero (0 para finalizar).....: " );
			ok = scanf( "%lu", &ingresos ) == 1 && ingresos >= 0;
			while( (ch = getchar()) != EOF && ch != '\n');
 
			if( ok == 0 ){
				printf( "\n El ingreso es incorrecto, intentelo nuevamente...\n\n\t");
				printf("Pulse una tecla para continuar..." ); getchar();
			}
 
		}while( !ok );
 
		contador++;
		mas_numeros = (int*)realloc(numeros, contador * sizeof(int));
 
		if( mas_numeros != NULL ){
			numeros = mas_numeros;
			numeros[contador-1] = ingresos;
		}
		else{
			safeFree( numeros );
			puts( "Error en asignar memoria....." );
			exit(1);	
		}
	}while( ingresos != 0 );	
 
	limpiarPantalla();
 
	printf ("\n Numeros introducidos.....: ");
	for( n=0; n < contador - 1; n++ )
		printf( "%d ",numeros[n] );
 
	ordenar( contador, numeros );
 
	safeFree( numeros );
}
 

Espero como siempre deseen ayudarme.-
Saludos.

Hola, ¿cómo están?
Parece que lo logré, el programa que dejo es para practicar es obvio que podía haber asignado toda la memoria de una vez y esta de más aclararles que utilizo un arreglo de una dimensión como si fuera de dos.

#include <stdio.h>
#include <stdlib.h>
 
 
int main( int argc, char **argv ){
	int fil=1, col=5, i, j, x=col, *tmp=NULL ;
	int a[] = {33,18,5,2,19,306,202,88,31,64};
	int *ptrArr = (int *)malloc( fil*col*sizeof(int) );
 
	if( ptrArr != NULL ){
		fil++;
		x += col;
 
		tmp = (int *)realloc( ptrArr, x );
		if( tmp != NULL ){
			ptrArr = tmp;
			tmp = NULL;
			x=0;
			for( i=0; i<fil; i++ ){
				for( j=0; j<col; j++ ){
					ptrArr[fil*i+j] = a[x];	
					printf( "%3d ", ptrArr[fil*i+j] );
					x++;
				}
				printf("\n");
			}
		}else{
			printf( "\n No se pudo reasignar memoria" );
			return EXIT_FAILURE;
		}
	}else{
		printf( "\n No se pudo reservar memoria" );
		return EXIT_FAILURE;
	}
 
	free( ptrArr );
	ptrArr = NULL;
 
	return EXIT_SUCCESS;
 
}
 

Saludos.

Tienes dos errores importantes en tu ultimo codigo:

- Primero, la notacion fil * i + j no accede al array como 2D, esto sobreescribe en cada iteracion del bucle externo lo asignado previamente, puedes comprobar esto imprimiendo el array a parte, lo que buscas es i * col + j.

- Segundo, solo reservas espacio para siente enteros no para diez, afortunadamento solo logras asignar siente enteros al array debido al error mencionado, caso contrario estarias desbordando el array.

Hola, geeke, ¿cómo estas? -
El primer error totalmente de acuerdo, en el segundo casi no tengo dudas que estoy equivocado porque desde que participas en mis post todo lo tuyo fueron aciertos, pero igual tengo la siguiente duda:
x en la línea 6 se iguala a col que vale 5, o sea, que para mí en la línea 8 reserva memoria para 5 enteros, luego en la línea 12 a la variable x le sumo col por lo tanto en la línea 14 x vale 10 y para mi criterio reservo memoria para 10 enteros, me podías decir en que estoy equivocado. -
Saludos.

No olvides que realloc reserva memoria en bytes, esto quiere decir que estas reasignando solo diez bytes, es decir espacio para dos enteros mas, teniendo en cuenta que un int es equivalente a cuatro bytes en un sistema de 32 bits. Por eso es importante no olvidar multiplicar size * sizeof(int).

Saludos.