なんとなくやりたいことは判るのですが、質問の前にご自分でテスト実行されてはいかがでしょうか。
流石に動かないでしょうし、回答側も判断に苦しむでしょう。
・Doに対応するLoopが無い
・IfIfってなんぞや
・環境が不明

判断するところのコードはこんな感じでしょうけど。

If a2="" and a3="" then

        S2. Enabled = False

        S3. Enabled = False

Elseif a2<>"" and a3="" then

        S2. Enabled = True

        S3. Enabled = False

End if

レコード一つ読む毎にこれを判断しているのなら、わざわざDo～Loopを使う必要は無いのでは？


（これってそもそもDBからデータ引っ張ってこれてるのかなぁ？）