Код:

Код

long double angle;  angle = DegToRad( StrToFloat( MaskEdit1 -> Text ) ); ....  AngleGrid -> Cells[1][0] = FloatToStr( Sin( angle ) );  AngleGrid -> Cells[1][1] = FloatToStr( Cos( angle ) );

При вводе имеем:
0: sin = 0, cos =1
90: sin = 1, cos = -2.7
180: sin = -5.4, cos = -1

К примеру на углах 45, 135, 405 все в норме.

Что-то я не помню таких функций, как Sin и Cos... хмм...
А может, ошибка появляется при отображении? Ну, места там не хватает... или при преобразовании что-то случается...

насколько я помню они радианы принимают... + что за странное название ( с большой буквы )? в math.h с маленькой...

Цитата(_hunter @ 5.1.2006, 16:19 )

насколько я помню они радианы принимают... + что за странное название ( с большой буквы )? в math.h с маленькой...

Я в BDS 2006. Там есть Math.hpp и Math.h, из первой - с большой буквы, из второй - с маленькой. Но замена их на маленькие результата не меняет. По крайней мере в лучшую сторону.

Цитата(BreakPointMAN @ 5.1.2006, 16:17 )

А может, ошибка появляется при отображении? Ну, места там не хватает... или при преобразовании что-то случается...

Все преобразования приведены выше.
Вряд ли там места не хватает. На 45 все окей, как и в 0.
Добавлено @ 16:28

Цитата(_hunter @ 5.1.2006, 16:19 )

насколько я помню они радианы принимают...

Я им их и передаю. DegToRad преобразует. Преобразует правильно.
Проверялся так:

Код

Edit5 -> Text =  FloatToStr( DegToRad( StrToFloat( MaskEdit1 -> Text ) ) );

Пишет нормальные значения PI, PI/2.

а если напрямую написать?

Код

Sin(angle * PI / 180)

Это сообщение отредактировал(а) Illuminaty - 5.1.2006, 16:30

Не помогает.

а что выводит при явном задании значения переменной angle ?

Код

angle=180.0;

Это сообщение отредактировал(а) Illuminaty - 5.1.2006, 16:45

стоп. все правтльно оно выдает.
при 180 син не -5.4 а -5.4 * 10^(-20) ( т.е. 0 )
напиши что-то типа AngleGrid -> Cells[1][0] = AnsiString().sprintf("%.3f", Sin( angle ) );

У меня в BC++B6 результаты совершенно адекватные...

Код

long double angle;  angle = DegToRad(180);  AngleGrid -> Cells[1][0] = FloatToStr( sin( angle ) );  AngleGrid -> Cells[1][1] = FloatToStr( cos( angle ) );

Выполнение этого кода даёт нам
sin: 1,2246...........E-16
cos: 1

Цитата(_hunter @ 5.1.2006, 16:57 )

напиши что-то типа AngleGrid -> Cells[1][0] = AnsiString().sprintf("%.3f", Sin( angle ) );

"Что-то типа" не помогло. Помогло вот что:

Код

AngleGrid -> Cells[1][0] = FloatToStr( RoundTo( Sin( angle ) , -10 ) );

Огромное тебе спасибо !!!
Добавлено @ 17:16
Хотя каждый раз Round использовать не хочется

а что значит не помогло?
минус мешает?
так его убрать можно...
или вообще так:
if ( abs(sin) < 0.00000000001 )
sin = 0;
написать
а потом sprintf ему...

У косинуса при - был -0.000
Минус тут не при чем. Там 1 должна быть

похоже sprintf() с long double-ами не работает. приводи перед конвертацией к doublе -- всеравно все знаки пользователю вряд ли нужны...
AnsiString().sprintf("%0.3f", (double)(Sin(DegToRad(angle))));

_hunter сработало. Слу, а что лучше в плане быстродействия и грамотности кода?
RoundTo или через sprintf?

интересный вопрос...
sprintf точно не меделеннее ( потому как при RounTo/FormatFloat выполняются аналогичные операции ) а возможно и быстрее -- новое значение не создается ( RounTo )
в любом случае нужно эксперимент ставить
а в полане грамотности ( а точнее красоты кода ) sprintf явно лучше

_hunter а с точки зрения грамотности будет выгоднее вынести это преобразование в отдельную функцию? При условии, что мне надо меня формат, т.е. кол-во отображаемых знаков после запятой?

