Elysian Shadows

Posted: **Sun Dec 27, 2009 5:42 am**

Is it OK to write single-statemented do whiles without the curly brace (like I do for my fors, ifs, whiles, etc.).

Example:

Code: Select all

do
{
	Something();
}
while (something);

As....

Code: Select all

do
	Something();

while (something);

Posted: **Sun Dec 27, 2009 7:07 am**

Yeah, I guess you could do that in Code::Blocks in Ubuntu:

Code: Select all

#include <iostream>
#include <cstdio>

class SomeClass {

    bool quit;

    public:

    SomeClass();

    void Something();

    bool getQuit();

};

SomeClass::SomeClass()
{
    quit = false;
}

void SomeClass::Something() {

    printf("strange");
}

bool SomeClass::getQuit() {

    return quit;
}

SomeClass SomeClass1;

int main()
{
    do
        SomeClass1.Something();

    while (!SomeClass1.getQuit());
}

Posted: **Sun Dec 27, 2009 8:27 am**

It will work in other compilers and operating systems as well.

Posted: **Sun Dec 27, 2009 10:18 am**

It's just as fine as doing:

Code: Select all

if() something;
else something;

Some programmers don't like that and always use curly braces for one-liners, but I think that's just being anal. If you asked me, I would say it's perfectly legit.

Posted: **Sun Dec 27, 2009 2:04 pm**

I always use braces. I once unconsciously commented out the body of an if-statement and asked myself for some hours why nothing was working.

Posted: **Sun Dec 27, 2009 2:41 pm**

GyroVorbis wrote:It's just as fine as doing:
Code: Select all
if() something;
else something;
Some programmers don't like that and always use curly braces for one-liners, but I think that's just being anal. If you asked me, I would say it's perfectly legit.

Agreed

Posted: **Sun Dec 27, 2009 6:55 pm**

GyroVorbis wrote:It's just as fine as doing:
Code: Select all
if() something;
else something;
Some programmers don't like that and always use curly braces for one-liners, but I think that's just being anal. If you asked me, I would say it's perfectly legit.

Yeah, it's legit, but I personally put braces just because it's easier for me to see a problem.

Posted: **Sun Dec 27, 2009 10:10 pm**

XianForce wrote:
GyroVorbis wrote:It's just as fine as doing:
Code: Select all
if() something;
else something;
Some programmers don't like that and always use curly braces for one-liners, but I think that's just being anal. If you asked me, I would say it's perfectly legit.
Yeah, it's legit, but I personally put braces just because it's easier for me to see a problem.

This

Posted: **Mon Dec 28, 2009 12:17 pm**

hurstshifter wrote:
XianForce wrote:
GyroVorbis wrote:It's just as fine as doing:
Code: Select all
if() something;
else something;
Some programmers don't like that and always use curly braces for one-liners, but I think that's just being anal. If you asked me, I would say it's perfectly legit.
Yeah, it's legit, but I personally put braces just because it's easier for me to see a problem.
This

I do this as well. I often add lines of debug code to my conditionals, and that never ends well if you didn't use braces for the one-liner. It also improves readability (for me).

Posted: **Mon Dec 28, 2009 5:55 pm**

hurstshifter wrote:
XianForce wrote:
GyroVorbis wrote:It's just as fine as doing:
Code: Select all
if() something;
else something;
Some programmers don't like that and always use curly braces for one-liners, but I think that's just being anal. If you asked me, I would say it's perfectly legit.
Yeah, it's legit, but I personally put braces just because it's easier for me to see a problem.
This

This.

Posted: **Tue Dec 29, 2009 8:58 am**

Thanks everyone.

By the way, newbie1234, what's all this about a "must be" object orientation. The language doesn't force me to use it so why should I when I don't need to?

Posted: **Tue Dec 29, 2009 10:35 am**

Master Jake wrote:Thanks everyone.

By the way, newbie1234, what's all this about a "must be" object orientation. The language doesn't force me to use it so why should I when I don't need to?

I'm just being a dick.

Posted: **Mon Jan 04, 2010 11:17 am**

GyroVorbis wrote:It's just as fine as doing:
Code: Select all
if() something;
else something;
Some programmers don't like that and always use curly braces for one-liners, but I think that's just being anal. If you asked me, I would say it's perfectly legit.

I use something similar, but it helps me differentiate the condition and action:

Code: Select all

if()
    something;
else
    something;

Posted: **Mon Jan 04, 2010 11:54 am**

Ginto8 wrote:
Code: Select all
if()
    something;
else
    something;

I use that method too, but I also put an additional space between the if, else-if, and else statements like:

Code: Select all

if (conidition)
	result;

else if (condition)
	result;

else
	result;

Just helps me.

Elysian Shadows

Omitting Curly Brace In Do-While Statement

Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement

Re: Omitting Curly Brace In Do-While Statement