在同一個控制器中創建編輯功能laravel

Question

因此，我在我的控制器中有一個創建功能，如下所示，我的路線是這樣的，我的問題是有沒有辦法讓我在不同的創建和編輯條件相同的功能都具有相當類似的編碼。有人能啓發我嗎？

class ManageAccountsController extends Controller 
{ 
    public function index() { 
     $users = User::orderBy('name')->get(); 
     $roles = Role::all(); 

     return view('manage_accounts', compact('users', 'roles')); 
    } 

    public function update() 
    { 
      // process the form here 

    // create the validation rules ------------------------ 
     $rules = array(
     'name'    => 'required',      // just a normal required validation 
     'email'   => 'required|email|unique:users',  // required and must be unique in the user table 
     'password'   => 'required|min:8|alpha_num', 
     'password_confirm' => 'required|same:password',   // required and has to match the password field 
     'mobile'   => 'required', 
     'role_id'   => 'required' 
     ); 

    // do the validation ---------------------------------- 
    // validate against the inputs from our form 
     $validator = Validator::make(Input::all(), $rules); 

    // check if the validator failed ----------------------- 
     if ($validator->fails()) { 

     // redirect our user back to the form with the errors from the validator 

      $input = Input::except('password', 'password_confirm'); 

      $input['autoOpenModal'] = 'true'; //Add the auto open indicator flag as an input. 
      return redirect() 
      ->back() 
      ->withInput($input) 
      ->withErrors($validator); 

     } else { 
     // validation successful --------------------------- 

     // user has passed all tests! 
     // let user enter the database 

     // create the data for our user 
      $user = new User; 
      $user->name  = Input::get('name'); 
      $user->email = Input::get('email'); 
      $user->password = Hash::make(Input::get('password')); 
      $user->mobile = Input::get('mobile'); 
      $user->role_id = Input::get('role_id'); 

     // save our user 
      $user->save(); 

     // redirect ---------------------------------------- 
     // redirect our user back to the form so they can do it all over again 
      Session::flash('flash_message', 'User successfully added!'); 

      return redirect()->back(); 

     } 
    } 
} 

routes.php文件

Route::get('manage_accounts', 'ManageAccountsController@index'); 
Route::post('manage_accounts', 'ManageAccountsController@update'); 

Answer 1

爲什麼不嘗試將一些代碼移出控制器？如果您要使用存儲庫，那麼您將能夠封裝您的一些邏輯，以便將其用於這兩種功能。

此外，您可以在不將所有額外代碼寫入控制器的情況下處理所有這些驗證 - 請參閱http://laravel.com/docs/5.0/validation#form-request-validation。

起初這似乎有點矯枉過正，但一旦你掌握了它，你的代碼將變得更易於管理和擴展。

（更多關於這些我會全力推薦傑弗里路的Laracasts https://laracasts.com/ - 幫我當我學習Laravel很多）

// routes.php 

// http://laravel.com/docs/5.0/controllers#restful-resource-controllers 
Route::resource('manage_accounts', 'ManageAccountsController'); 


// ManageAccountsController.php 

use Illuminate\Http\Request; 
use Illuminate\Routing\Controller; 

class ManageAccountsController extends Controller 
{ 

    public $userRepository; 


    public function __construct(UserRepository $userRepository) 
    { 
     $this->userRepository = $userRepository; 
    } 

    public function index() { 

     $users = User::orderBy('name')->get(); 
     $roles = Role::all(); 
     return view('manage_accounts', compact('users', 'roles')); 
    } 

    public function store(StoreUserRequest $request) 
    { 
     // validation already handled using this: http://laravel.com/docs/5.0/validation#form-request-validation 
     $this->userRepository->upsert($request) 

     Session::flash('flash_message', 'User successfully added!'); 

     return redirect()->back(); 

    } 

    public function update(StoreUserRequest $request, $id) 
    { 
     // validation already handled using this: http://laravel.com/docs/5.0/validation#form-request-validation 
     $this->userRepository->upsert($request, $id) 

     Session::flash('flash_message', 'User successfully updated!'); 

     return redirect()->back(); 

    } 
} 


// UserRepository.php 
class UserRepository { 



    public function upsert($data, $id = null) 
    { 

      // You will also need something like this 
      if(isset($data['id'])) 
      { 
       $user = $this->user->find($data['id']); 
      } 
      else { 
       $user = new User; 
      } 


      $user->name  = $data['name']; 
      $user->email = $data['email']; 
      $user->password = Hash::make($data['password']); 
      $user->mobile = $data['mobile']; 
      $user->role_id = $data['role_id']; 

      // save our user 
      $user->save(); 

      return $user; 
     } 
    } 

} 

請在這裏使用的代碼作爲指導（我在寫這匆忙，它肯定會包含錯誤）。快速閱讀存儲庫，我認爲這應該都是有道理的。

這裏的基本前提是分離出想要重用的代碼，而不是將其全部壓縮到同一個函數中。

希望這有助於！

Answer 2

更新或創建

嘗試updateOrCreate（）雄辯創建或更新匹配屬性的記錄。

讀取API文檔udateOrCreate()

您的代碼會像：

Model::updateOrCreate(['id' => $id], ['firstField' => 'value', 'secondField' => 'value']); 

注：第一個參數是要找到匹配的，並要保存第二數據的。

希望這是有幫助的。