в отдельную функцию смысла нет ( разве что читаемость немного повысить )
можно в том месте, где задается число знаков и формировать строку формата:
formatStr = "%0." + IntToStr(precision) + "f";
AnsiString().sprintf(formatStr, (double)(Sin(DegToRad(angle))));

Окей. Спасибо. Так и сделаю.

Хм, пишу вот так:

Код

Prec = "%." +  IntToStr(PrecComboBox -> ItemIndex)+"f ";

Выдает:

Цитата

[C++ Error] Unit2.cpp(39): E2277 Lvalue required

или

Код

Prec = "%." + IntToStr( PrecComboBox -> ItemIndex ) + "f";

Выдает:

Цитата

[C++ Error] [C++ Error] Unit2.cpp(269): E2085 Invalid pointer addition

Объявленно, как:

Код

char Prec[5];

Потому что слева у тебя char...а справа AnsiString...

Писать нужно так:

Код

Prec = ("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f").c_str();

или так...

Код

Prec = AnsiString("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f").c_str();

Либо вообще так:

Код

AnsiString Prec;   Prec = "%." + IntToStr( PrecComboBox -> ItemIndex ) + "f"; //и уже дальше  sprintf(Prec.c_str(), ...., ...);

Это сообщение отредактировал(а) Partizan - 6.1.2006, 18:36

Цитата(Partizan @ 6.1.2006, 18:33 )

Либо вообще так:

Это в моем случае получается несколько не рационально.
Воспользовался вариантом:

Цитата(Partizan @ 6.1.2006, 18:33 )

Prec = AnsiString("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f").c_str();

Эх, никак не могу привыкнуть, что AnsiString это не простая строка, а целый класс...

Спасибо.
Добавлено @ 18:43
Стоп. Выдает все равно [C++ Error] Unit2.cpp(39): E2277 Lvalue required

а...ну дык пральна...
если уж ты остановился на том варианте переопредели Prec так

Код

char *Prec;

+ не понятно чем AnsiString не нравится. почему бы ее и не использовать?

Цитата(_hunter @ 6.1.2006, 18:56 )

+ не понятно чем AnsiString не нравится. почему бы ее и не использовать?

Когда подставляю в формат, мне пишет несоответствие типов cont char * и AnsiString.

Цитата(Partizan @ 6.1.2006, 18:49 )

если уж ты остановился на том варианте переопредели Prec так

Пробовал.
Ругается вот так: [C++ Error] Unit2.cpp(39): E2235 Member function must be called or its address taken
Т.е. мы как-то не правильно в указатель пишем.

Цитата

Цитата(_hunter @  6.1.2006,  18:56 ) + не понятно чем AnsiString не нравится. почему бы ее и не использовать? Когда подставляю в формат, мне пишет несоответствие типов cont char * и AnsiString.

когда подставляешь в формат не забывай вызывать !!!c_str()!!!
Prec.c_str() - это у тебя должно быть в формате в случае AnsiString Prec;

Цитата

Цитата(Partizan @  6.1.2006,  18:49 ) если уж ты остановился на том варианте переопредели Prec так Пробовал. Ругается вот так: [C++ Error] Unit2.cpp(39): E2235 Member function must be called or its address taken Т.е. мы как-то не правильно в указатель пишем.

гммм...щас посмотрим...
E2235 Member function must be called or its address taken
A reference to a member function must be called, or its address must be taken with & operator....
возможно ты опечатался...
лично у меня следующий тест проходит на ура:

Код

char *Prec; float MyVar = 1.099999; Prec = ("%." + IntToStr( 2 ) + "f").c_str(); char *Buffer = new char[strlen(Prec)]; sprintf(Buffer, Prec, MyVar ); ShowMessage(Buffer); delete[] Buffer;

Цитата(Partizan @ 6.1.2006, 19:58 )

Prec = ("%." + IntToStr( 2 ) + "f").c_str();

Вот тут выкидывает на таком же коде.

используя ансистринг

Код

int intPrec = 2;//точность   double value = 1.099999;   AnsiString ansiPrec = "%." + IntToStr( intPrec ) + "f";   AnsiString rez;//результат тут   rez.sprintf(ansiPrec.c_str(),value);

