Posts by thomas_naveen

You are passing by value. Try passing by reference.

nested switch?

Kindly let us know whether your issue is resolved?

If so, mark the thread as solved.

Think about the flow.

How was the goto working?

Does the while loop take the flow the same way?

You need the flow to reach back to the cin.

For that you need to include the cin also in the loop.

You are getting these linker errors, since the constructors given in the error cannot be found by the linker.

Try implementing them.

Credit(int, double, double, string);
Saving(int, double, double);
Checking(int, double, double, int);

Memory should be allocated before you use the pointer for storing data.

In the current code you are taking in data to a pointer which could be pointing anywhere and then pointing that pointer to a fresh memory location.

You lose the input data in this way.

Make the 7th line your 5th line and check out the results.

int days[12]={31,28,31,30,31,30,31,31,30,31,30,31};

Try with the above modified days array.

void move_left();
void move_mid();
void move_right();

Put the three lines at the top of the file or in a header file, move.h

and say

#include "move.h"

at the top of the file

I am assuming that move_right is aslo visible for compilation.

You need to define the functions at the top of the file
or

in a header file and then include the header file.

If it is a csv file, I believe you can read it just like a text file.

Try allocating memory to your dir variable. Similar to the command variable.

28-31

void setY(int n)
           {            
                    x = n;
           }

This should be

void setY(int n)
           {            
                    y = n;
           }

Please check for more errors.

fmaxf for floats.

fmax takes only two variables as input.

So you can only use fmax(a,b)

For three variables a,b,c
use something like

d = fmax(a,b);

e = fmax(c,d)

e will be the max of a,b,c

You are expecting smallResult to be populated by the sumOdd function without calling the function.

For the code in sumOdd to be executed , you need to call the sumOdd function from the main function.

Better check out line 98 as well. It looks very similar to the bug you have identified.

Take alook at the strcpy function.

The C string will have a terminating null character.

In the for loop,

for (indexer = 0;others[indexer];indexer++)

when the index (indexer) reaches the index of this terminating null character , the loop will exit.

Please use the CODE tags properly.

double stands for double precision floating point number in C++

You have followed some thinking process to arrive at the answer, right?

Now convert this into pseudo code and finally code.

If you post atleast the pseudo code here, you will get some help.

Have you set the path variable?

You can debug build by setting the verbosity.

Check your compiler documentation.

You need to forward declare the Checking class.

Can you clearly and concisely state your problem?

Try setting the path too.

Check the size of the library on disk and whether you have the necessary permissions to use it?

You could consider strtok instead of sscanf.

I hope the large sized arrays are justified. You could end up corrupting your stack.

This works out to a negative overtime , right?

Try giving a hourly pay of 50.

I have seen this problem before of large arrays on stack.

Just declare it as a global.

Didn't you see the first reply to your post?

asctime returns a character pointer.

The currentTime function returns a character.

There is a mismatch here that needs to be fixed or taken into consideration.

Code & calculation looks fine.

NOTE: uniondue should be defined as a static const. (optional)

If deduction is greater than hours * rate, your overtime will be negative.

Can you post your inputs?

Please post your code.

Check out this tutorial.

I have no idea about any books on the subject.

However, several books have been listed in the reference section of the given tutorial

No. You will be calling seeding only after you have finished processing all your input parameters from command line.

Line 38 is outside the for loop where you are processing your command line parameters.

Please take a look at how the c_str function is used.

What I meant was:

comment out seeding() on line 27.

Add the following on line 38

if(count)
{
       seeding();

}

You can call the function immediately before you call cleanup either by itself or after checking for count.

Welcome to the club!!

However, it is fairly easy to identify SEG faults once you can competently use a debugger. Compile with -g option to use binary with debugger.

There are some cases where even the debugger can't help you.

Thorough analysis of the code only can help you there.

You are having a problem in line 32-36.

Instead of iterating through the linked list and adding your element at the end, you are pointing your head node to the new node and the new node points to your head.

All the intermediate nodes are lost. You are left with a LL of two nodes.

My apologies. I was looking at the original post. I believe you have got it right in the post after AD's post.

Are you allocating memory to Index?

I can see "Index[Counter] = new"

But I don't see "Index = new" anywhere or it's equivalent.

I would like to see Event.h

Can you please upload the same?

Also, posting in bits and pieces of code is most irritating.

You are posting it because you don't know what the error is. Then how do you expect yourself to understand what part of code is causing the problem. Just because the compiler is pointing to one line doesn't make that line the culprit.

Issue is with your while loop. It was not exiting properly.

Change

while(!myFile.eof())

to

while(getline(myFile,line))

and comment out the getline inside the while loop. (no need for two getline function calls)

Spelling mistake at 64

Employee::Employee()

70,71

firstName = first;

lastName = last;

You are assigning a character to a character pointer.

An approach would be to change this to take char pointers as input and use a string function to copy.

Allocate memory for the pointers in the constructor.

74-77 variable is mis-spelt. Declaration of functions in Employee.h and Employee.cpp don't match.

Before calling this function make sure that memory is allocated to the firstName pointer.

Employee:: setfirstName (char *fisrt)// should be first
#
{
#
strcpy_s ( firstName, first);
#

Continue like this and get back with corrected code.

Change the getmax implementation as

template <class T>
T pair<T>::getmax (){

In main use

pair<int> myobject(100,75);

Also rename pair as pair1, as there is a naming conflict in std namespace. (else remove using and use std:: as required)

you can use a function like

float calcPyramid(int length,int height)

First of all you are passing radius to calcPrism and calcPyramid.

In those functions, the calculation is not based on radius.

The compiler fails to give any error , because you have the length,height and width defined as globals.

Try using a debugger like gdb to identify the exact crash and the reason.

Take a look here.

Good catch WaltP.

@mrinal.s2008

Please check up the usage of ostream_iterator's * and ++ operators and how to add elements.

Check whether your stream is open using is_open().
Use eos as the exit criteria for the istream_iterator.

I wanted to check whether there is a naming conflict since the code has

using namespace std;

It could be something specific to the version that is being used.