What I did was writing $(1\, 2)(3\, 4)(1\, 3)(2\, 4)$ as $\bigl(\begin{smallmatrix} 1& 2 &3 & 4\\ 2& 1 &4 & 3 \end{smallmatrix}\bigr)\bigl(\begin{smallmatrix} 1& 2 &3 &4 \\ 3& 4& 1 & 2 \end{smallmatrix}\bigr)$ which is equal to $(1 4)(2 3)$
Is what I'm doing right ? And isn't there any other method for it ?
What you have done is correct.
Alternatively, you could make use of the fact that disjoint cycles commute and, instead, compute $(34)(12)(24)(13)$. This makes little if any difference at all.