チームメンバー選択画面(得点者、アシスト者を選択する画面)の 得点者ボタン押下時に強制終了
To:中村さん
本件対応しました。
下記リポジトリにコミットしたのでレビューお願いします。(Rev.19) https://svn.sourceforge.jp/svnroot/versus/branch/user/31141
指定リポジトリのベースは30971対応済みのソースをベースに作成しました。
以上、よろしくお願いします。
対応詳細
TeamMemberSelectActivity.java チームIDから全チームメンバーの名前を取得する処理で、 返却されたCursorから取得メンバー分のString領域を確保する際に使用している 関数cursor.getColumnCount()で強制終了。 カラム数ではなくデータ数を取得する必要があるので cursor.getCount()を使用する。
ただの関数名間違いな気がします。
To:中村さん
お疲れ様です。 上記ソースレビューですが、保留とさせてください。 現象が直ってませんでした。
チケットはさいとうに変更しておきます。
To:中村さん
お疲れ様です。 本件、再度修正しましたのでレビューお願いします。
(rev.26) https://svn.sourceforge.jp/svnroot/versus/branch/user/31141
<修正クラス> /branch/user/31141/src/jp/co/versus/PlayPointEditorActivity.java /branch/user/31141/src/jp/co/versus/TeamMemberSelectActivity.java
<修正概要> ・/branch/user/31141/src/jp/co/versus/PlayPointEditorActivity.java →TeamMemberSelectActivity画面でデータを取得できない場合、 返却されたRESULT_CENCELEDを受けてトーストを表示する処理を追加。
・/branch/user/31141/src/jp/co/versus/TeamMemberSelectActivity.java →チームIDから所属メンバーを取得するクエリのwhare句でExceptionが発生していたので修正。 以下のようなクエリを作成するよう変更。 修正前> SELECT _id, name FROM menber WHARE menberid = 1,2,3 修正後> SELECT _id, name FROM menber WHARE menberid = 1 OR menberid = 2 OR menberid = 3
→上記whare句を作成する関数の追加。
→表示されたメンバーをタップするとExceptionが発生していたのでonClickもついでに修正。
以上 よろしくお願いします。
To:しょうた 以下、指摘事項です。 [TeamEditorActivity.java] Cursorがクローズされてません。クローズするようにお願いします。 (元々のコードがクローズされてませんでしたが。。。)
To:なかむらさん
クローズ処理を追加し下記リビジョンでTrunkへコミットしました。
Rev.33
https://svn.sourceforge.jp/svnroot/versus/trunk/src
<前提条件> チーム名、チームメンバーともに未設定
<再現手順> 1.追加ボタン 2.何も設定せず画面下部OKボタン押下 3.チーム1を選択 4.得点者ボタン押下
以上の手順でCursorIndexOutOfBoundsExceptionが発生。
<補足> 手順4.でアシストボタン押下時も同様の現象が発生。