Thread: trouble with classes.

edit: actually I just figured it out. The problem was the struct. apparently it can't go there. New question. How can I use a struct in a class? Or should I just make a whole new class for that data?

edit again: Well, I feel special. I just forgot a semi-colon after the struct. So, I answered my own questions. I may have more later though!

Ok, So this isn't specific to classes, but since I have this thread open I'll go ahead and post it in here

I have a float being passed into a function. The float is a combination of two numbers, the first being year, second being date. I have it set up to be year.date, so say 1998.07 would be july 1998.
I need some help seperating the two though(if it's possible) This is what I have

Code:

    float year, month;
    year = yearMonth/1;
    month = yearMonth % year;

where yearMonth is being passed into the function as something like 1998.07.
the error is as follows

Code:

136 C:\Dev-Cpp\add_data.cpp      invalid operands of types `float' and `float' to binary `operator%'

Keep in mind that I'm getting this yearMonth number from inside of a different function, so I can't pass it as two seperate numbers(not that I can see anyway).
Any help is appreciated!


edit:
Also:
I'm trying to save the info that I get to a file. The file should be named by the number of the month in a folder by year.

Code:

dataFile.open("C:\\"year "\\"month".dat", ios::out);

gives the error

Code:

207 C:\Dev-Cpp\add_data.cpp     invalid operands of types `const char[4]' and `float' to binary `operator+'

I don't get it, is there a way to have a dynamic file out location?

Code:

dataFile.open( ("C:\\" + year + "\\" + month + ".dat").c_str() , ios::out);

assuming a std::string ... you may want to use some string streams too ... are year and month numbers? You'll have to convert them to std:: strings search around for functions to do it.

try doubles for your other problem

I'm not sure why you want dates in that format... but regardless of that:

Code:

year = static_cast<int>(YearMonth);
month = static_cast<int>(YearMonth - static_cast<int>(YearMonth));

The error you are getting, btw, comes from the fact you can only use the modulos operator with integrals.

EDIT: Just noticed something...

You are declaring year and month as floats. I suppose you don't want that. They are integrals. Integers... at most long integers, if you really want to work that far in the future

Change those declarations to int.

Originally Posted by System_159

Could I use pointers to store the values of year and month in a struct somewhere? We haven't gone over pointers in class yet, but I've read over the tutorial a few times and I think that's what they're about.

See my previous reply to fix your problem.

Regardless, pointer have other uses. Although, invariably lead to acessing a variable value. It's just that pointers would be an unecessary complication from what I can see of what you are trying to achieve.

You can use a class or struct though. But since you haven't gone through those also, it's best we leave it alone for now.

ok, I found a way around my problem, but of course ran into another one.
The solution that I came up with was to use stringstreams to turn my float into a string. then string.erase() to seperate it into two different strings. here's an example of what I mean:

Code:

#include <iostream>
#include <fstream>
#include <iomanip>
#include <string>
#include <sstream>

using namespace std;

int main()
{
    string syear, smonth;
    string syearMonth = "1998.06";
    syear = syearMonth;
    smonth = syearMonth;
    syear.erase(4);
    smonth.erase(1,5);
    cout << syear << endl;
    cout << smonth << endl;
    system("pause");
    return(0);
}

The problem I run into is that the zero comes back as a 1. Could anybody tell me why this is?

edit: Mario, I just realized why the code you gave earlier didn't work for me, you had YearMonth, where I had yearMonth. So it seems I don't need the above code. But it still makes me wonder why that happened, if anybody knows. Thanks for all the help and suggestions so far!

My gosh, sorry to keep editing, it surely makes for a mess, but I just had a problem with your tidbit Mario. Since I'm now using stringstreams and not strings I get this error:

Code:

214 C:\Dev-Cpp\add_data.cpp      no match for 'operator+' in '"C:\\" + yearString'

> edit: Mario, I just realized why the code you gave earlier didn't work for me, you had YearMonth, where I had yearMonth.

Well other than that, there is somehing else fundamentally wrong with my code. If you try it with the variable renamed correctly it will give you 0 for the month. The formula is wrong because it's not taking into account the common logarithm for the extracted decimal.

The correct formula would then be:

Code:

year = static_cast<int>(YearMonth);
month = static_cast<int>((YearMonth - static_cast<int>(YearMonth)) * 100);

But then as anon pointed out you will still have a problem. Decimal types are not geared towards precision. On my compiler, for instance, a float declared as 1874.07 becomes 1974.06995... and this is not good. Your month would be 6, instead of 7.

So, if you really insist in having dates stored like that, stringstreams is indeed the way to go.

setprecision. It's a manipulator declared in the iomanip header.

Well, no. Manipulators work only with the iostream objects (cin and cout). setprecision() works only with std::cout.