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

    +78

    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
    PreparedStatement statement = createStatement();

for (Value value: values) {

	if (statement == null) {
		statement = createStatement();
	}

	try {
		if (statement.isClosed()) {
			statement = createStatement();
		}
	} catch (Exception e) {
		logger.error("...", e);
	}

	// ...
}

    "бог любит троицу"

    Запостил: user654321, 23 Сентября 2011

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

    • ava ingenuus 23.09.2011 10:56 # 0

      Про addBatch(sql) видимо не слышал ни разу...
      Не понятно в каком контекте этот код, но наверняка можно было вообще без 100500 запросов обойтись.
      Ответить

      • ava user654321 23.09.2011 11:03 # 0

        Тут просле try-catch сразу идет выполнение стэйтмента.
        А вся эта конструкция запускается, если собранный руками multi-insert вывалил эксепшен.
        Ответить

    • ava absolut 23.09.2011 13:15 # 0

      почему в catch() нету 
      statement = createStatement();
      ?
      Ответить

    • ava gegMOPO4 23.09.2011 14:41 # 0

      На месте ... наверняка есть и statement = null, и statement.close(). Но да, выглядит странно.
      Ответить

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