Задача по книге Р. Лафоре

@UMD · Регистрация: 22.02.2015

Пример по книге Лафоре... (Система Windows 10/64bit.)
Я знаю что на форуме есть решения но я хочу своим умом понять и решить, к тому же код там написан извольте промолчать... .
Я лишь хочу чтобы мне указали на мои ошибки!
Кстати как я понял из задания проблему нужно решить именно как массив символов, в моем понимении это char[]...
(... Вам нужно будет обработать денежную строку как массив символов)

Проблема 1:
В main() он придерается к этому куску кода:

C++

1	while ((ch != 'n') \|\| (ch != 'N'))

после ввода 'n' или 'N' или любого другого сивола зациклевается все...
если ввести так:

C++

1	while ((ch != 'n') && (ch != 'N'))

выход осуществляется но повторить цикл невозможно, после ввода любого другого символа зациклевается все...

Проблема 2:
В этом куске не происходит копирование из char в char.

C++

1 2	if ((strmoney[i] >= '49' && strmoney[i] <= '57') \|\| strmoney[i] == '.') strtemp[i] = strmoney[i];

переменная

C++

1	strtemp[i];

выводит просто мусор: MMMMMMMMMMMMMMMMMMMMMMMMMM... .

Проблема 3:
в функции

C++

1	atof(strmoney)

нет вещественной точки, если есть запятая, то на его место выводится точка.
Например, ввод: 123,465,789.25
Вывод: 123.465
Если без запятых...
Например, ввод: 123465789.25
Вывод: 123465789

вот собственно и код:

C++

#include <iostream>
#include <cstdlib>
using namespace std;
//================================================================
class money
{
private:
    double Dmoney;
public:
    money() : Dmoney(0)
    {}
    money(double Dm) : Dmoney(Dm)
    {}
    double MS_told(char[]);
};
//================================================================
const int SIZE = 22;
//================================================================
double money::MS_told(char strmoney[])
{
    double msmoney = 0.0;
    char strtemp[SIZE];
    for (int i = 0; i < strlen(strmoney); i++)
    {
        if ((strmoney[i] >= '49' && strmoney[i] <= '57') || strmoney[i] == '.')
            strtemp[i] = strmoney[i];
        strtemp[i] = '\0';
    }
    msmoney = atof(strmoney);
        // Вывод для проверки...
    cout << "MS_told strmoney = " << strmoney << endl;
    cout << "strtemp = " << strtemp << endl;
    cout << "msmoney = " << msmoney << endl;
 
    return msmoney;
}
//================================================================
int main()
{
    setlocale(LC_ALL, "RUSSIAN");
    char strmoney[SIZE], ch = NULL;
    double Mmoney = 0.0;
    money m;
    cout << "Программа перевода денежной строки из char в double." << endl;
    while ((ch != 'n') || (ch != 'N'))
    {
        cout << "Введите денежную строку: "; cin.get(strmoney, SIZE);
        Mmoney = m.MS_told(strmoney);
        cout << "Результат вывода программы: " << Mmoney << endl;
        cout << "Продолжить ввод данных(y/n)?: "; cin >> ch;
    }
 
    cin.get();
    return 0;
}

@TheCalligrapher · 24.05.2019, 18:52

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

(strmoney[i] >= '49' && strmoney[i] <= '57')

Что такое '49' и '57'? Что вы хотели сказать этим синтаксисом?

@UMD · 24.05.2019, 19:27 **[ТС]**

это код ASCII. где '49' это цифта 0, соответственно '57' это 9
тоесть нужно скопировать в локальную переменную только цифры.
Для дальнейшей перечи в msmoney типа double.

Добавлено через 6 минут

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

msmoney = atof(strmoney);

Произошла очепятка

msmoney = atof(strtemp); правельней

@proc3nt · 25.05.2019, 14:14

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

for (int i = 0; i < strlen(strmoney); i++)

чтобы использовать функцию strlen( ) нужно подключить директиву препроцессора в начале программы..

C++

1	#include <string.h>

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

if ((strmoney[i] >= '49' && strmoney[i] <= '57') || strmoney[i] == '.')
strtemp[i] = strmoney[i];
strtemp[i] = '\0';

здесь зачем ты заполняешь буквально все элементы массива strtemp[ ] значением нуль терминала!? т.к. это присвоение произойдет при любом раскладе выполнения или не выполнения кода в инструкции if.. лучше напиши после самого блока кода for строку..

