normalize-logging

This commit is contained in:
xiaomlove
2021-04-26 20:37:17 +08:00
parent 58c6aa58e8
commit 63a2c71cb6
18 changed files with 225 additions and 109 deletions

View File

@@ -45,7 +45,8 @@ class Test extends Command
public function handle()
{
$rep = new ExamRepository();
$r = $rep->listUserExamProgress(1);
// $r = $rep->assignToUser(3);
echo new A;
dd($r);
}
}

View File

@@ -8,6 +8,7 @@ use App\Http\Resources\UserResource;
use App\Repositories\ExamRepository;
use App\Repositories\UserRepository;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
class ExamController extends Controller
{
@@ -110,4 +111,13 @@ class ExamController extends Controller
return $this->success($resource);
}
public function progress(Request $request)
{
$result = $this->repository->getUserExamProgress(Auth::id());
$resource = new ExamUserResource($result);
return $resource;
// dd($resource->response()->getData(true));
return $this->success($resource);
}
}

View File

@@ -9,6 +9,8 @@ use Illuminate\Http\Resources\Json\JsonResource;
class ExamUserResource extends JsonResource
{
public $preserveKeys = true;
/**
* Transform the resource into an array.
*
@@ -21,10 +23,14 @@ class ExamUserResource extends JsonResource
'id' => $this->id,
'status' => $this->status,
'status_text' => $this->statusText,
'created_at' => $this->created_at->format('Y-m-d H:i:s'),
'created_at' => formatDatetime($this->created_at),
'progress' => $this->when($this->progress, $this->progress),
'begin' => formatDatetime($this->begin),
'end' => formatDatetime($this->end),
'user' => new UserResource($this->whenLoaded('user')),
'exam' => new ExamResource($this->whenLoaded('exam')),
];
}
}

View File

@@ -19,7 +19,7 @@ class UserResource extends JsonResource
'email' => $this->email,
'username' => $this->username,
'status' => $this->status,
'added' => $this->added,
'added' => formatDatetime($this->added),
'class' => $this->class,
'class_text' => $this->class_text,
'avatar' => $this->avatar,

View File

@@ -0,0 +1,22 @@
<?php
namespace App\Logging;
use Monolog\Formatter\LineFormatter;
class NexusFormatter
{
public function __invoke($logger)
{
foreach ($logger->getHandlers() as $handler) {
$handler->setFormatter($this->formatter());
}
}
protected function formatter()
{
$format = "[%datetime%] [" . REQUEST_ID . "] %channel%.%level_name%: %message% %context% %extra%\n";
return tap(new LineFormatter($format, null, true, true), function ($formatter) {
$formatter->includeStacktraces();
});
}
}

View File

@@ -115,15 +115,9 @@ class User extends Authenticatable
}
public function exams(): \Illuminate\Database\Eloquent\Relations\BelongsToMany
{
return $this->belongsToMany(Exam::class, 'exam_users', 'uid', 'exam_id')->withTimestamps();
}
public function examDetails(): \Illuminate\Database\Eloquent\Relations\HasMany
public function exams()
{
return $this->hasMany(ExamUser::class, 'uid');
}
}

View File

@@ -2,6 +2,7 @@
namespace App\Providers;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\ServiceProvider;
use Illuminate\Http\Resources\Json\JsonResource;
@@ -25,5 +26,6 @@ class AppServiceProvider extends ServiceProvider
public function boot()
{
// JsonResource::withoutWrapping();
DB::connection(config('database.default'))->enableQueryLog();
}
}

View File

@@ -9,9 +9,6 @@ use App\Models\Torrent;
use App\Models\User;
use Carbon\Carbon;
use Illuminate\Support\Arr;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Log;
use Illuminate\Database\Capsule\Manager as Capsule;
class ExamRepository extends BaseRepository
{
@@ -102,7 +99,7 @@ class ExamRepository extends BaseRepository
$logPrefix = "uid: $uid";
$exams = $this->listValid();
if ($exams->isEmpty()) {
Log::info("$logPrefix, no valid exam.");
do_log("$logPrefix, no valid exam.");
return $exams;
}
$user = User::query()->findOrFail($uid, ['id', 'username', 'added', 'class']);
@@ -110,11 +107,15 @@ class ExamRepository extends BaseRepository
$filtered = $exams->filter(function (Exam $exam) use ($user, $logPrefix) {
$filters = $exam->filters;
if (empty($filters->classes)) {
Log::info("$logPrefix, exam: {$exam->id} no class");
do_log("$logPrefix, exam: {$exam->id} no class");
return false;
}
if (!in_array($user->class, $filters->classes)) {
Log::info("$logPrefix, user class: {$user->class} not in: " . json_encode($filters));
do_log("$logPrefix, user class: {$user->class} not in: " . json_encode($filters));
return false;
}
if (!$user->added) {
do_log("$logPrefix, user no added time", 'warning');
return false;
}
@@ -122,20 +123,20 @@ class ExamRepository extends BaseRepository
$registerTimeBegin = $filters->register_time_range[0] ? Carbon::parse($filters->register_time_range[0])->toDateString() : '';
$registerTimeEnd = $filters->register_time_range[1] ? Carbon::parse($filters->register_time_range[1])->toDateString() : '';
if (empty($registerTimeBegin)) {
Log::info("$logPrefix, exam: {$exam->id} no register_time_begin");
do_log("$logPrefix, exam: {$exam->id} no register_time_begin");
return false;
}
if ($added < $registerTimeBegin) {
Log::info("$logPrefix, added: $added not after: " . $registerTimeBegin);
do_log("$logPrefix, added: $added not after: " . $registerTimeBegin);
return false;
}
if (empty($registerTimeEnd)) {
Log::info("$logPrefix, exam: {$exam->id} no register_time_end");
do_log("$logPrefix, exam: {$exam->id} no register_time_end");
return false;
}
if ($added > $registerTimeEnd) {
Log::info("$logPrefix, added: $added not before: " . $registerTimeEnd);
do_log("$logPrefix, added: $added not before: " . $registerTimeEnd);
return false;
}
@@ -151,15 +152,19 @@ class ExamRepository extends BaseRepository
*
* @param $uid
* @param int $examId
* @param null $begin
* @param null $end
* @return mixed
*/
public function assignToUser($uid, $examId = 0)
public function assignToUser($uid, $examId = 0, $begin = null, $end = null)
{
$logPrefix = "uid: $uid, examId: $examId, begin: $begin, end: $end";
if ($examId > 0) {
$exam = Exam::query()->find($examId);
} else {
$exams = $this->listMatchExam($uid);
if ($exams->count() > 1) {
do_log(last_query());
throw new \LogicException("Match exam more than 1.");
}
$exam = $exams->first();
@@ -170,9 +175,18 @@ class ExamRepository extends BaseRepository
$user = User::query()->findOrFail($uid);
$exists = $user->exams()->where('exam_id', $exam->id)->exists();
if ($exists) {
throw new \LogicException("exam: {$exam->id} already assign to user: {$user->id}");
throw new \LogicException("Exam: {$exam->id} already assign to user: {$user->id}");
}
$result = $user->exams()->save($exam);
$data = [
'exam_id' => $exam->id,
];
if ($begin && $end) {
$logPrefix .= ", specific begin and end";
$data['begin'] = $begin;
$data['end'] = $end;
}
do_log("$logPrefix, data: " . nexus_json_encode($data));
$result = $user->exams()->create($data);
return $result;
}
@@ -224,40 +238,59 @@ class ExamRepository extends BaseRepository
'index' => $indexId,
'value' => $value,
];
Log::info('[addProgress]', $data);
do_log('[addProgress] ' . nexus_json_encode($data));
return $examUser->progresses()->create($data);
}
public function listUserExamProgress($uid, $status = null)
public function getUserExamProgress($uid, $status = null)
{
$logPrefix = "uid: $uid";
$query = ExamUser::query()->with(['exam', 'user'])->where('uid', $uid)->orderBy('exam_id', 'desc');
if ($status) {
$query->where('status', $status);
}
$result = $query->paginate();
$idArr = array_column($result->items(), 'id');
$examTable = (new Exam())->getTable();
$progressTable = (new ExamProgress())->getTable();
$progressSum = Capsule::table($progressTable)
->join($examTable, "$progressTable.exam_id", '=', "$examTable.id")
->whereIn("$progressTable.exam_user_id", $idArr)
->whereRaw("$progressTable.created_at >= $examTable.begin and $progressTable.created_at <= $examTable.end")
->groupByRaw("$progressTable.exam_user_id, $progressTable.`index`")
->selectRaw("$progressTable.exam_user_id, $progressTable.`index`, sum($progressTable.`value`) as `index_sum`")
$examUsers = $query->get();
if ($examUsers->isEmpty()) {
return null;
}
if ($examUsers->count() > 1) {
do_log("$logPrefix, user exam more than 1.", 'warning');
}
/** @var ExamUser $examUser */
$examUser = $examUsers->first();
/** @var Exam $exam */
$exam = $examUser->exam;
$logPrefix .= ", exam: " . $exam->id;
if ($examUser->begin) {
$logPrefix .= ", begin from examUser: " . $examUser->id;
$begin = $examUser->begin;
} elseif ($exam->begin) {
$logPrefix .= ", begin from exam: " . $exam->id;
$begin = $exam->begin;
} else {
do_log("$logPrefix, no begin");
return null;
}
if ($examUser->end) {
$logPrefix .= ", end from examUser: " . $examUser->id;
$end = $examUser->end;
} elseif ($exam->end) {
$logPrefix .= ", end from exam: " . $exam->id;
$end = $exam->end;
} else {
do_log("$logPrefix, no end");
return null;
}
$progressSum = $examUser->progresses()
->where('created_at', '>=', $begin)
->where('created_at', '<=', $end)
->selectRaw("`index`, sum(`value`) as sum")
->groupBy(['index'])
->get();
$map = [];
foreach ($progressSum as $value) {
$map[$value->exam_user_id][$value->index] = $value->index_sum;
}
foreach ($result as &$item) {
$item->progress_value = $map[$item->id] ?? [];
}
return $result;
do_log("$logPrefix, query: " . last_query() . ", progressSum: " . $progressSum->toJson());
$examUser->progress = $progressSum->pluck('sum', 'index')->toArray();
return $examUser;
}