OK so what, I guess, is the new added node here in my code then? I thought that the statement:
curNode = curNode->next;
moved the pointer to the next placeholder, ready to store a new value. Am I missing something here?
@thomas_naveen I implemented the changes you suggested. You can see the new code down below. I changed the struct to a class (made everything public for now, just for simplicity for the time being). I understand the need to change to **curNode in the function call, so I have a pointer to my pointer. That makes sense to me. What I don't understand now is why it has effectively changed nothing in my end result. Still just getting 1 and 9 in my output.
Sorry for being dim, I'm new to this and for some reason ListNodes just seem very abstract to me. Thanks for your help.
#include <iostream>
using namespace std;
class ListNode {
public:
double value;
ListNode *next;
void add( double, ListNode** );
bool isMember( double );
};
void ListNode::add( double x, ListNode **curNode ) {
(*curNode)->next = new ListNode; // Creates a ListNode at the end of the list
(*curNode) = (*curNode)->next; // Points to that ListNode
(*curNode)->next = 0; // Prevents it from going any further
(*curNode)->value = x;
//cout << (*curNode)->value << endl;
}
int main() {
const int SIZE = 5;
double testValue[SIZE] = { 1, 3, 5, 7, 9 };
double* tv = testValue;
ListNode *head;
ListNode *curNode;
head = new ListNode; …
1) Your file "max2sat_data" is present in the current directory with read permissions. 2) What is build_clauses returning? Is memory allocated to clauses when you are calling print_clauses?
To solve your problem, remove the else part which returns 1.
Add a initialised (true) flag in the function. Whenever an alphabet is encountered wich does not match your two conditions , set the value of the flag to a new value (false).
After the entire string is checked, use the flag to return 1 or some other value.
Lack of indentation makes this code impossible to read.
It looks very much like you have one closing brace too many before the first error. It is difficult to say exactly since you have not posted your entire code.
Your project must also include three separate files: a header file, an implementation file, and a driver/main file.
As mentioned in the problem statement, it is your project. So you should be doing it yourself and approaching us for any issues that you face in the project.
Try implementing it and post any issues that you face.
else if not the first line, but the first element ( we need to print out variable amount of * based on the line number)
output linenumber-1 * and then an &.
else (now worry about placing the & at different locations inside the pattern.) print out the & based on the relative position of j to i and also it should not be the first line.
char letters[6];
int mem[3];
int i, j;
letters[0]=65;//A
letters[1]=66;//B
letters[2]=71;//G
letters[3]=77;//M
letters[4]=78;//N
letters[5]=82;//R
letters[6]=86;//V
You are accessing the array boundary here. Change 6 to 7 in the array declaration.
jephthah commented: doh! look at how many "veteran posters" missed that! good catch :)+6
Take a look at how you are allocating memory in your ListNode<Type>.
It might be a good idea to explicitly allocate memory for your pointers there and ensure that the memory is allocated on the heap, rather than assuming that the memory is allocated somewhere.