C++

1	strtemp[strlen(strmoney)]='\0';

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

if ((strmoney[i] >= '49' && strmoney[i] <= '57') || strmoney[i] == '.')

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

это код ASCII. где '49' это цифта 0, соответственно '57' это 9

имхо, здесь у тебя используются не ASCII коды, а строки из двух символов-цифр.. если надо ASCII коды цифр, то так прямо и указывай..

C++

1	if ((strmoney[i] >= '0' && strmoney[i] <= '9') \|\| strmoney[i] == '.')

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

C++

1 2	strtemp[j]=strmoney[i]; j++;

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

while ((ch != 'n') || (ch != 'N'))

такое условие в корне неверное, т.к. оно несовместимое.. лучше использовать в конце блока кода while условие if..

C++

if(ch=='n' || ch=='N')
{
    break;
}
else if(ch=='y')
{
    continue;
}
else
{
    break;
}

но суть дела не в этом.. тебе требуется очищать буфер перед вводом новой строки, т.к. после ввода символа ch и нажатия клавиши Enter в буфере будет находится символ переноса строки "\n", который будет автоматически записываться в массив strmoney..

C++

1	cin.ignore()

вобщем вот весь измененный код.. желаю успехов в его заборе (тьфу-ты.. разборе)..

C++

#include <iostream>
#include <string.h>
#include <cstdlib>
using namespace std;
//================================================================
class money
{
private:
    double Dmoney;
public:
    money() : Dmoney(0)
    {}
    money(double Dm) : Dmoney(Dm)
    {}
    double MS_told(char[]);
};
//================================================================
const int SIZE = 22;
//================================================================
double money::MS_told(char strmoney[])
{
    double msmoney = 0.0;
    char strtemp[SIZE];
    for (int i=0,j=0;i<(int)strlen(strmoney);i++)
    {
        if ((strmoney[i] >= '0' && strmoney[i] <= '9') || strmoney[i] == '.')
        {
            strtemp[j]=strmoney[i];
            j++;
        }
    }
    strtemp[strlen(strmoney)]='\0';
 
    msmoney = atof(strtemp);
        // Вывод для проверки...
    cout << "MS_told strmoney = " << strmoney << endl;
    cout << "strtemp = " << strtemp << endl;
    cout << "msmoney = " << msmoney << endl;
 
    return msmoney;
}
//================================================================
int main()
{
    setlocale(LC_ALL, "RUSSIAN");
    char strmoney[SIZE], ch;
    int f=0;
    double Mmoney = 0.0;
    money m;
 
    cout<<"Программа перевода денежной строки из char в double." << endl;
 
    while(true)
    {
        cout<<"\nВведите денежную строку: ";
        if(f==0)
        {
            f=1;
        }
        else
        {
            cin.ignore();
        }
        cin.get(strmoney,SIZE);
        Mmoney=m.MS_told(strmoney);
        cout<<"Результат вывода программы: "<<Mmoney<<endl;
        cout<<"Продолжить ввод данных(y/n)?: ";
        cin>>ch;
 
        if(ch=='n' || ch=='N')
        {
            break;
        }
        else if(ch=='y')
        {
            continue;
        }
        else
        {
            break;
        }
    }
 
    return 0;
}

@UMD · 25.05.2019, 16:31 **[ТС]**

На сколько мне известно функция strlen(); в C++ помещена в пространство имен std;
Или как минимум в моем компиляторе, в заголовочном файле <iostream> уже включен <cstring>
В конечном счете я с вами согласен, лучше подключать <cstring>, так как не во всех компиляторах это реализовано.
(Хочу заметить, я не умничаю, просто читал об этом, если же ошибаюсь прошу наставить на путь истинный).

здесь я понял, так же понял про ASCII коды, и указал прямо цифры, спасибо!

C++

1
2
3

if ((strmoney[i] >= '0' && strmoney[i] <= '9') || strmoney[i] == '.') 
        strtemp[i] = strmoney[i]; 
strtemp[i + 1] = '\0';

что касается strtemp[strlen(strmoney)]='\0'; я реализовал так: strtemp[i + 1] = '\0'; тоже заработало
но ваш способ наглядней, и легко читаем, хоть и длиннее

здесь я сам не понял почему, но все заработало после ввода cin >> strmoney;

