C#でのDB(Oracle)のupdateがうまくいきません

Question

###前提・実現したいこと
C#でOracleに接続し、データベースを操作するプログラムの書き方を勉強中の身です。

http://www.webcodeexpert.com/2014/01/how-to-create-3-tier-application-in.html?m=1

上記サイトを参考に行っているのですが、update文の箇所のみ上手くいかず、困っています。
(delete、insertなど他の操作はできています)

※上記サイトはSQLServer向けのため、接続コードなど、一部変更を加えました。

###発生している問題・エラーメッセージ

エラーなし。

SQLの実行が上手くいっていないようです。
(C#を介さず実行したところ、問題なくバインドも実行もできました)

デバッグで確認しましたが、「result = cmd.ExecuteNonQuery();」部分でSQLが実行され、変更するレコード数が「result」に格納されるはずが、値が０になり、変更も反映されていませんでした。

バインド変数のパラメーター部分には値が入っていることを確認済です。
DALファイルのUpdateBookRecordを呼び出している箇所を見ても何も気づけませんでした。

色々試しましたが、何分知識も経験も足らず、自己解決は難しそうなため、修正点を教えていただければ幸いです……。
(原因箇所が該当ソースと違う可能性が高い場合はそれだけでも教えていただければ助かります)

###該当のソースコード

//…DAL.csファイル

public int UpdateBookRecord(BooksDetails_BEL objBEL)
{

int result;

try
{
string sql=@"
UPDATE YY_BOOKDETAILS
SET BOOKNAME = :BOOKNAME
,AUTHOR = :AUTHOR
,PUBLISHER = :PUBLISHER
,PRICE = :PRICE
,WHERE BOOKID = :BOOKID
";

OracleCommand Add = new OracleCommand (sql,con);

cmd.Parameters.Add("BOOKID", objBEL. BookId);
cmd.Parameters.Add("BOOKNAME", objBEL. BookName);
cmd.Parameters.Add("AUTHOR", objBEL. Author);
cmd.Parameters.Add("PUBLISHER, objBEL. Publisher);
cmd.Parameters.Add("PRICE", objBEL Price) ;


if (con.State == ConnectionState.Closed)
 {
con.Open()
 }

result = cmd.ExecuteNonQuery();

cmd.Dispose ();

if (result> 0)
 {
return result;
 }
else
 {
return 0;
 }
 
}
}

(インデントが消えて見にくくなってしまい、すいません)

###補足情報(言語/FW/ツール等のバージョンなど)
言語/C#
ツール/visual studio (2010)
DB/Oracle (10g)
oracle.dataaccess version=2.112.1.0

Comments

[seastar3]

Oracleのテーブルの形式とサンプルデータを示してください。

Answer 1

ExecuteNonQuery()の結果が0ということは、クエリは走行したが、そのクエリで影響を受けたレコードは0件ということです。
つまり、クエリは正常に動作していますが、パラメータの指定に誤りがあり、求めた条件でクエリが走行していないでしょうね。

何も指定しなかった場合、記されているSQLであれば、パラメータの指定順序は出現順でなければなりません。
つまり、名前でのバインドは行われません。

cmd.Parameters.Add("BOOKNAME", objBEL.BookName);
cmd.Parameters.Add("AUTHOR", objBEL.Author);
cmd.Parameters.Add("PUBLISHER", objBEL.Publisher);
cmd.Parameters.Add("PRICE", objBEL.Price) ;
cmd.Parameters.Add("BOOKID", objBEL.BookId);

OracleCommandを生成した直後に以下コードを記せば、名前でバインドしますので、パラメータの指定順序を気にする必要がなくなります。

cmd.BindByName = true;

Answer 2

単純に

cmd.Parameters.Add("BOOKID", objBEL. BookId);
cmd.Parameters.Add("BOOKNAME", objBEL. BookName);
cmd.Parameters.Add("AUTHOR", objBEL. Authir);
cmd.Parameters.Add("PUBLISHER, objBEL. Publisher);
cmd.Parameters.Add("PRICE", objBEL Price) ;

の箇所が

cmd.Parameters.Add("BOOKID", objBEL.BookId);
cmd.Parameters.Add("BOOKNAME", objBEL.BookName);
cmd.Parameters.Add("AUTHOR", objBEL.Authir);
cmd.Parameters.Add("PUBLISHER, objBEL.Publisher);
cmd.Parameters.Add("PRICE", objBEL.Price) ;

ではないかと思います。

Comments

[YURU-POTATO]

すいません、補足です。

こちらは会社で与えられ課題でして、USBの持ち込み等できないため、印刷・手書きしたものをスマホアプリで読み取りテキスト化し、修正を加えたものです。

実際のコードにスペルミスや余計な空白はなさそうです。(こちらのソースでご指摘いただいた箇所は後ほど修正します)

回答ありがとうございました。

>>C♯・Oracleが使用可能な方。

もし可能でしたら、サイトのプログラムをを試してみていただきたいです。
問題なく実行できるようでしたら、updateに関連する箇所のソースを見せていただけないでしょうか。
私自身が問題を把握できていないため回答しにくいとは思いますが非常に困っていますのでご協力お願いします。