Coding practice

[shaanxxx]

I alway had a question in mind whether to have two function in our
programme or one function with "if statement"
for two task having some part common.
I try to explain through example

-First implementation
Node * GetNextFile(ListState state)
{

... //common statements for file and directory processing
... // File specific statement

}

Node * GetNextDIR(ListState state)
{

... //common statements for file and directory processing
... // Directory specific statements

}
-Second implementation
Node * GetNext(ListState state , NodeType type)
{

... //common statements for file and directory processing

if (type == FILE_TYPE)
{
... // File specific statement
}
else
{
... // Directory specific statements
}

}

Let me know your thoughts on this

I think if common processing statements are less , we can go first
implementation .

[Richard Bos]

shaanxxx <sh******@yahoo.comwrote:

I alway had a question in mind whether to have two function in our
programme or one function with "if statement"
for two task having some part common.

It Depends.

HTH; HAND.

Richard

(No, really. It depends. Not just on the task at hand, but also on the
system(s) you intend to use.)

[shaanxxx]

On Dec 24, 4:07*pm, [e-mail] (Richard Bos) wrote:

shaanxxx <[e-mail]:

I alway had a question in mind whether to have two function in our
programme or one function with "if statement"
for two task having some part common.

It Depends.

HTH; HAND.

Richard

(No, really. It depends. Not just on the task at hand, but also on the
system(s) you intend to use.)

How it could be dependent on system ?
(my thoughts are , first implementation will have more code because of
code duplication. This duplication of code be avoided using by
replacing with function. This will result into more CPU cost . Its
like if i try to save memory , i lose on cpu cycle)

[Richard Bos]

shaanxxx <sh******@yahoo.comwrote:

On Dec 24, 4:07=A0pm, [e-mail] (Richard Bos) wrote:

shaanxxx <[e-mail]:

I alway had a question in mind whether to have two function in our
programme or one function with "if statement"
for two task having some part common.

It Depends.

HTH; HAND.

Richard

(No, really. It depends. Not just on the task at hand, but also on the
system(s) you intend to use.)

How it could be dependent on system ?

Well, take only your own example. On some systems it would make sense to
treat a directory as a specialised kind of file; on others, it would
not.

Richard

[Jack Klein]

On Mon, 24 Dec 2007 04:09:32 -0800 (PST), shaanxxx
<sh******@yahoo.comwrote in comp.lang.c:

On Dec 24, 4:07*pm, [e-mail] (Richard Bos) wrote:

shaanxxx <[e-mail]:

I alway had a question in mind whether to have two function in our
programme or one function with "if statement"
for two task having some part common.

It Depends.

HTH; HAND.

Richard

(No, really. It depends. Not just on the task at hand, but also on the
system(s) you intend to use.)

How it could be dependent on system ?
(my thoughts are , first implementation will have more code because of
code duplication. This duplication of code be avoided using by
replacing with function. This will result into more CPU cost . Its
like if i try to save memory , i lose on cpu cycle)

Where are the profile results that prove that your program is too
slow?

Where are the memory requirements and statistics for your program as
it is, proving that is uses too much memory?

Where are the test results proving that your program is complete,
correct, and meets all of its requirements?

If you don't have all of these things, you are wasting your time
thinking about memory versus execution speed tradeoffs.

--
Jack Klein
Home: http://JK-Technology.Com
FAQs for
comp.lang.c http://c-faq.com/
comp.lang.c++ http://www.parashift.com/c++-faq-lite/
alt.comp.lang.learn.c-c++
http://www.club.cc.cmu.edu/~ajo/docs/FAQ-acllc.html

[Jack Klein]

On Mon, 24 Dec 2007 01:57:45 -0800 (PST), shaanxxx
<sh******@yahoo.comwrote in comp.lang.c:

I alway had a question in mind whether to have two function in our
programme or one function with "if statement"
for two task having some part common.
I try to explain through example

-First implementation
Node * GetNextFile(ListState state)
{

... //common statements for file and directory processing
... // File specific statement

}

Node * GetNextDIR(ListState state)
{

... //common statements for file and directory processing
... // Directory specific statements

}
-Second implementation
Node * GetNext(ListState state , NodeType type)
{

... //common statements for file and directory processing

if (type == FILE_TYPE)
{
... // File specific statement
}
else
{
... // Directory specific statements
}

}

Let me know your thoughts on this

Both of your approaches are wrong.

I think if common processing statements are less , we can go first
implementation .

If you think that, do that. Get lost counting statements and lose
sight of important things.
T1 common(T2 param)
{
/* ... */
}

T1 type_one(T2 param)
{
/* ... */
}

T1 type_two(t2 param)
{
/* ... */
}

T1 some_func(t2 param, t3 other)
{
common(param);
if (other == /* ... */ )
{
type_one(param);
}
else
{
type_two(param);
}
/* ... */
}

Search "refactoring".

--
Jack Klein
Home: http://JK-Technology.Com
FAQs for
comp.lang.c http://c-faq.com/
comp.lang.c++ http://www.parashift.com/c++-faq-lite/
alt.comp.lang.learn.c-c++
http://www.club.cc.cmu.edu/~ajo/docs/FAQ-acllc.html