C++

    while (ch != 'n' && ch != 'N')
    {
        cout << "Введите денежную строку: "; cin >> strmoney;
        Mmoney = m.MS_told(strmoney);
        cout << "Результат вывода программы: " << Mmoney << endl;
        cout << "Продолжить ввод данных(y/n)?: "; cin >> ch;
    }

Вообщем спасибо вам большое за разъяснение, но есть одно НО...
Что в вашем примере, что в моем примере, функция atof(); не передает точку например:
Ввод: 123456.55
Вывод: 123456
Если есть запятая после нее(запятой) не выводится нечего
Ввод: 123,456.55
Вывод 123
Я склоняюсь к тому что фукнция atof() не работает правильно через фунцию MS_told();
Потому что когда я запускаю функцию atof() в main() пример:
Ввод: 123,456.78
Вывод 123.456
Если же ввести с точкой:
Ввод: 123.45
Вывод: 123
Я предполагаю это связано с setlocale(,,); но разобраться я не смог в этом вопросе.

@proc3nt · 26.05.2019, 09:43

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

здесь я понял

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

if ((strmoney[i] >= '0' && strmoney[i] <= '9') || strmoney[i] == '.')
strtemp[i] = strmoney[i];
strtemp[i + 1] = '\0';

по-моему, ты ничего не понял.. я же сказал ранее, что для массива strtemp должны идти свои индексы (например, j а не i)..
и присвоение нуль терминального символа должно происходить после всего цикла for, а не после if.. например, так..

C++

int j=0;
 
for (int i=0;i<(int)strlen(strmoney);i++)
{
    if ((strmoney[i] >= '0' && strmoney[i] <= '9') || strmoney[i] == '.')
    {
        strtemp[j]=strmoney[i];
        j++;
    }
}
 
strtemp[j]='\0';

и только тогда у тебя будут пропускаться буквы в исходной строке и записываться только цифры и точка во временную строку..
проверь свой код и мой.. замечаешь разницу!?

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

Что в вашем примере, что в моем примере, функция atof(); не передает точку

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

Я склоняюсь к тому что фукнция atof() не работает правильно через фунцию MS_told();

это не вина функции atof.. просто выводить значение msmoney нужно с помощью манипуляторов..

C++

1
2
3

#include <iomanip>
...
cout << setiosflags(ios::fixed)<<setprecision(8) << "msmoney = " << msmoney << endl;

смотри скриншот..

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

Если есть запятая после нее(запятой) не выводится нечего

а какой смысл тестирования программы с запятой, если она (запятая) не копируется во временную строку!? ведь именно временная строка преобразуется в число типа double.. да и вообще функция atof вроде работает только с точкой.. число может заканчиваться любым символом, который не может являться частью числа с пла*вающей точкой. например, этим символом может быть пробел, знак пунктуации, отличный от точки, буква, отличная от «Е» или «е»..

@TheCalligrapher · 27.05.2019, 22:14

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

это код ASCII. где '49' это цифта 0, соответственно '57' это 9

Код ASCII для '0' и '9' - это 49 и 57. У вас же в коде написано '49' и '57'. Это никакие не "коды ASCII".

Задача по книге Р. Лафоре

Решение

Решение

@TheCalligrapher Вездепух 10820 / 5841 / 1585 Регистрация: 18.10.2014 Сообщений: 14,496
	24.05.2019, 18:52	2
	Сообщение от UMD (strmoney[i] >= '49' && strmoney[i] <= '57') Что такое `'49'` и `'57'`? Что вы хотели сказать этим синтаксисом? 0

@UMD 0 / 0 / 0 Регистрация: 22.02.2015 Сообщений: 9
	24.05.2019, 19:27 [ТС]	3
	это код ASCII. где '49' это цифта 0, соответственно '57' это 9 тоесть нужно скопировать в локальную переменную только цифры. Для дальнейшей перечи в msmoney типа double. Добавлено через 6 минут Сообщение от UMD msmoney = atof(strmoney); Произошла очепятка msmoney = atof(strtemp); правельней 0

@TheCalligrapher Вездепух 10820 / 5841 / 1585 Регистрация: 18.10.2014 Сообщений: 14,496
	27.05.2019, 22:14	7
	Сообщение от UMD это код ASCII. где '49' это цифта 0, соответственно '57' это 9 Код ASCII для `'0'` и `'9'` - это `49` и `57`. У вас же в коде написано `'49'` и `'57'`. Это никакие не "коды ASCII". 0