Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. The Lounge
  3. PQOTD

PQOTD

Scheduled Pinned Locked Moved The Lounge
csharpquestion
59 Posts 31 Posters 0 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • N Nish Nishant

    |

    Regards, Nish


    Blog: voidnish.wordpress.com Latest article: C++ 11 features in Visual C++ 2013 Preview

    R Offline
    R Offline
    Rage
    wrote on last edited by
    #17

    Yep, but this is a workaround, not a best practice though.

    ~RaGE();

    I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Entropy isn't what it used to.

    1 Reply Last reply
    0
    • M Marc Clifton

      Kornfeld Eliyahu Peter wrote:

      you have to switch the order

      Yes indeed. Or use the bitwise operator. Marc

      Kornfeld Eliyahu PeterK Offline
      Kornfeld Eliyahu PeterK Offline
      Kornfeld Eliyahu Peter
      wrote on last edited by
      #18

      I do not like that change from C/C++...I mean the | and || changes... X|

      I'm not questioning your powers of observation; I'm merely remarking upon the paradox of asking a masked man who he is. (V)

      "It never ceases to amaze me that a spacecraft launched in 1977 can be fixed remotely from Earth." ― Brian Cox

      F J 2 Replies Last reply
      0
      • R Rage

        MISRA[^] Rule 33 & 34:

        The right hand operand of a || shall not contain side effects
        The operands of a logical || shall be primary expressions

        The example you gave is the very reason for the above two rules.

        ~RaGE();

        I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Entropy isn't what it used to.

        B Offline
        B Offline
        BobJanova
        wrote on last edited by
        #19

        There's actually some nice concise uses of

        var result = DoSomething() && DoSomethingThatDependsOnIt();
        var result = DoSomething() || HandleFailureCase();

        It's even better in languages where you can use non-boolean types with the boolean operators.

        R F 2 Replies Last reply
        0
        • M Marc Clifton

          Programming Quiz of the day. I just wrote this code (C#), which did something very unexpected. What was it that it unexpectedly did?

          		bool more = ProcessFlyouts();
          		more = more || ProcessCarrierAnimations();
          

          How would you change it to "do the right thing?" Marc

          K Offline
          K Offline
          Keith Barrow
          wrote on last edited by
          #20

          Though Nish's and OG's examples will work, I think it is symantically better to do this:

          bool more = ProcessFlyouts();
          bool foo = ProcessCarrierAnimations();
          more = more || foo;

          Less terse code (normally bad) - but explicit in calling ProcessCarrierAnimations.

          PB 369,783 wrote:

          I just find him very unlikeable, and I think the way he looks like a prettier version of his Mum is very disturbing.[^]

          P 1 Reply Last reply
          0
          • M Marc Clifton

            Programming Quiz of the day. I just wrote this code (C#), which did something very unexpected. What was it that it unexpectedly did?

            		bool more = ProcessFlyouts();
            		more = more || ProcessCarrierAnimations();
            

            How would you change it to "do the right thing?" Marc

            P Offline
            P Offline
            PIEBALDconsult
            wrote on last edited by
            #21

            more |= ProcessCarrierAnimations(); ?

            You'll never get very far if all you do is follow instructions.

            1 Reply Last reply
            0
            • M Marc Clifton

              You should be able to glean what the programmer intends to happen from the code. :) Marc

              B Offline
              B Offline
              Bassam Abdul Baki
              wrote on last edited by
              #22

              Marc Clifton wrote:

              You should be able to glean what the programmer intends to happen from the code. :)

              Riiiiiiiiiiight!!! :)

              Web - BM - RSS - Math - LinkedIn

              1 Reply Last reply
              0
              • B BobJanova

                There's actually some nice concise uses of

                var result = DoSomething() && DoSomethingThatDependsOnIt();
                var result = DoSomething() || HandleFailureCase();

                It's even better in languages where you can use non-boolean types with the boolean operators.

                R Offline
                R Offline
                Rage
                wrote on last edited by
                #23

                Let's say I have never seen these two lines of code, OK ?

                ~RaGE();

                I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Entropy isn't what it used to.

                1 Reply Last reply
                0
                • M Marc Clifton

                  You should be able to glean what the programmer intends to happen from the code. :) Marc

                  S Offline
                  S Offline
                  Simon_Whale
                  wrote on last edited by
                  #24

                  Have you been to QA lately? :laugh:

                  Every day, thousands of innocent plants are killed by vegetarians. Help end the violence EAT BACON

                  1 Reply Last reply
                  0
                  • M Marc Clifton

                    Programming Quiz of the day. I just wrote this code (C#), which did something very unexpected. What was it that it unexpectedly did?

                    		bool more = ProcessFlyouts();
                    		more = more || ProcessCarrierAnimations();
                    

                    How would you change it to "do the right thing?" Marc

                    A Offline
                    A Offline
                    Andy Brummer
                    wrote on last edited by
                    #25

                    var more = more || initalizeMore(); is a common practice in javascript.

                    Curvature of the Mind now with 3D

                    1 Reply Last reply
                    0
                    • Kornfeld Eliyahu PeterK Kornfeld Eliyahu Peter

                      I do not like that change from C/C++...I mean the | and || changes... X|

                      I'm not questioning your powers of observation; I'm merely remarking upon the paradox of asking a masked man who he is. (V)

                      F Offline
                      F Offline
                      Forogar
                      wrote on last edited by
                      #26

                      This isn't a change from C++. It works the same way for both operators. Doesn't it? Now I'm going to have to read up on it, dammit!

                      - I would love to change the world, but they won’t give me the source code.

                      Kornfeld Eliyahu PeterK 1 Reply Last reply
                      0
                      • B BobJanova

                        There's actually some nice concise uses of

                        var result = DoSomething() && DoSomethingThatDependsOnIt();
                        var result = DoSomething() || HandleFailureCase();

                        It's even better in languages where you can use non-boolean types with the boolean operators.

                        F Offline
                        F Offline
                        Forogar
                        wrote on last edited by
                        #27

                        Ouch! && Ouch! || Ouch!

                        - I would love to change the world, but they won’t give me the source code.

                        1 Reply Last reply
                        0
                        • M Marc Clifton

                          Programming Quiz of the day. I just wrote this code (C#), which did something very unexpected. What was it that it unexpectedly did?

                          		bool more = ProcessFlyouts();
                          		more = more || ProcessCarrierAnimations();
                          

                          How would you change it to "do the right thing?" Marc

                          P Offline
                          P Offline
                          Pete OHanlon
                          wrote on last edited by
                          #28

                          Now, I'm not going to look at the other answers before I post so I don't know if others have got it right, or if there are additional nuggets of information. If ProcessFlyouts returns true then the second test isn't carried out - it's a conditional OR statement. So, if you want ProcessCarrierAnimations to be evaluated, convert it to

                          more = more | ProcessCarrierAnimations();

                          1 Reply Last reply
                          0
                          • F Forogar

                            This isn't a change from C++. It works the same way for both operators. Doesn't it? Now I'm going to have to read up on it, dammit!

                            - I would love to change the world, but they won’t give me the source code.

                            Kornfeld Eliyahu PeterK Offline
                            Kornfeld Eliyahu PeterK Offline
                            Kornfeld Eliyahu Peter
                            wrote on last edited by
                            #29

                            You have to! Otherwise you may end up with some mess if updating C/C++ to C# (what I'm doing just now)...

                            I'm not questioning your powers of observation; I'm merely remarking upon the paradox of asking a masked man who he is. (V)

                            "It never ceases to amaze me that a spacecraft launched in 1977 can be fixed remotely from Earth." ― Brian Cox

                            F 1 Reply Last reply
                            0
                            • K Keith Barrow

                              Though Nish's and OG's examples will work, I think it is symantically better to do this:

                              bool more = ProcessFlyouts();
                              bool foo = ProcessCarrierAnimations();
                              more = more || foo;

                              Less terse code (normally bad) - but explicit in calling ProcessCarrierAnimations.

                              PB 369,783 wrote:

                              I just find him very unlikeable, and I think the way he looks like a prettier version of his Mum is very disturbing.[^]

                              P Offline
                              P Offline
                              PIEBALDconsult
                              wrote on last edited by
                              #30

                              "Eliminate needless local variables." :-D

                              You'll never get very far if all you do is follow instructions.

                              1 Reply Last reply
                              0
                              • R Rage

                                MISRA[^] Rule 33 & 34:

                                The right hand operand of a || shall not contain side effects
                                The operands of a logical || shall be primary expressions

                                The example you gave is the very reason for the above two rules.

                                ~RaGE();

                                I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus Entropy isn't what it used to.

                                P Offline
                                P Offline
                                PIEBALDconsult
                                wrote on last edited by
                                #31

                                I never follow other peoples' rules. :cool:

                                You'll never get very far if all you do is follow instructions.

                                S 1 Reply Last reply
                                0
                                • Kornfeld Eliyahu PeterK Kornfeld Eliyahu Peter

                                  You have to! Otherwise you may end up with some mess if updating C/C++ to C# (what I'm doing just now)...

                                  I'm not questioning your powers of observation; I'm merely remarking upon the paradox of asking a masked man who he is. (V)

                                  F Offline
                                  F Offline
                                  Forogar
                                  wrote on last edited by
                                  #32

                                  Nah! As long as it compiles...

                                  - I would love to change the world, but they won’t give me the source code.

                                  Kornfeld Eliyahu PeterK 1 Reply Last reply
                                  0
                                  • F Forogar

                                    Nah! As long as it compiles...

                                    - I would love to change the world, but they won’t give me the source code.

                                    Kornfeld Eliyahu PeterK Offline
                                    Kornfeld Eliyahu PeterK Offline
                                    Kornfeld Eliyahu Peter
                                    wrote on last edited by
                                    #33

                                    Yeah! It compiles. After then you can rub your head to find where the bug is!

                                    I'm not questioning your powers of observation; I'm merely remarking upon the paradox of asking a masked man who he is. (V)

                                    "It never ceases to amaze me that a spacecraft launched in 1977 can be fixed remotely from Earth." ― Brian Cox

                                    J 1 Reply Last reply
                                    0
                                    • M Marc Clifton

                                      Programming Quiz of the day. I just wrote this code (C#), which did something very unexpected. What was it that it unexpectedly did?

                                      		bool more = ProcessFlyouts();
                                      		more = more || ProcessCarrierAnimations();
                                      

                                      How would you change it to "do the right thing?" Marc

                                      Richard Andrew x64R Offline
                                      Richard Andrew x64R Offline
                                      Richard Andrew x64
                                      wrote on last edited by
                                      #34

                                      How about:

                                      more &= ProcessCarrierAnimations();

                                      The difficult we do right away... ...the impossible takes slightly longer.

                                      1 Reply Last reply
                                      0
                                      • M Marc Clifton

                                        Programming Quiz of the day. I just wrote this code (C#), which did something very unexpected. What was it that it unexpectedly did?

                                        		bool more = ProcessFlyouts();
                                        		more = more || ProcessCarrierAnimations();
                                        

                                        How would you change it to "do the right thing?" Marc

                                        J Offline
                                        J Offline
                                        Jorgen Andersson
                                        wrote on last edited by
                                        #35

                                        Is it just me that would have done it as a oneliner?

                                        Wrong is evil and must be defeated. - Jeff Ello[^]

                                        P 1 Reply Last reply
                                        0
                                        • J Jorgen Andersson

                                          Is it just me that would have done it as a oneliner?

                                          Wrong is evil and must be defeated. - Jeff Ello[^]

                                          P Offline
                                          P Offline
                                          PIEBALDconsult
                                          wrote on last edited by
                                          #36

                                          That could make setting a breakpoint on the second call difficult. :shrug:

                                          You'll never get very far if all you do is follow instructions.

                                          J 1 Reply Last reply
                                          0
                                          Reply
                                          • Reply as topic
                                          Log in to reply
                                          • Oldest to Newest
                                          • Newest to Oldest
                                          • Most Votes


                                          • Login

                                          • Don't have an account? Register

                                          • Login or register to search.
                                          • First post
                                            Last post
                                          0
                                          • Categories
                                          • Recent
                                          • Tags
                                          • Popular
                                          • World
                                          • Users
                                          • Groups