Есть число 1000000, нужно его отформатировать в 1,000,000

В общем, нужно создать аналог функции number_format (PHP)

Я написал что-то близкое (и оно даже работает), но нужно её максимально упростить (самое главное сделать БЕЗ рекурсии, как в примере)
или это самый оптимальный вариант


Может есть готовые решения на С++ (с использованием STL/CRT или ещё чего, только без MFC)


Это сообщение отредактировал(а) suxxor - 27.7.2011, 23:36

Добавлено @ 01:51
ну и перевернуть строку

Это сообщение отредактировал(а) baldina - 28.7.2011, 02:20

а зачем такое условие, вроде достаточно


Цикл лучше сделать 
do { 
  ... 
} while (lNumber);
а то 0 не выведет.

И в самом конце строку не забыть терминировать.
szBuffer[iCnt] = 0;

О, посоны, я вас люблю!

Сделал так, вроде всё правильно, если чё не так укажите!


Это сообщение отредактировал(а) suxxor - 28.7.2011, 06:30

1.

сравнивать ULONGLONG на предмет меньше ли оно нуля не имеет смысла, а сам нуль чем не число ???

2.

это непринципиально, но так будет правильней.
3.
И потом, я бы этот знак передавал бы в параметрах, а по-умолчанию сделал бы ',':

а вот рабочий вариант не привязанный к windows'у
http://liveworkspace.org/code/67370a62a951...ef8428df42ea5c2

stringstream + locale и не надо никаких велосипедных циклов, вы чего.

Хммм.... пример приведи

borisbn, спасибо

а как поступать с отрицательными числами?  if(lNumber < 0) тогда сохраняем значение отрицательности и на выходе прибавляем к началу szBuffer минус, как это реализовать?

Это сообщение отредактировал(а) suxxor - 28.7.2011, 11:45

suxxor, http://stackoverflow.com/questions/4163243...-separator-in-c читани для разгона.

Добавлено через 12 минут и 3 секунды
suxxor, а потом это http://blog.p-jansson.com/2011/02/grouping...rator-in-c.html - более гибкий подход.

Вот еще текст: http://en.allexperts.com/q/C-1040/2008/8/T...d-Separator.htm Хотя, тут более общая информация про всю эту байду.

spyswamp, locale выдаст формат зависимый от настроек в ОС, если я не ошибаюсь

suxxor, ты инфу по ссылкам почитал (в особенность по второй)?

