コードの複製 - swift3

Question

swift3 noobはこちらですので、これは愚かな質問であればよろしくお願いいたします。

私はこの問題を数回見つけましたが、解決できませんでした。コードの重複を避けるにはどうすればいいですか？

@IBAction func logOutButton(_ sender: Any) { 

    var userEntered = usernameText.text! 

    let appDelegate = UIApplication.shared.delegate as! AppDelegate 
    let context = appDelegate.persistentContainer.viewContext 

    let request = NSFetchRequest<NSFetchRequestResult>(entityName: "Users") 

    request.predicate = NSPredicate(format: "username = %@", "xyz") 

    do { 

     let results = try context.fetch(request) 

    } catch { 

     print ("Delete failed") 
    } 

} 

@IBAction func submitButton(_ sender: Any) { 
    var users = [String]() 
    var userEntered = usernameText.text! 
    let appDelegate = UIApplication.shared.delegate as! AppDelegate 
    let context = appDelegate.persistentContainer.viewContext 


    let request = NSFetchRequest<NSFetchRequestResult>(entityName: "Users") 


    let newValue = NSEntityDescription.insertNewObject(forEntityName: "Users", into: context) 

    newValue.setValue(userEntered, forKey: "username") 

    request.returnsObjectsAsFaults = false 

    do { 
     try context.save() 

ご覧のとおり、以下のコードは両方のボタンに複製されています。どうすればこれを避けることができますか？私はメソッドでそれを記述しようとしましたが、何か間違ったことをしているに違いありませんし、多くのエラーをスローします。

var userEntered = usernameText.text! 
    let appDelegate = UIApplication.shared.delegate as! AppDelegate 
    let context = appDelegate.persistentContainer.viewContext 


    let request = NSFetchRequest<NSFetchRequestResult>(entityName: "Users") 

だけNSFetchRequestを返すreturn関数を作るあなた

Answer 1

ありがとうございます。

func getFetchRequest() -> NSFetchRequest { 
    let appDelegate = UIApplication.shared.delegate as! AppDelegate 
    let context = appDelegate.persistentContainer.viewContext 
    let request = NSFetchRequest<NSFetchRequestResult>(entityName: "Users") 
    return request 
} 

Answer 2

次に、あなたの行動は

@IBAction func logOutButton(_ sender: Any) { 
    let request = getRequest() 
    request.predicate = NSPredicate(format: "username = %@", "xyz") 
    do { 
     let results = try context.fetch(request) 
    } catch { 
     print ("Delete failed") 
    } 
}