C#

JelleM wrote:

I don't really think the code is 100% ok, but can somebody help?

// Not enough context to determine what this line does.
bill.Rows.Clear();
 
// Create a parameterised SQL Command
SqlCommand myCommand = new SqlCommand("SELECT * FROM orders WHERE date = @date", connect);
myCommand.Parameters.AddWithValue("@date", DateTime.Parse(listOrders.Text.ToString().Substring(0, 19)));
 
// Exectute the query
SqlDataReader myReader = myCommand.ExecuteReader();
while (myReader.Read())
{
    // Assign amounts and products the results of the query
    amounts = myReader["amount"].ToString();
    products = myReader["productid"].ToString();
}
// Close the query - NOTE: If there is more than one row returned from the query all but
// the last row will have been lost by this point.
myReader.Close();
 
// These two lines are redundant - they do nothing useful.
string[] product = new string[products.Split(' ').Length];
string[] amount = new string[amounts.Split(' ').Length];
 
// If there were rows returned from the query then split the data up on a space character.
// If there were no rows returned from the query then products and amounts will be null
// and the code will fail.
product = products.Split(' ');
amount = amounts.Split(' ');

// Loop over the products array 
for (int i = 0; i <= product.Length; i++)
{
    // Not enough context to say what this line does.
    bill.Rows.Add(amount[i], product[i], 1, 2);
}

public partial class Form2 : Form
{

public int d, e, f;
public Form2(int a, int b, int c)
{
 d = a;
 e = b;
 f = c;
InitializeComponent();
}

...

public partial class Form1 : Form
{

public int a, b, c;
public Form1()
{
InitializeComponent();
}

something_OnClick()
{
Form2 frm2 = new Form2(a, b, c);
frm2.ShowDialog();
}

private void comboBox1_TextChanged(object sender, EventArgs e)
{
    ComboBox c = (ComboBox)sender;
    label1.Text = c.Text;
}

------------------------------
"The Soapbox has been so ..."