Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > comp.lang.c > #164134

Re: A question about style

Show all headers | View raw

Oğuz <oguzismailuysal@gmail.com> writes:

> I have this if-else thingy:
>
> if (!expr[expr_length - 1].is_number) {
>   if (skip_adj[expr[expr_length - 1].value][op])
>     return false;
> }
> else if (expr_length > 2 && !expr[expr_length - 2].is_number) {
>   if (skip_alt[expr[expr_length - 2].value][op])
>     return false;
>   else if (expr[expr_length - 3].is_number
>       && expr[expr_length - 3].value < expr[expr_length - 1].value
>       && skip_alt_asc[expr[expr_length - 2].value][op])
>     return false;
> }
>
> (`expr' is a fixed-size array of structs, I increase `expr_length'
> as I fill it in.  `skip_*' are two dimensional arrays of booleans,
> but I don't have any problem regarding them.)
>
> And it looks ugly;  barely readable even with comments.  I want it
> to be readable at least.  [...]

Putting the code in the context of a function body (with a
'return 5;' at the end just to pick a value obviously not
the same as 'false'):

    int
    original_code(){
      if (!expr[expr_length - 1].is_number) {
        if (skip_adj[expr[expr_length - 1].value][op])
          return false;
      }
      else if (expr_length > 2 && !expr[expr_length - 2].is_number) {
        if (skip_alt[expr[expr_length - 2].value][op])
          return false;
        else if (expr[expr_length - 3].is_number
            && expr[expr_length - 3].value < expr[expr_length - 1].value
            && skip_alt_asc[expr[expr_length - 2].value][op])
          return false;
      }
      return 5;
    }

Assuming no major changes to the code (so stipulating any changes
be kept local to this one function), and taking 'expr' to be an
array of type 'Expression', I would try something along these
lines:

    int
    possible_revision(){
        Expression *e = expr + expr_length;

        if(  e[-1].is_number  ){
            if(  skip_adj[ e[-1].value ][op]  )  return  false;

        } else if(  expr_length > 2  &&  e[-2].is_number  ){
            if(
                skip_alt[ e[-2].value ][op]  ||  (
                    e[-3].is_number               &&
                    e[-3].value  <  e[-1].value   &&
                    skip_alt_asc[ e[-2].value ][op]
                )
            ){
                return  false;
            }
        }

        return  5;
    }

To me this way of writing is cleaner and easier to take in.  I
have the sense that I can follow what is being done, even if
exactly what is being accomplished for the caller isn't evident.
Choosing a good name for the function might to a long way towards
addressing the latter question.

Different people have different reactions to various lexical
styles, so that aspect may be somewhat off putting here.  If so
then perhaps the example will spark some other ideas.  My sense
for this question is that a judicious choice of where to insert
white space, both horizontal and vertical, is a key element of
making the code easier to take in and comprehend.

Back to comp.lang.c | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

A question about style Oğuz <oguzismailuysal@gmail.com> - 2021-12-30 09:18 +0300
  Re: A question about style Öö Tiib <ootiib@hot.ee> - 2021-12-30 04:42 -0800
    Re: A question about style Öö Tiib <ootiib@hot.ee> - 2021-12-30 04:47 -0800
      Re: A question about style James rock <jamesroack12@gmail.com> - 2021-12-30 15:40 +0000
      Re: A question about style Oğuz <oguzismailuysal@gmail.com> - 2021-12-30 19:24 +0300
  Re: A question about style Manfred <noname@add.invalid> - 2021-12-30 18:34 +0100
  Re: A question about style Bart <bc@freeuk.com> - 2021-12-30 18:14 +0000
  Re: A question about style Bonita Montero <Bonita.Montero@gmail.com> - 2021-12-30 19:43 +0100
    Re: A question about style Meredith Montgomery <mmontgomery@levado.to> - 2021-12-30 16:56 -0300
  Re: A question about style Paul N <gw7rib@aol.com> - 2021-12-30 12:15 -0800
    Re: A question about style Oğuz <oguzismailuysal@gmail.com> - 2021-12-31 09:05 +0300
  Re: A question about style Tim Rentsch <tr.17687@z991.linuxsc.com> - 2021-12-30 14:01 -0800
    Re: A question about style Oğuz <oguzismailuysal@gmail.com> - 2021-12-31 09:11 +0300
  Re: A question about style Keith Thompson <Keith.S.Thompson+u@gmail.com> - 2021-12-30 16:22 -0800
  Re: A question about style Jens Schweikhardt <usenet@schweikhardt.net> - 2022-01-02 13:28 +0000

csiph-web