создание sql запросов из строкового массива

Question

Мой интерфейс-

Я пытаюсь развернуть вложенный подзапрос-

select * from jobs where (location='delhi' or location='Mumbai') and profile in(select profile from jobs where profile='CompScience');

где для каждого из отмеченных флажков я хочу добавить это к условию. Например, если отмечены флажки Дели, Мумбаи, CompScience

запрос будет -

select * from jobs where (location='delhi' or location='Mumbai') and profile in(select profile from jobs where profile='CompScience'); 

это моя попытка-

private void button1_Click(object sender, EventArgs e)
{
    String location=null;
    string profile-null;

    if (checkBox1.Checked == true)
    {
        location+= checkBox1.Text;
    }

    if (checkBox2.Checked == true)
    {
        location += checkBox2.Text;
    }

    if (checkBox3.Checked == true)
    {
        location += checkBox3.Text;
    }

    if (checkBox4.Checked == true)
    {
        profile += checkBox4.Text;
    }

    if (checkBox5.Checked == true)
    {
        profile += checkBox5.Text;
    }

    if (checkBox6.Checked == true)
    {
        profile += checkBox6.Text;
    }

    //MessageBox.Show(location);

    db_CONNECT();
    conn.Open();

    try
    {
        String query = "select * from jobs where(location= 'delhi' or location = 'Mumbai') and profile in(select profile from jobs where profile = 'CompScience');";
        OracleCommand comm2 = new OracleCommand(selectquery, conn);
        OracleDataAdapter MyAdapter = new OracleDataAdapter();//adapter acts as interface btw database and dataset(which is collectio of tables)
        MyAdapter.SelectCommand = comm2;
        DataTable dTable = new DataTable();//datatable represents a single table in database 
        MyAdapter.Fill(dTable);
        dataGridView1.DataSource = dTable;
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }

    conn.Close();
}

Я попытался объединить строки и затем извлечь из них отдельные элементы.

Edit-

private void button1_Click(object sender, EventArgs e)
{
    db_CONNECT();

    try
    {
        CheckBox[] Locations = { checkBox1, checkBox2, checkBox3 };
        CheckBox[] Profiles = { checkBox4, checkBox5, checkBox6 };

        string locs = string.Join(" or ", Locations.Where(c => c.Checked).Select(x => $"location = '{x.Text}'"));
        string profs = string.Join(" or ", Profiles.Where(c => c.Checked).Select(x => $"profile = '{x.Text}'"));
        string query = $"select * from jobs where ({locs}) and profile in(select profile from jobs where {profs})";

        OracleCommand comm2 = new OracleCommand(query, conn);
        OracleDataAdapter MyAdapter = new OracleDataAdapter();//adapter acts as interface btw database and dataset(which is collectio of tables)
        MyAdapter.SelectCommand = comm2;
        DataTable dTable = new DataTable();//datatable represents a single table in database 
        MyAdapter.Fill(dTable);
        dataGridView1.DataSource = dTable;
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message);
    }

    conn.Close();
}

Answer 1

List<string> locations = new List<string>();

Я бы изменил button1_Click метод следующим образом:

private void button1_Click(object sender, EventArgs e)
{       
    if (checkBox1.Checked == true)
    {
        locations.Add(checkBox1.Text);
    }
    else
    {
        locations.Remove(checkBox1.Text);
    }
    // and so on for other locations
}

Затем вы можете создать команду запроса следующим образом (это пример только для locations, для profile вы должны сделать то же самое):

var locationsString = string.Join(", ", locations.Select(l => $"'{l}'"));  // this gives you, e.x. 'Delhi', 'Mumbai'

var query = "";
if (locations.Any())
{
    query = $"select * from jobs where(location in {locationsString }) and profile in(select profile from jobs where profile = 'CompScience');";
}
else
{
    query = $"select * from jobs where profile in(select profile from jobs where profile = 'CompScience');";
}

Answer 2

Продолжая ответ Дмитрия, я бы предложил еще больше упростить это до

.

// ... do locations AND profiles the way Dmitry suggested

// start out with a generic query
StringBuilder querybuilder = new StringBuilder("SELECT * FROM jobs WHERE 1 = 1");

if (locations.Any())
{
    var locationsString = string.Join(", ", locations.Select(l => $"'{l}'"));
    querybuilder.AppendFormat(" AND location IN ({0})", locationsString);
}

if (profiles.Any())
{
    var profilesString = string.Join(", ", profiles.Select(l => $"'{l}'"));
    querybuilder.AppendFormat(" AND profile IN ({0})", profilesString);
}

// ...

OracleCommand comm2 = new OracleCommand(querybuilder.ToString(), conn);

Поймать все WHERE 1 = 1 - это общепринятый способ создания динамически составленных запросов, позволяющий значительно упростить условия добавления предложений переменных к вашему запросу.

Answer 3

Вы можете иметь массив своих флажков и присоединяться к проверенным текстам, используя string.Join():

CheckBox[] Locations = {Checkbox1, CheckBox2, CheckBox3};
CheckBox[] Profiles = {Checkbox4, CheckBox5, CheckBox6};

string locs = string.Join(" or ", Locations.Where(c => c.Checked).Select(x => $"location = '{x.Text}'");
 string profs = string.Join(" or ", Profiles.Where(c => c.Checked).Select(x => $"profile = '{x.Text}'");
string result = $"select * from jobs where ({locs}) and profile in(select profile from jobs where {profs})";

Если у вас есть флажки в родительском контейнере, например, в групповом окне или на панели, вы можете даже сделать какэто:

CheckBox[] Locations = locPanel.Controls.OfType<CheckBox>().ToArray();
CheckBox[] Profiles = profPanel.Controls.OfType<CheckBox>().ToArray();