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. this (another C# programmer rant)

this (another C# programmer rant)

Scheduled Pinned Locked Moved The Lounge
csharpvisual-studiocomquestion
41 Posts 26 Posters 8 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.
  • M Marc Clifton

    Code:

    	public override string ToString()
    	{
    		string s = String.Empty;
    
    		for (long i = 0; i < this.k; ++i)
    		{
    			s += this.data\[i\].ToString() + " ";
    		}
    
    		return s;
    	} // ToString()
    

    I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

    Thyme In The Country Interacx My Blog

    D Offline
    D Offline
    Douglas Troy
    wrote on last edited by
    #13

    that.sucks;


    :..::. Douglas H. Troy ::..
    Bad Astronomy |VCF|wxWidgets|WTL

    1 Reply Last reply
    0
    • L leonej_dt

      Agreed. this, Self and Me are keywords that serve no purpose.

      To those who understand, I extend my hand. To the doubtful I demand: Take me as I am. Not under your command, I know where I stand. I won't change to fit yout plan. Take me as I am.

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

      They serve limited purpose, but rarely do any harm.

      1 Reply Last reply
      0
      • L leonej_dt

        Agreed. this, Self and Me are keywords that serve no purpose.

        To those who understand, I extend my hand. To the doubtful I demand: Take me as I am. Not under your command, I know where I stand. I won't change to fit yout plan. Take me as I am.

        D Offline
        D Offline
        dandy72
        wrote on last edited by
        #15

        leonej_dt wrote:

        Agreed. this, Self and Me are keywords that serve no purpose.

        I wouldn't go that far. I avoid 'this' like the plague as a general rule, but I'll still use it as a parameter to a function, for example. What's your alternative in that case?

        L 1 Reply Last reply
        0
        • D dandy72

          leonej_dt wrote:

          Agreed. this, Self and Me are keywords that serve no purpose.

          I wouldn't go that far. I avoid 'this' like the plague as a general rule, but I'll still use it as a parameter to a function, for example. What's your alternative in that case?

          L Offline
          L Offline
          leonej_dt
          wrote on last edited by
          #16

          Ah, I forgot that. Sorry.

          To those who understand, I extend my hand. To the doubtful I demand: Take me as I am. Not under your command, I know where I stand. I won't change to fit yout plan. Take me as I am.

          1 Reply Last reply
          0
          • P PIEBALDconsult

            I do it just to piss people off. :-D

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

            PIEBALDconsult wrote:

            I do it just to piss people off.

            You're one step closer to "Outlaw" status.

            "Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
            -----
            "...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001

            1 Reply Last reply
            0
            • M Marc Clifton

              Code:

              	public override string ToString()
              	{
              		string s = String.Empty;
              
              		for (long i = 0; i < this.k; ++i)
              		{
              			s += this.data\[i\].ToString() + " ";
              		}
              
              		return s;
              	} // ToString()
              

              I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

              Thyme In The Country Interacx My Blog

              L Offline
              L Offline
              Le centriste
              wrote on last edited by
              #18

              I would be more concern of naming a member variable "k". Why do you hate "this" so much?

              1 Reply Last reply
              0
              • M Marc Clifton

                Code:

                	public override string ToString()
                	{
                		string s = String.Empty;
                
                		for (long i = 0; i < this.k; ++i)
                		{
                			s += this.data\[i\].ToString() + " ";
                		}
                
                		return s;
                	} // ToString()
                

                I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

                Thyme In The Country Interacx My Blog

                M Offline
                M Offline
                Member 96
                wrote on last edited by
                #19

                Almost certainly it's to kick in intellisense. I do it all the time; it's a form of insurance to gurantee you are typing the correct variable name. At worst it's a difference of esthetics, it makes no other difference.


                "It's so simple to be wise. Just think of something stupid to say and then don't say it." -Sam Levenson

                1 Reply Last reply
                0
                • S Shog9 0

                  I think his complaint is that this.k could be easily written simply as k. Although yes, k is a terrible name for a class member.

                  ----

                  You're right. These facts that you've laid out totally contradict the wild ramblings that I pulled off the back of cornflakes packets.

                  M Offline
                  M Offline
                  Marc Clifton
                  wrote on last edited by
                  #20

                  Shog9 wrote:

                  Although yes, k is a terrible name for a class member.

                  In the context of the class, it makes sense. Combinations(n, k), where "n" and "k" are the common math terms for these concepts. Marc

                  Thyme In The Country Interacx My Blog

                  1 Reply Last reply
                  0
                  • S StevenWalsh

                    i use letters all the time for couters in a for loop, do you hate that?

                    for(int i = 0; i < this.k; i++){
                    MarcClifton.Anger++;
                    }

                    Einstein argued that there must be simplified explanations of nature, because God is not capricious or arbitrary. No such faith comforts the software engineer. -Fred Brooks

                    M Offline
                    M Offline
                    Marc Clifton
                    wrote on last edited by
                    #21

                    StevenWalsh wrote:

                    i use letters all the time for couters in a for loop, do you hate that?

                    I do that all the time. And I start with "i", harking back to the days of Fortran, where "i" was the first of several letters that defaulted to integer type. Pretty crazy. Marc

                    Thyme In The Country Interacx My Blog

                    1 Reply Last reply
                    0
                    • M Marc Clifton

                      Code:

                      	public override string ToString()
                      	{
                      		string s = String.Empty;
                      
                      		for (long i = 0; i < this.k; ++i)
                      		{
                      			s += this.data\[i\].ToString() + " ";
                      		}
                      
                      		return s;
                      	} // ToString()
                      

                      I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

                      Thyme In The Country Interacx My Blog

                      S Offline
                      S Offline
                      Single Step Debugger
                      wrote on last edited by
                      #22

                      No C++ programmers in that thread. This.IsGreat(true); Personally, I really hate to see “::someMetod() “ in the code. Why it is so difficult for someone to type the base class name?!? - especially if you have a multiple inheritance.

                      The narrow specialist in the broad sense of the word is a complete idiot in the narrow sense of the word. Advertise here – minimum three posts per day are guaranteed.

                      modified on Thursday, September 18, 2008 2:19 PM

                      S 1 Reply Last reply
                      0
                      • M Marc Clifton

                        Code:

                        	public override string ToString()
                        	{
                        		string s = String.Empty;
                        
                        		for (long i = 0; i < this.k; ++i)
                        		{
                        			s += this.data\[i\].ToString() + " ";
                        		}
                        
                        		return s;
                        	} // ToString()
                        

                        I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

                        Thyme In The Country Interacx My Blog

                        A Offline
                        A Offline
                        Adam Maras
                        wrote on last edited by
                        #23

                        Uh... those of you that are playing the IntelliSense card... Alt+Right Arrow is your friend ;)

                        1 Reply Last reply
                        0
                        • M Marc Clifton

                          Code:

                          	public override string ToString()
                          	{
                          		string s = String.Empty;
                          
                          		for (long i = 0; i < this.k; ++i)
                          		{
                          			s += this.data\[i\].ToString() + " ";
                          		}
                          
                          		return s;
                          	} // ToString()
                          

                          I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

                          Thyme In The Country Interacx My Blog

                          J Offline
                          J Offline
                          jbarton
                          wrote on last edited by
                          #24

                          I am not a fan of "this." as a prefix (coming from a C++ background). The place that I work at does new development in C# and has decided to use the Microsoft StyleCop to check the code style. One of the rules of the StyleCop is that member access must be prefixed by "this.". Apparently this is the Microsoft recommended way of indicating member variable access. John

                          K 1 Reply Last reply
                          0
                          • S Shog9 0

                            I've seen comments... probably here... from folk who use it to make it obvious they're using member variables (replacing the old m_ prefix, essentially). I don't care for it either.

                            ----

                            You're right. These facts that you've laid out totally contradict the wild ramblings that I pulled off the back of cornflakes packets.

                            K Offline
                            K Offline
                            Kevin McFarlane
                            wrote on last edited by
                            #25

                            Shog9 wrote:

                            I don't care for it either.

                            Microsoft StyleCop enforces use of this on private members. StyleCop is being adopted more and more within Microsoft so expect more of it going forward. To be honest I don't care much either way - though to date I've used the _ for private fields.

                            Kevin

                            1 Reply Last reply
                            0
                            • M Marc Clifton

                              Code:

                              	public override string ToString()
                              	{
                              		string s = String.Empty;
                              
                              		for (long i = 0; i < this.k; ++i)
                              		{
                              			s += this.data\[i\].ToString() + " ";
                              		}
                              
                              		return s;
                              	} // ToString()
                              

                              I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

                              Thyme In The Country Interacx My Blog

                              M Offline
                              M Offline
                              Mark Salsbery
                              wrote on last edited by
                              #26

                              Visual Studio generated code uses it everywhere. Drives me nuts coming from a C++ background.

                              Mark Salsbery Microsoft MVP - Visual C++ :java:

                              P 1 Reply Last reply
                              0
                              • J jbarton

                                I am not a fan of "this." as a prefix (coming from a C++ background). The place that I work at does new development in C# and has decided to use the Microsoft StyleCop to check the code style. One of the rules of the StyleCop is that member access must be prefixed by "this.". Apparently this is the Microsoft recommended way of indicating member variable access. John

                                K Offline
                                K Offline
                                Kevin McFarlane
                                wrote on last edited by
                                #27

                                jbarton wrote:

                                Apparently this is the Microsoft recommended way of indicating member variable access.

                                Basically, StyleCop is being used more and more across Microsoft teams so expect more of it going forward. It's the Java house style btw. Note that StyleCop is customisable though. You can disable that rule for your team.

                                Kevin

                                J 1 Reply Last reply
                                0
                                • M Marc Clifton

                                  Code:

                                  	public override string ToString()
                                  	{
                                  		string s = String.Empty;
                                  
                                  		for (long i = 0; i < this.k; ++i)
                                  		{
                                  			s += this.data\[i\].ToString() + " ";
                                  		}
                                  
                                  		return s;
                                  	} // ToString()
                                  

                                  I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

                                  Thyme In The Country Interacx My Blog

                                  R Offline
                                  R Offline
                                  Ri Qen Sin
                                  wrote on last edited by
                                  #28

                                  I simply use it because it helps me figure out what's a class member or not, although laziness is also a good justification. :)

                                  So the creationist says: Everything must have a designer. God designed everything. I say: Why is God the only exception? Why not make the "designs" (like man) exceptions and make God a creation of man?

                                  1 Reply Last reply
                                  0
                                  • K Kevin McFarlane

                                    jbarton wrote:

                                    Apparently this is the Microsoft recommended way of indicating member variable access.

                                    Basically, StyleCop is being used more and more across Microsoft teams so expect more of it going forward. It's the Java house style btw. Note that StyleCop is customisable though. You can disable that rule for your team.

                                    Kevin

                                    J Offline
                                    J Offline
                                    jbarton
                                    wrote on last edited by
                                    #29

                                    I wish I could disable this warning. It was a decision from higher up that StyleCop would be used pretty much as is (a few warnings have been turned off, but for the most part they are all on). There is a check-in policy that says that there can't be any StyleCop warnings based on the global team settings. I can't turn it off just for my machine or I would get warnings and the check-in would fail.

                                    K 1 Reply Last reply
                                    0
                                    • J jbarton

                                      I wish I could disable this warning. It was a decision from higher up that StyleCop would be used pretty much as is (a few warnings have been turned off, but for the most part they are all on). There is a check-in policy that says that there can't be any StyleCop warnings based on the global team settings. I can't turn it off just for my machine or I would get warnings and the check-in would fail.

                                      K Offline
                                      K Offline
                                      Kevin McFarlane
                                      wrote on last edited by
                                      #30

                                      Too bad for you. Personally I'm not much fussed by this rule. There are many things that irritate me more. :)

                                      Kevin

                                      T 1 Reply Last reply
                                      0
                                      • D Dan Neely

                                        In vs2k3, intilisense didn't start as soon as you began typing. IF you weren't sure what the variable was called you had to type this. to make it come up. Not needed int 2k8, I never used 2k5; could just be an old bad habit.

                                        Today's lesson is brought to you by the word "niggardly". Remember kids, don't attribute to racism what can be explained by Scandinavian language roots. -- Robert Royall

                                        L Offline
                                        L Offline
                                        Lost User
                                        wrote on last edited by
                                        #31

                                        In 2k5 it also immediately comes up

                                        1 Reply Last reply
                                        0
                                        • M Marc Clifton

                                          Code:

                                          	public override string ToString()
                                          	{
                                          		string s = String.Empty;
                                          
                                          		for (long i = 0; i < this.k; ++i)
                                          		{
                                          			s += this.data\[i\].ToString() + " ";
                                          		}
                                          
                                          		return s;
                                          	} // ToString()
                                          

                                          I HATE WHEN PROGRAMMERS USE THIS. this.k??? Give me a FB!!!! That doesn't even make sense from a "I'm saving keystrokes by prefixing with "this." so Intellisense kicks in." Marc

                                          Thyme In The Country Interacx My Blog

                                          L Offline
                                          L Offline
                                          leppie
                                          wrote on last edited by
                                          #32

                                          Man that is ugly code! Grrrrrr. Why do people never use string.Join ?

                                          xacc.ide - now with TabsToSpaces support
                                          IronScheme - 1.0 alpha 4a out now (29 May 2008)
                                          ((lambda (x) `((lambda (x) ,x) ',x)) '`((lambda (x) ,x) ',x))

                                          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