да короче, трудно мне с шаблонами STL работать, очень трудно ;( 
По мне лучше этот вариант

Это сообщение отредактировал(а) suxxor - 28.7.2011, 14:40

Если трудности вызваны не запретом начальства, то стоит взять книжку и почитать о STL. Нет в нем ничего сложного. Просто подход несколько неожиданный для классического программиста.

Это сообщение отредактировал(а) bsa - 29.7.2011, 11:38

bsa, проблем  начальством нет - потому что начальства нет... Учу по книге STL для профессионалов от Николая Джусьютиса, всё никак ниасилю

suxxor, я благодаря вам baldina, и borisbn, сохранил в свою личную библотеку тоже такую функцию.
Если, конечно не последуют претесты от вышеназванных кодеров по лицензионным соглашениям  

Если интересует заключительный вариант, то вот он.
Переносимый, универсальный на любой целочисленный тип (включая отрицательные числа).
Вывод тоже универсальный: как в юникод, так и в обычную строку.
Возвращает длину выведенной строки.

тестирование здесь:
http://liveworkspace.org/code/19f5212fafcc...0f437555a875e98

зы: если какие замечания по коду, велкам.

volatile, а чего б сразу не взять модуль

и избавиться в цикле от

?

P.S. Лицензия MIT 


Это сообщение отредактировал(а) borisbn - 2.8.2011, 08:25

А для кого писалось это?

Код - жесть. Уязвим и опасен, я бы сказал, как минимум. Ботай яростно STL, ибо это никогда ревью не пройдет. 

Я думал над этим. Тут есть одна маленькая зоковыка. 
Если число является самым маленьким отрицательным, то этот фокус не проходит.
например:
signed char c = -128;
c = -c;
здесь  с не станет +128 




не опасней обычных itoa(); sprintf(); и т.д.
но универсален, в отличии от последних.

volatile, именно поэтому означенные вами функции не рекомендованы к использованию и являются deprecated.

spyswamp, я прекрасно этими функциями пользуюсь, и никогда у меня лично с ними проблем не было.
этот код я писал не для ревью, а для личных нужд.
И меня в плане "безопасности" он вполне устраивает.

Напишите пример безопасной, но такой-же эффективной, удобной, и универсальной функции, я буду только рад.
  
Ваши ссылки на применение потоковых функций с применением locale я просмотрел.
Считаю что это ни универсально, ни удобно, ни эффективно. Имхо.

ЗЫ: Я вообще не очень люблю потоковый вывод (сейчас адепты строгого С++ забросают помидорами).

Это сообщение отредактировал(а) volatile - 2.8.2011, 13:32

borisbn : volatile 0 : 1
где тут смайлик с посыпанием головы пеплом ?

Ваша функция не эффективна, не удобна, не универсальна. Ее завалить можно в 1 прием извне. RTFM.

А по STL я скажу проще: если используете C++, то и пишите на C++ с использованием соответствующего функционала. До вас уже сотни людей все изучили до подноготной, обернули, написали и соптимизировали. Не надо тут заливать всем в уши ваш бред про любовь и нелюбовь.

Дальнейшую дискуссию с вами продолжать не вижу смысла. У вас даже "имхо" неумное оказалось.

Кем ? M$-ом ? Если бы мы их слушали, то мы (программисты) каждый год меняли бы технологии (COM, ActiveX, MTS, .Net, Silverlight, etc. - всех не упомнишь).




поддерживаю. 

spyswamp, запиши это

в Си++ с потоками

borisbn, по printf() и ее безопасности читани интернеты, а потом уже поддерживай.

По второму вопросу: ты реально думаешь, что я не смогу на потоках это сделать? 

Добавлено через 9 минут и 40 секунд
Это если ты скажешь "да":



Debug: http://dl.dropbox.com/u/943861/projects/tr...f-vs-stream.png

А вообще для таких вещей уже давно есть boost::format().

spyswamp, есть у меня знакомый электрик, некоторые вещи иногда он делает не по правилам, не отключая напряжение. это не значит, что напряжение безопасно. но это не значит что он неосторожен: как раз он достаточно осторожен, что бы иногда (когда это эффективнее) работать так.
а потоки в STL не верх совершенства. хотя бы потому, что непросто создать универсальный интерфейс. было б не так, не появились бы подпорки boost::lexical_cast и boost::format.

volatile, borisbn, а вот форматирование чисел с разделителями в STL делается таки довольно изящно и удобно.

вобщем хорошш спорить   

Да, изящно!   

volatile, не вырывай из контекста. borisbn попросил - я написал. И будьте внимательнее, товарищ велосипедист: "А вообще для таких вещей уже давно есть boost::format()."

не говори мне, что мне делать, а я не скажу (дальше - в интернетах)


я, наверное, неправильно выразился. я не имел в виду, что этого сделать нельзя. я и сам могу это сделать. я как раз "поддерживал" то, что это будет

как, собственно и вышло.


1. Попридержи язык с оскорблениями
2. У меня, в отличие от тебя, есть необходимость собирать проект на многих компьютерах, а не на одном своём рабочем. Ради простого проекта по тому же самому переводу широты/долготы в строку я буст подключать (а главное такскать его за собой) не собираюсь

Это сообщение отредактировал(а) borisbn - 2.8.2011, 16:25

borisbn, у меня проекты собираются под 4 версии винды и 2 дистрибутива linux. А все любители printf() и raw-указателей уже давно кастрированы. Чем еще хочешь померяться?

Сыроежка, ты где? ))  

Так ребята, давайте на этом остановимся. Честное слово надоело.

spyswamp, вашу точку зрения я понял.

Мне просто нужна такая функция для того чтобы можно было применять форматный вывод, без привлечения потоков.
Часто пишу небольшие утилитки вообще без <iostream> и иже с ним. Утилитки получаются небольшими, быстрыми, и удобными в написании.

Суть сводится к тому применять ли вообще в С++ функции подобные sprinyf ();
Я их применяю, и в обозримом будущем отказываться от них не собираюсь.
Вы их опасаетесь. Это ваше право, и вашу точку зрения я понимаю (и уважаю).
Всё.

  

volatile, *handshake* Пожалуй, да. Не будем друг другу настроение портить.  

Всё. Не соримся. Я не буду огульно говорить, что стримы плохо ( только с доказательством  )... Мало того, можно привести множество примеров, где без них просто не обойтись. Никак. Но и Вы должны признать, что 1) если знаешь, что делаешь 2) протестировал на куче вариантах 3) требуется быстродействие, то в замене стримов "велосипедами" типа sprintf нет ничего предосудительного.

Короче, если обходиться без оскорблений, то высказывание своей точки зрения, а главное - выслушивание чужой - это не ссора, а спор. А в споре, как известно...

Ещё короче, я, также как и Вы, против ссор, но за спор 
*handshake* ?

borisbn, sure yeah! *handshake*

Ну вот и славно...
В качестве мишени для насмешек и придирок выставляю свой старый код.
Там еще и гото используется. (действует как красная тряпка на многих).

Ох, боюсь низко пасть в глазах общественности...

Ну да и ладно, щаз покажу, чего уж там. 

Компилить под студией.
u64 - это unsigned __int64


Ревнители стогого С++ и знатоки стандартов, можете забросать меня тухлыми яйцами. Я не обижусь.
Даже боюсь представить сколько пунктов стандарта здесь нарушено...

В качестве небольшого оправдания (если вообще такое возможно оправдать!) могу сказать,
что код написан очень давно, работал долго и никогда не подводил.

в прямых руках и кремниевые ножи рулят)))