1

tenho essa função php (setBet) que cadastra as apostas no meu sistema baseado na BetFair, o fato é que estou querendo expandir o negócio e provavelmente algum apostador vai fazer uma aposta ao mesmo tempo que outro. A função abaixo Cadastra primeiro o cabeçalho da aposta na tabela "bet" e com a função sleep(1) eu espero 1 segundo para a inserção da query logo após, com a função SELECT MAX(id) eu seleciono o id que acabei de cadastrar para usar na foreignkey da tabela "bet_matches" que cadastra qual partida o cliente apostou, e o id da aposta dele.

MEU PROBLEMA : por usar a função sleep(1) e a MAX(id) eu com certeza vou ter algum problema no futuro, de mistura de apostas ou coisa parecida.. então, como posso fazer para melhorar essa função ?

public function setBet($punter_name, $punter_phone, $value, $colab_id, $array_match_and_bet)
{
    // Cadastrar Cabeçalho e Pegar ID Aposta
    $query =    "SET TIME_ZONE = '-03:00';";
    $query .= "INSERT INTO bet(id, punter_name, punter_phone, value, colab_id, date_time, possible_prize) values(NULL, '$punter_name', '$punter_phone', '$value', '$colab_id', now(), 0);";
    $data = mysqli_multi_query(Conexao::conectar(), $query) or die(Conexao::conectar());

    // Se Dados For Verdadeiro, Cabeçalho cadastrado
    if ($data){
        // Espera um segundo e Pega o id que acabamos de gerar
        sleep(1);
        $query = "SELECT MAX(id) FROM bet";
        $data = mysqli_query(Conexao::conectar(), $query);
        $line = mysqli_fetch_assoc($data);
        $last_id = $line["MAX(id)"];
        // Hora de cadastrar as partidas selecionadas Bitch
        sort($array_match_and_bet);
        $count = count($array_match_and_bet);
        for ($i=0; $i < $count ; $i++) { 
            $match_and_bet = explode('-', $array_match_and_bet[$i]);
            // var_dump($match_and_bet);
            $id_match = $match_and_bet[0];
            $type_bet = $match_and_bet[1];
            $odd_at_time = $match_and_bet[2];
            // Vetor para ser utilizado no calculo do premio possivel
            $odds[] = $match_and_bet[2];
            $query = "INSERT INTO `bet_matches` SET match_id = $id_match, type_bet = '$type_bet', bet_id = $last_id, datetime_add = now(), odd_at_time = $odd_at_time";
            // var_dump($query);
            $data = mysqli_query(Conexao::conectar(), $query);
            if (!$data) {
                exit();
            }
        }
        // Calcula a cotacao
        $possible_prize = Bet::calcBetPossiblePrize($odds) * $value;
        // Guarda o Premio Possivel
        Bet::setPossiblePrize($possible_prize, $last_id);

        if ($data) {
                header("Location: ../bet_details.php?bet_id=".$last_id."&msg=betMakeSuccess");
            }
            else{
                header("Location: ../bet_details.php?bet_id=".$last_id."&msg=betMakeDuplicate");
            }

    }
    else
        return 0;   
}

EDITADO: SEGUE MINHA FUNÇÃO Conexao::conectar()

public static function conectar()
{
    $con = mysqli_connect('localhost', 'root', 'vagrant', 'boleiros') or trigger_error(mysql_error(),E_USER_ERROR);

    if (mysqli_connect_errno())
    {
        echo "Failed to connect to MySQL: " . mysqli_connect_error();
        //you need to exit the script, if there is an error
        exit();
    }
    return $con;
    // var_dump($con);
}
9
  • 2
    Porque o uso do sleep?
    – Miguel
    Commented 17/06/2016 às 17:28
  • por que as vezes rola um delay no registro da query, e se a query não tiver sido executada ainda quando chegar a execução do MAX(id) ele pega o id da aposta anterior, aí embola tudo .. pq ele cadastra a nova aposta junto à aposta anterior :/
    – Yuri Foxx
    Commented 17/06/2016 às 17:32
  • 2
    Vai sim, porque o próprio script php não vai avançar sem o passo anterior estar concluido. Perdia-se toda a lógica de um programa se assim fosse. Imagine que declara que x=1 e depois faz y=x+2, obviamente para que o y seja criado precisa de haver um x...
    – Miguel
    Commented 17/06/2016 às 17:49
  • 2
    O @Miguel está certo, não faz sentido o próximo passo ser executado sem que o anterior tenha sido concluído. Commented 17/06/2016 às 17:52
  • 1
    @YuriFoxx por essa lógica o que lhe garantiria que 1 segundo bastava?
    – Miguel
    Commented 17/06/2016 às 17:53

1 Resposta 1

3

É muito importante que guarde a conecção numa variável para não estar sempre a conectar-se. Eu não sei como tem a sua conecção mas tente assim:

....
$conn = Conexao::conectar(); 
$query = "SET TIME_ZONE = '-03:00';";
$conn->query($query) or die('Erro na definição da timezone, tente mais tarde');
$query = "INSERT INTO bet (id, punter_name, punter_phone, value, colab_id, date_time, possible_prize) values(NULL, '$punter_name', '$punter_phone', '$value', '$colab_id', now(), 0);";
$data = $conn->query($query) or die('Erro na inserção, tente mais tarde');
$lastId = $conn->insert_id; // aqui guarda este e só este id, desta conecção que inseriu o novo dado

$query = "SELECT * FROM bet WHERE id=" .$lastId;
$data = mysqli_query($conn, $query);
    ....
14
  • 1
    Editei um pequeno detalhe em cima @YuriFoxx, na declaração de $lastId
    – Miguel
    Commented 17/06/2016 às 17:55
  • 1
    Nem precisa do if($data) {, porque $data ou existe ou é die()
    – Miguel
    Commented 18/06/2016 às 10:17
  • 1
    Sim, se guardar a conexão numa variável, tudo o que fizer na base de dados com aquela conexão só vai pertencer áquela conecção
    – Miguel
    Commented 22/06/2016 às 13:20
  • 1
    Editei o die('Erro na inserção, tente mais tarde') em cima, não sei se chegou a ver... Mas assim vai ter o que queria, o ultimo id inserido por aquela conecção... Ecusa de ter o sleep assim. E depois dentro do if pode alterar $data = mysqli_query(Conexao::conectar(), $query); para $data = mysqli_query($conn, $query);
    – Miguel
    Commented 22/06/2016 às 13:22
  • 1
    Pode editar a pergunta e colocar lá o seu metodo conectar() sff
    – Miguel
    Commented 22/06/2016 às 13:34

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .