[SOLUCIONADO] Problema con Codigo C++

[albertofdez92]

Hola, el otro dia me pasaron un codigo para hacer un sorteo por grupos, e modificado algunas cosas, y me daban error pero las e ido solucionando, el problema que tengo ahora es con printf que me da un error y no caigo porque. este es el error que me indica.

http://www.casimages.es/i/140505013410333844.png.html

[vangodp]

si no pones el source completo no sabemos lo que te pasa.

[albertofdez92]

lo siento, este es el codigo entero.

#include <cstdlib>
#include <iostream>
#include <conio.h>
#include <time.h>

using namespace std;

int main(int argc, char *argv[])
{
//NOTA ESTE ES UN SISTEMA QUE GENERE SORTEO DE 8 EQUIPOS SI QUIERE INCLUIR
//MAS EQUIPOS NECESITA CAMBIAR LAS VARIABLES DE LOS VECTORES Y LOS CICLOS
//WHILE, FOR

//Signfica que se crea un arreglo con 8 equipos
//Y cada equipo tendra un total de 30 letras
//para el nombre, ejemplo: Alemania
char *paises[30] ={"Teresa", "MariaM", "David", "IreneS", "Olivia", "SergioF", "Alberto", "ElenaA", "SaraA", "JavierR",
"Daniel", "MariC", "CarmenG", "AlvaroA"};

//Variable que llevara el control de los paises que ya salieron en
//sorteo de los grupos
//Se uso la configuracion los primeros cuatro primer grupo
//los ultimos cuatro son el segundo grupo
int participantes[14];
//Variable que contara los paises que han participado o que
//ya salieron en el sorteo
int contadorpaises = 0;
int i =0;

//Se declara las variables para generar el numero aleatorio
int numeroaleatorio = 0;
int numeroobtenido =0;
bool numerocorrecto = true;

//Se ejecuta hasta que el contador sea 8
//por lo cual si el contador es menor de 8 significa
//que no sean creado todo los grupos
while(contadorpaises <14)
{
//Se especifica el numero correcto
numerocorrecto = true;
//Se crea el numero aleatorio
srand (time (NULL));
//Se crea los numeros aleatorios
//del 1 al 8
numeroaleatorio =rand() % 14;

//Se recorren los paises que ya estan agregados en
//el torneo
for(i=0; i<contadorpaises;i++)
{
//Si el numero aleatorio es un pais que ya agregado en
//no se puede volver a usar por lo cual se valida
if(numeroaleatorio == participantes[i])
{
//El numero no es correcto y se termina el ciclo FOR
numerocorrecto =false;
break;
}
}

//Si el numero que se genero es correcto
if(numerocorrecto ==true)
{
//Se agrega el pais a un grupo
participantes[contadorpaises] = numeroaleatorio;
contadorpaises = contadorpaises + 1;
}
}

//Variable funciona para recorrer los grupos y mostrarlos al usuario
contadorpaises = 1;

//Se imprimen los grupos en pantalla para que el usuario pueda verlos
for(i=0; i<22;i++)
{
if(i==0)
{
//Se imprime el primer grupo
printf("PRIMER GRUPO \n");
}
if(i==4)
{

//Se imprime el primer grupo
printf("SEGUNDO GRUPO \n");
}
//Se muestra el participante
printf("Equipo %d -- %s\n", contadorpaises, paises[participantes[i]]);
contadorpaises = contadorpaises + 1;
}

system("PAUSE");
return EXIT_SUCCESS;
}

[vangodp]

Supongo que si el programa arranca seras capaz de seguir¿No? Tu problema es por que printf no anda?
Pues si estas compilando en c++ debes incluir cstdio #include<cstdio> o en vez de usar printf usas cout que para eso esta #include<iostream>.
Como veo que hay librerías de c++ estas obligado a guardarlo como .cpp y usar el compilador de cpp.
Las librerías son otro punto, en c usas stdio.h para usar printf pero en cpp le quitas la h y pon una c delante, quedando como dije antes cstdio.
Librerías como stdlib.h stdio.h quedan como cstdlib y cstdio, no es obligado a ponerlas así pero es mejor por que esas son las de cpp.
Si sigues con problemas aquí estamos ;)

