Java будет работать эта настройка потоков или что я могу делать неправильно - PullRequest
0 голосов
/ 23 ноября 2011

Я немного неуверен и должен получить совет.
У меня есть:

public class MyApp extends JFrame{

И оттуда я делаю;

MyServer = new MyServer (this);
MyServer.execute();

MyServer - это:

public class MyServer extends SwingWorker<String, Object> {   

MyServer выполняет listen_socket.accept() в doInBackground()

и при соединении создает новый

class Connection implements Runnable {

У меня есть любимый DbHelper это синглтон

Содержит соединение Sqlite.Я инициирую его в приведенном выше MyApp
и полностью передаю ссылки в мой runnable:

class Connection implements Runnable {

Мой вопрос: что произойдет, если будет два одновременных read или `write?
Здесь я думал, что все методы в синглтоне синхронизированы, и
поместит все вызовы в очередь в ожидании блокировки синхронизированного метода.

Будет ли это работать или что я могу изменить?

public final class DbHelper {

    private boolean initalized = false;
    private String HomePath = "";
    private File DBFile;

    private static final String SYSTEM_TABLE = "systemtable";   
    Connection con = null;
    private Statement stmt;
    private static final DbHelper instance = new DbHelper ();

    public static DbHelper getInstance() {

        return instance;

    }

    private DbHelper () {

        if (!initalized)
        {
            initDB();

            initalized = true;
        }
    }

    private void initDB()
    {
        DBFile = locateDBFile();
        try {

            Class.forName("org.sqlite.JDBC");

            // create a database connection
            con = DriverManager.getConnection("jdbc:sqlite:J:/workspace/workComputer/user_ptpp");

        } catch (SQLException e) {
            e.printStackTrace();
        } catch (ClassNotFoundException e) {
            e.printStackTrace();
        }
    }

    private File locateDBFile()
    {
        File f = null;
        try{
            HomePath = System.getProperty("user.dir");
            System.out.println("HomePath: " + HomePath);
            f = new File(HomePath + "/user_ptpp");
            if (f.canRead())
                return f;
            else
            {
                boolean success = f.createNewFile();
                if (success) {
                    System.out.println("File did not exist and was created " + HomePath);
                    // File did not exist and was created
                } else {
                    System.out.println("File already exists " + HomePath);

                    // File already exists

                }
            }
        } catch (IOException e) {
             System.out.println("Maybe try a new directory. " + HomePath);
            //Maybe try a new directory.
        }
        return f;
    }

    public String getHomePath()
    {
        return HomePath;
    }



    public synchronized String getSelectedSystemTableColumn( String column) {

        String query = "select "+ column + " from " + SYSTEM_TABLE ;
        try {
            stmt = con.createStatement(ResultSet.TYPE_FORWARD_ONLY, ResultSet.CONCUR_READ_ONLY);
            ResultSet rs = stmt.executeQuery(query);
            while (rs.next()) {

                String value = rs.getString(column);

                if(value == null || value == "")
                    return "";
                else
                    return value;
            }

        } catch (SQLException e ) {
            e.printStackTrace();
            return "";
        } finally {

        }
        return "";

    }

}

1 Ответ

1 голос
/ 23 ноября 2011

Классы в java.sql должны быть поточно-ориентированными, в соответствии со спецификацией:

http://docs.oracle.com/javase/1.3/docs/guide/jdbc/spec/jdbc-spec.frame9.html

Мы требуем, чтобы все операции над всеми объектами java.sql были многопоточный безопасный и способный правильно справляться с несколькими потоки одновременно вызывают один и тот же объект. Некоторые драйверы могут позволить более одновременное выполнение, чем другие. Разработчики могут полностью принять на себя одновременное исполнение; если водитель требует какой-либо формы Синхронизация, это обеспечит это. Единственная разница видна разработчик будет, что приложения будут работать с пониженным параллелизм.

Вряд ли это будет хорошей идеей. Вы должны пересмотреть этот дизайн. Я думаю, что объединение соединений и закрытие их в максимально узкой области было бы лучшей идеей.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...