Comment transposer deux arguments d'une fonction en Python?

Comment puis-je échanger deux arguments dans un appel à une fonction Python?

Si je mets pointl'espace entre ces deux arguments:

self.assertEqual(json.loads(some.data), json_data)

puis M-t( transpose-words), j'obtiens:

self.assertEqual(json.loads(some.json), data_data)

Par contre avec CMt ( transpose-sexps) j'obtiens:

self.assertEqual(json.loadsjson_data, (some.data))

Ce que je veux c'est:

self.assertEqual(json_data, json.loads(some.data))

Y a-t-il une commande qui fera ça?

C'est quelque chose que je voulais aussi avoir depuis longtemps, et j'ai trouvé ici une motivation pour y travailler. Ce n'est probablement pas très robuste, mais au premier essai, il semble couvrir les cas que j'ai essayés:

(defun my/calculate-stops ()
  (save-excursion
    (let ((start
           (condition-case e
               (while t (backward-sexp))
             (error (point))))
          stops)
      (push start stops)
      (condition-case e
          (while t
            (forward-sexp)
            (when (looking-at "\\s-*,")
              (push (point) stops)))
        (error (push (point) stops)))
      (nreverse stops))))

(defun my/transpose-args ()
  (interactive)
  (when (looking-at "\\s-") (backward-sexp))
  (cl-loop with p = (point)
           with previous = nil
           for stop on (my/calculate-stops)
           for i upfrom 0
           when (<= p (car stop)) do
           (when previous
             (let* ((end (cadr stop))
                    (whole (buffer-substring previous end))
                    middle last)
               (delete-region previous end)
               (goto-char previous)
               (setf middle (if (> i 1) (- (car stop) previous)
                              (string-match "[^, \\t]" whole 
                                            (- (car stop) previous)))
                     last (if (> i 1) (substring whole 0 middle)
                            (concat (substring whole (- (car stop) previous) middle)
                                    (substring whole 0 (- (car stop) previous)))))
               (insert (substring whole middle) last)))
           (cl-return)
           end do (setf previous (car stop))))

J'utilise une variation de ce transpose-sexpsqui regarde le cas que vous décrivez et transpose les choses séparées par des virgules, ou tout simplement fait régulièrement transpose-sexps. Il laisse également le curseur en place au lieu de le faire glisser vers l'avant, ce qui est un peu différent mais j'aime personnellement.

(defun my-transpose-sexps ()
  "If point is after certain chars transpose chunks around that.
Otherwise transpose sexps."
  (interactive "*")
  (if (not (looking-back "[,]\\s-*" (point-at-bol)))
      (progn (transpose-sexps 1) (forward-sexp -1))
    (let ((beg (point)) end rhs lhs)
      (while (and (not (eobp))
                  (not (looking-at "\\s-*\\([,]\\|\\s)\\)")))
        (forward-sexp 1))
      (setq rhs (buffer-substring beg (point)))
      (delete-region beg (point))
      (re-search-backward "[,]\\s-*" nil t)
      (setq beg (point))
      (while (and (not (bobp))
                  (not (looking-back "\\([,]\\|\\s(\\)\\s-*" (point-at-bol))))
        (forward-sexp -1))
      (setq lhs (buffer-substring beg (point)))
      (delete-region beg (point))
      (insert rhs)
      (re-search-forward "[,]\\s-*" nil t)
      (save-excursion
        (insert lhs)))))

Dans les modes qui utilisent SMIE, transpose-sexpdevrait fonctionner correctement pour ce cas. Ils échoueront quand le symbole infixe (aka "séparateur") n'est pas un ,(ou un ;) mais un mot (par exemple and).

Donc, mon avis est que la commande pour cela est transpose-sexpet quand cela ne fonctionne pas correctement, je le considère comme un bug (mais un bug qui peut être difficile et / ou prendre du temps à corriger et à faible priorité, alors ne le faites pas ' t retenez votre souffle). La façon de le résoudre est de définir forward-sexp-functionune fonction qui saura que "après avoir vu une virgule, je saute tout autour de l'argument".