> 他のフォームを呼び出すのは、すべて
> form2.showdialog でよくなるのでは？

モーダルにすると、子画面を表示した後で呼び出し元を最小化するといった事ができず、
ユーザー操作の妨げになります。「他の画面を操作されては困る」といった場合に
モーダルにするのは良いですが、それ以外の場合は極力モードレスにすることが望ましいです。
https://www.weblio.jp/content/%E3%83%A2%E3%83%BC%E3%83%89%E3%83%AC%E3%82%B9%E3%83%80%E3%82%A4%E3%82%A2%E3%83%AD%E3%82%B0


また、Form2.ShowDialog に頼った実装だと、引数付きコンストラクタを利用できなくなるので、
初期化処理などが制限されてしまいがちです。そのため、自分はモーダルなら

'呼び出し側の処理 

Using f As New Form2(引数1, 引数2)

　　If f.ShowDialog() = DialogResult.Cancel Then

        Return

    End If

    Me.Label1.Text = f.ResultText   '独自プロパティで結果を返す 

End Using

というパターンもしくは

Dim selectedFiles As String() = Form2.ShowFilesDialog(targetFolder)    '自作の Shared メソッド 

If selectedFiles.Length = 0 Then

    Return

End If

な形式にすることがあります。
後者の場合、外部から New Form2() できないよう、コンストラクタを Private にします。