Неправильный insert()

[pitersky]

Для таких ветвлений кмк лучше switch использовать. он как-то понагляднее

[Владимир Максимов]

Цитата:

Сообщение от pitersky

Для таких ветвлений кмк лучше switch использовать. он как-то понагляднее

Вот уж чего здесь точно не нужно! Вообще-то, здесь вообще нет ветвления. Просто последовательное создание разрядности.

str 1024  strDuration;
;

// Сначала формируем секунды
// Ограничение на меньше 59999 - смысла не имеет, поскольку в противном случае
// нужна еще ветка по формированию часов и минут, если значение больше 59999
if (duration > 999)
{
    strDuration = Int2Str(duration div 1000) + " с ";
}

// Теперь формируем миллисекунды
strDuration += Int2Str(duration mod 1000) + " мс";

testPerfomanceTable.Duration2 = strDuration;

Другими словами, никакого ветвления. Просто последовательное формирование разрядов от старших к младшим

[pitersky]

Цитата:

Сообщение от Владимир Максимов

Вот уж чего здесь точно не нужно! Вообще-то, здесь вообще нет ветвления. Просто последовательное создание разрядности.

если нет ветвления, то switch по определению не нужен. а нужен он там, где ветвление есть и его не обойти. как будто десяток вложенных else if выглядит лучше...

[Владимир Максимов]

Цитата:

Сообщение от pitersky

если нет ветвления, то switch по определению не нужен. а нужен он там, где ветвление есть и его не обойти. как будто десяток вложенных else if выглядит лучше...

Проблема в том, что если здесь использовать swith, то это будет switch (true). Надеюсь, Вы не предлагаете 59999 case писать А конструкция switch (true) однозначно указывает на ошибку проектирования. Такая конструкция - это то же самое, что и десяток вложенных else if, но "вид сбоку" И то и другое выглядит "не естесственно". Как следствие, возникает желание проверить логику на предмет отказа от ветвления. Проверил. Не нужна она в данном случае.