Mensajes

Reconstruyendo el post:
Aqui dejo el codigo completo: http://codepad.org/Cfv3Qp9G
dejo claro que es para ayudar al compañero Gaspi. 

Jajaja, gracias por eso, me ahorraste trabajo

selecciona el texto y luego escoges el lenguaje

la etiqeuta code de apertura quedará

Código [Seleccionar] Expandir

[code=cpp][/code]

Sale todo raro si pongo eso, me cambia el código y se ve todo verde:

void GenerarFixture(Equipo vectorequipos[], tipofixture *&fixture, int &fechastotales, int &PartidosPorFecha, int &totalequipos, tipofixture *&puntero1, tipofixture *&punterofin,tipofixture *&punteroinicial) //Genera el fixture y además me devuelve algunos parámetros necesarios para saber tamaños
{
    int i=0;
    int r=1;

    int local = 0;
    int visitante = 0;
   int y=0;
   int vec[99];
   int g=0;
   LlenarVector(vectorequipos, i);

...

para que codigo sean más legibles usa etiquetas GeSHi (mira mi firma)

Está seleccionada esa opción :S, supongo que lo cambió usted  gracias

Hola, cómo va todo?
Estoy con un trabajo bastante largo, y tengo un pequeño problema al ordenar una lista.
La función ordenar la llamo desde lo siguiente

Código [Seleccionar] Expandir

void GenerarFixture(Equipo vectorequipos[], tipofixture *&fixture, int &fechastotales, int &PartidosPorFecha, int &totalequipos, tipofixture *&puntero1, tipofixture *&punterofin,tipofixture *&punteroinicial) //Genera el fixture y además me devuelve algunos parámetros necesarios para saber tamaños
{
    int i=0;
    int r=1;

    int local = 0;
    int visitante = 0;
   int y=0;
   int vec[99];
   int g=0;
LlenarVector(vectorequipos, i);


   if(i!=0){
   cout<<"Generando el fixture"<<endl;

   totalequipos = i+1;
   int t= totalequipos%2;

if (t==0){
       fechastotales=totalequipos-1;
}else fechastotales= totalequipos;

   PartidosPorFecha = totalequipos/2;

   crearlista(fixture);

   if(fechastotales!=0){

   int w=0;
   int l=1;
   while (l<=fechastotales){
       fixture=new(tipofixture);

       while(r<=totalequipos/2){

                   generarPartido(totalequipos, l, r, local, visitante);
                   ponerenlista(fixture,vectorequipos,local,visitante,i);
                   r++;
           };

       Aleatoriafecha(l,fechastotales,fixture,w,vec);
           if(puntero1==NULL && punteroinicial==NULL){ // Probablemente esta asignación esté mal hecha!
               puntero1=new(tipofixture);
               punteroinicial=new(tipofixture);
               puntero1=fixture;
               punteroinicial=fixture;
           };

       l++;

       fixture->infofixture.fuesimulado=false;

       if( l <= fechastotales){
       fixture=fixture->siguiente;
       }else punterofin=fixture->siguiente;




   };

   };

   cout<<"..."<<endl;
   cout<<"Generado!"<<endl;
   //                   <-----------------              acá llamaría a la función ordenarlista

 } else cout<<"En el archivo no hay equipos participantes!"<<endl; //si el archivo está vacío.

}


Donde puntero1 y punteroinicial pretendo que sean punteros al primer nodo de la lista fixture.

Aquí las estructuras para la lista (sé que el typedef está de más):

Código [Seleccionar] Expandir

typedef struct{
   char equipo[31];
   int resultado;
   int paramA;
   int paramB;
   int puntos;
}tipopartido;
typedef struct{ // local y visitante tienen almacenado un array tipopartido por fecha
   tipopartido local[99];
   tipopartido visitante[99];
   }partidosfixture;

typedef struct{ //nodo de información de la lista
   int fecha;
   partidosfixture tabla[99];
   bool fuesimulado;
}info;

typedef struct tipofixture{ // la lista del fixture
info infofixture;
tipofixture *siguiente;
}tipofixture;


La función anterior funciona correctamente, pero cuando llamo a la función burbuja que creé para ordenar la lista, explota (no por error, sino porque hace comparaciones de NULL):

Código [Seleccionar] Expandir

void OrdenarLista(int fechastotales, tipofixture *&puntero1){

   cout<<"Ordenando por fecha..."<<endl;
   tipofixture *auxiliar= NULL;
   if(auxiliar=NULL)
   {
     auxiliar=new(tipofixture);
   };
   
   auxiliar=puntero1;
   int aux=0;
   if(puntero1->siguiente==NULL){ //comprobé si era por esto que explotaba, y sí, justamente es nulo.
       cout<<"es nulo";
   }else cout<<" sigue"<<endl;

   while(puntero1!=NULL){
   for(int j=0;j<=(fechastotales-1);j++){
            auxiliar=auxiliar->siguiente;
           if(puntero1->infofixture.fecha>auxiliar->infofixture.fecha){
               aux=puntero1->infofixture.fecha;
               puntero1->infofixture.fecha=auxiliar->infofixture.fecha;
               auxiliar->infofixture.fecha=aux;
           }
       } puntero1=puntero1->siguiente;
   }



La idea es ordenar la lista por fecha en forma ascendente (ya que en aleatoriafecha estoy pasando numeros al azar desordenados), y que me quede ordenada por puntero1.
Fechastotales es el tamaño de la lista, puntero1, como dije, debería apuntar al primer nodo de la lista. La lista tiene más de un nodo, lo comprobé con salidas por pantalla, pero el error surge porque puntero1->siguiente es NULL, no entiendo porqué, quizá asigné mal el puntero.
Necesito ayuda urgente!
GRACIAS.



Uso CodeBlocks, y como dato aparte puntero1 y punteroinicial los inicié en el main como nulos. Tengo también un puntero para el final de la lista. La lista que uso en todo el programa se llama fixture.


EDITO: parece que tampoco me estaría rellenando la lista fixture, solo llena con la fecha en cada nodo, y no con los nombres (eso sólo lo hace en el primer nodo); alguna idea?
Esta es la función que uso:

Código [Seleccionar] Expandir

void ponerenlista(tipofixture *&fixture, Equipo vectorequipos[], int &local, int &visitante, int i) //Para poner en lista tengo que revisar los índices.
{

   int a=0;


   buscarlocal(fixture,vectorequipos,local,visitante,i,a);
   buscarvisitante(fixture,vectorequipos,visitante,i,a);

}

void buscarlocal (tipofixture *&fixture, Equipo vectorequipos[], int local, int visitante, int i,int &a){ //Como el vector de equipos está ordenado por índice, hago una búsqueda binaria.

int inicio = 0;
int fin = i;
bool encontrado = false;
int pos = 0;
while (inicio<=fin && !encontrado){

   pos = (inicio+fin)/2;


   if(vectorequipos[pos].indice == local){


       encontrado=true;
       strncpy(fixture->infofixture.tabla[a].local[a].equipo, vectorequipos[pos].nombre,31);
       fixture->infofixture.tabla[a].local[a].paramA = vectorequipos[pos].paramA;
       fixture->infofixture.tabla[a].local[a].paramB = vectorequipos[pos].paramB;

       a++;
   } else if (vectorequipos[pos].indice>local){
           fin=pos-1;
           }else inicio = pos+1;

}
}
void buscarvisitante(tipofixture *&fixture, Equipo vectorequipos[], int visitante, int i, int &a){ //Búsqueda binaria para el visitante

int inicio = 0;
int fin = i;
bool encontrado = false;
int pos = 0;
while (inicio<=fin && !encontrado){
   pos = (inicio+fin)/2;
   if(vectorequipos[pos].indice == visitante){

       encontrado=true;
       strncpy(fixture->infofixture.tabla[a].visitante[a].equipo, vectorequipos[pos].nombre,31);
       fixture->infofixture.tabla[a].visitante[a].paramA = vectorequipos[pos].paramA;
       fixture->infofixture.tabla[a].visitante[a].paramB = vectorequipos[pos].paramB;
       cout<<fixture->infofixture.tabla[a].visitante[a].equipo<<endl;


       a++; // sumo acá el a porque ya terminé esta sección
   } else if (vectorequipos[pos].indice>visitante){
           fin=pos-1;
           }else inicio = pos+1;
}
}




De nuevo, nada tira error, todo compila, sigo pensando que es algo de la lista

Pueden cerrar el tema, cree otro con un problema más específico.

En aquellos casos en los que necesites modificar el valor del puntero (liberar memoria, modificar el tamaño de la memoria reservada, etc), SI. En estos casos es un imperativo para que los cambios se reflejen fuera de la función.

En el resto de casos no es necesario y puede ser incluso contraproducente. Lo más sano es facilitar a cada función la cantidad mínima de datos necesaria.

Quizás, una de las pegas de la clase "string" es que no admite unicode... pero para eso está la clase "wstring". En cualquier caso, ambas clases forman parte del estándar... y aunque en la librería estándar existen bugs, no es motivo para no usarlos. Básicamente porque cualquier librería externa que tengas que usar (incluso las propias del sistema operativo) también están expuestas a fallos de programación y no por ello dejas de usarlas. Además, la librería estándar suele tener algoritmos altamente optimizados, por lo que suele ser la mejor opción para programas de uso general.

Un saludo.

Se agradece la ayuda nuevamente.
Genial, ya hice los respectivos cambios y vi que cometí varios errores de lógica muy tontos como comparar solo con un =. Procedo a una depuración línea a línea a ver si consigo algo.

http://www.mediafire.com/download/47n2996mw204vcs/Trabajo.zip

[ni la más mínima idea idea]

Cuando usas un puntero como argumento, todo lo que modifiques que se refiera a la memoria apuntada por el puntero se verá reflejado fuera de la función... pero el puntero, como variable, es algo local, luego modificar la posición de memoria apuntada por el puntero es un cambio local y no se refleja fuera de la función. No se si me explico.

Mmm, si se explica. Debería cambiar el resto de las funciones en las que pasé una lista como parámetro?

Me permito la licencia de decir públicamente que tu profesor no tiene ni la más mínima idea idea de programación. Las clases de la librería estándar están ahí por algo. Y su implementación es bastante eficiente y estable. Otra cosa es que, claro, al no tener mucha idea sobre programación orientada a objetos, prefiera que programéis lo más parecido a C posible... pero eso, como he dicho, no habla muy bien del nivel de tu profesor.

La materia no cubre objetos, por eso las listas las hacemos de esa manera.
Le discutiré eso pues mi profesor es graduado de la facultad más prestigiosa en ingeniería de mi país, donde estoy estudiando yo ahora, y sé que tiene sus razones para evitar que usemos los String.

Imagínate que el arreglo admite dos elementos y estás rellenando el segundo. Al hacer "i++", i pasa a valer "2" y en la siguiente línea intenas escribir en el elemento con índice "2", es decir, estarías escribiendo en una posición de memoria no válida.

Ahora capté lo que me quería decir, gracias, error solucionado.


He dejado ya el link de descarga en Mediafire del trabajo entero y compilable.

[Las funciones no deben terminar con ";"]

Creo que puede ser algún buffer que sea demasiado pequeño. También te falta el return de la función main.

saludos.

Verdad lo del main, no lo había notado, quizá por eso no pasa del primer bucle. Muchas gracias.

Las funciones no deben terminar con ";"

Esto sucede con la copia que has puesto de "LlenarVector", por ejemplo.

Estás programando en C++, no deberías usar los includes que acaben en ".h"

Esas cabeceras son de C. Puedes cargar la cabecera propia de C++ poniendo el prefijo "C" y eliminando el ".h" ( #include <cmath> en vez de #include <math.h> )

Dado que estás programando en C++, sería más cómodo y seguro para tí usar la clase "string" en vez de "char*" para almacenar cadenas.

En C++ no hace falta usar "typedef" para declarar estructuras:



Los índices no los estás actualizcodecorrectamente:

Fíjate que en la línea anterior estás actualizando "i", luego la línea marcada está modificando el índice del siguiente registro.

Cuidado al inicializar los arreglos:


"fechastotales" es una variable. Esta forma de definir los arreglos no es estándar y puede darte problemas. Es más seguro crear un arreglo de tamaño fijo y más grande de lcodeesario o recurrir amemoria dinámica.

La función AleatoriaFecha no funciona

Si revisas el código, verás que "vec" únicamente va a contener un elemento SIEMPRE... en la función no hay ningún bucle.

La función "crearlista" no funciona



En primer lugar, el nombre de la función no se corresponde con el funcionamiento teorico de la función, ya que no crea nada, en tal caso inicializa un puntero. En segundo lugar, esta función no hace ABSOLUTAMENTE NADA, "tipofixture", si bien es un puntero, es una variable local. Esto quiere decir que cualquier cambio que realices sobre el puntero en sí (ojo, no sobre la memoria "apuntada" ), es un cambio que va a desaparecer al salir el código de la función. Si tu idea es que el cambio del puntero se refleje fuera de la función tienes que usar un puntero doble:

Resumen

Tu código tiene bastantes inconsistencias. Deberías revisarlo con más detenimiento.

Ah si, y sería de agradecer que, para la siguiente ocasión, el código que facilitas pueda ser compilado... no es demasiado agradable tener que deducir el funcionamiento de tu programa únicamente viendo líneas de código.

Un saludo

Muchísimas gracias por su tiempo leyendo todo y dándome útiles consejos. Paso a explicar algunas cosas:
el Typedef lo uso de costumbre, es bueno saber que no hace falta.
Las cadenas de tipo string no las tengo permitidas usar, el profesor nos dijo que eran sumamente inestables y que prefería que usemos char.
El índice quería que empezara en 1, por eso el i++ previo.
Es verdad lo de vec, debería haber definido 'w' en el ciclo de Generarfixture y pasarlo por referencia.
Puede usted profundizar un poco lo que dijo al final sobre la función crearlista? Entiendo porqué es inútil, pero tenía entendido que las listas y los array siempre pasan por referencia.
De nuevo, muchas gracias y tendré en cuenta la crítica final, mañana cuando pueda revisar todo, edito el tema subiendo la librería y las funciones faltantes.
Que tenga un buen día

Hola, mi primer post en el foro, mil disculpas si rompo alguna regla, corríjanme si lo hago.
Miren, acabo de terminar un trabajo práctico para mi facultad, es programación básica, nada muy complicado, pero depende de éste que apruebe la materia.
El problema radica en que el programa me compila de lo mejor, pero al ejecutarlo explota, no se porqué, estuve revisando y pienso que pude haber definido mal alguna lista, o hecho alguna mala comparación entre cadenas char, pero no sé. El código me parece muy extenso como para copiarlo y pegarlo, son alrededor de 500 líneas, pero copiaré lo que pueda estar mal.
El trabajo se trata de hacer un fixture partiendo de un archivo de texto, que fue proporcionado junto con una funciones para facilitar el trabajo.
Includes y estructuras:

Código [Seleccionar] Expandir

#include <iostream>
#include "torneo.h"// lo dio el profesor, contiene las funciones leerEquipo, generarPartido y simularPartido (no las incluiré)

#include <cstdlib>
#include <math.h>
#include <string.h>
#include <time.h>
#define BUFFER_SIZE 1024
using namespace std;

typedef struct{ //acá entran  los datos del archivo
    short indice;
char nombre[31];
int paramA;
int paramB;
} Equipo; //para el array que va a tener cada partido
typedef struct{
    char equipo[31];
    int resultado;
    int paramA;
    int paramB;
    int puntos;
}tipopartido;

typedef struct{ // local y visitante tienen almacenado un array tipopartido por fecha
    tipopartido local[99];
    tipopartido visitante[99];
    }partidosfixture;
typedef struct{ // Para mostrar la tabla de posiciones uso otro array en vez de la lista, es más fácil de ordenar
    int puntos;
    char nombre[31];
}posiciones;
typedef struct{ //nodo de información de la lista
    int fecha;
    partidosfixture tabla[99];
    bool fuesimulado;
}info;

typedef struct tipofixture{ // la lista del fixture
info infofixture;
tipofixture *siguiente;
}tipofixture;

Acá están las funciones hechas por mí, no agrego las previas declaraciones, deben tener varios errores básicos, agradezco cualquier crítica y consejo que me ayude a mejorar.

Main:

Código [Seleccionar] Expandir

int main() {
    int totalequipos= 0;
    posiciones vectortabla[99];
    tipofixture *fixture;
    tipofixture *listanueva = NULL;
int fechastotales=0;
    int v=0;
int a;
int PartidosPorFecha;
Equipo vectorequipos[99];
int tamanio= 0;

while (a!= 7) {
cout<<"FIXTURE 2014"<<endl;
cout<<"Ingrese un número"<<endl;
cout<<"1. Generar Fixture"<<endl;
cout<<"2. Simular Partidos"<<endl;
cout<<"3. Ver Equipo"<<endl;
cout<<"4. Ver Tabla de Posiciones"<<endl;
cout<<"5. Guardar Fixture"<<endl;
cout<<"6. Cargar Fixture"<<endl;
if (v==0){
cout<<"7. Salir"<<endl;
}
cin>>a;
int localA, localB, visitanteA, visitanteB, golesLocal, golesVisitante;
        if(a=1){ //con el switch explotaba, no sé porqué
            GenerarFixture(vectorequipos, fixture, fechastotales, PartidosPorFecha,totalequipos);
            listanueva = fixture;
            OrdenarLista(listanueva);
            cout<<"Su fixture se ha generado con éxito"<<endl;
        }
    else if (a=2){
                menusimularpartido (fixture,listanueva, fechastotales,PartidosPorFecha,vectortabla,tamanio);
    } else if (a=4){
            cout<<"ingrese una fecha, de querer ver la tabla hasta la ultima fecha simulada ingrese 0"<<endl;
            int m;
            cin>>m;
            if(recorrerlista(listanueva,m)){
                    cout<<"No se simularon partidos hasta esa fecha";
            }else if(m=0) { cout<<"No existe una fecha 0!";}
            else MostrarTabla (listanueva,m,totalequipos,vectorequipos);



    } else if (a=3){

                MostrarEquipo(listanueva,PartidosPorFecha,fechastotales);

    }else if(a=5){ //tengo que reemplazar el archivo de existir, y crearlo de no figurar
        FILE *archivo = fopen ("equipos.txt", "w");
        for(int k=1;k<=totalequipos/2;k++){
                    int f=0;

                    fwrite((const void*)& f,sizeof(int)*1,1,archivo);
                    fwrite((const void*) & listanueva->infofixture.tabla[f].local[f].paramA,sizeof(int)*1,1,archivo);
                    fwrite((const void*) & listanueva->infofixture.tabla[f].local[f].paramB,sizeof(int)*1,1,archivo);
                    fwrite(listanueva->infofixture.tabla[f].local[f].equipo,sizeof(char)*31,1,archivo);
                    fwrite(listanueva->infofixture.tabla[f].visitante[f].equipo,sizeof(char)*31,1,archivo);

                    f++;
                }
                cout<<"Guardado con éxito";
                fclose(archivo);

    } else if (a=6){
           vaciarlista(fixture);
           vaciarlista(listanueva);
           GenerarFixture(vectorequipos,fixture,fechastotales,PartidosPorFecha,totalequipos);
            "Se ha cargado y generado un nuevo fixture!"
};



};
if(a=7)
        vaciarlista(fixture);
        vaciarlista(listanueva);

}

GenerarFixture (el objetivo de esta función es formar un fixture completo de l campeonato de fútbol, para eso usé la lista FIXTURE, y posteriormente la ordenada LISTANUEVA). La función generarPartido lo que hace es mediante un índice me devuelve los equipos que se enfrentarán:

Código [Seleccionar] Expandir

void GenerarFixture(Equipo vectorequipos[], tipofixture *fixture, int &fechastotales, int &PartidosPorFecha, int &totalequipos) //Genera el fixture y además me devuelve algunos parámetros necesarios para saber tamaños
{
     int i=0;
     int r=1;

     int local = 0;
     int visitante = 0;

LlenarVector(vectorequipos, i);
    if(i!=0){
    cout<<"Generando el fixture"<<endl;

    totalequipos = i+1;
    int t= totalequipos%2;

if (t=0)
        fechastotales=totalequipos-1;
else fechastotales= totalequipos;

    PartidosPorFecha = totalequipos/2;

    crearlista(fixture);

    if(fechastotales!=0){

    for(int l=1;l<=fechastotales;l++){
            Aleatoriafecha(l,fechastotales,fixture); //hace aleatoria la selección de fechas. Posterior ordenamiento.

        while(r<=totalequipos/2){

                    generarPartido(totalequipos, l, r, local, visitante);
                    ponerenlista(fixture,vectorequipos,local,visitante,i);


            };
        fixture->infofixture.fuesimulado=false; // Para evitar varias simulaciones.
        fixture=fixture->siguiente;

        };



    };
    cout<<"..."<<endl;
    cout<<"Generado!"<<endl;

  } else cout<<"En el archivo no hay equipos participantes!"<<endl; //si el archivo está vacío.
}

Acá hay funciones que fueron necesarias para crear el fixture, la función leerEquipo fue proporcionada por el profesor, como dije antes, y lo que hace es leer el archivo sacando los parámetros allí indicados:

Código [Seleccionar] Expandir

void LlenarVector(Equipo vectorequipos[], int &i) { //usando la función dada "leerequipo" lleno el vector y devuelvo el tamaño.
     FILE *archivo = fopen ("equipos.txt", "r");
char nombre[31];
int pA, pB;


while (leerEquipo(archivo, nombre, pA, pB))
{

vectorequipos[i].paramA = pA; //estos dos son parámetros usados para simular partidos, son enteros.
vectorequipos[i].paramB = pB;
strncpy(vectorequipos[i].nombre, nombre, 31);
i++;
         vectorequipos[i].indice = i;
};
fclose (archivo);




};
void Aleatoriafecha(int l, int fechastotales, tipofixture *fixture){
    srand(time(NULL));//Para que no repita en caso de fixture nuevo
    int vec[fechastotales];
    int x= 1+(rand()%fechastotales+1); //las fechas arracan en 1
    int w=0;
    vec[w]=x;


    if (FechaEncontrada(vec,fechastotales,x)){
            Aleatoriafecha(l,fechastotales,fixture);//si la fecha ya había sido usada, genera otro número aleatorio. Evito repetición de fechas.

    } else {
         fixture->infofixture.fecha=x; //si no encontró la fecha en el array, entonces se puede meter la fecha ahí
         w++;





}
}

bool FechaEncontrada(int vec[],int fechastotales, int x){
    bool encontrada=false;
    for(int t=0; t<=fechastotales-1; t++){ //que sea tamaño fechastotales-1 (el -1 va porque fechas totales arranca de 1) hace que se estén generando todas las fechas sin excepción
        if(vec[t]=x){
            encontrada=true;
        }
    }

return encontrada;



}
void crearlista(tipofixture *fixture){ //creo listas
fixture=NULL;

};

void ponerenlista(tipofixture *fixture, Equipo vectorequipos[], int &local, int &visitante, int i) //Para poner en lista tengo que revisar los índices.
{
    int a=0;
buscarlocal(fixture,vectorequipos,local,visitante,i,a);
buscarvisitante(fixture,vectorequipos,visitante,i,a);
};

void buscarlocal (tipofixture *fixture, Equipo vectorequipos[], int local, int visitante, int i,int &a){ //Como el vector de equipos está ordenado por índice, hago una búsqueda binaria.

int inicio = 0;
int fin = i;
bool encontrado = false;
int pos = 0;
while (inicio<=fin && !encontrado){
    pos = (inicio+fin)/2;
    if(vectorequipos[pos].indice == local){
        encontrado=true;
        strncpy(fixture->infofixture.tabla[a].local[a].equipo, vectorequipos[pos].nombre,31);
        fixture->infofixture.tabla[a].local[a].paramA = vectorequipos[pos].paramA;
        fixture->infofixture.tabla[a].local[a].paramB = vectorequipos[pos].paramB;
    } else if (vectorequipos[pos].indice>local){
            fin=pos-1;
            }else inicio = pos+1;

};
}
void buscarvisitante(tipofixture *fixture, Equipo vectorequipos[], int visitante, int i, int &a){ //Búsqueda binaria para el visitante
int inicio = 0;
int fin = i;
bool encontrado = false;
int pos = 0;
while (inicio<=fin && !encontrado){
    pos = (inicio+fin)/2;
    if(vectorequipos[pos].indice == visitante){
        encontrado=true;
        strncpy(fixture->infofixture.tabla[a].visitante[a].equipo, vectorequipos[pos].nombre,31);
        fixture->infofixture.tabla[a].visitante[a].paramA = vectorequipos[pos].paramA;
        fixture->infofixture.tabla[a].visitante[a].paramB = vectorequipos[pos].paramB;
        a++; // sumo acá el a porque ya terminé esta sección
    } else if (vectorequipos[pos].indice>visitante){
            fin=pos-1;
            }else inicio = pos+1;
};
};




Acá la parte de simular partido, simularPartido devuelve por referencia goleslocal y golesvisitante:

Código [Seleccionar] Expandir

void menusimularpartido(tipofixture *fixture, tipofixture *listanueva,int fechastotales, int &PartidosPorFecha, posiciones vectortabla[], int &tamanio){
    int fechasimular;
    if(fechastotales>0){
    cout<<"Hasta que fecha desea simular los partidos?"<<endl;
    cin>>fechasimular;
    if(fechastotales<fechasimular && fechasimular>0){
        comprobarsimulacion(fixture,fechasimular,fechastotales,PartidosPorFecha,vectortabla,listanueva,tamanio);
    } else cout<< "El campeonato no llega hasta esa fecha (o se ha ingresado un valor mas chico que 1)!"; //si se escribe una fecha no permitida, lo indica.
    };

}


void comprobarsimulacion(tipofixture *fixture, int fechasimular, int fechastotales, int &PartidosPorFecha, posiciones vectortabla[], tipofixture *listanueva, int &tamanio){
    int i;
    if (recorrerlista(fixture,fechasimular)){
        cout<<"Ya hay fechas simuladas, desea sobreescribir los datos?"<<endl;
        cout<<"1.Si"<<endl;
        cout<<"2.No"<<endl;

        cin>>i;
        if (i=1){
            simularfecha(fixture,listanueva,fechasimular,PartidosPorFecha,vectortabla,tamanio);
        };
    } else simularfecha(fixture,listanueva,fechasimular,PartidosPorFecha,vectortabla, tamanio);


}

bool recorrerlista(tipofixture *fixture, int fechasimular){
    tipofixture *listaux=fixture;
    int a=1;
    bool t= false;
    while (a<=fechasimular){//si yo ingreso una fecha por ejemplo 4, y ya la primera está simulada, esto va a devolver verdadero para evitar sobreescribir la simulación anterior.
    if(listaux->infofixture.fuesimulado){
            t=true;
    } else listaux=listaux->siguiente; // como la lista está ordenada por fecha, no tiene sentido seguir, lo dejo porque no molesta.
    a++;
    }
    vaciarlista(listaux);
    return t;
}

void vaciarlista (tipofixture *listaux){//libero listas
    tipofixture *aux;
        while (listaux!=NULL){
            aux=listaux;
            listaux=listaux->siguiente;
            delete(aux);
        }
}
void simularfecha(tipofixture *fixture, tipofixture *listanueva, int fechasimular, int &PartidosPorFecha, posiciones vectortabla[], int &tamanio){

    int goleslocal=0;
    int golesvisitante=0;
    int totalpartidos= fechasimular*PartidosPorFecha;
    int a=1;
    while(fechasimular>=a){
        for(int x=1;x>=PartidosPorFecha;x++){
            int w=0;
            simularPartido(fixture->infofixture.tabla[w].local[w].paramA,fixture->infofixture.tabla[w].local[w].paramB,fixture->infofixture.tabla[w].visitante[w].paramA,fixture->infofixture.tabla[w].visitante[w].paramB,goleslocal,golesvisitante);
            if(goleslocal>golesvisitante){ //va a sumar los puntos por partido dependiendo del resultado, para luego ordenar más fácilmente.
            listanueva->infofixture.tabla[w].local[w].puntos=3;
            listanueva->infofixture.tabla[w].visitante[w].puntos=0;
            }else if (goleslocal<golesvisitante) {
            listanueva->infofixture.tabla[w].local[w].puntos=0;
            listanueva->infofixture.tabla[w].visitante[w].puntos=3;
            } else if (goleslocal=golesvisitante) { listanueva->infofixture.tabla[w].local[w].puntos=1;
            listanueva->infofixture.tabla[w].visitante[w].puntos=1;
            }
            listanueva->infofixture.tabla[w].local[w].resultado=goleslocal;
            listanueva->infofixture.tabla[w].visitante[w].resultado=golesvisitante;
            w++;

        }
        listanueva=listanueva->siguiente;
        a++;

    }




}



Lo final, acá hago la tabla de posiciones, ordenando un array y mostrando por pantalla los parámetros que indico allí. También acumulo 3 ptos. por partidos ganados.

Código [Seleccionar] Expandir

void MostrarEquipo(tipofixture *listanueva,int PartidosPorFecha, int fechastotales){
    tipofixture *mostrar = listanueva;
    char equipo[31];
    cout<<"ingrese el nombre del equipo que quiere buscar"<<endl;
    cin>>equipo;
    while (mostrar!=NULL){
        for(int x=1; x<=PartidosPorFecha; x++){
            int t=0;
            if(mostrar->infofixture.tabla[t].local[t].equipo[31]=equipo[31]){
                cout<<mostrar->infofixture.tabla[t].local[t].equipo<<"  "<< mostrar->infofixture.tabla[t].local[t].resultado<<"-"<<mostrar->infofixture.tabla[t].visitante[t].equipo<<"   "<<mostrar->infofixture.tabla[t].visitante[t].resultado<<endl;
            } else if (mostrar->infofixture.tabla[t].visitante[t].equipo[31]=equipo[31]){
                cout<<mostrar->infofixture.tabla[t].local[t].equipo<<"  "<< mostrar->infofixture.tabla[t].local[t].resultado<<"-"<<mostrar->infofixture.tabla[t].visitante[t].equipo<<"   "<<mostrar->infofixture.tabla[t].visitante[t].resultado<<endl;
                } else cout<<"ERROR: Parece que ha ingresado mal el nombre del equipo"<<endl;
        }
        mostrar=mostrar->siguiente;
    }
    vaciarlista(mostrar);
    }


void OrdenarLista(tipofixture *listanueva){
    tipofixture *listaordenada = NULL;
    while (listanueva!=NULL){
        tipofixture *ptr=listanueva;
        listanueva=listanueva->siguiente;
        ReinsertarEnLista(listaordenada,ptr);

    }
listanueva=listaordenada;
vaciarlista(listaordenada);

}

void ReinsertarEnLista(tipofixture *listaordenada, tipofixture *nuevo){
    tipofixture *ptr=listaordenada;
    tipofixture *ant;
    while (ptr!=NULL&&ptr->infofixture.fecha>nuevo->infofixture.fecha){
        ant = ptr;
        ptr=ptr->siguiente;
    }
nuevo->siguiente=ptr;
if(ptr=listaordenada){
    listaordenada=nuevo;
} else ant->siguiente=nuevo;

}

void MostrarTabla(tipofixture *listanueva, int m, int totalequipos, Equipo vectorequipos[]){
    posiciones vec[totalequipos-1];
    for (int x=1;x>=totalequipos;x++){
        int w=0;
        strncpy(vec[w].nombre,vectorequipos[w].nombre,31);
        w++;
    }
    while (listanueva!=NULL){
        acumularpuntaje (vec, listanueva, totalequipos);
        ordenarpuntaje(vec,totalequipos);
    }
    for (int l=1;l<=totalequipos;l++){
        int p=0;
        cout<<vec[p].nombre<<":"<< vec[p].puntos<<endl;
}

}

void acumularpuntaje (posiciones vec[],tipofixture *listanueva,int totalequipos){
    tipofixture *ptr=listanueva;
    while (ptr!=NULL){
        for(int x=1;x<=(totalequipos/2);x++){
            int w=0;
            if(ptr->infofixture.tabla[w].local[w].equipo[31]=vec[w].nombre[31]){
                    vec[w].puntos= vec[w].puntos+ptr->infofixture.tabla[w].local[w].puntos;

            } else if (ptr->infofixture.tabla[w].visitante[w].equipo[31]=vec[w].nombre[31]){
                    vec[w].puntos= vec[w].puntos+ptr->infofixture.tabla[w].visitante[w].puntos;
        }
    }
    ptr=ptr->siguiente;
    }
}
void ordenarpuntaje(posiciones vec[], int totalequipos){
    int aux=0;
    for(int j=0; j<=totalequipos-1; j++){
        for (int i=0; i<=totalequipos-1-j; i++){
            if(vec[i].puntos<vec[i+1].puntos){
                aux=vec[i].puntos;
                vec[i].puntos=vec[i+1].puntos;
                vec[i+1].puntos=aux;

            }
        }
    }


}




Uso CodeBlocks, lo probé en una pc con XP, Vista y W7, pero en todas cuando corro me deja de funcionar, con el simple mensaje de "El programa ha dejado de funcionar".

Muchísimas gracias a aquel que se haya gastado en leer todo esto, de verdad necesito una mano, cualquier pregunta voy a estar al tanto y a partir de hoy voy a intentar hacerme activo en el foro.
Intenté ser los más claro posible al programar.
Si a alguien le interesa las consignas del trabajo, acá están en un .pdf
https://groups.google.com/group/ayedea/attach/9902d97dd397670f/TP_1.pdf?part=0.0.1&authuser=0