c++ getting error

babag · 07-04-2008, 03:34 PM

getting the following error compiling c++:

Code:

[babag@office disk]$ g++ LMP_Process-20.cpp -g -o LMP_Process
LMP_Process-20.cpp: In function ‘int main()’:
LMP_Process-20.cpp:488: error: expected `)' before ‘!’ token
LMP_Process-20.cpp:490: error: expected `;' before ‘)’ token

here's the offending code:

Code:

		else if ((sensorW / AspectRatio) !< sensorH) {				// Keep height, trim width

			(CropX     = (CtrX - ((sensorH * AspectRatio) / 2))), 	(CropY = 0));
			(CropW     = sensorW),					(CropH = sensorH);
			(CanvasW   = CropW),					(CanvasH = CropH);
			(CanvasPos = "center");

		}

line 488 is the 'else if' line.

the code compiles if i comment this passage out. this is a copied section of code. the only difference in this bit and the original is the '!' in the first line. in other words, one says:

Code:

		else if ((sensorW / AspectRatio) < sensorH) {

and the other says:

Code:

		else if ((sensorW / AspectRatio) !< sensorH) {

any tips appreciated.

thanks,
BabaG

harry edwards · 07-04-2008, 03:49 PM

To my knowledge ‘!<’ is not a valid operator, did you mean not equal, which is ‘!=’.

dmail · 07-04-2008, 03:52 PM

Quote:

Originally Posted by harry edwards

To my knowledge ‘!<’ is not a valid operator, did you mean not equal, which is ‘!=’.

!< == >= or !(foo<bar)

Quote:

The message you have entered is too short. Please lengthen your message to at least 10 characters.

Ok I need to post some text

edit: that does not actually say what I wanted (and I could have used || instead of 'or').Yes you are correct that operator is not defined, but you can use the same logic as I posted

babag why are you using so many parentheses and the comma operator?

babag · 07-04-2008, 04:30 PM

Quote:

Originally Posted by dmail

!< == >= or !(foo<bar)

i don't get this. what does it say?

Quote:

Originally Posted by dmail

babag why are you using so many parentheses and the comma operator?

mainly because i only barely know anything about what i'm doing here but it works, everywhere but in this bit.

thanks,
BabaG

dmail · 07-04-2008, 05:22 PM

Quote:

Originally Posted by babag

i don't get this. what does it say?

That would be my failed attempt at operator humour then.

What I was saying is that the operator you tried to use is not defined so you can replace your logic using two methods (could use more)

Code:

		else if ((sensorW / AspectRatio) >= sensorH) {

or

		else if ( ! ( (sensorW / AspectRatio) < sensorH) ) {

babag · 07-04-2008, 11:47 PM

Code:

	if ((sensorW / AspectRatio) < sensorH) {				// Keep width, trim height

		(CropX     = 0), 					(CropY = CtrY - ((sensorW / AspectRatio) / 2));
		(CropW     = sensorW),					(CropH = (sensorW / AspectRatio));
		(CanvasW   = CropW),					(CanvasH = CropH);
		(CanvasPos = "center");

	}

	else if ((sensorW / AspectRatio) >= sensorH) {				// Keep height, trim width

		(CropX     = (CtrX - ((sensorH * AspectRatio) / 2))), 	(CropY = 0));
		(CropW     = sensorW),					(CropH = sensorH);
		(CanvasW   = CropW),					(CanvasH = CropH);
		(CanvasPos = "center");

	}

now produces this:

Code:

LMP_Process-20.cpp: In function ‘int main()’:
LMP_Process-20.cpp:490: error: expected `;' before ‘)’ token

in the 'else if' line:

Code:

(CropX     = (CtrX - ((sensorH * AspectRatio) / 2))), 	(CropY = 0));

commenting out the line, the compile works.

EDIT:
ok. this compiled:

Code:

(CropX     = CtrX - ((sensorH * AspectRatio) / 2)),	(CropY = 0);

thanks,
BabaG