create CRUD, search employee by thiennguyen84 · Pull Request #1 · thiennguyen84/Employee

thiennguyen84 · 2018-02-06T12:30:03Z

No description provided.

hapo-truongpm · 2018-02-08T14:28:04Z

app/Http/Controllers/EmployeeController.php

+    //
+    public function show(){
+    	$employees = DB::table('employees')->orderBy('name')->paginate(10);	
+    	return view('index',['employees'=>$employees]);


nên dùng Eloquent String thay vì Query Builder https://laravel.com/docs/5.5/eloquent tuy QB có tốc độ truy vấn nhanh hơn nhưng khi câu query dài, phức tạp thì ES sẽ dễ hiểu hơn

@TruongPhan16, @thiennguyen84 ngoài ra thì dùng Eloquent Engine thì dữ liệu trả về mới là các Model Object đã extend từ Eloquent, lúc đó mới sử dụng được các method định nghĩa trong model và các method hỗ trợ bởi Eloquent

hapo-truongpm · 2018-02-08T14:29:11Z

app/Http/Controllers/EmployeeController.php

+    	],
+    	[
+    		'email.unique'=>'Email đã có người sử dụng',
+    	]);


nên tách ra file request riêng https://laravel.com/docs/5.5/validation#form-request-validation

@TruongPhan16 e làm cái controller mới r nhé.@@

hapo-truongpm · 2018-02-08T14:29:35Z

app/Http/Controllers/EmployeeController.php

+	    $employee->name = $request->name;
+	    $employee->address = $request->address;
+	    $employee->birth_day = $request->birthday;
+	    $employee->email = $request->email;


birthday là 1 từ

birth day e để kiểu date nên k nhập được kí tự

tức là birthday là 1 từ thôi chứ k phải birth_day

@TruongPhan16 @@.

hapo-truongpm · 2018-02-08T14:31:29Z

app/Http/Controllers/EmployeeController.php

+use Validator;
+use File;
+
+class EmployeeController extends Controller


nên dùng controller resource cho CRUD https://laravel.com/docs/5.5/controllers#resource-controllers

@TruongPhan16 đang training nên cần giải thích thêm tại sao ?

@chungth e làm cái controller mới r nhé.@@

hapo-truongpm · 2018-02-08T14:32:15Z

app/Http/Controllers/EmployeeController.php

+    }
+
+    public function delete($id){
+    	$employee = Employee::find($id);


sử dụng findOrFail vì khi không tìm thấy sẽ trả về trang 404

hapo-truongpm · 2018-02-08T14:32:47Z

app/Http/Controllers/EmployeeController.php

+    	$searchs = Employee::where('name','like','%'.$name.'%')->orderBy('name')->paginate(10);
+    	return view('search',['searchs'=>$searchs,'name'=>$name]);
+    }
+}


function này có thể gộp chung vào index

@TruongPhan16 tách như thế này có thể refactoring để search bằng ajax ?

@chungth thế là e tách ra như này có phải gộp lại k a. :v

hapo-truongpm · 2018-02-08T14:33:57Z

resources/views/add.blade.php

+    </div>
+	@endif
+	<div class="col-lg-6 col-lg-offset-3">
+		<form action = "{{route('add')}}" method="POST" enctype="multipart/form-data">


thiếu space giữa {{}}

@TruongPhan16 thiếu space là sao a

@thiennguyen84 tức là thiếu dấu cách ấy

hapo-truongpm · 2018-02-08T14:34:35Z

resources/views/add.blade.php

+	  		</div>
+	  		<div class="form-group">
+	    		<label for="email">Email:</label>
+	    		<input type="email" class="form-control" name= "email" value="@gmail.com" required="">


value="@gmail.com" là như thế nào nhỉ?

@TruongPhan16 e cho sẵn value là @gmail.com để chỉ cần nhập phần tên đầu thôi

thế hotmail hay các mail khác thì sao?

e đặt value vậy thôi chứ người ta có thể xóa value nhập hotmail khác mà a.:v

hapo-truongpm · 2018-02-08T14:35:14Z

resources/views/edit.blade.php

