12月7日(月)1、2コマ目

今日、やったこと

新課題「かんたんオークションサイト構築(Java版)」

今日のホワイトボード

<A>タグについて

2ページ目の商品一覧画面にて、商品名のリンクをクリックすると、商品を選択できます。

リンクは作るにはHTMLの<A>タグを使います。<A>タグにはクリック後移動するURLを指定するhref属性があります。

図　<A>タグ


なお、リンククリックはHTTPのGETコマンドでリンク先をリクエストします。


前回の課題について

課題「かんたんオークションサイト(C#版)」を採点して気が付いた点を。

UserDAOクラスのGetCount()メソッド

ダメ１　全件検索実行後、検索結果のユーザー名、パスワードを1件づつチェック

まず、以下のSQLを実行。

　SELECT * FROM ユーザーマスタ

つぎに、検索結果のユーザー名、パスワードをチェック。

ユーザー名、パスワードが引数と一致するならカウントアップ。

while(reader.Next) {

　if(reader["ユーザー名"].Equals(引数のユーザー名)　&&　

　　reader["パスワード"].Equals(引数のパスワード)) {

　　count++;

　}

}


ダメ２　検索実行、検索結果をカウントアップ

まず、以下のSQLを実行。

　SELECT * FROM ユーザーマスタ 

　WHERE ユーザー名=引数のユーザー名 AND パスワード=引数のパスワード

つぎに検索結果の行数をカウントアップ。

while(rs.Next) {

　count++;

}


ダメ３　検索実行、検索結果をListに格納してListの要素数を返す

まず、以下のSQLを実行。

　SELECT * FROM ユーザーマスタ 

　WHERE ユーザー名=引数のユーザー名 AND パスワード=引数のパスワード

つぎに検索結果をListに格納。

while(rs.Next) {

　list.Add(new User(・・));

}

Listの要素数を返す。

return list.Count;


おすすめ　グループ関数count()を使う

実行するSQLはこれ。

　SELECT count(*) FROM ユーザーマスタ 

　WHERE ユーザー名=引数のユーザー名 AND パスワード=引数のパスワード

検索結果は条件に合う行数。これが欲しい値。


BidDAOクラスのInsert()メソッド

応札データテーブルをロックします。（してない人も結構いた）

SQLは以下です。

　SELECT*FROM 応札データ WHERE 商品ID=選択商品のID FOR UPDATE;

このSQLを実行するCommandオブジェクトの名前ですが、

　OracleCommand cmdLock = new OracleCommand("SELECT・・", con);

あたりなら良いのですが、

　OracleCommand cmdRock = new OracleCommand("SELECT・・", con);

な人がちょこちょこおられました。

違いが判るでしょうか？

メソッドの戻り値で条件分岐

戻り値がbool型なメソッドを実行し、その結果で条件分岐する場合です。

嫌なかんじ１

bool result = メソッド実行;

if(result == true)
{
　　結果がtrueのときの処理
}
else
{
　　結果がfalseのときの処理
}

このあと、変数resultは使っていない

嫌なかんじ２

bool result = メソッド実行;

if(result == true)
{
　　結果がtrueのときの処理
}
else if(result == false)
{
　　結果がfalseのときの処理
}

このあと、変数resultは使っていない

こうしてほしい

if(メソッド実行)
{
　　結果がtrueのときの処理
}
else
{
　　結果がfalseのときの処理
}

「いやなかんじ１」では、変数resultは使っていないに等しいです。無駄は省きましょう。
「いやなかんじ２」では、bool型の場合、trueかfalseのいづれかです。
else ifでわざわざfalseかチェックする必要はありません。
elseにきたときはfalseです。

今回の課題も内容的にはほぼ同じです。
この辺を改善していただけることを期待しています。

コメント

新しいコメントは書き込めません。

このブログの人気の投稿

11月16日(月)1、2コマ目

イメージ
今日、やったこと 新課題「C#版簡単オークションサイト」の説明 今日のホワイトボード 処理結果を伝える 以前から話している内容そのままです。とくに真新しい内容はありません。 図　処理結果を伝える [データベース]列の初期値 INSERT INTO実行時に挿入値が未指定の場合にデフォルト値が登録されるようにするには、図のようにdefaultを使って初期値を指定します。 図　応札日時列の初期値は現在日時(systimestampで取得できる) なお、列制約構文での制約、さらに初期値を指定する場合は下図の順になります。 create table テーブル名( 　列名　データ型　 default 初期値  制約 ) 入札処理 入札前の最高値取得は1人だけしかできないようにしなければならない。 入札最高値より高い金額しか入札できません。 まず、入札最高値をチェックする必要がありますが、下図のようにAさん、Bさんがほぼ同じタイミングでチェック（ロックしなければできる）し、先にBさんがDBに書き込むとAさんはエラーにする必要がありますが、Aさんのチェック時では最高値以上であるため、エラーにできません。 そこで、まず入札データテーブルの対象商品のデータをすべてロックし、ロックできた人がだけその後の処理（最高値取得、チェック、データ書き込み）ができるようにしてください。 図　最高値は1人だけしか取得できないようにするために NotFilledExceptionについて シーケンス図に書いてありましたが、ちゃんと説明をしていなかったので。 未入力項目がある場合は、NotFilledExceptionをスローするとかいてあります。 〇NotFilledExceptionクラスをつくる まず、NotFilledExceptionクラスですが、自作例外クラスです。Exceptionを継承してください。 図　自作例外クラスNotFilledExceptionをつくる 〇空白チェックのながれ tryブロック内は正常系をメインに実装し、catchブロックでエラー内容に応じた処理を書くのがtry～catchの一般的な実装です。 図　空白チェックのながれ ページ遷移 ログインページのOnPost()は以下のようになります。 ポイントは OnPost()の戻り値はIActionResult 遷移...
続きを読む

11月2日(月)1、2コマ目

イメージ
今日、やったこと .Net版異常時の対応 今日のホワイトボード 処理は2種類の終わり方がある 処理が正常に実行された「正常終了」となにがしかの問題が発生した「異常終了」。 異常終了も図のように2パターンに分類できる。 図　正常終了と異常終了 この2パターンの異常終了を通知する方法を.Netでは以下のようにすることにする。 ユーザーに通知して、 リカバリしてほしい エラー 戻り値で通知 ユーザーに通知しても、 リカバリが期待できない エラー 例外で通知 tryブロックについて 基本的に例外はスローしっぱなし。スローされた例外は上位へ伝播させる。 ただし、例外時に実行すべき処理がある場合のみ、例外をキャッチしていい。 これを今後の基本ルールにします。 図　tryブロックについて 今日のサンプルソース Userクラス ユーザー情報受け渡し用クラス。 UserInsertResultEnum 列挙型。ユーザー登録結果を通知するために用意。 UserDAOクラス 全件検索(SelectAll())、登録(Insert())メソッドのみ。 Serviceクラス ビジネスロジックレイアのクラス。今回はなくてもよかったけど。 Index.cshtml Index.cshtml.cs
続きを読む

10月13日(火)1、2コマ目

イメージ
きょう、やったこと Javaトランザクション(ロック) 演習課題 今日のホワイトボード おさらい Javaでトランザクション実装 基本は下図。 図　トランザクション実装の基本パターン try with resourcesを使うと下図のようになる。 図　try with resourcesでトランザクション実装 くどいけど、オラクルは クローズ時に未コミットデータはコミットする ので注意！！ ネットには結構、 クローズ時に未コミットデータはロールバックする と書かれている。 ロック ロックとは、データを排他制御するための仕組み。 データをロックした人（接続）しか、対象データにアクセスすることができなくなる。 ロック済みデータを他の人（接続）がロックすると、ロック解除待ちになる。 DBMSは更新系SQLを実行する際、対象行をロックする。よって、自分でロックする必要は基本的にない。 ただ場合によっては明示的にロックする必要がある。 ロックするには、 ロック対象行を検索するSELECTにFOR UPDATEをつけて実行する だけ。 ロックを解除するには、 コミットまたはロールバック を実行する。 図　ロック 演習課題について 一度に複数商品の在庫引当ができるところがややこしいポイント。 ボタンクリック時にdoPost()メソッドには、すべてのテキストボックス入力内容が送信されるため、どの商品に対する入力かを判断する必要がある。 ヒント あくまでも一案です。このとおりに作る必要はありません。 〇数量入力用テキストボックスの名前は「 quantity_商品ID 」にする これで名前からどの商品のテキストボックスかがわかる。 〇[doPost()にて]まず、送信データのすべての名前を取得 request.getParameterNames() で取得できる。 〇[doPost()にて]取得した名前がテキストボックスのものか判断 名前が「quantity_xxx」なら、数量入力用テキストボックス。 Stringクラスのmatches()メソッドで名前(String型)が「quantity_xxx」とマッチするかチェック。 　 取得した名前.matches("^quantitiy_.+$") quantity_で始まり(^quantity_) 任意の文字が1文字以...
続きを読む