Is putting tasks inside the 'body' task of uvm_sequence class a good practice?

In reply to piyushkamalakar:

I have a bunch of tasks for read and write transactions. I was wondering if there is any way I can put them inside the uvm_sequence body task. Is it a good practice to do this? Thank you!

Encapsulating functionality in tasks is always a good coding practice. Where you are call these tasks depends on what your objective is.