+        </ul>
+    </div>
+	@endif
+	<div class="row">


nên hiện error vào input tương ứng thay vì hiện tất cả

hapo-truongpm · 2018-02-08T14:35:37Z

resources/views/edit.blade.php

+				{{ csrf_field() }}
+			<div class="col-lg-6">
+				<div class="avata_name">
+				 	Avata


class sai convention avata-name

hapo-truongpm · 2018-02-08T14:36:33Z

resources/views/edit.blade.php

+	@endif
+	<div class="row">
+		<form action = "{{route('edit',$employee->id)}}" method="POST" enctype="multipart/form-data">
+				{{ csrf_field() }}


sử dụng method PUT hoặc PATCH khi update cho đúng chuẩn RESTful

@TruongPhan16 mà nếu đặt PUT PATH thì để form up ảnh riêng à anh

{{ method_field('PUT') }}
thêm cái này vào anh ơi. Mà không phải tạo form up ảnh riêng đâu anh.

hapo-truongpm · 2018-02-08T14:41:07Z

resources/views/index.blade.php

+    	@foreach ($employees as $value)
+      <?php $dem++; ?>
+      <tr>
+        <td>{{$dem}}</td>


Nên dùng biến tiếng anh thay vì tiếng việt

Không cần đoạn <?php $dem++; ?> cho $dem++ ở td đầu tiên là được

Blade template có hỗ trợ @php @endphp thay cho thẻ đóng mở php

hapo-truongpm · 2018-02-08T14:43:13Z

resources/views/index.blade.php

+        <td>{{$dem}}</td>
+        <td>{{$value->name}}</td>
+        <td><img src="http://localhost:8000/img/{{$value->image}}" width="40" height="35px" alt=""></td>
+        <td>{{$value->email}}</td>


Sử dụng asset helper thay vì viết cả url ra như này https://laravel.com/docs/5.5/helpers#method-asset

Đừng style css inline

hapo-truongpm · 2018-02-08T14:44:09Z

resources/views/index.blade.php

+    </tbody>
+  </table>
+  <div class="row">{{$employees->links()}}</div>
+</div>


Như này thì phần phân trang sẽ không ra chính giữa đúng k nhỉ

@TruongPhan16 Quy định là ra giữa a???

@thiennguyen84 ra giữa cho dễ nhìn thôi

hapo-truongpm · 2018-02-08T14:45:05Z

resources/views/search.blade.php

+        <td>{{$value->address}}</td>
+        <td><div class="edit"><a href="{{route('edit',$value->id)}}" class="btn btn-info"><span class="ti-pencil-alt"></span></a></div></td>
+        <td><form action="/task/{{ $value->id }}" method="POST">
+          {{ csrf_field() }}


truyền route name vào action thay vì hard code

hapo-truongpm · 2018-02-08T14:46:15Z

routes/web.php

+Route::get('search',[
+	'as'=>'search',
+	'uses'=>'EmployeeController@search'
+]);


Tất cả route trên có thể thay bằng Route::resource cho ngắn gọn

@TruongPhan16 thế là phải làm lại route à a.@@

@thiennguyen84 cách của em ko sai nhưng @TruongPhan16 có giới thiệu cái Route::resource để sau em định nghĩa nó sẽ ngắn gọn và dễ hơn, em đọc trong laravel doc nhé

@chungth ok a

chungth

Các file trong folder

public/img là file được người dùng upload lên ?

Những file đó thuộc về dữ liệu của app nên cần ignore khỏi git repo

hapo-anhht · 2018-02-09T09:05:07Z

resources/views/edit.blade.php

+	@endif
+	<div class="row">
+		<form action = "{{route('edit',$employee->id)}}" method="POST" enctype="multipart/form-data">
+				{{ csrf_field() }}


{{ method_field('PUT') }}
thêm cái này vào anh ơi. Mà không phải tạo form up ảnh riêng đâu anh.

hapo-anhht · 2018-02-09T09:06:45Z

resources/views/header.blade.php

    <div class="container">
        <div class="navbar-header">
