[sabret00the]

can i do?

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

[Xenon]

i think you forgot a { } pair

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

[sabret00the]

but if i put them additionals curly brackets in, won't i close the conditionals? before it's acheive what i want it to?

[sabret00the]

heres an example of the code i'm trying to ask if will work or not

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

[Xenon]

if you don't close the brackets, the else will be regarded to the inner if's and not to the outer if as you want to have

[sabret00the]

ok sorry to bother you but i don't get it, just edited my last post for a clarity, basically the conditional starts inside of the if/else and ends outside of it. yet if i close them, then it's not gonna perform what i'm asking it to do?

[Xenon]

now you have confused me.....

i don't get what you want, but from what you said before, that code is correct:

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

[Velocd]

I've clarified your code.

It's extremely recommended to put brackets for if, else, loops, and functions under the statement. Also, when verifying if a variable exists, use isset(). If you just want to see if a variable has a value, check with the ! (bang) operator in a boolean statement. For strings, use empty().

Here is your code. I'm not sure what you're trying to achieve with the empty IFs, but you were missing the last bracket.

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

[sabret00the]

thank you all, it's very much appreciated, velo, what i was trying to acheive started with the $flood variable and ended with the "INSERT INTO confessions SET..." bit basically i was trying to find out if i needed to repeat the whole code or if i could just use the conditional based on an admin cp option of letting guests confess or not.

[Velocd]

Just a minor correction..

I had:

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

The ! (bang) operator, which could literally be translated to "not", is equivalent to empty().

Not using ! is equivalent to !empty(), or just placing a variable between the () in a statement.

The code should have been:

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

[sabret00the]

thanks it's appreciated.

not to be a pain but why do you recommend putting all curly brackets under the statement?

[Xenon]

Originally Posted by sabret00the

thanks it's appreciated.

not to be a pain but why do you recommend putting all curly brackets under the statement?

well, it's the vb3 coding standard, and also a well know coding standart in real programming languages like C++.

it makes the code better readable, and especially makes it easier to find if you have missed a bracket

[sabret00the]

i'll have to try and get used to it, it's just as i'm learning it, i'm being taught to do it

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

(which i find alot easier to read)

rather than

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

arghh feels like such a waste of space.

[dstruct2k]

The way I've always coded, I've done this:

Block Disabled:      (Update License Status)
Suspended or Unlicensed Members Cannot View Code.

Bit of both.

[Xenon]

*gg*
everyone bringing in his own coding standards