JujaLabs / sqlcmd

0 stars 5 forks source link

Выделить интерфейс DatabaseManager и добавить его реализацию в виде класса InMemoryDatabaseManager #108

Closed danilkuznetsov closed 6 years ago

danilkuznetsov commented 6 years ago
  1. Переименовать класс DatabaseManager в JDBCDatabaseManager

  2. Выделить интерфейс DatabaseManager из существующего класса JDBCDatabaseManager.

интерфейс DatabaseManager должен содержать все публичные методы класса DatabaseManager.

public boolean connect(String database, String user, String password)
public void close()
public String[] getTableNames()
public DataSet[] getTableData
public boolean insert(String tableName, DataSet dataset)
public boolean delete(String tableName, int id)
public boolean update(String tableName, int id)
  1. Заменить тип данных везде где используется класс JDBCDatabaseManager на интерфейс DatabaseManager Проверить что тесты работают и нет поломок :)

  2. Добавить еще одну реализацию DatabaseManager в классе InMemoryDatabaseManager. Реализовать все методы, кроме метода public boolean update(String tableName, int id). Этот метод мы сделаем позднее.

Класс InMemoryDatabaseManager покрыть тестами.

Shtramak commented 6 years ago

@danilkuznetsov Данил, по ходу public boolean update(String tableName, int id) ты случайно в таску добавил... Ну, как бы метод можно оставить и пока что в нем сделать UnsupportedOperationException, но сигнатуру точно нужно поменять...

danilkuznetsov commented 6 years ago

@Shtramak я пока оставил как есть, ты прав, как мы и договаривались, его можно не делать Мы сделаем рефакторинг, но в будущем думаю что не надо с ребятами такое делать :)