проблема с типом float.

[tridentxp]

есть кусок кода ниже. вот: если переменную "a" объявить unsigned int, то в окне получим округлённое значение типа float, а если её объявить float - ом то всё идёт отлично. чо за беда ?
//---------------------------------------------------------------------------

void __fastcall TForm1::frequencyChange(TObject *Sender)
{float f,f1,freq;

unsigned int a;
AnsiString s;
for(a=1;a<11;a++)if ((frequency->Text[a])!='.') s+=frequency->Text[a];
freq=StrToFloat(s);if (freq==0) goto end;
a=60000000/freq;
f=60000000/a;f1=60000000/(a+1);
  if ( (freq-f)>(freq-f1) ) f=f1;
actual_freq->Caption="Actual frequency : "+ FloatToStr(f); //
end:
}
//---------------------------------------------------------------------------

[SkaT]

Наверное как обычно, в математической операции (любой) нужно применять переменные (константы) одного типа, иначе компилятор начнет преобразовывать тип по своему усмотрению. Соответственно результат будет отличаться от "ожидаемого".

[tridentxp]

ну да, по ходу это и произошло, может просто есть какие типичные настройки, запрещающие преобразование типа?...

[SkaT]

Не, это наврядли.
Попробуй указатели типа, может прокатит.
Не помню где используются, в голове каша.
ну типа: f=60000000.0/(float)a
Или что-то в этом духе.

[Prz777]

Переменную а необходимо объявить float, т.к. она получается результатом деления a=60000000/freq; А потом если нужно получить целое применить округление.

[tridentxp]

прикооллл. а как такое понимать ?
void __fastcall TForm1::frequencyChange(TObject *Sender)
{float f,f1,freq;//  ,a

unsigned int int_freq,a;
AnsiString s;
for(a=1;a<11;a++)if ((frequency->Text[a])!='.') s+=frequency->Text[a];
freq=StrToFloat(s);if (freq==0) goto end;
int_freq=StrToInt(s);

a=(60000000/int_freq);

f=60000000/(float)a;f1=60000000/ (float) (a+1);
actual_freq->Caption="Actual frequency : "+ FloatToStr(f)+"  a="+IntToStr(a)+" f1="+FloatToStr(f1);

  if ( (freq-f)>(freq-f1) ) f=f1;

end:
}

[SkaT]

А что не ток-то, ты напиши.
Вроде всё верно. f1, отличается от калькулятора компа только в пятой цифре после запятой.
Что не нравится???
Если целое значение результата   f=60000000/(float)a
То думаю компилятору (float) не достаточно.

[tridentxp]

60 000 000 / 53571 = 1120,0089600716805734445875567005 - дробное число, а выводится без дробной части.

[tridentxp]

    бред какой то - ща с дробной частью выдаёт - мистика...

[SkaT]

Я ещё у тебя не вижу инициализации строковой переменной s.
Лучше самому следить за начальными значениями переменных а не доверять компилятору, и тем более компу.


Отклоняясь от темы.
Уже несколько раз показывали передачу по "кабельному".
Ученый, установил по всему миру кучу генераторов "RND".
Если в мире всё "в рамках", то "RND" даёт 50% нулей и 50% единиц.
Но не за долго до катастроф начинается отклонение от 50/50.


Так что "мыслеформы" влияют на работу процессора...
[size=78%][/size]

Кстати, проверил на Делфи, такой проблемы нет (в данном контексте).

[tridentxp]

Я ещё у тебя не вижу инициализации строковой переменной s.

5 ая строчка : AnsiString s;

[Prz777]

Введи новую переменную float b; для вычисления b=(60000000/int_freq);. Переменная int a; для a=1;a<11;a++

[tridentxp]

b=(60000000/int_freq);

тут прикол в том, что эта переменная и должна быть unsigned int - она будет перекидываца в 32-ух битный таймер внутри камня - задавая тем самым коэффициент деления. и вся эта котовасия с разными типами - стремление подобрать как можно ближе частоту желаемой.

[Prz777]

После b=(60000000/int_freq); нужно применить округление до  меньшего. На Delphi это было-бы так a:=Round(b);. Далее использовать a.

[tridentxp]

сейчас уже другой прикол - через раз выдаёт то с дробной частью, то без неё.

[Prz777]

Долго долбил. Без дробной части получалось если вводить 1,10,100,1000 и т.д. Остальные все с дробной частью.

[SkaT]

Даже при небольших значениях частоты дробная часть отваливается?
Или только при значениях выше 6.7MГц

[tridentxp]

чот я под вечер тупить начал - вроде на месте дробная часть.... надо срочно горлышко смочить...

[drfaust]

60000000

преепиши в 60000000.0 - не знаю как в паскале (дельфи), а вот в С/С++ желательно явно указать тип константы, разные компилеры могут по разному преобразовывать типы (один 60 лямов преобразует в float/double, другой наоборот а преобразует в инт.)

И да тут f=60000000/a;f1=60000000/(a+1); у тебя а - целое, т.е. 60000000 - тоже целое, таким образом делим целое на целое получаем целое (без всяких дробей), а уже потом преобразовываем оператором присваивания во float. Если с этим floaf f что-то ещё делаешь, то могут появится погрешности в виде к-то десятых/тысячных.

[Prz777]

делим целое на целое получаем целое (без всяких дробей)

К примеру 3:2=1,5. Тип переменной результата должен быть float или double.

[tridentxp]

drfaust, сенькью, на всякий пожарный прописал константу дробной. вощем забадать удалось через известную на руси точку - пятую:
Caption="Actual frequency :"+ FloatToStr(60000000.0/a), а не FloatToStr(f).
теперь дробная часть отображается.

[drfaust]

Caption="Actual frequency :"+ FloatToStr(60000000.0/a), а не FloatToStr(f).

f=60000000.0/float(a);будет правильнее, я говорил, что паскаль я намертво забыл, а поведение С/С++ в этом случае будет зависеть от компилера(порядок преобразования типом в стандарте не описан).

[SkaT]

Кстати у функции FloatToStr() возможно есть параметр "формат строки". В паскале вроде есть. Должна быть возможность указать количество знаков после запятой, в строке. Тогда, даже в случае "целого" числа, будет показывать с нулями, это выглядит более корректно.