Software Best Practices

[strvt]

Коллега прислал выдержку из нашего кода. В исходниках было немного разнесено, но на одном экране таки умещалось:

private static void deleteHourly(PreparedStatement stmt, Timestamp start, Timestamp end, int curveNum) throws SQLException
{
stmt.setInt(1, curveNum);
stmt.setInt(2, PositionValuationConstants.TIME_PERIOD_HOURLY);
stmt.setTimestamp(3, start);
stmt.setTimestamp(4, end);
stmt.execute();
}

private static void deleteHalfHourly(PreparedStatement stmt, Timestamp start, Timestamp end, int curveNum) throws SQLException
{
stmt.setInt(1, curveNum);
stmt.setInt(2, PosValConstants.TIME_PERIOD_TYPE_IND_HALFHOURLY);
stmt.setTimestamp(3, start);
stmt.setTimestamp(4, end);
stmt.execute();
}


private static void deleteDaily(PreparedStatement stmt, Timestamp start, Timestamp end, int curveNum) throws SQLException
{
stmt.setInt(1, curveNum);
stmt.setInt(2, PositionValuationConstants.TIME_PERIOD_DAILY);
stmt.setTimestamp(3, start);
stmt.setTimestamp(4, end);
stmt.execute();
}

Я выделил отличия.

Comments

[strvt.livejournal.com]

Теоретически два из трёх методов скорее всего были написаны ещё когда мы программировали в Штатах. Правда всё равно индусом :). Он у нас был большим специалистом по Джаве.

[kanstantin.livejournal.com]

А ну так, очевидно, знакомые того вашего индусского парня только-только начали подключатся пока что :-) Думаю, через какое-то время они с таким подходом просто горы свернут!