@Bhavita_Patil, the easiest would be if you post some geometry so we can find out if your cylinders are split in half, eg. because they where imported from iges…
Also, are you testing against brep faces or the underlying surface ? Are you sure that face1.IsClosed(0) is testing the correct direction ?
Btw. you might check these scripts to merge the cylinders.
_
c.