metaclass: (Default)
[personal profile] metaclass
Есть такой оккультный стек-трейс: https://gist.github.com/def1fa8666a8ac83130a.
Причина в том, что в деконструктор массивов кложури вида (let [sql & params] some-array] передан пустой массив и строка sql в таком случае становится null и оный null передается в jdbc драйвер.
Строка на которой валится NPE, выглядит следующим образом:
protected boolean checkForEscapes(String sql) {
sql = sql.toLowerCase();

А вопрос, вообще говоря, в следующем: почему никто не проверяет параметр на валидность хотя бы в одной точке входа в одной из трех либ (clojure.java.jdbc, apache dbcp или jaybird)?

Выглядит так, как будто нормальную проверку параметров на валидность на границах API в жабе делать не принято.

PS: [livejournal.com profile] artureg упорно убеждает меня, что "пусть валится", потому что проверка должна быть "в другом слое". Судя по всему, все разработчики думают, что обработка ошибок должна быть в другом слое :)

Date: 2012-10-10 08:41 am (UTC)
From: [identity profile] metaclass.livejournal.com
Мой код вообще nil не содержит, а содержит сознательно помещенный туда пустой массив в качестве параметра для clojure.java.jdbc/with-query-results

Date: 2012-10-10 12:18 pm (UTC)
From: [identity profile] zmila.livejournal.com
так у тебя:
(let [[sql & params] some-array] 
  (call-java-shit sql ...) )
или
(let [[sql & params] some-array] 
  (when sql      
    (call-java-shit sql ...) ) )
?
:)


Date: 2012-10-10 01:00 pm (UTC)
From: [identity profile] metaclass.livejournal.com
Первое, но не у меня, а в либе.

Profile

metaclass: (Default)
metaclass

April 2017

S M T W T F S
      1
2345678
9101112 131415
16171819202122
23242526272829
30      

Most Popular Tags

Style Credit

Expand Cut Tags

No cut tags
Page generated Aug. 13th, 2025 04:09 pm
Powered by Dreamwidth Studios