1. Java / Говнокод #16045

    +68

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    class CircuitBreaker
{
    boolean broken = false;
    CircuitBreaker() {}
    private void breakCircuit()
    {
        this.broken = true;
    }
    private boolean isBroken()
    {
        return this.broken;
    }
}

private boolean writeAssetsToDisk()
{
    CircuitBreaker breaker = new CircuitBreaker();
    writeBase64EncodedAssetToDisk(breaker, "...", getPath(...));
    writeBase64EncodedAssetToDisk(breaker, "...", getPath(...));
    writeBase64EncodedAssetToDisk(breaker, "...", getPath(...));
    writeBase64EncodedAssetToDisk(breaker, "...", getPath(...));
    return !breaker.isBroken();
}

private void writeBase64EncodedAssetToDisk(CircuitBreaker breaker, String base64String, String filename)
{
    if (breaker.isBroken()) {
        return;
    }
    ...
    try
    {
        ...
    }
    catch (IOException e)
    {
        breaker.breakCircuit(); return;
    }
    ...
}

    Используй исключения, Люк. Фрагмент из Amazon Mobile Ads SDK.

    Запостил: chaoswithin, 23 Мая 2014

    Комментарии (3) RSS

    • ava guest 23.05.2014 13:58 # +1 

      try { /* ... */ } 
catch (IOException e) { breaker.breakCircuit(); return; }


      Нас попросили использовать исключения... Мы использовали как могли
      Ответить

    • ava bormand 23.05.2014 14:00 # +1

      > class CircuitBreaker
      C-C-COMBO BREAKER
      Ответить

      • ava bormand 23.05.2014 18:21 # 0

        Кстати, в c# есть подобный класс:

        http://msdn.microsoft.com/en-us/library/system.threading.cancellationtoken%28v=vs.110%29.aspx

        Так что может быть в асинхронной лапше этот комбобрекер и имеет смысл. Вот только CancellationToken можно передавать всяким ReadAsync'ам, а хрень из топика - нет.
        Ответить

    Добавить комментарий