из одной строки перекопировать в другую только цифры

[KAMLS]

Да, я её уже смотрел, но ведь там еще точку надо пропустить кроме цифр.

[Alex11223]

isdigit(ch) || isточка(ch)

[KAMLS]

Вот при такой компоновке кода компилятор ошибок не показывает. Но при исполнении выдаёт ошибку.

#include<iostream>
#include<conio.h>
#include<cstdlib>
#include<math.h>
#include<iomanip>
using namespace std;
class money
{
private:
	int a;
	long double summa;
	char str [23];
public:
	money (): summa(0), a(0)
	{}
	long double mstold()
	{
		char str1[23];
		cout<<"Введите денежную строку: \n";
		cin.get(str1, 23);
		for(int j=0; j<23; j++)
		{
			if ( isdigit (str1[j])==1 || str1[j]=='.') // если перые символы строки str - число
			{
				str[a]=str1[j];
				a++;
			}
			
		}
		str[a]='\0';
		long double summa=atof(str);
		return summa;	
	}
};
int main()
{
	setlocale(LC_ALL,"rus");
	long double b;
	money m1;
	b=m1.mstold();
	cout<<setiosflags(ios::fixed)
		<<setiosflags(ios::showpoint)
		<<setprecision(2)
		<<b;
	getch();
	return 0;
}

Подскажите пожалуйста что не так?

[Alex11223]

Так какую ошибку-то?

И == 1 не надо.
Во-первых в С/С++ и так можно писать if (1), if (2), if (!0) и т.п. вместо if (true)
Во-вторых isdigit возвращает "A value different from zero" когда цифра. Это не обязательно 1.

[KAMLS]

Если вам не трудно, попытайтесь запустить программу у себя. скриншот ошибки прикреплен.

[Alex11223]

Да должно работать по идее, ну только ==1 убрать, иначе результат может быть неправильным.
http://rextester.com/JAQA86161

Наверно вы что-то странное вводите/передаете, добавьте вывод str1[j] перед проверкой.

UPD: а, так а зачем цикл до 23? Надо до strlen.

[KAMLS]

Да, с циклом согласен, но 23 это от начала да так и осталось, тут за идею борьба идет))
Скопировал ваш код, в отдельной программе его построил и запустил, работает. Только содержимое массива сделал $333.34
Выводит 333.34 т.е. как задумано.
Скорректировал свою программу, т.е. сделал копию вашей, выводит
333.00
Почему - ваще не втыкаю!!!
Полная копия! Только мне getch(); ставить приходится, иначе ничего увидеть не успеваю.
А вот когда программа не скорректирована, т.е. когда надо вводить то же самое $333.34 выдает показанную ошибку. т.е. компилятор молчит, а ошибка вылетает в процессе исполнения.
Такие дела.

#include<iostream>
#include<conio.h>
#include<cstdlib>
#include<math.h>
#include<iomanip>
using namespace std;
class money
{
private:
	int a;
	long double summa;
	char str [23];
public:
	money (): summa(0), a(0)
	{}
	long double mstold()
	{
		char str1[23]="$333.34";
		for(int j=0; j<23; j++)
		{
			if ( isdigit (str1[j]) || str1[j]=='.') // если перые символы строки str - число
			{
				str[a]=str1[j];
				a++;
			}
			
		}
		str[a]='\0';
		long double summa=atof(str);
		return summa;	
	}
};
int main()
{
	setlocale(LC_ALL,"rus");
	long double b;
	money m1;
	b=m1.mstold();
	cout<<setiosflags(ios::fixed)
		<<setiosflags(ios::showpoint)
		<<setprecision(2)
		<<b;
	getch();
	return 0;
}

Вот моя скорректированная программа, которая вместо 333.34 выдает 333.00
А в не скорректированной ==1 убрал, толку нет также. Может VS 2088 барахлит?

[KAMLS]

Такие странные дела..
Убрал в своей скорректированной программе setlocale(LC_ALL,"rus");
и стало срабатывать как надо 333.34!!!
При чем тут setlocale(LC_ALL,"rus");?!!

[Alex11223]

Так я ж сказал почему. Цикл идет дальше конца нормального содержимого строки и читает всякий мусор, на который срабатывает какая-то отладочная проверка корректности переданного символа внутри isdigit.

А при

char str1[23]="$333.34";

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

int arr[23] = { 41, 42 };

[KAMLS]

Переделал j<23 на j<strlen(str1) и стало работать с вводом значения а не с установленным в программе.
Но осталась блин проблема с setlocale(LC_ALL,"rus"); пишет .00 вместо .34
Подскажите пожалуйста почему?