October 27th, 2004

jagainu

идиоты

Была маленькая функция проверки правильности текущей записи. То есть нам нужна запись. Мы говорим функции, она проверяет текущую запись. Если мы уже стоим на нужной, не делает ничего. Если на другой, подчитывает нужную. Проверяется текущая запись конечно же по ключевому полю.
Босс с ещё одной нашей просветлённой программисткой наткнулись на ситуацию, когда ключевое поле оказалось неверным. То есть значения в нём были от одной записи, а во всех остальных полях от другой.
Вместо того, чтобы найти причину, что было бы самым правильным решением...
Вместо того, чтобы спихнуть эту задачу на меня, как лучшего дебаггера фирмы...
И даже вместо того, чтобы заменить вызов этой функции на безусловную считку нужной записи...
...эти сабж изменили всю функцию, убрав из неё оптимизирующую проверку. Которая конечно же вызывалась из десятка других мест. И некоторые из этих мест пеерестали работать, потому что запись там находилась в процессе изменения и из-за безусловной перечитки все изменения теперь сбрасываются.
И когда я указал этим сабжам на возникшую проблему, они ответили, что исходный баг они найти не смогли и даже не помнят, где он был. Так что теперь они выполнят работу в 10 раз больше, отыскав все места где вызывается данная функция и внеся в них такие поправки, что на момент перечитки записи в ней не будет несохранённых изменений.
Убил бы.
  • Current Mood
    infuriated infuriated