Java 信号量不调用release（）

我已经在Android应用程序中实现了一个线程，它每分钟都被调用一次，进程的调用通过一个报警管理器进行

@Override
public void onReceive(Context context, Intent intent) {
    try {
        PowerManager powerManager = (PowerManager) context.getSystemService(Context.POWER_SERVICE);
        PowerManager.WakeLock wakeLock;
        if (powerManager != null) {
            wakeLock = powerManager.newWakeLock(PowerManager.PARTIAL_WAKE_LOCK, "Send-data");
            wakeLock.acquire(10 * 60 * 1000L);
            new Thread(new SendPolicyData(context)).start();
            wakeLock.release();
        }
    } catch (Exception e) {
        wil.WriteFile("1)AlarmSendData - Exception: " + e.toString());
    }
}

线程中包含的代码从数据库中提取一组数据，这些数据必须通过POST发送到服务器，通过信号量控制对资源的访问

@SuppressWarnings("ResultOfMethodCallIgnored")
public class SendPolicyData implements Runnable {
    private static final WriteInLogFile wil = new WriteInLogFile();
    private final Context ctx;

public SendPolicyData(Context ctx) {
    this.ctx = ctx;
}

public void run() {
    try {
        SemaphoreDTS.getInstance().goIn();

        Authentication singleCall = new Authentication();
        Utilities utilities = new Utilities(ctx);
        DbGest dbGest = DbGest.getInstance(ctx);

        if (utilities.checkConnection()) {
            int lastProcessedID = -1;
            int attempts = 0;

            Cursor cursor = dbGest.freeQuery("SELECT id, label, value, priority FROM " + TABLE_DATATOSEND + " ORDER BY priority,dateIns");
            if (cursor != null) {
                cursor.moveToFirst();
                if (cursor.getCount() > 0) {
                    do {
                        int toProcessID = cursor.getInt(cursor.getColumnIndex("id"));
                        String value = cursor.getString(cursor.getColumnIndex("value"));
                        String labelString = cursor.getString(cursor.getColumnIndex("label"));

                        if (lastProcessedID == toProcessID) {
                            if (attempts <= 5) {
                                attempts++;
                                Thread.sleep(3000);
                            } else {
                                attempts = 0;
                                dbGest.changeDTSRecordPriority(toProcessID);
                            }
                        }

                        switch (labelString) {
                            case "photo":
                               //DO STUFF
                             break;
                        lastProcessedID = toProcessID;
                    } while (cursor.moveToNext());
                }
                cursor.close();
            }
        }
    } catch (Exception e) {
        SemaphoreDTS.getInstance().goOut();
        wil.WriteFile("7)SendPolicyData - Exception: " + e.toString());
    } finally {
        SemaphoreDTS.getInstance().goOut();
    }
    SemaphoreDTS.getInstance().goOut();
}

在我所做的测试中，信号量经常被阻塞，出于某种原因，为了能够执行新的捕获，没有调用必要的释放

我想我写的代码是正确的，我不明白我错在哪里。

在此代码块中：

catch (Exception e) {
        SemaphoreDTS.getInstance().goOut();
        wil.WriteFile("7)SendPolicyData - Exception: " + e.toString());
    } finally {
        SemaphoreDTS.getInstance().goOut();
    }
    SemaphoreDTS.getInstance().goOut();

您总是有两个调用

.gout（）

，因为

最后

块总是会调用。

---

当您调用.release（）（在您的方法

.gout（）

）时，信号量将获得可用的许可，即1个许可，而您的信号量将获得2个许可。我想问题从这里开始。尝试在任何地方删除调用“.gout（）方法，但不要在finally（）中删除。

是否确实需要在函数goIn（）和goOut（）中指定“synchronized”？如果是，为什么？

catch (Exception e) {
        SemaphoreDTS.getInstance().goOut();
        wil.WriteFile("7)SendPolicyData - Exception: " + e.toString());
    } finally {
        SemaphoreDTS.getInstance().goOut();
    }
    SemaphoreDTS.getInstance().goOut();