¡Buenas DickGumshoe!

Tu problema es sencillode resolver.
La variable fila no la tienes inicializada al principio, si no que la inicializas dentro del bucle, cuando en realidad deberias hacerlo fuera.


Si fila=7 lo metes dentro del bucle, fila solo toma los valores 7 y 6.....
Ademas, tu variable fila es un int, eso significa que si el bucle se repite 8 veces, fila sera igual a "-1"......asi que deberas poner un:

¡Espero haberte ayudado, un saludo!

PD: Deberias tabular y comentar el codigo, asi te sera mas facil a ti y a los demas ver donde estan los fallos.

Hola

La solución es muy fácil, lo que tienes que hacer es que "fila" sea un array con tantos campos como columnas haya y en cada posición almacene la fila por la que va esa columna. Al principio inicializas todas las posiciones a 7 y si por ejemplo insertas en la columna "col" pues fila[col]-- y listo.

A esto tendrías que añadirle un control para que no se pueda insertar en las columnas que ya estén llenas.

Saludos

Hola

La idea es crearte un array que tenga tantas posiciones como columnas tenga el tablero, en este caso 8. Pues cada una de esas posiciones (que corresponden a las columnas del tablero) indicará dónde se insertará la ficha. Te lo explico con un ejemplo:

#include<stdio.h>
#include<stdlib.h>
 
int main (void)
{ 
   int M[8][8];
 
  int i, j, col, jugador, s, fila[8];
 
   for(i=0; i<8; i++){
       for(j=0; j<8; j++){
           M[i][j] = 0;
       }
       fila[i] = 7;
   }
 
   jugador = 1;
 
   do{    
 
 
   for(i=0; i<8; i++){
       for(j=0; j<8; j++){
           printf("%d  ",M[i][j]);                 
       }   
       printf("\n");                                  
   }
 
   printf("Introduce la columna (1-8): ");
   scanf("%d", &col);
 
   col = col-1; 
 
   M[fila[col]][col] = jugador;
 
   fila[col]--; //
 
 
   printf("Introduzca un 0 si abandona o un 1 si quiere seguir jugando: ");
   scanf("%d", &s);
   jugador = 3-jugador;
   }
   while(s==1); 
 
 
system("pause");    
return 0;
}
 

Si tienes más dudas ya sabes.

Saludos

Hola

La idea es crearte una función (no se si habrás llegado a ese tema) a la que le pasas por parámetros la matriz (el tablero) y solo tienes que comprobar si hay 4 posiciones consecutivas, en horizontal, vertical o diagonal, con el mismo valor (1 si en esa posición ha puesto el jugador 1 o 2 si ha puesto el jugador 2) y comprobar también que ese valor no es 0 (el tablero está inicializado a 0)

Coge un papel y un boli y dibújate el tablero y al lado del tablero el número de las filas y de las columnas para que veas las comprobaciones que tienes que hacer, es fácil pero lía un poco.

Si no te sale te pongo una posible solución

Saludos

Hola

Claro, esa matriz.

Un ejemplo sería:

int finPartida(int m[FILAS][COLUMNAS])
{
    int i, j;
    for(i = FILAS-1; i >= 3; i--)
        for(j = COLUMNAS-1; j >= 3; j--)
            if((m[i][j] == m[i-1][j] && m[i][j] == m[i-2][j] && m[i][j] == m[i-3][j] && m[i][j] != 0) || /*Aqui comprobamos si hay 4 iguales en la misma fila*/
               (m[i][j] == m[i][j-1] && m[i][j] == m[i][j-2] && m[i][j] == m[i][j-3] && m[i][j] != 0) || /*Aqui comprobamos si hay 4 iguales en la misma columna*/
               (m[i][j] == m[i-1][j-1] && m[i][j] == m[i-2][j-2] && m[i][j] == m[i-3][j-3] && m[i][j] != 0)) /*Aqui comprobamos si hay 4 iguales en la diagonal principal*/
                return 1;
 
    for(i = FILAS-1; i >= 3; i--)
        for(j = 0; j <= COLUMNAS-3; j++)
            if(m[i][j] == m[i-1][j+1] && m[i][j] == m[i-2][j+2] && m[i][j] == m[i-3][j+3] && m[i][j] != 0) /*Aqui comprobamos si hay 4 iguales en la diagonal secundaria*/
                return 1;
 
    return 0;
}
 

