Кілька коментарів до коду.
1. Глобальні змінні - зло. А в цьому коді вони ще й не мають жодного сенсу, бо db та SQL перестворюються в обох функціях.
2. Ви перевідкриваєте з'єднання в обох функціях, що неефективно.
Ці проблеми легко вирішити, створивши клас, що інкапсулює з'єднання.
class Connection:
def __init__(self, dbname = 'db.db'):
self.database = sqlite3.connect(dbname)
self.cursor = database.cursor()
#commit тут не потрібен, жодних змін не внесено
def create_table(self, ...):
#не треба start_db і глобальні змінні, використовуємо self
...
def create_column(self, ...):
#не треба start_db
...
if __name__ == '__main__':
conn = Connection()
conn.create_table('table_1')
conn.create_column('table_1', 'column_1', 'INTEGER')
3. Ви передаєте в БД невичищені стрічки, що є антипатерном безпеки (гугліть Bobby Tables). Найкращий спосіб уникнути проблем - використовувати знаки питання чи іменні підставлення:
SQL.execute("""CREATE TABLE IF NOT EXISTS ? (
id INTEGER
)""", (table_name,))
args = {"table_name":table_name, "column_name":column_name, "column_class":column_class}
SQL.execute("""ALTER TABLE :table_name ADD COLUMN :column_name :column_class""", args)