если не нравится замени на char

Код

int intPrec = 2;   double value = 1.099999;   char *charPrec = ("%." + IntToStr( intPrec ) + "f").c_str();   AnsiString rez;   rez.sprintf(charPrec,value);

Все должно ботать

Цитата(TIGERоX @ 6.1.2006, 22:12 )

char *charPrec = ("%." + IntToStr( intPrec ) + "f").c_str();

Не катит. Таже фигня.
Может это уже особенности BDS, но факт остается фактом.

Цитата(TIGERоX @ 6.1.2006, 22:12 )

используя ансистринг

Такой вариант работает, но не хочу я каждый раз вызывать лишнюю функцию.
В общем, сделал вот так:

Код

*Prec = * ("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f").c_str();

Добавлено @ 22:39
Блин, а так AccessViolation выкидывает
Добавлено @ 22:51

Код

Prec = ("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f").c_str();

- Компилится, но когда подставляю формат, появляются какие-то квадраты

НУ МАТЬ МОЯ ЖЕНЩИНА!..

Код

Prec = ("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f").c_str();

Нельзя, блин, так, блин, делать... блин... ибо "("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f")" это временный объект... Нельзя взять его char*-представление, и потом использовать этот адрес для инициализации указателя... Временный объект будет разрушен после выполнения этой строки, и по указателю фиг знает что будет лежать, блин.

А ты весь код приведи....вот и разберёмся...

Цитата(BreakPointMAN @ 6.1.2006, 23:14 )

Нельзя, блин, так, блин, делать... блин... ибо "("%." + IntToStr( PrecComboBox -> ItemIndex ) + "f")" это временный объект... Нельзя взять его char*-представление, и потом использовать этот адрес для инициализации указателя... Временный объект будет разрушен после выполнения этой строки, и по указателю фиг знает что будет лежать, блин.

Сразу надо было говорить

Сделал так:

Код

AnsiString PrecStr = "%." + IntToStr( PrecComboBox -> ItemIndex ) + "f";  Prec = PrecStr.c_str();

Так, вроде бы со всем разобрался...
Но есть такой глюк.
Есть несколько вот таких функций, висящих на разных кнопках и изменяющих разные Edit'ы. (*)

Код

void __fastcall TForm2::ExpBtnClick(TObject *Sender) {  try   {  ExpEditRes -> Text = AnsiString().sprintf( Prec, (double) Exp( StrToFloat( ExpEditPower -> Text ) ) );  }  catch ( Exception& )   {  Application -> MessageBox( "Слишком большая степень." , "Ошибка" , MB_OK );  } }

Есть список, из 9 элементов, котоый дает уже знакомую точность. Вот у него идет обраюотка на OnChange.
Вот функция

Код

void __fastcall TForm2::PrecComboBoxChange(TObject *Sender) {  AnsiString PrecStr = "%0." + IntToStr( PrecComboBox -> ItemIndex ) + "f";  Prec = PrecStr.c_str();  Form2->Caption = Prec;  AngleBtnClick( Sender );  LogBtnClick( Sender );  ExpBtnClick( Sender ); }

Как видно, она вызывает вот те самые функции (*)
Так вот, когда я выбираю, все значения в этих функциях изменяют значения при выводе на новую точность. Все это работает.
Но, вот если я выбрал точность, начиная с 4 ItemIndex или %0.3f, то если нажать на кнопку, все поля, которые должны пересчитаться - обнуляются. Why?
Внутри функций Prec используется только как:

Код

LogEdit10 -> Text = AnsiString().sprintf( Prec, (double) Log10( x ) );

Т.е. Prec не ПЕРЕОПРЕДЕЛЯЕТСЯ в них.

странно. по коду граблей не видно... нужно отладчиком походить.

а что будет если так:
LogEdit10 -> Text = AnsiString().sprintf("%0.3f", (double) Log10( x ) );
написать?

Так все остается, как надо.
Странно, но Prec изменяет свое значение при клике на кнопки. Почему? Фиг знает, ведь она там не изменяется.

Я так чувствую, что где-то я очень глупо указатель использую.
Вот прикладываю исходник.

Это сообщение отредактировал(а) Mal Hack - 10.1.2006, 19:01

Присоединённый файл ( Кол-во скачиваний: 3 )
 Unit2.cpp 10,54 Kb