0

По началу таблица пуста:

введите сюда описание изображения

Затем заполняю ее информацией

введите сюда описание изображения

И Вот этот код срабатывает только на первую запись, то есть код не должен создавать еще одну запись с таким же id_match. Цикл foreach не перебирает все записи, а только первую, почему? И как сделать чтоб цикл перебирал все записи из таблицы? Может проблема в запросе?

public function BetsOne(Request $request)
{
    $input = $request->all();
    $bets = Bet::all()->where('user',Auth::id());

    if($bets->isEmpty()){
        $info = InfoUser::where('user_id',Auth::id())->first()->balans;
        if($request->summa <= $info){
            $bet = new Bet();
            $bet->fill($input);
            $bet->save(); //записыаем все с формы в табилцу bets

            $iu = InfoUser::where('user_id',Auth::id())
                ->decrement('balans', $request->summa);

            $match = Match::find($request->id_match);
            $match->balans_one = $match->balans_one + $request->summa; //перезаписываем поле balans_one
            $proc = ceil($match->balans_one * 100 / ($match->balans_one+$match->balans_two));
            $koff = round(100/$proc, 2);//коэффициент
            $match->koof_one = $koff;
            if($match->balans_two != 0){
                $proc_two = 101 - $proc;
                $koff_two = round(100/$proc_two,2);
                $match->koof_two = $koff_two;
            }
            if($match->save()){
                Session::flash('store', 'Ваша ставка принята!');
                return redirect()->route('home');
            }
        }else{
            Session::flash('danger','Недостаточно средств на вашем счету!');
            return redirect()->route('home');
        }
    }else{
        foreach($bets as $b){
            if($b->id_match != $request->id_match){
                $info = InfoUser::where('user_id',Auth::id())->first()->balans;
                if($request->summa <= $info){
                    $bet = new Bet();
                    $bet->fill($input);
                    $bet->save(); //записыаем все с формы в табилцу bets

                    $iu = InfoUser::where('user_id',Auth::id())
                        ->decrement('balans', $request->summa);

                    $match = Match::find($request->id_match);
                    $match->balans_one = $match->balans_one + $request->summa; //перезаписываем поле balans_one
                    $proc = ceil($match->balans_one * 100 / ($match->balans_one+$match->balans_two));
                    $koff = round(100/$proc, 2);//коэффициент
                    $match->koof_one = $koff;
                    if($match->balans_two != 0){
                        $proc_two = 101 - $proc;
                        $koff_two = round(100/$proc_two,2);
                        $match->koof_two = $koff_two;
                    }
                    if($match->save()){
                        Session::flash('store', 'Ваша ставка принята!');
                        return redirect()->route('home');
                    }
                }else{
                    Session::flash('danger','Недостаточно средств на вашем счету!');
                    return redirect()->route('home');
                }
            }else{
                Session::flash('danger','Вы уже сделали ставку на этот матч');
                return redirect()->route('home');
            }
        }
    }
}
5
  • Может проблема в строке if($b->id_match != $request->id_match)? Там у вас значение 1 и поэтому только 1 раз отрабатывает код
    – TheMY3
    30 апр 2018 в 10:50
  • @YaroslavMolchan нет не потому, просто в else стоит return и функция прекращает свою работу. 30 апр 2018 в 10:54
  • @Digital Core Кстати.. да, возможно 30 апр 2018 в 10:59
  • @Digital Core А в каком else? Можете уточнить? И на что заменить return? 30 апр 2018 в 11:39
  • @DigitalCore да, верно, не заметил это.
    – TheMY3
    30 апр 2018 в 12:50

1 ответ 1

1

Вы должны построить функцию таким образом, чтоб завершение работы функции return вызывалось только после выполнения всех задач.

function Function_Name ($arg) {
    foreach(var t in $arg){
        if(t)
            return; // выход из цикла, и функции с значением NULL
    }
    var $result = ...
    return $result; // завершение работы функции - возврат результата
    }

Этот код завершится при первом вхождении true в arg - если вам нужно обработать весь массив аргументов, не вызывайте в цикле return

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.