Applet Orchard

Clean Code: Learn to write clean, maintainable and robust code

[ad_1]
Clean Code: Learn to write clean, maintainable and robust code.

In this video, I take a poorly-written piece of code and refactor in a step-by-step fashion. I also teach you some productivity tips along the way that helps you write code fast with Visual Studio.

If you enjoy my teaching style and want to learn more from me, check out my courses on Udemy:

C# Intermediate: Classes, Interfaces and Object-oriented Programming

C# Advanced: Take Your C# Skills to the Next Level

Clean Code: The Art of Writing Beautiful C# Code

C# Developers: Double Your Coding Speed

The Complete Backbone.js Course from Beginner to Advanced

Connect with me on social media:

My blog

Facebook

Twitter

Posted

in

by

appletorchard

Tags:

Comments

34 responses to “Clean Code: Learn to write clean, maintainable and robust code”

  1. ChrisUMB Avatar
    ChrisUMB

    When you were fixing the SQLInjection bit, that was already fixed. He enclosed the entire string entry in a string, both at the beginning AND end, and I feel like you realized that when you went to put the final apostrophe in.

    Reply
  2. silver1000n2 Avatar
    silver1000n2

    Your coding isn't any better.

    Reply
  3. Rabi Moshe Hizskia Avner goldenberg Shekelstein Avatar
    Rabi Moshe Hizskia Avner goldenberg Shekelstein

    I hate the var thing.
    No reason to use it its much better in my idea to do:
    MemoryReader mem = GetMemory();
    than
    var mem = GetMemory();

    With the var type you look at it and u have no idea what get memory does without look at it which wastes time.

    Reply
  4. King Parodije Avatar
    King Parodije

    call me old fashioned, but I like to have all code on one place.

    Reply
  5. blipman17 Avatar
    blipman17

    Can I just note that there's an "inline variable" shortcut nowdays. that does exactly what you want to do at 8:50? It's "Ctrl + R & Ctrl + I." This does not seem like much, but it also works for methods.

    Reply
  6. Unique GOD **i** Avatar
    Unique GOD **i**

    is there a way to create shortcut codes for example just typing "main" we get "public static void main(String[] args){}"
    like sublimetext 3 in vs code

    Reply
  7. Nick Cardinal Avatar
    Nick Cardinal

    Solid work my friend, thank you!

    Reply
  8. james Dudley Avatar
    james Dudley

    he used or instead of '||'

    Reply
  9. Joshua Weaver Avatar
    Joshua Weaver

    oh did someone mention the Using thing? Everyone trying to be a clean coder making a filthy mess of the comments.

    Reply
  10. Joshua Weaver Avatar
    Joshua Weaver

    Learned some things. I will soon be coding in VS. So thanks. I am not sure I will move away from declaring my variables at the top in one section though. They can be a sectioned outline for proofing the Cleaner work section. I like to set them early which also helps debugging, cleaning, and possibly removing them to make it modular code.

    Reply
  11. Sudikinoko Avatar
    Sudikinoko

    Is there a reason to not use the service class as a static class? Its a little bit a pain in the eye to see the code seperated just to instantiate a new class or a I missing something?

    Reply
  12. Saied Hassaninia Avatar
    Saied Hassaninia

    If the var reader = command.ExecutreReader(); is outside the try, does and an exception is thrown does it still go into the finally block?

    Reply
  13. Valerii Bobrovskyi Avatar
    Valerii Bobrovskyi

    індуський гавнокод

    Reply
  14. Marwan Iddouche Avatar
    Marwan Iddouche

    i don't know if that is available on visual studio
    i want to have that vertical line that cross conditions
    if(something){
    | code ;
    | code;
    | code;
    |}
    else{
    | code;
    | code;
    |}
    any help pleas !

    Reply
  15. brockalicous merk Avatar
    brockalicous merk

    Guy's if you're a student go to JetBrains apply for a student license and you can get all kinds of stuff and Resharper is in that pack. So you don't need to buy ReSharper!!

    Reply
  16. Pramod Sutar Avatar
    Pramod Sutar

    Great content with well explanation. Keep it up.

    Reply
  17. Madolite Avatar
    Madolite

    I generally don't like var (because it's less readable if you have a lot of them), but I realize that it's useful as long as it's used sparingly, e.g. in place of really long class names like SqlConnection. Nice tip. 🙂

    Reply
  18. Vishal Kotecha Avatar
    Vishal Kotecha

    Advertising Re-sharper?

    Reply
  19. kevindt12 Avatar
    kevindt12

    Rename is the best thing i have ever found out i justt bined it to F2 use it alease every 10 min

    Reply
  20. VelezBiH Avatar
    VelezBiH

    Thank you for sharing this is very informative.

    Reply
  21. Peter Buchmann Avatar
    Peter Buchmann

    Well, not that bad. But how would you make your code testable?
    This would be the next step.
    And one bad habit: Using local variables starting with an underscore is ugly. Underscores are ugly anyway and should not be used in C# (stylecop SA1310: http://stylecop.soyuz5.com/SA1310.html).

    Reply
  22. 楊漢軒 Avatar
    楊漢軒

    hey yo Mosh! Nice concept and tutorial! Thanks!

    Reply
  23. Tarek Siala Avatar
    Tarek Siala

    like it

    Reply
  24. Anthony Berent Avatar
    Anthony Berent

    Is there a bug in the final code? What happens if executeReader() throws? It looks to me as if the connection is never closed in this case.

    Reply
  25. Ronald Mariah Avatar
    Ronald Mariah

    Aren't you still executing the command out of the try block. Exception can still occur. And the connection will not be closed.

    Reply
  26. Gregg Wonderly Avatar
    Gregg Wonderly

    You should be using the using(){} construct for anything implementing IDisposable. try{}finally{} doesn't work right here if the query is malformed.

    This is a misleading solution because it does not completely cover the needed error checking and handling, at all.

    Practically, you should never teach people to code methods like this. Instead, they should be coding a BaseDAO class for their application which encompasses all of the behaviors they need plus the use of the appropriate database specific details such as the need for "as" on aliases or any other details. Such utility functions make it much easier to then create an interface for your BaseDAO functionality, and then use "unity" to plug in an appropriate implementation for each database that your application might need to use.

    Reply
  27. Marcsello Hooves Avatar
    Marcsello Hooves

    You should include into the title "apsx With Visual Studio + Seccurity tips"… You are talking about SQL injection protection, which is a) Have nothing to do with being clean, maintainable or robust, 2) you solved it using something that only works in that aspx thingy. SQL Injection protection sure important, but I'm not came here to study about it in a language that I hope I'll never have to use. (sorry, I stopped watching here)

    This is a good video for sure… but the title is a bit misleading… ^^

    P.S: why do you call someone a hacker, who can use SQL injection?

    Reply
  28. hojo70 Avatar
    hojo70

    Most of your points are pedantic.

    Reply
  29. Priya SivaKumar Avatar
    Priya SivaKumar

    Helped a lot for the freshers like me..Thanks Mosh.

    Reply
  30. Khalid Mohammad Avatar
    Khalid Mohammad

    the opening music, It is not strange to me

    Reply
  31. Christopher Mayer Avatar
    Christopher Mayer

    Insightful an useful content. Thank you sir!

    Reply
  32. mamounfj mamounfj Avatar
    mamounfj mamounfj

    This video is better than paid courses or reading books.

    Reply
  33. Hazarth Avatar
    Hazarth

    To be honest, I hate the "var" thing. I think it makes the code less readable. Sure, it might help with something like var connection = new SqlConnection(…), because that one is obvious, but var reader = command.ExecuteReader(); is a horrible thing to do in my opinion. I now have no idea what type it is that ExecuteReader() even returns. It might be a SqlDataReader, but if i Didn't know that upfront, it might as well be a StreamReader of any other kind. And since the function says "execute" and "Reader" it sounds like some kind of reader was already executed, so do I expect a string? or an IEnumerable<> of some type? Without previous knowledge of SqlDataClient lib I would have no idea what is going on and it would take extra time to process that or google that.
    There are places where var makes sense and saves time and space… but it's not a one in all solution and shouldn't be used that way.
    Who agrees?

    Reply
  34. krige Avatar
    krige

    You can "extract a method" from a selected lines of code using the CTRL+R+M shortcut. You can surround a selected number of rows with a try-finally block using the CTRL+ALT+J shortcut, although I would rather use the "using" statement instead of the try-finally block. I don't see any reason why defining StockService as a class property: I would rather define the method GetProductQuantityFromStock as static method. I don't see any reason why private properties should be prefixed with underscore. Resharper can move a method in another class for you, I don't remember the shortcut, maybe F6?

    Reply

Leave a Reply

Your email address will not be published. Required fields are marked *