Solved

How do I modify a C++ class to handle unitialized variables?

Posted on 2010-11-12
9
351 Views
Last Modified: 2012-05-10
How do I modify the  attached C++ class to handle unitialized variables?  It is a symbol table and I want it to display an error such as "Uninitialized Variable y" for expressions like:
(x/y), x = 1;

#include <string>

#include <vector>

using namespace std;



#include "symboltable.h"



void SymbolTable::insert(string variable, double value)

{

    const Symbol& symbol = Symbol(variable, value);

    elements.push_back(symbol);

}



double SymbolTable::lookUp(string variable) const

{

    for (int i = 0; i < elements.size(); i++)

		if (elements[i].variable == variable)

			return elements[i].value;

    return -1;

}

Open in new window

0
Comment
Question by:NSing9
  • 5
  • 3
9 Comments
 
LVL 53

Expert Comment

by:Infinity08
ID: 34126816
Right now, your SymbolTable::lookUp function is returning -1 when the symbol is not found.

If you want to generate an exception, here are two options :

(a) let SymbolTable::lookUp throw an exception instead of returning -1.

(b) let the calling code throw an exception when -1 is returned.

Either way, make sure to catch the exception somewhere, and handle it appropriately (by displaying an error message eg.).

If you have specific issues with how exceptions are implemented in code, here's a nice basic tutorial about them :

        http://www.cplusplus.com/doc/tutorial/exceptions/

If your problem is something else, please let us know what exactly.
0
 

Author Comment

by:NSing9
ID: 34127695
Thanks.  I think that is where the problem lies.  I don't know how to implement that portion of the code.  I have read the tutorial about exceptions a few time already, but still can't seem to implement the correct code.
0
 
LVL 53

Expert Comment

by:Infinity08
ID: 34127962
Could you post what you tried here ? That way, it'll be easier for me to see where the problem lies.
0
 

Author Comment

by:NSing9
ID: 34128141
Yes, here are the four files that I have been working with.  I have tired making the changes in both the main class and variable class.
Test.cpp
DivideByZeroException.h
symboltable.cpp
variable.cpp
0
Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

 
LVL 53

Accepted Solution

by:
Infinity08 earned 300 total points
ID: 34128162
That's mostly looking good.

All that is missing, is a catch block for exceptions of the type UninitializedException (just like the catch block you already have for exceptions of the type DivideByZeroException).

Btw, you didn't post the UninitializedException.h file, but I assume it's very similar to the DivideByZeroException.h file.

If you still get errors or other problems, could you please post the exact error message(s) here ?
0
 
LVL 8

Expert Comment

by:davesgonebananas
ID: 34128175
You're calling throw UninitializedException() where that looks to be a class. You probably need to create an object of type UnitializedException and throw that instead.

throw new UnitializedException();
0
 
LVL 53

Expert Comment

by:Infinity08
ID: 34128202
>> throw new UnitializedException();

I would NOT recommend throwing a pointer - especially not a pointer to dynamically allocated memory (that can cause all kinds of issues - eg. if the cause of the exception was an out-of-memory condition).

Instead, throw objects whenever you can (and catch them by reference in order to avoid unnecessary copies).

So, the current code is just fine.
0
 

Author Closing Comment

by:NSing9
ID: 34128276
It finally worked.  I had too many windows open in my compiler.  I had a catch block for exceptions of the type UninitializedException but I had entered it in the wrong file.  I also had a UninitializedException.h, but I attached the wrong one to my comments.  It is working perfectly now.  Thanks for your help!
0
 
LVL 53

Expert Comment

by:Infinity08
ID: 34130047
Great :)
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
C++ get user from AD  (VS6) 7 53
Grammars for C C++ and java 1 114
C++ mouse_event mouse look 7 69
VS2015 Redefinition errors 4 27
An Outlet in Cocoa is a persistent reference to a GUI control; it connects a property (a variable) to a control.  For example, it is common to create an Outlet for the text field GUI control and change the text that appears in this field via that Ou…
This tutorial is posted by Aaron Wojnowski, administrator at SDKExpert.net.  To view more iPhone tutorials, visit www.sdkexpert.net. This is a very simple tutorial on finding the user's current location easily. In this tutorial, you will learn ho…
The goal of this video is to provide viewers with basic examples to understand opening and writing to files in the C programming language.
The goal of this video is to provide viewers with basic examples to understand opening and reading files in the C programming language.

911 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

19 Experts available now in Live!

Get 1:1 Help Now