IF greater than 90 then minus 90 or if less than 90 add 90

BokJr

Here is the code I have so far, Im trying to return an answer in new_axis however is A is less than 90 I need to add 90 and vice versa, if A is more than 90 I need to minus 90, can someone help?

@ qreal S = ui->sphere->text().toDouble();
qreal C = ui->cylinder->text().toDouble();
qreal A = ui->axis->text().toDouble();
qreal result (C+S);
qreal answer (A<90then+90orA>90then-90);
ui->new_sphere->setText(QString::number(result));
ui->new_cylinder->setText(QString::number(result));
ui->new_axis->setText(QString::number(answer));@

SGaist

Hi,

@
if ( A < 90) {
result += 90;
} else if (A > 90) {
result -= 90;
}
@

If A == 90 then do nothing, is that right ?

This question is not really related to Qt, rather basic C/C++. I'd recommend getting a good book about it

BokJr

yes thats correct if A=90 then do nothing.

I am learning basic C/C++ currently, literally started 3 days ago.

Thank you, I plan to get a book shortly, where I live I have to order this so waiting on it to come. for now you have been a great help.

So to implement this in my current working, I have the following

@void Transpose::on_pushButton_clicked()
{
qreal S = ui->sphere->text().toDouble();
qreal C = ui->cylinder->text().toDouble();
qreal A = ui->axis->text().toDouble();
qreal result (C+S);
qreal answer if (A < 90);
result += 90;
else if (A > 90);
result -= 90;
ui->new_sphere->setText(QString::number(result));
ui->new_cylinder->setText(QString::number(result));
ui->new_axis->setText(QString::number(answer));
}@

but getting an error, what am I doing wrong?

SGaist

The code is, take again a look at my example (I should have used another variable name)

[quote author="BokJr" date="1417082246"]
@
qreal answer if (A < 90); << not valid hence the rest also falls apart
result += 90;
else if (A > 90);
result -= 90;
ui->new_sphere->setText(QString::number(result)); << why not use a QDoubleSpinBox ?
ui->new_cylinder->setText(QString::number(result)); << same question
ui->new_axis->setText(QString::number(answer)) << ditto;
}@
[/quote]
Since you are a beginner, don't remove the {} from if/for/while construct even if they are one liner, that will avoid some easy made mistakes.

BokJr

The double spin box isn't in use because I want separate answers from S, C and A.

basically I want it to run as follows that if the user inputs a value into the following

qreal S = ui->sphere->text().toDouble();
qreal C = ui->cylinder->text().toDouble();
qreal A = ui->axis->text().toDouble();

then is runs the calculation c+s=new_sphere,
new_cylinder= negative value of original C value
and the A either rotates from 90 to 180 or 180 to 90 (but if the total is below 90 then I need to add 90 and if it's above 90 then I need to subtract 90)

So if the original S= -2.00
the original C=+1.00
and the A=75

new_sphere= -1.00
new_cylinder= -1.00
new_axis= 165 (because its below 90 I added 90 to the total.)

BokJr

Oops that is a newbie mistake, I could use spine boxes :)

BokJr

So i've come to this

@void Transpose::on_pushButton_clicked()
{
qreal S = ui->sphere->text().toDouble();
qreal C = ui->cylinder->text().toDouble();
qreal A = ui->axis->text().toDouble();
qreal answer (C+S);
qreal total (-C);
qreal result;
if (A <= 90) {
result +=90;
} else if (A >= 90) {
result -=90;
}
ui->new_sphere->setText(QString::number(answer));
ui->new_cylinder->setText(QString::number(total));
ui->new_axis->setText(QString::number(result));
}@

But the new_axis doesn't calculate correctly?

Binary91

In both conditions, you have 'is equal 90' ... (if(less or equal 90) else if(greater or equal 90)), so if 'A' is exactly 90, only the first block will be executed. Maybe you want minus 90 for a value of 90? Then you have to change the condition into:
@if(A < 90)
result += 90;
else if(A >= 90) // or simply else without a condition
result -= 90;@

BokJr

yep I fixed it prior, thanks, the webpage wouldn't let me reply that I had fixed it though.

Binary91

Great! So if your code works, you can change the thread title prepending [SOLVED], so everyone knows it ;-)