ZsgsDesign / NOJ

app/Http/Controllers/Contest/BoardController.php

Braces +12 added lines, -10 removed lines

@@ -37,7 +37,7 @@ 
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
             if($clearance == 3){
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else{
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
@@ -83,7 +83,7 @@ 
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
             if($clearance == 3){
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else{
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
@@ -116,7 +116,9 @@ 
         }
 
         $editor_left_width = $accountModel->getExtra(Auth::user()->id, 'editor_left_width');
-        if(empty($editor_left_width)) $editor_left_width='40';
+        if(empty($editor_left_width)) {
+            $editor_left_width='40';
+        }
 
         return view('contest.board.editor', [
             'page_title'=>"Problem Detail",
@@ -153,7 +155,7 @@ 
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
             if($clearance == 3){
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else{
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
@@ -168,10 +170,10 @@ 
             if($i != 0) {
                 if($r['score'] == $contestRank[$i-1]['score'] && $r['penalty'] == $contestRank[$i-1]['penalty']){
                     $r['rank'] = $contestRank[$i-1]['rank'];
-                }else{
+                } else{
                     $r['rank'] = $i + 1;
                 }
-            }else{
+            } else{
                 $r['rank'] = 1;
             }
         }
@@ -212,7 +214,7 @@ 
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
             if($clearance == 3){
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else{
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
@@ -250,7 +252,7 @@ 
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
             if($clearance == 3){
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else{
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
@@ -285,7 +287,7 @@ 
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
             if($clearance == 3){
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else{
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
@@ -310,7 +312,7 @@ 
         if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
             if($clearance == 3){
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else{
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }

Spacing +38 added lines, -38 removed lines

@@ -34,10 +34,10 @@ 
         $contestModel=new ContestModel();
         $clearance=$contestModel->judgeClearance($cid, Auth::user()->id);
         $basicInfo=$contestModel->basic($cid);
-        if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
-            if($clearance == 3){
+        if (!$clearance || time()<strtotime($basicInfo['begin_time'])) {
+            if ($clearance==3) {
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else {
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
@@ -50,7 +50,7 @@ 
         if ($remainingTime<=0) {
             $remainingTime=0;
         }
-        if($basicInfo['public'] && !$basicInfo['audit_status']){
+        if ($basicInfo['public'] && !$basicInfo['audit_status']) {
             return Redirect::route('contest.detail', ['cid' => $cid]);
         }
         return view('contest.board.challenge', [
@@ -83,15 +83,15 @@ 
         $accountModel=new AccountModel();
         $clearance=$contestModel->judgeClearance($cid, Auth::user()->id);
         $basicInfo=$contestModel->basic($cid);
-        if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
-            if($clearance == 3){
+        if (!$clearance || time()<strtotime($basicInfo['begin_time'])) {
+            if ($clearance==3) {
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else {
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
         $basicInfo=$contestModel->basic($cid);
-        if($basicInfo['public'] && !$basicInfo['audit_status']){
+        if ($basicInfo['public'] && !$basicInfo['audit_status']) {
             return Redirect::route('contest.detail', ['cid' => $cid]);
         }
         $contest_name=$contestModel->contestName($cid);
@@ -122,8 +122,8 @@ 
             ];
         }
 
-        $editor_left_width = $accountModel->getExtra(Auth::user()->id, 'editor_left_width');
-        if(empty($editor_left_width)) $editor_left_width='40';
+        $editor_left_width=$accountModel->getExtra(Auth::user()->id, 'editor_left_width');
+        if (empty($editor_left_width)) $editor_left_width='40';
 
         return view('contest.board.editor', [
             'page_title'=>"Problem Detail",
@@ -157,15 +157,15 @@ 
         $contestModel=new ContestModel();
         $clearance=$contestModel->judgeClearance($cid, Auth::user()->id);
         $basicInfo=$contestModel->basic($cid);
-        if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
-            if($clearance == 3){
+        if (!$clearance || time()<strtotime($basicInfo['begin_time'])) {
+            if ($clearance==3) {
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else {
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
         $basicInfo=$contestModel->basic($cid);
-        if($basicInfo['public'] && !$basicInfo['audit_status']){
+        if ($basicInfo['public'] && !$basicInfo['audit_status']) {
             return Redirect::route('contest.detail', ['cid' => $cid]);
         }
         $contest_name=$contestModel->contestName($cid);
@@ -176,14 +176,14 @@ 
 
         // To determine the ranking
         foreach ($contestRank as $i => &$r) {
-            if($i != 0) {
-                if($r['score'] == $contestRank[$i-1]['score'] && $r['penalty'] == $contestRank[$i-1]['penalty']){
-                    $r['rank'] = $contestRank[$i-1]['rank'];
-                }else{
-                    $r['rank'] = $i + 1;
+            if ($i!=0) {
+                if ($r['score']==$contestRank[$i-1]['score'] && $r['penalty']==$contestRank[$i-1]['penalty']) {
+                    $r['rank']=$contestRank[$i-1]['rank'];
+                } else {
+                    $r['rank']=$i+1;
                 }
-            }else{
-                $r['rank'] = 1;
+            } else {
+                $r['rank']=1;
             }
         }
         $rankFrozen=$contestModel->isFrozen($cid);
@@ -220,15 +220,15 @@ 
         $contestModel=new ContestModel();
         $clearance=$contestModel->judgeClearance($cid, Auth::user()->id);
         $basicInfo=$contestModel->basic($cid);
-        if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
-            if($clearance == 3){
+        if (!$clearance || time()<strtotime($basicInfo['begin_time'])) {
+            if ($clearance==3) {
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else {
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
         $basicInfo=$contestModel->basic($cid);
-        if($basicInfo['public'] && !$basicInfo['audit_status']){
+        if ($basicInfo['public'] && !$basicInfo['audit_status']) {
             return Redirect::route('contest.detail', ['cid' => $cid]);
         }
         $contest_name=$contestModel->contestName($cid);
@@ -262,14 +262,14 @@ 
         $contestModel=new ContestModel();
         $clearance=$contestModel->judgeClearance($cid, Auth::user()->id);
         $basicInfo=$contestModel->basic($cid);
-        if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
-            if($clearance == 3){
+        if (!$clearance || time()<strtotime($basicInfo['begin_time'])) {
+            if ($clearance==3) {
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else {
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
-        if($basicInfo['public'] && !$basicInfo['audit_status']){
+        if ($basicInfo['public'] && !$basicInfo['audit_status']) {
             return Redirect::route('contest.detail', ['cid' => $cid]);
         }
         $contest_name=$contestModel->contestName($cid);
@@ -300,14 +300,14 @@ 
         $contestModel=new ContestModel();
         $clearance=$contestModel->judgeClearance($cid, Auth::user()->id);
         $basicInfo=$contestModel->basic($cid);
-        if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
-            if($clearance == 3){
+        if (!$clearance || time()<strtotime($basicInfo['begin_time'])) {
+            if ($clearance==3) {
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else {
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
-        if($basicInfo['public'] && !$basicInfo['audit_status']){
+        if ($basicInfo['public'] && !$basicInfo['audit_status']) {
             return Redirect::route('contest.detail', ['cid' => $cid]);
         }
         $contest_name=$contestModel->contestName($cid);
@@ -324,19 +324,19 @@ 
         ]);
     }
 
-    public function analysis($cid){
+    public function analysis($cid) {
         $contestModel=new ContestModel();
         $clearance=$contestModel->judgeClearance($cid, Auth::user()->id);
         $basicInfo=$contestModel->basic($cid);
-        if (!$clearance || time() < strtotime($basicInfo['begin_time'])) {
-            if($clearance == 3){
+        if (!$clearance || time()<strtotime($basicInfo['begin_time'])) {
+            if ($clearance==3) {
                 return Redirect::route('contest.admin', ['cid' => $cid]);
-            }else{
+            } else {
                 return Redirect::route('contest.detail', ['cid' => $cid]);
             }
         }
         $basicInfo=$contestModel->basic($cid);
-        if($basicInfo['public'] && !$basicInfo['audit_status']){
+        if ($basicInfo['public'] && !$basicInfo['audit_status']) {
             return Redirect::route('contest.detail', ['cid' => $cid]);
         }
         $contest_name=$contestModel->contestName($cid);

app/Http/Controllers/Contest/IndexController.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
         $filter["rated"]=isset($all_data["rated"]) ? $all_data["rated"] : null;
         $filter["anticheated"]=isset($all_data["anticheated"]) ? $all_data["anticheated"] : null;
         $filter["practice"]=isset($all_data["practice"]) ? $all_data["practice"] : null;
-        $return_list=$contestModel->list($filter,Auth::check()?Auth::user()->id:0);
+        $return_list=$contestModel->list($filter, Auth::check() ?Auth::user()->id : 0);
         $featured=$contestModel->featured();
         if (is_null($return_list)) {
             if (isset($all_data["page"]) && $all_data["page"]>1) {
@@ -66,7 +66,7 @@ 
         $contestModel=new ContestModel();
         $groupModel=new GroupModel();
         $clearance=Auth::check() ? $contestModel->judgeClearance($cid, Auth::user()->id) : 0;
-        $basic = $contestModel->basic($cid);
+        $basic=$contestModel->basic($cid);
         if (Auth::check()) {
             $contest_detail=$contestModel->detail($cid, Auth::user()->id);
             $registration=$contestModel->registration($cid, Auth::user()->id);

app/Babel/Extension/noj/Submitter.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
             "spj_version" => null,
             "spj_config" => null
         ];
-        if($probBasic["spj"] && $probBasic["spj_version"]){
+        if ($probBasic["spj"] && $probBasic["spj_version"]) {
             $submit_data["spj_version"]=$probBasic["spj_version"];
             $submit_data["spj_config"]=$probBasic["spj_lang"];
         }

app/Babel/Extension/noj/JudgeClient.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
     }
     public function getLanguageConfigByLanguage($language, $spj=false)
     {
-        return $this->getLanguageConfigByKey($language.($spj?'_lang_spj_config':'_lang_config'));
+        return $this->getLanguageConfigByKey($language.($spj ? '_lang_spj_config' : '_lang_config'));
     }
     public function getLanguageConfigByKey($key)
     {

app/Models/Search/ProblemSearchModel.php

Spacing +8 added lines, -8 removed lines

@@ -13,20 +13,20 @@
 
     public function search($key)
     {
-        $result = [];
-        if(strlen($key) >= 2){
-            $ret = self::where('pcode', $key)
-                ->orWhereRaw('MATCH(`title`) AGAINST (? IN BOOLEAN MODE)',[$key])
+        $result=[];
+        if (strlen($key)>=2) {
+            $ret=self::where('pcode', $key)
+                ->orWhereRaw('MATCH(`title`) AGAINST (? IN BOOLEAN MODE)', [$key])
                 ->select('pcode', 'title')
                 ->limit(120)
                 ->get()->all();
-            if(!empty($ret)){
-                $result += $ret;
+            if (!empty($ret)) {
+                $result+=$ret;
             }
         }
-        $problemModel = new ProblemModel();
+        $problemModel=new ProblemModel();
         foreach ($result as $p_index => $p) {
-            if($problemModel->isBlocked($p['pid']) || $problemModel->isHidden($p["pid"])){
+            if ($problemModel->isBlocked($p['pid']) || $problemModel->isHidden($p["pid"])) {
                 unset($result[$p_index]);
             }
         }

app/Admin/Controllers/SubmissionController.php

Spacing +9 added lines, -9 removed lines

@@ -83,11 +83,11 @@ 
         $grid->column('sid', "ID")->sortable();
         $grid->time("Time");
         $grid->memory("Memory");
-        $grid->verdict("Verdict")->display(function ($verdict) {
+        $grid->verdict("Verdict")->display(function($verdict) {
             return '<i class="fa fa-circle '.$this->color.'"></i> '.$verdict;
         });
         $grid->language("Language");
-        $grid->submission_date("Submission Date")->display(function ($submission_date) {
+        $grid->submission_date("Submission Date")->display(function($submission_date) {
             return date("Y-m-d H:i:s", $submission_date);
         });
         ;
@@ -97,11 +97,11 @@ 
         $grid->jid("JID");
         $grid->coid("COID");
         $grid->score("Raw Score");
-        $grid->filter(function (Grid\Filter $filter) {
-            $filter->column(6, function ($filter) {
+        $grid->filter(function(Grid\Filter $filter) {
+            $filter->column(6, function($filter) {
                 $filter->like('verdict');
             });
-            $filter->column(6, function ($filter) {
+            $filter->column(6, function($filter) {
                 $filter->equal('cid', 'Contest ID');
                 $filter->equal('uid', 'User ID');
                 $filter->equal('pid', 'Problem ID');
@@ -138,14 +138,14 @@ 
         $show->coid('COID');
         $show->vcid('VCID');
         $show->score();
-        $show->share()->using(['No','Yes']);
+        $show->share()->using(['No', 'Yes']);
         return $show;
     }
 
     private function codify($field, $lang=null)
     {
-        $field->unescape()->as(function ($value) use ($field,$lang) {
-            $field->border = false;
+        $field->unescape()->as(function($value) use ($field, $lang) {
+            $field->border=false;
             $hash=md5($value);
             if (blank($value)) {
                 $value=" ";
@@ -186,7 +186,7 @@ 
     {
         $form=new Form(new EloquentSubmissionModel);
         $form->model()->makeVisible('password');
-        $form->tab('Basic', function (Form $form) {
+        $form->tab('Basic', function(Form $form) {
             $form->display('sid');
             $form->text('time')->rules('required');
             $form->text('memory')->rules('required');

app/Admin/routes.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     $router->resource('contests', ContestController::class);
     $router->resource('groups', GroupController::class);
 
-    Route::group(['prefix' => 'babel'], function (Router $router) {
+    Route::group(['prefix' => 'babel'], function(Router $router) {
         $router->get('/', 'BabelController@index')->name('admin.babel.index');
         $router->get('installed', 'BabelController@installed')->name('admin.babel.installed');
         $router->get('marketspace', 'BabelController@marketspace')->name('admin.babel.marketspace');

app/Models/Eloquent/MessageModel.php

Spacing +25 added lines, -25 removed lines

@@ -17,11 +17,11 @@ 
      */
     public static function send($config)
     {
-        $message = new MessageModel;
-        $message->sender = $config['sender'];
-        $message->receiver = $config['receiver'];
-        $message->title = $config['title'];
-        $message->content = $config['content'];
+        $message=new MessageModel;
+        $message->sender=$config['sender'];
+        $message->receiver=$config['receiver'];
+        $message->title=$config['title'];
+        $message->content=$config['content'];
         /*
         if(isset($config['reply'])){
             $message->reply = $config['reply'];
@@ -30,7 +30,7 @@ 
             $message->reply = $config['allow_reply'];
         }
         */
-        $message->official = 1;
+        $message->official=1;
         $message->save();
         return true;
     }
@@ -61,8 +61,8 @@ 
      */
     public static function allowReply($id)
     {
-        $message = static::where('id',$id)->first();
-        if(empty($message)){
+        $message=static::where('id', $id)->first();
+        if (empty($message)) {
             return false;
         }
         return $message['allow_reply'] ? true : false;
@@ -77,8 +77,8 @@ 
      */
     public static function list($uid)
     {
-        return static::join('users','message.sender','=','users.id')
-            ->where('receiver',$uid)
+        return static::join('users', 'message.sender', '=', 'users.id')
+            ->where('receiver', $uid)
             ->select(
                 'message.id as id',
                 'users.name as sender_name',
@@ -103,13 +103,13 @@ 
      */
     public static function read($mid)
     {
-        $message = static::find($mid);
-        if(!empty($message)){
-            $message->unread = 0;
+        $message=static::find($mid);
+        if (!empty($message)) {
+            $message->unread=0;
             $message->save();
         }
-        return static::join('users','message.sender','=','users.id')
-            ->where('message.id',$mid)
+        return static::join('users', 'message.sender', '=', 'users.id')
+            ->where('message.id', $mid)
             ->select(
                 'message.id as id',
                 'users.name as sender_name',
@@ -133,7 +133,7 @@ 
      */
     public static function allRead($uid)
     {
-        return static::where('receiver',$uid)
+        return static::where('receiver', $uid)
             ->update(['unread' => 0]);
     }
 
@@ -161,20 +161,20 @@ 
      */
     public static function remove($messages)
     {
-        $del_count = 0;
-        if(is_array($messages)){
+        $del_count=0;
+        if (is_array($messages)) {
             foreach ($messages as $mid) {
-                $message = static::find($mid);
-                if(!empty($message)){
+                $message=static::find($mid);
+                if (!empty($message)) {
                     $message->delete();
-                    $del_count ++;
+                    $del_count++;
                 }
             }
-        }else{
-            $message = static::find($messages);
-            if(!empty($message)){
+        } else {
+            $message=static::find($messages);
+            if (!empty($message)) {
                 $message->delete();
-                $del_count ++;
+                $del_count++;
             }
         }
         return $del_count;

Braces +1 added lines, -1 removed lines

@@ -170,7 +170,7 @@
                     $del_count ++;
                 }
             }
-        }else{
+        } else{
             $message = static::find($messages);
             if(!empty($message)){
                 $message->delete();

app/Models/ProblemModel.php

Spacing +35 added lines, -35 removed lines

@@ -48,11 +48,11 @@ 
             $prob_detail["pdf"]=false;
             $prob_detail["viewerShow"]=false;
             $prob_detail["file_ext"]=null;
-            if($prob_detail['file'] && !blank($prob_detail['file_url'])){
-                $prob_detail["file_ext"]=explode('.',basename($prob_detail['file_url']));
+            if ($prob_detail['file'] && !blank($prob_detail['file_url'])) {
+                $prob_detail["file_ext"]=explode('.', basename($prob_detail['file_url']));
                 $prob_detail["file_ext"]=end($prob_detail["file_ext"]);
                 $prob_detail["pdf"]=Str::is("*.pdf", basename($prob_detail['file_url']));
-                $prob_detail["viewerShow"]= blank($prob_detail["parsed"]["description"]) &&
+                $prob_detail["viewerShow"]=blank($prob_detail["parsed"]["description"]) &&
                                             blank($prob_detail["parsed"]["input"]) &&
                                             blank($prob_detail["parsed"]["output"]) &&
                                             blank($prob_detail["parsed"]["note"]);
@@ -194,9 +194,9 @@ 
 
     private function inteliAudit($uid, $content)
     {
-        if (strpos($content, '```')!==false){
+        if (strpos($content, '```')!==false) {
             $userSolutionHistory=DB::table("problem_solution")->where(['uid'=>$uid])->orderByDesc('updated_at')->first();
-            if (!empty($userSolutionHistory) && $userSolutionHistory["audit"]==1){
+            if (!empty($userSolutionHistory) && $userSolutionHistory["audit"]==1) {
                 return 1;
             }
         }
@@ -292,7 +292,7 @@ 
     {
         // $prob_list = DB::table($this->table)->select("pid","pcode","title")->get()->all(); // return a array
         $submissionModel=new SubmissionModel();
-        $preQuery=DB::table($this->table)->where('hide','=',0);
+        $preQuery=DB::table($this->table)->where('hide', '=', 0);
         if ($filter['oj']) {
             $preQuery=$preQuery->where(["OJ"=>$filter['oj']]);
         }
@@ -446,7 +446,7 @@ 
 
         if (!empty($data["sample"])) {
             foreach ($data["sample"] as $d) {
-                if(!isset($d['sample_note'])) $d['sample_note']=null;
+                if (!isset($d['sample_note'])) $d['sample_note']=null;
                 DB::table("problem_sample")->insert([
                     'pid'=>$pid,
                     'sample_input'=>$d['sample_input'],
@@ -492,7 +492,7 @@ 
 
         if (!empty($data["sample"])) {
             foreach ($data["sample"] as $d) {
-                if(!isset($d['sample_note'])) $d['sample_note']=null;
+                if (!isset($d['sample_note'])) $d['sample_note']=null;
                 DB::table("problem_sample")->insert([
                     'pid'=>$pid,
                     'sample_input'=>$d['sample_input'],
@@ -507,7 +507,7 @@ 
 
     public function discussionList($pid)
     {
-        $paginator = DB::table('problem_discussion')->join(
+        $paginator=DB::table('problem_discussion')->join(
             "users",
             "id",
             "=",
@@ -526,10 +526,10 @@ 
             'users.name',
             'users.id as uid'
         ])->paginate(15);
-        $list = $paginator->all();
-        foreach($list as &$l){
-            $l['updated_at'] = $this->formatTime($l['updated_at']);
-            $l['comment_count'] = DB::table('problem_discussion_comment')->where('pdid','=',$l['pdid'])->count();
+        $list=$paginator->all();
+        foreach ($list as &$l) {
+            $l['updated_at']=$this->formatTime($l['updated_at']);
+            $l['comment_count']=DB::table('problem_discussion_comment')->where('pdid', '=', $l['pdid'])->count();
         }
         return [
             'paginator' => $paginator,
@@ -572,7 +572,7 @@ 
 
     public function discussionDetail($pdid)
     {
-        $main = DB::table('problem_discussion')->join(
+        $main=DB::table('problem_discussion')->join(
             "users",
             "id",
             "=",
@@ -591,12 +591,12 @@ 
             'users.name',
             'users.id as uid'
         ])->get()->first();
-        $main['created_at'] = $this->formatTime($main['created_at']);
+        $main['created_at']=$this->formatTime($main['created_at']);
         $main['content']=clean(Markdown::convertToHtml($main["content"]));
 
-        $comment_count = DB::table('problem_discussion_comment')->where('pdid','=',$pdid)->count();
+        $comment_count=DB::table('problem_discussion_comment')->where('pdid', '=', $pdid)->count();
 
-        $paginator = DB::table('problem_discussion_comment')->join(
+        $paginator=DB::table('problem_discussion_comment')->join(
             "users",
             "id",
             "=",
@@ -615,11 +615,11 @@ 
             'users.name',
             'users.id as uid'
         ])->paginate(10);
-        $comment = $paginator->all();
-        foreach($comment as &$c){
+        $comment=$paginator->all();
+        foreach ($comment as &$c) {
             $c['content']=clean(Markdown::convertToHtml($c["content"]));
-            $c['created_at'] = $this->formatTime($c['created_at']);
-            $c['reply'] = DB::table('problem_discussion_comment')->join(
+            $c['created_at']=$this->formatTime($c['created_at']);
+            $c['reply']=DB::table('problem_discussion_comment')->join(
                 "users",
                 "id",
                 "=",
@@ -647,20 +647,20 @@ 
                 'users.name',
                 'users.id as uid'
             ])->get()->all();
-            foreach($c['reply'] as $k=>&$cr){
+            foreach ($c['reply'] as $k=>&$cr) {
                 $cr['content']=clean(Markdown::convertToHtml($cr["content"]));
-                $cr['reply_uid'] = DB::table('problem_discussion_comment')->where(
+                $cr['reply_uid']=DB::table('problem_discussion_comment')->where(
                     'pdcid',
                     '=',
                     $cr['reply_id']
                 )->get()->first()['uid'];
-                $cr['reply_name'] = DB::table('users')->where(
+                $cr['reply_name']=DB::table('users')->where(
                     'id',
                     '=',
                     $cr['reply_uid']
                 )->get()->first()['name'];
-                $cr['created_at'] = $this->formatTime($cr['created_at']);
-                if($this->replyParent($cr['pdcid'])!=$c['pdcid']){
+                $cr['created_at']=$this->formatTime($cr['created_at']);
+                if ($this->replyParent($cr['pdcid'])!=$c['pdcid']) {
                     unset($c['reply'][$k]);
                 }
             }
@@ -675,19 +675,19 @@ 
 
     public function replyParent($pdcid)
     {
-        $reply_id=DB::table('problem_discussion_comment')->where('pdcid','=',$pdcid)->get()->first()['reply_id'];
-        $top=DB::table('problem_discussion_comment')->where('pdcid','=',$reply_id)->get()->first()['reply_id'];
-        if(isset($top)){
+        $reply_id=DB::table('problem_discussion_comment')->where('pdcid', '=', $pdcid)->get()->first()['reply_id'];
+        $top=DB::table('problem_discussion_comment')->where('pdcid', '=', $reply_id)->get()->first()['reply_id'];
+        if (isset($top)) {
             return $this->replyParent($reply_id);
-        }else{
+        } else {
             return $reply_id;
         }
     }
 
     public function pcodeByPdid($dcode)
     {
-        $pid = DB::table('problem_discussion')->where('pdid','=',$dcode)->get()->first()['pid'];
-        $pcode = $this->pcode($pid);
+        $pid=DB::table('problem_discussion')->where('pdid', '=', $dcode)->get()->first()['pid'];
+        $pcode=$this->pcode($pid);
         return $pcode;
     }
 
@@ -708,11 +708,11 @@ 
 
     public function pidByPdid($pdid)
     {
-        $pid = DB::table('problem_discussion')->where('pdid','=',$pdid)->get()->first()['pid'];
+        $pid=DB::table('problem_discussion')->where('pdid', '=', $pdid)->get()->first()['pid'];
         return $pid;
     }
 
-    public function addComment($uid,$pdid,$content,$reply_id)
+    public function addComment($uid, $pdid, $content, $reply_id)
     {
         $pid=$this->pidByPdid($pdid);
         $pdcid=DB::table('problem_discussion_comment')->insertGetId([
@@ -731,6 +731,6 @@ 
   
     public function isHidden($pid)
     {
-        return DB::table('problem')->where('pid','=',$pid)->get()->first()['hide'];
+        return DB::table('problem')->where('pid', '=', $pid)->get()->first()['hide'];
     }
 }

Braces +7 added lines, -3 removed lines

@@ -446,7 +446,9 @@ 
 
         if (!empty($data["sample"])) {
             foreach ($data["sample"] as $d) {
-                if(!isset($d['sample_note'])) $d['sample_note']=null;
+                if(!isset($d['sample_note'])) {
+                    $d['sample_note']=null;
+                }
                 DB::table("problem_sample")->insert([
                     'pid'=>$pid,
                     'sample_input'=>$d['sample_input'],
@@ -492,7 +494,9 @@ 
 
         if (!empty($data["sample"])) {
             foreach ($data["sample"] as $d) {
-                if(!isset($d['sample_note'])) $d['sample_note']=null;
+                if(!isset($d['sample_note'])) {
+                    $d['sample_note']=null;
+                }
                 DB::table("problem_sample")->insert([
                     'pid'=>$pid,
                     'sample_input'=>$d['sample_input'],
@@ -679,7 +683,7 @@ 
         $top=DB::table('problem_discussion_comment')->where('pdcid','=',$reply_id)->get()->first()['reply_id'];
         if(isset($top)){
             return $this->replyParent($reply_id);
-        }else{
+        } else{
             return $reply_id;
         }
     }