Два раза вызываешь getColumnAt. Помимо производительности, тут ещё засада с многопоточностью: если параллельно обнулять значение, то ты тут NPE получишь.