KITTYさん、こんにちは。

PB７で、Retrieve処理はこのFOR文の前で行っていることを前提にはなします。
違ってる場合には、ごみレスです。

DataWindowは、SELECTしたときの情報をキー情報として、テーブルを更新します。
下記の処理だと、２つめのループで常に同一行（１?ls_max）への更新をしているはずです。
Commit後、Resetもしていないし。
キー情報へも値をセットしていたら、変更された内容でUPDATEされてしまうはずです。

対応としては、外側のループの中に、retrieveを入れるのが、素直だと思います。

ps
どっちかてーと。
DBがわで、バッチ(PL/SQLとか）にすべき処理だと思いますが。
２重ループ＋SELECT発行してるので、処理速度が遅くなりますし。
ネットワーク上を、SELECT文がループ回数分「飛ぶ」し。

＞簡単に記述しているため分かりにくいかもしれませんが、よろしくお願いします。
＞外側のループはデータを取得するための関数を呼び出す回数分回っています。
＞この書き方ではupdateを行なうたびに前のデータに上書きされてしまいます。
＞
＞
＞for l_count = 1 to l_max　
＞
＞　// ここで別のテーブルからデータを取得するための関数を呼び出します
＞　//（取得したデータはls_dataに格納されています）
＞
＞　/* 取得データ件数分データウィンドウにデータを格納します */
＞　for ls_count = 1 to ls_max
＞　 dw_1.Setitem(ls_count,ls_data[ls_count])
＞  next
＞　 /* 更新を行ないます */
＞　 Iret = dw_1.update(TRUE,TRUE)
＞　 /* ｴﾗｰでなかったらCOMMITを行ないます */
＞　 if Iret1 >0  then
＞　　 COMMIT using sqlca_mdb;
＞　 /* ｴﾗｰだったらROLLBACKを行ない処理終了です */
＞　 else
＞　　 ROLLBACK using sqlca_mdb;
＞　   return -1
＞　 end if
＞next
＞
＞
＞