すべてのコードパスが値を返すわけではありません。C＃ - プライベート文字列とforループ

Question

こんにちは、これを重複としてマークする前に、私は見て、他の人を試して、運がなかった。

すべてのコードパスが値を返すわけではないという文字列getBrandのエラーが引き続き発生します。以下は

private string getBrand(string id) 
{ 
    con.Open(); 
    SqlCommand cmd = con.CreateCommand(); 
    cmd.CommandType = CommandType.Text; 
    cmd.CommandText = "Select brand from tbl_products where productId = '" + id + "'"; 
    cmd.ExecuteNonQuery(); 
    con.Close(); 
    DataTable dt = new DataTable(); 
    SqlDataAdapter sda = new SqlDataAdapter(cmd); 
    sda.Fill(dt); 
    getBrand = dt.Rows[0][0].ToString(); 
}  

私はgetBrand Stringに渡す文字列「idは」からのクエリを実行したい取得する場所です。

for (int i = 0; i < salesGridView.Rows.Count; i++) 
{ 
    table2.AddCell(new Phrase(salesGridView[1, i].Value.ToString(), normFont)); 
    string id = salesGridView[0, i].Value.ToString(); 
    table2.AddCell(new Phrase(getBrand(id), normFont)); 
} 

Answer 1

dt.Rows[0][0].ToString();をメソッド名に保存しました。方法から次の行を返す必要があります。

return dt.Rows[0][0].ToString(); 

または、別の変数の名前に格納し、その変数を返します。このように：

var temp = dt.Rows[0][0].ToString(); 
return temp; 

Answer 2

あなたは、このようにそれを実行する必要があります。

private string getBrand(string id) 
    { 
     con.Open(); 
     SqlCommand cmd = con.CreateCommand(); 
     cmd.CommandType = CommandType.Text; 
     cmd.CommandText = "Select brand from tbl_products where productId = '" + id + "'"; 
     cmd.ExecuteNonQuery(); 
     con.Close(); 
     DataTable dt = new DataTable(); 
     SqlDataAdapter sda = new SqlDataAdapter(cmd); 
     sda.Fill(dt); 

     return dt.Rows[0][0].ToString(); 
    }