He visto mas errores pero necesito algo de tiempo para mirarlos.
Lo dejo aqui el código organizado para que si alguien tiene mas luces que yo te vaya ayudando en algo.

Código C++:

#include <cstdlib>
#include <iostream>
#include <conio.h>
#include <time.h>
#include <cstdio> // <-Esto le faltaba para el printf furular XD
using namespace std;
 
int main ( int argc, char * argv[] ) {
    //NOTA ESTE ES UN SISTEMA QUE GENERE SORTEO DE 8 EQUIPOS SI QUIERE INCLUIR
    //MAS EQUIPOS NECESITA CAMBIAR LAS VARIABLES DE LOS VECTORES Y LOS CICLOS
    //WHILE, FOR
    
    //Signfica que se crea un arreglo con 8 equipos
    //Y cada equipo tendra un total de 30 letras
    //para el nombre, ejemplo: Alemania
    char * paises[30] = {"Teresa", "MariaM", "David", "IreneS", "Olivia", "SergioF", "Alberto", "ElenaA", "SaraA", "JavierR",
                         "Daniel", "MariC", "CarmenG", "AlvaroA"
                        };
                        
    //Variable que llevara el control de los paises que ya salieron en
    //sorteo de los grupos
    //Se uso la configuracion los primeros cuatro primer grupo
    //los ultimos cuatro son el segundo grupo
    int participantes[14];
    //Variable que contara los paises que han participado o que
    //ya salieron en el sorteo
    int contadorpaises = 0;
    int i = 0;
    
    //Se declara las variables para generar el numero aleatorio
    int numeroaleatorio = 0;
    int numeroobtenido = 0;
    bool numerocorrecto = true;
    
    //Se ejecuta hasta que el contador sea 8
    //por lo cual si el contador es menor de 8 significa
    //que no sean creado todo los grupos
    while ( contadorpaises < 14 ) {
        //Se especifica el numero correcto
        numerocorrecto = true;
        //Se crea el numero aleatorio
        srand ( time ( NULL ) );
        //Se crea los numeros aleatorios
        //del 1 al 8
        numeroaleatorio = rand() % 14;
        
        //Se recorren los paises que ya estan agregados en
        //el torneo
        for ( i = 0; i < contadorpaises; i++ ) {
            //Si el numero aleatorio es un pais que ya agregado en
            //no se puede volver a usar por lo cual se valida
            if ( numeroaleatorio == participantes[i] ) {
                //El numero no es correcto y se termina el ciclo FOR
                numerocorrecto = false;
                break;
            }
        }
        
        //Si el numero que se genero es correcto
        if ( numerocorrecto == true ) {
            //Se agrega el pais a un grupo
            participantes[contadorpaises] = numeroaleatorio;
            contadorpaises = contadorpaises + 1;
        }
    }
    
    //Variable funciona para recorrer los grupos y mostrarlos al usuario
    contadorpaises = 1;
    
    //Se imprimen los grupos en pantalla para que el usuario pueda verlos
    for ( i = 0; i < 22; i++ ) {
        if ( i == 0 ) {
            //Se imprime el primer grupo
            printf ( "PRIMER GRUPO \n" );
        }
        
        if ( i == 4 ) {
        
            //Se imprime el primer grupo
            printf ( "SEGUNDO GRUPO \n" );
        }
        
        //Se muestra el participante
        printf ( "Equipo %d -- %s\n", contadorpaises, paises[participantes[i]] );
        contadorpaises = contadorpaises + 1;
    }
    
    system ( "PAUSE" );
    return EXIT_SUCCESS;
}
 

[albertofdez92]

joe muchas gracias, esque tengo bastante abandonado esto y ya hay cosas que no recuerdo, se que tengo varios errores mas, como estos:

C:\Users\afernandez\Downloads\main.cpp In function 'int main(int, char**)':

19 41 C:\Users\afernandez\Downloads\main.cpp [Warning] deprecated conversion from string constant to 'char*' [-Wwrite-strings]

[vangodp]

Es un aviso mas bien que un error, desaconsejando que uses puntero de esta forma char *p = "cadena"; ya que la cadena no la vas a poder modificar después.
Lo único que puedes hacer es apuntar a otra cadena.

[albertofdez92]

a vale entonces solo le falta el error del main


