Bueno, mi problema es el siguiente:
Desde que empecé con C, el profesor me recomendó usar Linux.
El tema es que SIEMPRE que uso punteros, y toco la memoria, cuando compilo me sale "Violación de segmento"...
El profesor me dijo "El problema no es Linux, el problema es el programador" ¬¬
En fin, no encuentro ayuda, sé que estoy haciendo algo mal, pero no sé qué... Sé que Linux no te deja tocar la memoria como si lo hace Windows (el cual no me genera problema con esto)...
En fin, acá dejo un código que justo estaba haciendo hoy, es muy simple, compila excelente, pero dice el famoso "violación de segmento"...
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
typedef struct {
char *nombre;
char *apellido;
char *fecha;
char *legajo;
char *tipDoc;
int DNI;
}Alumno;
int main(){
Alumno alu,alu2;
malloc(sizeof(alu.nombre));
malloc(sizeof(alu.apellido));
malloc(sizeof(alu.fecha));
malloc(sizeof(alu.legajo));
malloc(sizeof(alu.tipDoc));
strcpy(alu.nombre,"Esteban");
strcpy(alu.apellido,"Gonzalez");
strcpy(alu.fecha,"02111991");
strcpy(alu.legajo,"10662/3");
strcpy(alu.tipDoc,"DNI");
alu.DNI=36363023;
malloc(sizeof(alu2.nombre));
malloc(sizeof(alu2.apellido));
malloc(sizeof(alu2.fecha));
malloc(sizeof(alu2.legajo));
malloc(sizeof(alu2.tipDoc));
alu2=alu;
printf("El nombre del alu es: %s, el de alu2 es: %s\n",alu.nombre,alu2.nombre);
printf("El apellido del alu es: %s, el de alu2 es: %s\n",alu.apellido,alu2.apellido);
printf("La fecha del alu es: %s, el de alu2 es: %s\n",alu.fecha,alu2.fecha);
printf("El legajo del alu es: %s, el de alu2 es: %s\n",alu.legajo,alu2.legajo);
printf("El tipo de documento del alu es: %s, el de alu2 es: %s\n",alu.tipDoc,alu2.tipDoc);
printf("El DNI del alu es: %d, el de alu2 es: %d\n\n\n",alu.DNI,alu2.DNI);
strcpy(alu.nombre,"Jose");
printf("El nombre del alu es: %s, el de alu2 es: %s\n",alu.nombre,alu2.nombre);
printf("El apellido del alu es: %s, el de alu2 es: %s\n",alu.apellido,alu2.apellido);
printf("La fecha del alu es: %s, el de alu2 es: %s\n",alu.fecha,alu2.fecha);
printf("El legajo del alu es: %s, el de alu2 es: %s\n",alu.legajo,alu2.legajo);
printf("El tipo de documento del alu es: %s, el de alu2 es: %s\n",alu.tipDoc,alu2.tipDoc);
printf("El DNI del alu es: %d, el de alu2 es: %d\n",alu.DNI,alu2.DNI);
return 0;
}
Gracias de antemano...
Todas las líneas de malloc están mal, malloc es una función que devuelve un puntero a la memoria asignada, por lo cual deberías asignarlo a la variable que intentas asignarle espacio.
Hola
Además de que estás reservando el tamaño de un puntero
Saludos
Desde ya, MUCHÍSIMAS gracias, me ha vuelto loco durante todo el seminario este error... Ahora ya no me lo tira más, acá dejo el código modificado para que me digan si de la forma que lo solucioné está bien o es una falta de respeto a los ojos de un programador de C :P
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
typedef struct {
char *nombre;
char *apellido;
char *fecha;
char *legajo;
char *tipDoc;
int DNI;
}Alumno;
int main(){
Alumno alu,alu2;
alu.nombre= malloc(sizeof(alu.nombre));
alu.apellido= malloc(sizeof(alu.apellido));
alu.fecha= malloc(sizeof(alu.fecha));
alu.legajo= malloc(sizeof(alu.legajo));
alu.tipDoc= malloc(sizeof(alu.tipDoc));
strcpy(alu.nombre,"Esteban");
strcpy(alu.apellido,"Gonzalez");
strcpy(alu.fecha,"02111991");
strcpy(alu.legajo,"10662/3");
strcpy(alu.tipDoc,"DNI");
alu.DNI=36363023;
alu2.nombre= malloc(sizeof(alu2.nombre));
alu2.apellido= malloc(sizeof(alu2.apellido));
alu2.fecha= malloc(sizeof(alu2.fecha));
alu2.legajo= malloc(sizeof(alu2.legajo));
alu2.tipDoc= malloc(sizeof(alu2.tipDoc));
alu2=alu;
printf("El nombre del alu es: %s, el de alu2 es: %s\n",alu.nombre,alu2.nombre);
printf("El apellido del alu es: %s, el de alu2 es: %s\n",alu.apellido,alu2.apellido);
printf("La fecha del alu es: %s, el de alu2 es: %s\n",alu.fecha,alu2.fecha);
printf("El legajo del alu es: %s, el de alu2 es: %s\n",alu.legajo,alu2.legajo);
printf("El tipo de documento del alu es: %s, el de alu2 es: %s\n",alu.tipDoc,alu2.tipDoc);
printf("El DNI del alu es: %d, el de alu2 es: %d\n\n\n",alu.DNI,alu2.DNI);
strcpy(alu.nombre,"Jose");
printf("El nombre del alu es: %s, el de alu2 es: %s\n",alu.nombre,alu2.nombre);
printf("El apellido del alu es: %s, el de alu2 es: %s\n",alu.apellido,alu2.apellido);
printf("La fecha del alu es: %s, el de alu2 es: %s\n",alu.fecha,alu2.fecha);
printf("El legajo del alu es: %s, el de alu2 es: %s\n",alu.legajo,alu2.legajo);
printf("El tipo de documento del alu es: %s, el de alu2 es: %s\n",alu.tipDoc,alu2.tipDoc);
printf("El DNI del alu es: %d, el de alu2 es: %d\n",alu.DNI,alu2.DNI);
return 0;
}
Ahora estaría bien,te faltaba el cast:
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
typedef struct {
char *nombre;
char *apellido;
char *fecha;
char *legajo;
char *tipDoc;
int DNI;
}Alumno;
int main(){
Alumno alu,alu2;
alu.nombre= (char *)malloc(sizeof(alu.nombre));
alu.apellido= (char *)malloc(sizeof(alu.apellido));
alu.fecha= (char *)malloc(sizeof(alu.fecha));
alu.legajo= (char *)malloc(sizeof(alu.legajo));
alu.tipDoc= (char *)malloc(sizeof(alu.tipDoc));
strcpy(alu.nombre,"Esteban");
strcpy(alu.apellido,"Gonzalez");
strcpy(alu.fecha,"02111991");
strcpy(alu.legajo,"10662/3");
strcpy(alu.tipDoc,"DNI");
alu.DNI=36363023;
alu2.nombre= (char*) malloc(sizeof(alu2.nombre));
alu2.apellido= (char*)malloc(sizeof(alu2.apellido));
alu2.fecha= (char *)malloc(sizeof(alu2.fecha));
alu2.legajo= (char*)malloc(sizeof(alu2.legajo));
alu2.tipDoc= (char*)malloc(sizeof(alu2.tipDoc));
alu2=alu;
printf("El nombre del alu es: %s, el de alu2 es: %s\n",alu.nombre,alu2.nombre);
printf("El apellido del alu es: %s, el de alu2 es: %s\n",alu.apellido,alu2.apellido);
printf("La fecha del alu es: %s, el de alu2 es: %s\n",alu.fecha,alu2.fecha);
printf("El legajo del alu es: %s, el de alu2 es: %s\n",alu.legajo,alu2.legajo);
printf("El tipo de documento del alu es: %s, el de alu2 es: %s\n",alu.tipDoc,alu2.tipDoc);
printf("El DNI del alu es: %d, el de alu2 es: %d\n\n\n",alu.DNI,alu2.DNI);
strcpy(alu.nombre,"Jose");
printf("El nombre del alu es: %s, el de alu2 es: %s\n",alu.nombre,alu2.nombre);
printf("El apellido del alu es: %s, el de alu2 es: %s\n",alu.apellido,alu2.apellido);
printf("La fecha del alu es: %s, el de alu2 es: %s\n",alu.fecha,alu2.fecha);
printf("El legajo del alu es: %s, el de alu2 es: %s\n",alu.legajo,alu2.legajo);
printf("El tipo de documento del alu es: %s, el de alu2 es: %s\n",alu.tipDoc,alu2.tipDoc);
printf("El DNI del alu es: %d, el de alu2 es: %d\n",alu.DNI,alu2.DNI);
return 0;
}
Por cierto,una sugerencia,yo la fecha la pondria como int mas que como char... :rolleyes:
Un saludo
Gracias, gracias y nuevamente MIL gracias...
Con el tema de la fecha... Cuando lo pongo en int, long int o cualquier int que se me ocurra me tira el siguiente error de compilación:
Citarerror: invalid digit "9" in octal constant
Me suena rarisimo, no sé porqué dice que es octal, porque nada que ver... Por eso lo hice como cadena :P
La fecha déjala char* porque como piensas asignar 26/07/11 en un int? lo de octal es porque si le asignas a un int esto 09 el tomará el numero como octal y los números octales no incluyen el 9 (0-7).
Claro, vos decís que el error de octal me lo toma por el primer '0' del número?...
Claro, no lo pensé a eso...
De todas formas si, el hecho de las '/' es importante, aunque no hice una cadena muy grande como para que entren, jaja...
Igual son ejercicios pavos que estoy re/haciendo porque rindo un parcial y empecé de 0, jaja... Hay muchas cosas que las tenía bastante oxidadas :P
Ya de paso hago otra consulta:
Por ejemplo, yo definí una macro, supongamos:
#define T_INT 1;
si yo después, a esa macro la quiero comparar con una variable de tipo int también, que previamente fue ingresado su valor por teclado, ¿cómo se haría?...
Porque yo hice
if (valor == T_INT)
printf("te imprimo algo");
y me tira error : error: expected ')' before ';' token
#define T_INT 1;
#define T_FLOAT 2;
#define T_STR 3;
union T_union{
int ival;
float fval;
char *sval;
};
int main(){
int valor;
union T_union u;
u.ival=22;
u.fval=22.69;
u.sval=(char *)malloc(sizeof(u.sval));
strcpy(u.sval,"NO ME LEAS");
printf("Ingrese un valor para invocar la lectura:\t");
scanf("%d\n",&valor);
if (valor == T_INT)
printf("El valor de ival es: %d\n",u.ival);
if (valor == T_FLOAT)
printf("El valor de fval es: %f\n",u.fval);
if (valor == T_STR)
printf("El contenido de sval es: %s\n",u.sval);
return 0;
}
Ahí dejé el código para complicar menos las cosas...
Edito:ya lo explico el compañero naderST :P
De todos modos eso seria todo segun veo,si quieres puedes utilizar al operador new para asignar memoria,es de C++ pero es mas simple de asignar,aunque claro si el profe te dice que uses C yo no voy a decir lo contrario,asique dejaria el code asi si nadie tiene algo que añadir.
PD:Asignar la fecha como int a --->26072011,queda mas chapuzas pero bueno...Ademas no lo he probado,lo estoy diciendo por intuicion mas que otra cosa...xD
Un saludo
Si, es un seminario de lenguaje C, no nos permiten ni C++ ni C#.
Es más, nos hicieron hacer un trabajo final de manejos de .BMP y nos hicieron hacer todo por nuestra cuenta, nada de usar funciones predefinidas de BMP ni nada, todo manual...
Aprendí bastante, pero fue un dolor de cabeza importante, jaja...
Cita de: Triper0 en 26 Julio 2011, 22:26 PM
Si, es un seminario de lenguaje C, no nos permiten ni C++ ni C#.
Es más, nos hicieron hacer un trabajo final de manejos de .BMP y nos hicieron hacer todo por nuestra cuenta, nada de usar funciones predefinidas de BMP ni nada, todo manual...
Aprendí bastante, pero fue un dolor de cabeza importante, jaja...
Jaja me lo imagino :xD
Te recomiendo que leas mas sobre punteros, con respecto a #define fíjate que estas poniendo ';' al final y ahí no lleva ';'
Si, justamente lo que mas me cuesta son las listas en C... En Pascal las manejo de diez, pero con C y el tema de los punteros me agarro un lindo pedalin...
En C la conversión explicita no es necesaria y no se recomienda, en este lenguaje el idioma (convención informal) para reservar memoria es:
p = malloc(N * sizeof *p);
Donde N es el numero de elementos.
En tus programas (me refiero a Triper0) sigues con el mismo error que ya te mencionaron: reservas memoria para "sizeof(algun_puntero)" cuando el tamaño debería indicar el tamaño del objeto apuntado. Por ejemplo:
#include <stdio.h>
#include <stdlib.h>
#define LONG_MAX_LINEA 1024
int main(void)
{
char *p;
if ((p = malloc(LONG_MAX_LINEA)) == NULL)
return EXIT_FAILURE;
puts("Introduce una linea de texto:");
if (fgets(p, LONG_MAX_LINEA, stdin) == NULL)
return EXIT_FAILURE;
printf("Linea: %s", p);
free(p);
return EXIT_SUCCESS;
}
En este caso no se requiere de "sizeof *p" ya que es igual a "sizeof(char)" y este siempre es igual a uno.
En cuanto al ultimo programa (para practicar con uniones) tiene varios errores, el principal que se utiliza la unión como si fuera una estructura. Aquí lo mejor es conseguir (y seguir) un buen libro. Recomendaciones sobre ellos en uno de los temas fijos.
Un saludo
Cita de: rir3760 en 27 Julio 2011, 02:49 AM
En C la conversión explicita no es necesaria y no se recomienda, en este lenguaje el idioma (convención informal) para reservar memoria es:
p = malloc(N * sizeof *p);
Donde N es el numero de elementos.
En tus programas (me refiero a Triper0) sigues con el mismo error que ya te mencionaron: reservas memoria para "sizeof(algun_puntero)" cuando el tamaño debería indicar el tamaño del objeto apuntado. Por ejemplo:
#include <stdio.h>
#include <stdlib.h>
#define LONG_MAX_LINEA 1024
int main(void)
{
char *p;
if ((p = malloc(LONG_MAX_LINEA)) == NULL)
return EXIT_FAILURE;
puts("Introduce una linea de texto:");
if (fgets(p, LONG_MAX_LINEA, stdin) == NULL)
return EXIT_FAILURE;
printf("Linea: %s", p);
free(p);
return EXIT_SUCCESS;
}
En este caso no se requiere de "sizeof *p" ya que es igual a "sizeof(char)" y este siempre es igual a uno.
En cuanto al ultimo programa (para practicar con uniones) tiene varios errores, el principal que se utiliza la unión como si fuera una estructura. Aquí lo mejor es conseguir (y seguir) un buen libro. Recomendaciones sobre ellos en uno de los temas fijos.
Un saludo
Ese "N se refiere a número de elementos" cómo sería?... Por ejemplo, si yo tengo un puntero de tipo int, sería N=1
Y si tengo un char *nombre[10] sería N=10... ¿Así?
El tema de las uniones, seguramente tenga millones de errores, NUNCA use una unión, y justo vi un ejercicio sólo de unión, leí que era parecido a el struct pero no le encuentro la diferencia...
El libro que leí todo y seguí paso a paso, que fue uno de los que me recomendaron en la cátedra fue "C con ejemplos"...
Explica muy bien todo, pero no llega a cosas "muy avanzadas" como listas que es algo de lo cual necesito saber bien porque estoy seguro que en el parcial me van a tomar eso, además de obvio, para aprender más...
Desde ya muchísimas gracias también por el ejemplo y por todo ;)
Cita de: Triper0 en 27 Julio 2011, 04:09 AM
Ese "N se refiere a número de elementos" cómo sería?... Por ejemplo, si yo tengo un puntero de tipo int, sería N=1
Y si tengo un char *nombre[10] sería N=10... ¿Así?
No.
El numero de elementos que piensas utilizar. Por ejemplo si necesitas espacio para una cantidad N de enteros (determinado en tiempo de ejecución) utilizas (sin validación para que sea mas sencillo):
#include <stdio.h>
#include <stdlib.h>
int main(void)
{
int *p;
int num_elem;
int i;
printf("Indica el numero de elementos: ");
fflush(stdout);
scanf("%d", &num_elem);
/* Reserva de memoria */
p = malloc(num_elem * sizeof *p);
/* Lectura de datos */
for (i = 0; i < num_elem; i++){
printf("p[%d] == ", i);
fflush(stdout);
scanf("%d", p + i); /* Equivalente a "&p[i]" */
}
/* Algun proceso de datos ... */
/* Liberacion de memoria y salida del programa */
free(p);
return EXIT_SUCCESS;
}Un saludo
Cita de: rir3760 en 27 Julio 2011, 04:36 AM
No.
El numero de elementos que piensas utilizar. Por ejemplo si necesitas espacio para una cantidad N de enteros (determinado en tiempo de ejecución) utilizas (sin validación para que sea mas sencillo):
#include <stdio.h>
#include <stdlib.h>
int main(void)
{
int *p;
int num_elem;
int i;
printf("Indica el numero de elementos: ");
fflush(stdout);
scanf("%d", &num_elem);
/* Reserva de memoria */
p = malloc(num_elem * sizeof *p);
/* Lectura de datos */
for (i = 0; i < num_elem; i++){
printf("p[%d] == ", i);
fflush(stdout);
scanf("%d", p + i); /* Equivalente a "&p[i]" */
}
/* Algun proceso de datos ... */
/* Liberacion de memoria y salida del programa */
free(p);
return EXIT_SUCCESS;
}
Un saludo
Ahahaha, fenómeno!!!...
Millón de gracias nuevamente ;-)
estoy igual que el amigp! soy un programador novato! tengo problemas con algun for! me sale el errror violacion de segmento core generado! se que el problema debe estar en algun for que se esta saliendo del espacio y esta tocando la memoria o algo asi fue que me explicaron y he dado vueltas y nada que he podido resolver!!!! gracias espero una respuesta!!!!!
para la ayuda de un buen samaritano y programador!
#include <stdio.h>
#include <time.h>
//Bienvenido al juego tres en raya
void loop (char c[3][3]);
void Intro_Primera (char c[3][3]);
void tablero (char c[3][3]); // tablero para mostrar en pantalla
void Intro_Yo (char c[3][3]); //turno del jugador o usuario
void Intro_IA (char c[3][3]); //turno aleatorio o computadora
int ganador (char c[3][3]); //funcion para determinar quuien gana
int main () {
char c [3][3];
loop (c);
system ("pause");
return 0;
}
void loop (char c [3][3]){
int i, j;
i = 0;
Intro_Primera (c);
tablero(c);
do{
system ("cls");
tablero(c);
if (i % 2 == 0) {
Intro_Yo (c);
}
else {
Intro_IA (c);
}
j = ganador (c);
i++;
}while (i <= 9 && j == 2);
if (j == 0){
printf ("Muy Bien!! Has ganado!!\n\n");
}
else if (j == 1){
printf ("ohhh noo!! Has perdido!!");
}
else{
printf ("Has empatado!! Intentalo de nuevo!!");
}
}
void Intro_Primera (char c[3][3]){
int i, j;
char aux;
aux='1';
for (i = 0; 1 < 3; i++){
for (j = 0; j < 3; j++){
c[j] = aux++;
}
}
}
void Intro_Yo(char c[3][3]){
int i, j, k;
char aux;
do{
do{
printf ("Coloca una ficha: ");
fflush (stdin);
scanf ("%c",&aux);
}while (aux < '1' || aux > '9');
k = 0;
switch (aux){
case '1': {
i = 0;
j = 0;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '2': {
i = 0;
j = 1;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '3': {
i = 0;
j = 2;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '4': {
i = 1;
j = 0;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '5': {
i = 1;
j = 1;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '6': {
i = 1;
j = 2;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '7': {
i = 2;
j = 0;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '8': {
i = 2;
j = 1;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
case '9': {
i = 2;
j = 2;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
printf ("la posicion esta ocupada! Intentalo con otro numero!!\n\n");
}
break;
}
}
}while (k == 1);
c[j] = 'X';
}
void Intro_IA (char c[3][3]){
int i, j, k;
srand (time(NULL));
do{
i = rand() % 3;
j = rand() % 3;
k = 0;
if (c[j] == 'X' || c[j] == 'O'){
k = 1;
}
}while (k == 1);
c[j] = 'O';
}
void tablero (char c[3][3]){
int i, j;
for (i = 0; 1 < 3; i++){
for ( j = 0; j < 3; j++){
if (j < 2){
printf (" %c |",c[j]);
}
else {
printf (" %c ",c[j]);
}
}
if (i < 2){
printf ("\n-----------\n");
}
}
printf ("\n\n");
}
int ganador (char c[3][3]){
if (c[0][0] == 'X' || c[0][0] == 'O'){
if (c[0][0] == c[0][1] && c[0][0] == c[0][2]){
if (c[0][0] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
if (c[0][0] == c[1][0] && c[0][0] == c[2][0]){
if (c[0][0] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
}
if (c[1][1] == 'X' || c[1][1] == 'O'){
if (c[1][1] == c[0][0] && c[1][1] == c[2][2]){
if (c[1][1] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
if (c[1][1] == c[1][0] && c[1][1] == c[1][2]){
if (c[1][1] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
if (c[1][1] == c[2][0] && c[1][1] == c[0][2]){
if (c[1][1] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
if (c[1][1] == c[0][1] && c[1][1] == c[2][1]){
if (c[1][1] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
}
if (c[2][2] == 'X' || c[2][2] == 'O'){
if(c[2][2] == c[2][0] && c[2][2] == c[2][1]){
if (c[2][2] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
if (c[2][2] == c[0][2] && c[2][2] == c[1][2]){
if (c[2][2] == 'X'){
return 0; //ha ganado
}
else{
return 1; // ha perdido
}
}
}
return 2;
}