Коррекция перевода float > char[]

@insideone · Регистрация: 10.01.2010

Вечер добрый! Задача такова - из float'а получить строку, но после запятой нужно оставить N знаков. Собственно я вывожу FPS на экран если конкретнее, а так в общем и для других целей сгодится.
Делаю так:

C++

#define DOT_DIG_NUM 2
#define TEMP_SIZE 30
qString::qString(float newData){
    char newCharData[TEMP_SIZE] = {0};
/// start
    gcvt(newData, 1 + newData/10.0 + DOT_DIG_NUM, &newCharData[0]);
    int dot_pos = 0; while(newCharData[dot_pos] != '.') dot_pos++;
    if ( dot_pos >= TEMP_SIZE ) dot_pos = 0;
    char* AfterDot = &newCharData[dot_pos+1];
    for (int i = 0; i < DOT_DIG_NUM; i++)
    {
        if ( AfterDot[i] == 0 )
            AfterDot[i] = '0';
    }
    AfterDot[DOT_DIG_NUM] = 0;
/// end
    _new(&newCharData[0]);
}

При этом коде тут

C++

void qString::_new (const char* newData){
    if ( newData != NULL )
    {
        // Выделение памяти под полученную строку
        _size = strlen(newData) + 1; // тут

вылетает

Необработанное исключение в "0x00479982" в "WFAE.exe": 0xC0000005: Нарушение прав доступа при записи "0x00003034".

(адрес this - 0x00003030)
Впрочем ошибка не важна, хочется хороший алгоритм перевода написать, видно тот барахлит

А без него все работает отлично

@Nick Alte · 03.03.2010, 20:25

А чем не угодили sprintf и std::strstream?

@insideone · 03.03.2010, 20:40 **[ТС]**

Нужно быстро и безопасно

И вообще я не люблю пушкой по воробьям...

@easybudda · 04.03.2010, 01:12

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

Нужно быстро и безопасно

Сообщение от Nick Alte

А чем не угодили sprintf и std::strstream?

И в правду - уж куда быстрей и безопаснее? Я бы тоже примерно так сделал:

C

#include <stdio.h>
 
#define LEN 32
 
int main(void){
    int n;
    char template[LEN] = { 0 }, strval[LEN] = { 0 };
    double dval;
    
    printf("Double value: ");
    scanf("%lf", &dval);
    printf("Digits after point: ");
    scanf("%d", &n);
    
    snprintf(template, LEN - 1, "%%.%df", n);
    snprintf(strval, LEN - 1, template, dval);
    printf("As string: %s\n", strval);
    
    return 0;
}

Добавлено через 29 минут
вариант C++

C++

#include <iostream>
#include <sstream>
#include <string>
 
int main(){
    double dval;
    int n;
    std::string str;
    
    std::cout << "Double value: ";
    std::cin >> dval;
    std::cout << "Digits after point: ";
    std::cin >> n;
    
    std::stringstream ss;
    ss.setf(std::ios::fixed, std::ios::floatfield);
    ss.precision(n);
    ss << dval;
    ss >> str;
    
    std::cout << "As string: " << str << std::endl;
    
    return 0;
}

@insideone · 04.03.2010, 01:25 **[ТС]**

2 easybudda: Если я не ошибаюсь достаточно 2 строчек

По крайне мере это у меня работает верно

C++

1 2	char newCharData[TEMP_SIZE] = {0}; sprintf(&newCharData[0], "%." DOT_DIG_NUM "f", newData);

Быстра ли универсальная функция? Точно не быстрее узконаправленной (если качество кода одинаково). Про ненадежность просто читал где то и не раз. И то и другое важно - приложение игровое. Вернее даже... конечно безопасность вопрос сильный, и все же сейчас на первом плане скорость. К тому же мне просто интересно реализовать этот алгоритм без применения сторонних функций... Впринципе я то его и реализовал, но почему то ошибки вылетают

Не по теме:

А вообще чисто на подсознательном уровне эти printf функции кажутся консольными и мозолят глаза :( Интересно разработчики передовых трехмерных игр тоже их используют? :-D

Добавлено через 6 минут
Хм, вариант с std заинтересовал конечно, хотя вопрос... а накладные расходы на конструкцию

C++

1	std::stringstream ss;

и её убиение не убьют ли всю скорость? По крайне мере безопасности std я доверяю...

@CyBOSSeR · 04.03.2010, 03:11

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

C++

// ...
  ss.setf(std::ios::fixed, std::ios::floatfield);
  ss.precision(n);
  ss << dval;
// ...

Этот фрагмент можно переписать чуть проще:

C++

#include <iomanip>
// ...
  ss << std::setprecision(n) << std::fixed << dval;
// ...

@easybudda · 04.03.2010, 08:56

insideone, кстати, вот Вам ещё вариант. Не знаю, на сколько быстрый, крайне не устойчивый к плохим данным, но может, как идея, пригодится...

C

#include <stdio.h>
#include <math.h>
 
#define LEN 32
 
char *strReverse(char *s){
    char *h, *t, c;
    for ( t = s; *t; ++t )
        ;
    for ( h = s, t -= 1; h < t; ++h, --t ){
        c = *h;
        *h = *t;
        *t = c;
    }
    return s;
}
 
char *doubleToStr(double d, int n, char *s){
    char *p = s;
    long num = (long)( d * pow(10, n) );
    while ( n-- ){
        *p++ = num % 10 + '0';
        num /= 10;
    }
    *p++ = '.';
    do {
        *p++ = num % 10 + '0';
        num /= 10;
    } while ( num );
    *p = 0;
    return strReverse(s);
}
 
int main(void){
    int n;
    double dval;
    char buf[LEN];
    
    printf("Double value: ");
    scanf("%lf", &dval);
    printf("Digits after point: ");
    scanf("%d", &n);
    
    printf("As string: %s\n", doubleToStr(dval, n, buf));
    
    return 0;
}

@insideone · 04.03.2010, 16:03 **[ТС]**

Спасибо, становился на

C++

long qString::_pow(unsigned char x , unsigned char n){
    long res = 1;
    while ( n-- ) res *= x;
return res;
}
 
char *qString::str_reverse(char *s){
    char *h, *t, c;
    for ( t = s; *t; ++t );
    for ( h = s, t -= 1; h < t; ++h, --t )
    {
        c = *h;
        *h = *t;
        *t = c;
    }
return s;
}
 
char* qString::_long2str(long Number, char* Dest, bool AddDot){
    do {
        *Dest++ = Number % 10 + '0';
        Number /= 10;
    }
    while ( Number );
    if ( AddDot ) *Dest++ = '.';
return Dest;
}
 
char* qString::_float2str(float Number, unsigned char DotDig, char* Dest){
    long int_part = Number;
    long rest_part = (Number - int_part) * _pow(10, DotDig);
    _long2str(int_part, _long2str(rest_part, Dest, true));
return str_reverse(Dest);
}
 
qString::qString(float newData){
    char newCharData[TEMP_SIZE] = {0};
    _float2str(newData, 2, &newCharData[0]);
    _new(&newCharData[0]);
}

Evg · 04.03.2010, 16:29

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

Быстра ли универсальная функция?

А зачем тебе в данном месте скорость? Ты же это значение так или иначе собираешься печатать на экран, всё равно это место (печать на экран) у тебя отожрёт основное время. Так стОит ли экономить на спичках?

@insideone · 04.03.2010, 17:11 **[ТС]**

И вправду

Я не думал в таком ракурсе... Впрочем на рендере не сэкономить особо, приходится заботится об остальном

@Nick Alte 1661 / 1033 / 174 Регистрация: 27.09.2009 Сообщений: 1,945
	03.03.2010, 20:25	2
	А чем не угодили sprintf и std::strstream? 0

@insideone Модератор 3682 / 959 / 114 Регистрация: 10.01.2010 Сообщений: 2,550
	03.03.2010, 20:40 [ТС]	3
	Нужно быстро и безопасно И вообще я не люблю пушкой по воробьям... 0

Evg 21139 / 8155 / 628 Регистрация: 30.03.2009 Сообщений: 22,465 Записей в блоге: 30
	04.03.2010, 16:29	9
	Сообщение от insideone Быстра ли универсальная функция? А зачем тебе в данном месте скорость? Ты же это значение так или иначе собираешься печатать на экран, всё равно это место (печать на экран) у тебя отожрёт основное время. Так стОит ли экономить на спичках? 1

@insideone Модератор 3682 / 959 / 114 Регистрация: 10.01.2010 Сообщений: 2,550
	04.03.2010, 17:11 [ТС]	10
	И вправду Я не думал в таком ракурсе... Впрочем на рендере не сэкономить особо, приходится заботится об остальном 0

Коррекция перевода float > char[]

Решение