C:\Users\afernandez\Downloads\main.cpp In function 'int main(int, char**)':

[vangodp]

el int, y el char los puedes borrar.
sirven para cuando vas a pasar argumentos desde fuera del programa.
Si dejas a asi int main() no le va pasar nada al programa.

Pero lo que te esta diciendo es que en la función main esta pasando el error de antes del puntero.
Pero si no te deja compilar borra lo que tienes entre() de main como dije.
¿Por curiosidad que compilador/IDE usas?

[albertofdez92]

Dev-C++ 5.3.0.4

lo he dejado como dices int main () y sigue sin dejar, marca el error en #include <cstdlib>

Código PHP:

  #include <cstdlib>
#include <iostream>
#include <conio.h>
#include <time.h> 
#include <cstdio>
 
using namespace std;
 
int main()
{
    //NOTA ESTE ES UN SISTEMA QUE GENERE SORTEO DE 8 EQUIPOS SI QUIERE INCLUIR
    //MAS EQUIPOS NECESITA CAMBIAR LAS VARIABLES DE LOS VECTORES Y LOS CICLOS
    //WHILE, FOR
    
    //Signfica que se crea un arreglo con 8 equipos
    //Y cada equipo tendra un total de 30 letras
    //para el nombre, ejemplo: Alemania
    char *paises[30] ={"Teresa", "MariaM", "David", "IreneS", "Olivia",  "SergioF", "Alberto",  "ElenaA", "SaraA", "JavierR", 
    "Daniel", "MariC", "CarmenG", "AlvaroA"};
    
    //Variable que llevara el control de los paises que ya salieron en
    //sorteo de los grupos
    //Se uso la configuracion los primeros cuatro primer grupo
    //los ultimos cuatro son el segundo grupo
    int participantes[14];
    //Variable que contara los paises que han participado o que
    //ya salieron en el sorteo
    int contadorpaises = 0;
    int i =0;
    
    //Se declara las variables para generar el numero aleatorio
    int numeroaleatorio = 0;
    int numeroobtenido =0;
    bool numerocorrecto = true;
    
    //Se ejecuta hasta que el contador sea 8
    //por lo cual si el contador es menor de 8 significa
    //que no sean creado todo los grupos
    while(contadorpaises <14)
    {
            //Se especifica el numero correcto
            numerocorrecto = true;
            //Se crea el numero aleatorio
            srand (time (NULL));
            //Se crea los numeros aleatorios
            //del 1 al 8
            numeroaleatorio =rand() % 14;
 
            //Se recorren los paises que ya estan agregados en
            //el torneo
            for(i=0; i<contadorpaises;i++)
            {
                     //Si el numero aleatorio es un pais que ya agregado en
                     //no se puede volver a usar por lo cual se valida
                    if(numeroaleatorio == participantes[i])
                    {
                           //El numero no es correcto y se termina el ciclo FOR
                           numerocorrecto =false;
                           break;
                    }
            }
            
            //Si el numero que se genero es correcto
            if(numerocorrecto ==true)
            {
                    //Se agrega el pais a un grupo
                    participantes[contadorpaises] = numeroaleatorio;
                    contadorpaises = contadorpaises + 1;
            }                  
    }
    
    //Variable funciona para recorrer los grupos y mostrarlos al usuario
    contadorpaises = 1;
    
    //Se imprimen los grupos en pantalla para que el usuario pueda verlos
    for(i=0; i<22;i++)
    {
             if(i==0)
             {
                    //Se imprime el primer grupo
                    printf("PRIMER GRUPO \n");
             }
             if(i==4)
             {
                    
                    //Se imprime el primer grupo
                    printf("SEGUNDO GRUPO \n");
             }
             //Se muestra el participante
             printf("Equipo %d -- %s\n", contadorpaises, paises[participantes[i]]);
             contadorpaises = contadorpaises + 1; 
    }
 
    system("PAUSE");
    return EXIT_SUCCESS;
} 
   

[albertofdez92]

vale ya compila y ejecuta, solo faltaba una cosa que no me di cuenta yo, torpeza mia, muchas gracias.

[vangodp]

time la puedes cambiar también de time.h por ctime ;)
para mas información aquí las tienes :D
http://www.cplusplus.com/reference/