処理に応じて関数定義したい

Question

doneとfail文を別で関数定義したいのですが、.done.failのドットによってエラーになってしまいます。

コード

$('.search__btn').on('click', function () { //検索ボタンがクリックされた時
            comparison();

このように定義し、それにあった処理をまとめたいです。

コード
<script>
$(function(){
          var preSearchword = ''; //空の文字列を入れてあります
          var pageNum = 1; //変数に１を代入
          $('.search__btn').on('click', function () { //検索ボタンがクリックされた時
            comparison();
          });
          $('.search__btn').on('click',function(){
            ss();
          });
          $('.search__btn').on('click',function(){
            zz();
          });
            var searchword = $('.search__text__input').val(); //変数にバリューを取得した値を代入 文字を入力する部分
            if (preSearchword === searchword) { //if文でpreSearchwordとsearchwordが厳密に等しい場合の式を作り
              //console.log(pageNum)
              pageNum++; //１回目の検索結果と2回目クリックされる時の検索結果が同じ場合、１ページずつプラスしていきます
            } else { //そうでない場合
              pageNum = 1; //ページ数を１に戻します　最初のクリックの際は、falseを返します
              $('ul').empty(); //同時に下に表示していたulの子要素をempty()で消します
            }
            preSearchword = searchword; //preSearchwordにsearchwordを入れます
            // console.log(searchword);
            // console.log(searchword);
          function done(){
            $.ajax({
              url: 'https://app.rakuten.co.jp/services/api/BooksTotal/Search/20130522',
              type: 'GET',
              datatype: 'json',
              data: {
                keyword: searchword, //バリューを取得した変数を、dataのオブジェクトの中のvalueに入れます
                applicationId: '1019399324990976605',
                booksGenreId: '001',
                hits: 20, //検索でヒットした件数を指定
                page: pageNum //ページ数
              },
            })
            .done(function (data) { //doneで処理が完了した時
              console.log(data);
              //検索結果が0件の場合
              if (data.hits === 0) { //if文で、ヒット件数が0と厳密に等しい場合
                $('ul.lists').prepend('<p class="message">検索結果が見つかりませんでした<br>別のキーワードで検索してください</p>'); //listというクラスを持つulの中に、prependでクラスを入れ、pタグを表示させます
              }
              //通信が成功した時
              var booklist = ''; //変数に空の文字列を代入
              $.each(data.Items,function(index,item){ //$.eachで繰り返し処理を行い、　itemは、現在のアクセスのDOM（htmlなどを自由に操作する仕組み）を意味しています。
                // console.log(item);
                booklist += '<li class="lists__item">' + index + '<div class="lists__item__inner">' + '<a href="' + // booklistという変数に必要なクラスを加算代入しつつ、全ての要素に対して固有パラメータを入れます
                item.Item.itemUrl + '"class="lists__item__link" target="_blank">' + '<img src="' + item.Item
                .largeImageUrl + '" class="lists__item__img" alt="' + item.Item.title + '">' +
                '<p class="lists__item__detail">作品名：　' + item.Item.title + '</p>' + //itemは、関数の第二引数,Itemは、楽天の個別商品のバリュー,三つ目は、固有パラメーター
                '<p class="lists__item__detail">作者　：　' + item.Item.author + '</p>' +
                '<p class="lists__item__detail">出版社：　' + item.Item.publisherName + '</p>' + '</a>' +
                '</div>' + '</li>';
              })
              $('ul.lists').prepend(booklist); //listというクラスを持つulの中にprepend(変数)とし、入れます
            })
          }
          function fail(){
            .fail(function () { //通信失敗時は、fail文を使います
              $('.lists').after('<div class="error"></div>') //errorという新たなクラスをlistsの後ろ(中)に挿入します
              error = $('.error'); //変数にクラスを代入
              error.html('<p class="message">データ通信できませんでした<br>接続を確認してください</p>'); //変数に対して、htmlで書き換えて、文字を入力しないで検索をかけた場合と通信がうまくいかなかった時にエラー文を表示させます
            })
          }
        });
    </script>

Comments

[m.ts10806]

>.done.failのドットによってエラーになってしまいます。
https://teratail.com/help/question-tips#questionTips3-4-2
>表示されたエラーメッセージをそのままコピー&ペーストしましょう。自分でタイプしなおしたり、自分で解釈・要約しようとしてはいけません。

そもそもそのエラーとやらが提示されていないので、そこも含めて追記願います。
あと、実際のIDっぽいのが入ってますがこれは全世界に発信して大丈夫ですか？

Answer 1

中身だけ関数にしましょう。

function done(data){
  //中身省略
}

function fail(){
  // 中身省略
}

$.ajax({/* 中身略 */}).done(done).fail(fail);