Aquí debes cambiar FILAS y COLUMNAS por 8, yo lo he hecho genérico.

Entonces si devuelve 1 es que hay 4 en raya, si devuelve 0 sigue el juego. Ahora tienes que meter en el main la llamada a esta función, y dependiendo del valor de retorno se acabará el juego o no.

Me voy a poner a hacer una versión del juego utilizando lo que llevas hasta ahora, cuando lo termine lo pongo aquí.

Saludos

Hola

El error que te está dando está en la llamada a la función, tú tienes finPartida(); pero debes pasarle la matriz finPartida(M);

Te pongo mi programa y así lo comparas, y si tienes alguna duda pregunta.

#include <stdio.h>
 
#define FILAS 8
#define COLUMNAS 8
 
void mostrarTablero(int m[FILAS][COLUMNAS]);
int finPartida(int m[FILAS][COLUMNAS]);
 
int main(void)
{
    int m[FILAS][COLUMNAS];
    int i, j, col, jugador, s = 0;
    int fila[COLUMNAS];
 
    for(i = 0; i < 8; i++)
    {
        for(j = 0; j < 8; j++)
            m[i][j] = 0;
        fila[i] = 7;
    }
 
    jugador = 1;
 
    do
    {
        mostrarTablero(m);
 
        printf("\nTurno del jugador %d\n", jugador);
 
        do
        {
            printf("Introduce la columna (1-8): ");
            scanf("%d", &col);
            if(fila[col-1] < 0)
            {
                col = 0;
                printf("\nEsa columna esta llena\n\n");
            }
        }
        while(col < 1 || col > 8);
 
        col = col-1;
        m[fila[col]][col] = jugador;
        fila[col]--;
 
        jugador = 3 - jugador;
 
        s = finPartida(m);
    }
    while(s == 0);
 
    mostrarTablero(m);
 
    printf("\n\nGANADOR EL JUGADOR %d\n\n", 3 - jugador);
 
    while(getchar() != '\n');
    printf("Pulsa Enter para salir...");
    getchar();
    return 0;
}
 
 
void mostrarTablero(int m[FILAS][COLUMNAS])
{
    int i, j;
 
    printf("\n");
    for(i = 0; i < FILAS; i++)
    {
        printf("%d - ", i+1);
        for(j = 0; j < COLUMNAS; j++)
            printf("%i  ", m[i][j]);
        printf("\n");
    }
 
    printf("   ");
    for(j = 0; j < COLUMNAS; j++)
        printf("___");
 
    printf("\n    ");
    for(j = 0; j < COLUMNAS; j++)
        printf("%i  ", j+1);
    printf("\n\n");
}
 
 
int finPartida(int m[FILAS][COLUMNAS])
{
    int i, j;
    for(i = FILAS-1; i >= 3; i--)
        for(j = COLUMNAS-1; j >= 3; j--)
            if((m[i][j] == m[i-1][j] && m[i][j] == m[i-2][j] && m[i][j] == m[i-3][j] && m[i][j] != 0) ||
               (m[i][j] == m[i][j-1] && m[i][j] == m[i][j-2] && m[i][j] == m[i][j-3] && m[i][j] != 0) ||
               (m[i][j] == m[i-1][j-1] && m[i][j] == m[i-2][j-2] && m[i][j] == m[i-3][j-3] && m[i][j] != 0))
                return 1;
 
    for(i = FILAS-1; i >= 3; i--)
        for(j = 0; j <= COLUMNAS-3; j++)
            if(m[i][j] == m[i-1][j+1] && m[i][j] == m[i-2][j+2] && m[i][j] == m[i-3][j+3] && m[i][j] != 0)
                return 1;
 
    return 0;
}
 

Saludos

Hola

Tienes razón, en las 3 primeras columnas no funciona el vertical, hice el código a ojo sin probarlo y esto es lo que pasa, pero es fácil de arreglar.

Bueno me alegro de que te haya quedado todo claro.

Saludos

Edito: también pasa lo mismo con las horizontales en las 3 filas de arriba