Topic: Expected (

[FamousAdventurer77]

Nitpicky detail I'm missing, where would the parentheses go in here? I can't save the room because I keep getting Expected '(' for the oFruit.Visible line.

[code]
function hFruitTree_Interact()
{
  if oFruit.Visible = true; {
  Display("There is no need to climb the tree."); }
  else {
  Display("There is no need to climb the tree. Besides, there isn't any more ripe fruit to pick anyway.");
  }
}[/code]

[Atelier]

[code]function hFruitTree_Interact()
{

  if (oFruit.Visible == true) Display("There is no need to climb the tree.");

  else {
  Display("There is no need to climb the tree. Besides, there isn't any more ripe fruit to pick anyway.");
  }
}[/code]

Try this. Also, make sure you use the right amount of equal signs:

= sets, == checks

For example:

[code]if (oFruit.Visible == true) Display("I'm checking whether Fruit is visible with two equal signs");


//player enter room, or whatever

oFruit.Visible = false; //Here I'm setting a variable because it's just one equal sign.[/code]

Notice also in my top code, you can shorten this

[code]if oFruit.Visible = true; {
  Display("There is no need to climb the tree."); }[/code]

into this

[code]if (oFruit.Visible = true) Display("There is no need to climb the tree.");[/code]

Because you don't need the extra { } if there's only one expression after the if statement.

[FamousAdventurer77]

Ah, that's what I was missing....I had a feeling it was more than just a misplaced (. Thanks!

[FamousAdventurer77]

Er, problem...using this

[code]
function hFruitTree_Interact() {
if (oFruit.Visible == true) Display ("There is no need to climb the tree.");
else {
Display("There is no need to climb the tree. Besides, there isn't any more ripe fruit to pick anyway."); }
}
[/code]

turns the object invisible even though I set it to visible initially.

[Khris]

No it doesn't.
The problem lies somewhere else.

[FamousAdventurer77]

I'm getting really confused now: I deleted that bit of code and replaced it with the old one (not the one I first posted, just a simple one-line Display message) and made sure that oFruit was initially visible in the Room's objects panel.

It still isn't when I run the test game. What went wrong...? 

[code]if (oFruit.Visible = true) Display("There is no need to climb the tree.");[/code]  I got a parse error when I tried this line.

[ddq]

Try this. Also, make sure you use the right amount of equal signs:

= sets, == checks

For example:

[code]if (oFruit.Visible == true) Display("I'm checking whether Fruit is visible with two equal signs");[/code]

You should probably read up on some tutorials for AGS or for C-like programming languages in general.

[Calin Leafshade]

This isnt even nearly finished but it might help:

http://svn.thethoughtradar.com/AGSScript101/

[Khris]

If the object isn't visible, make sure you aren't turning it off somewhere else in the script.
It might also be off-screen or hidden behind a walkbehind.

[Atelier]

Try this. Also, make sure you use the right amount of equal signs:

= sets, == checks

For example:

[code]if (oFruit.Visible == true) Display("I'm checking whether Fruit is visible with two equal signs");[/code]

You should probably read up on some tutorials for AGS or for C-like programming languages in general.

Me or OP

[Khris]

The OP of course, ddq was quoting you to show that you already corrected the mistake.
Seems like you edited your "==" to "=" though which is wrong.

Again, = sets the variable to the left to whatever the result of the right side is.
== is used to compare two values or Strings.

[FamousAdventurer77]

The culprit was a walk-behind, code works after I fixed it and plugged it back in.

Yep I could use a scripting lesson or two since I haven't done it in years, the video tutorials are pretty much the best I got right now since I tend to learn the best from being shown the basics by an actual person then figuring out the rest on my own (opposed to strictly out of a book.)

Calin- great link! Looks like the perfect thing for me to read up on.

[McMonsterBrothers]

I'm getting really confused now: I deleted that bit of code and replaced it with the old one (not the one I first posted, just a simple one-line Display message) and made sure that oFruit was initially visible in the Room's objects panel.

It still isn't when I run the test game. What went wrong...? 

[code]if (oFruit.Visible = true) Display("There is no need to climb the tree.");[/code]  I got a parse error when I tried this line.

i dont know if this helps but i think that you must put the {}'s in there like this:
[code]if (oFruit.Visible == true); {
 Display ("There is no need to climb the tree.");

}
[/code]

[Iceboty V7000a]

No, if there is a single line of code you don't need the braces (though adding them there wouldn't hurt).

Also, your line shouldn't work either, as you've added a semi-colon that shouldn't be there:

if (oFruit.Visible == true); {
  Display ("There is no need to climb the tree.");
}

[McMonsterBrothers]

No, if there is a single line of code you don't need the braces (though adding them there wouldn't hurt).

Also, your line shouldn't work either, as you've added a semi-colon that shouldn't be there:

if (oFruit.Visible == true); {
  Display ("There is no need to climb the tree.");
}

oops my bad lol thanks dude