-            <img src="asset/images/hapo.jpg" class="hapo_img" alt="">
+            <img src="http://localhost:8000/asset/images/hapo.jpg" class="hapo_img" alt="">


tại sao chỗ này dùng link http: vậy anh :@@

@anhht-hapo a chạy serve nếu k sử dụng http://localhost:8000 thì n không nhận link css.@@

hapo-anhht · 2018-02-09T09:07:18Z

resources/views/index.blade.php

  <div class="row">
    @if(Session::has('thanhcong'))
-              <div class="alert alert-success thanh-cong">{{Session::get('thanhcong')}}</div>
+      <div class="alert alert-success thanh-cong">{{Session::get('thanhcong')}}</div>


sao tên class tiếng việt không dấu vậy anh

…c.@@

hapo-anhht · 2018-02-09T10:15:47Z

app/Http/Controllers/EmployController.php

+    {
+        //
+        $employee = new Employee();
+        if($request->hasFile('avata')){


avatar chứ ko phải avata đau a

nguyenvanpho · 2018-02-11T02:04:02Z

app/Http/Controllers/EmployController.php

+        return redirect('employs')->with('thanhcong','Xóa nhân viên thành công');
+    }
+
+    public function search(Request $request){


convention dòng này em ơi! {} 2 dấu đóng mở này phải nằm trên 1 dong trắng

nguyenvanpho · 2018-02-11T02:04:16Z

app/Http/Controllers/EmployeeController.php

@@ -13,12 +13,12 @@ class EmployeeController extends Controller
 {
    //
    public function show(){


nguyenvanpho · 2018-02-11T02:04:28Z

app/Http/Controllers/EmployeeController.php

+    	return view('admin.employee.index',['employees'=>$employees]);
    }

    public function add(){


nguyenvanpho · 2018-02-11T02:06:41Z

app/Http/Controllers/EmployeeController.php

+	    $employee->birthday = $request->birthday;
 	    $employee->email = $request->email;
 	    $employee->save();
    	return redirect('/show')->with('thanhcong','Thêm nhân viên thành công');


@thiennguyen84 dùng route('name') đi em.ko nên dùng đường dẫn url nữa

nguyenvanpho · 2018-02-12T08:23:18Z

app/Http/Controllers/EmployController.php

        $employee->email = $request->email;
        $employee->save();
-        return redirect('/employs')->with('thanhcong','Thêm nhân viên thành công');
+        return redirect()->route('employs.index')->with('thanhcong','Thêm nhân viên thành công');


@thiennguyen84 cho message này vào file lang em ơi. Đọc doc ở đây em nhé https://laravel.com/docs/5.5/localization
các chỗ tương tự em tự tìm rồi tự cho vào nhé, anh ko comment nữa nhé
(nguyên xi dòng anh chung review của anh. em xem của anh tham khảo nha)

thiennguyen84 added 3 commits February 6, 2018 18:29

create CRUD, search employee

4ddd3e1

fix layout, validator

b84b22f

Lam chuc nang upload file

3348e54

hapo-truongpm reviewed Feb 8, 2018

View reviewed changes

chungth suggested changes Feb 9, 2018

View reviewed changes

hapo-anhht reviewed Feb 9, 2018

View reviewed changes

thiennguyen84 added 2 commits February 9, 2018 16:40

fix theo review nhung con ham update theo REST con loi chua xu li duo…

c8d1225

…c.@@

ignore public/img

09c9e25

hapo-anhht reviewed Feb 9, 2018

View reviewed changes

fix su dung thu vien asset de hien thi avata

b285c9f

nguyenvanpho reviewed Feb 11, 2018

View reviewed changes

nguyenvanpho reviewed Feb 12, 2018

View reviewed changes

thiennguyen84 force-pushed the feature/buildEmployee branch from ded0b26 to 4fbfc41 Compare July 3, 2018 06:44

fix update theo chuan REST, fix space

5fb7bf1

thiennguyen84 force-pushed the feature/buildEmployee branch from 4fbfc41 to 5fb7bf1 Compare July 3, 2018 06:54

Conversation

thiennguyen84 commented Feb 6, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chungth left a comment

Choose a reason for hiding this comment

Uh oh!