Блог

Я работаю с dao, мой пользователь добавляет таблицу и ее поле. Мои программы проверяют, существует ли таблица, если да, то добавляется только поле, если таблица не существует, добавляется таблица, а затем добавляется поле.

Моя проблема возникает, когда я пытаюсь добавить второе поле. Это говорит мне, что таблица уже существует. Он не проходит через цикл надлежащим образом. вы можете помочь?

     string tablename = txtNameTable.Text.Trim();
        string fieldname = txtFieldName.Text.Trim();
        string size = txtSize.Text.Trim();

        //create a table
        myTable = clsDataSource.mydb.CreateTableDef(tablename);

        //create fields 
        if (cboFieldType.SelectedItem.ToString() == "Text")
        {
            myField = myTable.CreateField(fieldname, DAO.DataTypeEnum.dbText, size);
        }


        if (cboFieldType.SelectedItem.ToString() == "AutoNumber")
        {
            myField = myTable.CreateField(fieldname, DAO.DataTypeEnum.dbLong);
            myField.Attributes = (int)DAO.FieldAttributeEnum.dbAutoIncrField;
        }


        if (cboFieldType.SelectedItem.ToString() == "Number")
        {
            myField = myTable.CreateField(fieldname, DAO.DataTypeEnum.dbLong, size);
        }


        if (cboFieldType.SelectedItem.ToString() == "Date")
        {
            myField = myTable.CreateField(fieldname, DAO.DataTypeEnum.dbDate);
        }           


        //check if table exists
        bool tableExist = false;

        foreach (TableDef tb in clsDataSource.mydb.TableDefs)
        {
            if (tb.Name == myTable.Name)
            {
                myTable.Fields.Append(myField);
                tableExist = true;
                break;
            }
        }
            if (tableExist == false)
            {
                myTable.Fields.Append(myField);
                clsDataSource.mydb.TableDefs.Append(myTable);
            }



            //add primary key

            if (ckPK.Checked == true)
            {
                Index myIndex = myTable.CreateIndex("PrimaryKey");
                myField = myIndex.CreateField(fieldname);
                ((IndexFields)myIndex.Fields).Append(myField);
                myIndex.Primary = true;

                try
                {
                    clsDataSource.mydb.TableDefs[myTable.Name].Indexes.Append(myIndex);
                }
                catch (Exception)
                {
                    MessageBox.Show("This table already has a primary key", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                }
            }
            Activate(true, true, true);
        }
  

Хорошо, вот в чем дело. Если у вас есть несколько таблиц в вашем операторе foreach, то ваш оператор if с проверкой, является ли Findtable ложным, достигается только после того, как все таблицы повторяются. Следовательно, вы должны переместить свой оператор if внутрь цикла, чтобы он мог добавить новую таблицу в случае, если она не существует. Практически, вот как это должно выглядеть:

     List<String> existingTables = new List<String>();
    foreach (TableDef tb in clsDataSource.mydb.TableDefs)
        {
               if (tb.Attributes == 0 amp;amp; tb.Name == myTable.Name)
                {
                    clsDataSource.mydb.TableDefs[myTable.Name].Fields.Append(myField);
                    myTable.Fields.Append(myField);
                    clsDataSource.mydb.TableDefs.Append(myTable);
                }         
                else {       
                    existingTables.Add(MyTable.Name.ToString());
                }      
        }//end foreach
    foreach (var el in existingTables){
           MessageBox.Show("This table already exists: {0}", el)
    }//end foreach
  

Таким образом, вы избавляетесь от другого условия и сразу добавляете свою таблицу, если она соответствует всем параметрам. Более того, вы также избавляетесь от этого логического значения, которое в данном случае не требуется. Как только все таблицы будут повторены, вы выйдете из цикла.