From 018a0c7978af7b9cf697ce031b24b8139888554c Mon Sep 17 00:00:00 2001 From: iNPUTmice Date: Tue, 18 Nov 2014 03:10:59 +0100 Subject: better usage of synchronized in some places --- .../siacs/conversations/entities/Conversation.java | 6 +- .../services/XmppConnectionService.java | 154 +++++++++++---------- .../conversations/ui/ConversationFragment.java | 2 +- 3 files changed, 80 insertions(+), 82 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/eu/siacs/conversations/entities/Conversation.java b/src/main/java/eu/siacs/conversations/entities/Conversation.java index b1df7ec7..4e894b84 100644 --- a/src/main/java/eu/siacs/conversations/entities/Conversation.java +++ b/src/main/java/eu/siacs/conversations/entities/Conversation.java @@ -57,7 +57,7 @@ public class Conversation extends AbstractEntity { private Jid nextCounterpart; - protected ArrayList messages = new ArrayList<>(); + protected final ArrayList messages = new ArrayList<>(); protected Account account = null; private transient SessionImpl otrSession; @@ -148,10 +148,6 @@ public class Conversation extends AbstractEntity { } } - public void setMessages(ArrayList msgs) { - this.messages = msgs; - } - public String getName() { if (getMode() == MODE_MULTI && getMucOptions().getSubject() != null) { return getMucOptions().getSubject(); diff --git a/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java b/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java index e7bf5207..2ff29b19 100644 --- a/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java +++ b/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java @@ -139,7 +139,7 @@ public class XmppConnectionService extends Service { private MessageGenerator mMessageGenerator = new MessageGenerator(this); private PresenceGenerator mPresenceGenerator = new PresenceGenerator(this); private List accounts; - private CopyOnWriteArrayList conversations = null; + private final CopyOnWriteArrayList conversations = new CopyOnWriteArrayList(); private JingleConnectionManager mJingleConnectionManager = new JingleConnectionManager( this); private HttpConnectionManager mHttpConnectionManager = new HttpConnectionManager( @@ -486,27 +486,23 @@ public class XmppConnectionService extends Service { } }; - this.databaseBackend = DatabaseBackend - .getInstance(getApplicationContext()); + this.databaseBackend = DatabaseBackend.getInstance(getApplicationContext()); this.accounts = databaseBackend.getAccounts(); for (Account account : this.accounts) { account.initOtrEngine(this); this.databaseBackend.readRoster(account.getRoster()); } + initConversations(); this.mergePhoneContactsWithRoster(); - this.getConversations(); - getContentResolver().registerContentObserver( - ContactsContract.Contacts.CONTENT_URI, true, contactObserver); + getContentResolver().registerContentObserver(ContactsContract.Contacts.CONTENT_URI, true, contactObserver); this.fileObserver.startWatching(); - this.pgpServiceConnection = new OpenPgpServiceConnection( - getApplicationContext(), "org.sufficientlysecure.keychain"); + this.pgpServiceConnection = new OpenPgpServiceConnection(getApplicationContext(), "org.sufficientlysecure.keychain"); this.pgpServiceConnection.bindToService(); this.pm = (PowerManager) getSystemService(Context.POWER_SERVICE); - this.wakeLock = pm.newWakeLock(PowerManager.PARTIAL_WAKE_LOCK, - "XmppConnectionService"); + this.wakeLock = pm.newWakeLock(PowerManager.PARTIAL_WAKE_LOCK,"XmppConnectionService"); toggleForegroundService(); } @@ -878,21 +874,23 @@ public class XmppConnectionService extends Service { }); } - public List getConversations() { - if (this.conversations == null) { + private void initConversations() { + synchronized (this.conversations) { Hashtable accountLookupTable = new Hashtable<>(); for (Account account : this.accounts) { accountLookupTable.put(account.getUuid(), account); } - this.conversations = databaseBackend - .getConversations(Conversation.STATUS_AVAILABLE); + this.conversations.addAll(databaseBackend.getConversations(Conversation.STATUS_AVAILABLE)); for (Conversation conv : this.conversations) { Account account = accountLookupTable.get(conv.getAccountUuid()); conv.setAccount(account); - conv.setMessages(databaseBackend.getMessages(conv, 50)); + conv.addAll(0, databaseBackend.getMessages(conv, 50)); checkDeletedFiles(conv); } } + } + + public List getConversations() { return this.conversations; } @@ -923,12 +921,11 @@ public class XmppConnectionService extends Service { } } - public void populateWithOrderedConversations(List list) { + public void populateWithOrderedConversations(final List list) { populateWithOrderedConversations(list, true); } - public void populateWithOrderedConversations(List list, - boolean includeConferences) { + public void populateWithOrderedConversations(final List list, boolean includeConferences) { list.clear(); if (includeConferences) { list.addAll(getConversations()); @@ -992,60 +989,63 @@ public class XmppConnectionService extends Service { public Conversation findOrCreateConversation(final Account account, final Jid jid, final boolean muc) { - Conversation conversation = find(account, jid); - if (conversation != null) { - return conversation; - } - conversation = databaseBackend.findConversation(account, jid); - if (conversation != null) { - conversation.setStatus(Conversation.STATUS_AVAILABLE); - conversation.setAccount(account); - if (muc) { - conversation.setMode(Conversation.MODE_MULTI); - } else { - conversation.setMode(Conversation.MODE_SINGLE); - } - conversation.setMessages(databaseBackend.getMessages(conversation, - 50)); - this.databaseBackend.updateConversation(conversation); - } else { - String conversationName; - Contact contact = account.getRoster().getContact(jid); - if (contact != null) { - conversationName = contact.getDisplayName(); - } else { - conversationName = jid.getLocalpart(); + synchronized (this.conversations) { + Conversation conversation = find(account, jid); + if (conversation != null) { + return conversation; } - if (muc) { - conversation = new Conversation(conversationName, account, jid, - Conversation.MODE_MULTI); + conversation = databaseBackend.findConversation(account, jid); + if (conversation != null) { + conversation.setStatus(Conversation.STATUS_AVAILABLE); + conversation.setAccount(account); + if (muc) { + conversation.setMode(Conversation.MODE_MULTI); + } else { + conversation.setMode(Conversation.MODE_SINGLE); + } + conversation.addAll(0, databaseBackend.getMessages(conversation, 50)); + this.databaseBackend.updateConversation(conversation); } else { - conversation = new Conversation(conversationName, account, jid, - Conversation.MODE_SINGLE); + String conversationName; + Contact contact = account.getRoster().getContact(jid); + if (contact != null) { + conversationName = contact.getDisplayName(); + } else { + conversationName = jid.getLocalpart(); + } + if (muc) { + conversation = new Conversation(conversationName, account, jid, + Conversation.MODE_MULTI); + } else { + conversation = new Conversation(conversationName, account, jid, + Conversation.MODE_SINGLE); + } + this.databaseBackend.createConversation(conversation); } - this.databaseBackend.createConversation(conversation); + this.conversations.add(conversation); + updateConversationUi(); + return conversation; } - this.conversations.add(conversation); - updateConversationUi(); - return conversation; } public void archiveConversation(Conversation conversation) { - if (conversation.getMode() == Conversation.MODE_MULTI) { - if (conversation.getAccount().getStatus() == Account.State.ONLINE) { - Bookmark bookmark = conversation.getBookmark(); - if (bookmark != null && bookmark.autojoin()) { - bookmark.setAutojoin(false); - pushBookmarks(bookmark.getAccount()); + synchronized (this.conversations) { + if (conversation.getMode() == Conversation.MODE_MULTI) { + if (conversation.getAccount().getStatus() == Account.State.ONLINE) { + Bookmark bookmark = conversation.getBookmark(); + if (bookmark != null && bookmark.autojoin()) { + bookmark.setAutojoin(false); + pushBookmarks(bookmark.getAccount()); + } } + leaveMuc(conversation); + } else { + conversation.endOtrIfNeeded(); } - leaveMuc(conversation); - } else { - conversation.endOtrIfNeeded(); + this.databaseBackend.updateConversation(conversation); + this.conversations.remove(conversation); + updateConversationUi(); } - this.databaseBackend.updateConversation(conversation); - this.conversations.remove(conversation); - updateConversationUi(); } public void clearConversationHistory(Conversation conversation) { @@ -1075,23 +1075,25 @@ public class XmppConnectionService extends Service { } public void deleteAccount(Account account) { - for (Conversation conversation : conversations) { - if (conversation.getAccount() == account) { - if (conversation.getMode() == Conversation.MODE_MULTI) { - leaveMuc(conversation); - } else if (conversation.getMode() == Conversation.MODE_SINGLE) { - conversation.endOtrIfNeeded(); + synchronized (this.conversations) { + for (Conversation conversation : conversations) { + if (conversation.getAccount() == account) { + if (conversation.getMode() == Conversation.MODE_MULTI) { + leaveMuc(conversation); + } else if (conversation.getMode() == Conversation.MODE_SINGLE) { + conversation.endOtrIfNeeded(); + } + conversations.remove(conversation); } - conversations.remove(conversation); } + if (account.getXmppConnection() != null) { + this.disconnect(account, true); + } + databaseBackend.deleteAccount(account); + this.accounts.remove(account); + updateAccountUi(); + UIHelper.showErrorNotification(getApplicationContext(), getAccounts()); } - if (account.getXmppConnection() != null) { - this.disconnect(account, true); - } - databaseBackend.deleteAccount(account); - this.accounts.remove(account); - updateAccountUi(); - UIHelper.showErrorNotification(getApplicationContext(), getAccounts()); } private void removeStaleListeners() { diff --git a/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java b/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java index cf02c970..7c421d54 100644 --- a/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java +++ b/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java @@ -94,7 +94,7 @@ public class ConversationFragment extends Fragment { } }; protected ListView messagesView; - protected List messageList = new ArrayList<>(); + final protected List messageList = new ArrayList<>(); protected MessageAdapter messageListAdapter; protected Contact contact; private EditMessage mEditMessage; -- cgit v1.2.3