41
submitted 1 month ago* (last edited 1 month ago) by andioop@programming.dev to c/learn_programming@programming.dev

Besides some of the very, very obvious (don't copy/paste 100 lines of code, make it a function! Write comments for your future self who has forgotten this codebase 3 years from now!), I'm not sure how to write clean, efficient code that follows good practices.

In other words, I'm always privating my repos because I'm not sure if I'm doing some horrible beginner inefficiency/bad practice where I should be embarrassed for having written it, let alone for letting other people see it. Aside from https://refactoring.guru, where should I be learning and what should I be learning?

you are viewing a single comment's thread
view the rest of the comments
[-] milkisklim@lemm.ee 9 points 1 month ago

For following good practices, I highly recommend using a linter like ruff. I've learned a lot from it's explanations on why my code is bad.

Also I have tried to avoid using else statements.

[-] Hammerheart@programming.dev 5 points 1 month ago

how do you avoid using else, and why?

[-] milkisklim@lemm.ee 4 points 1 month ago* (last edited 1 month ago)

Disclaimer: I would only call my skill level as intermediate and would yield to any more senior developer here.

It's not a hard and fast rule, but you can usually write it without the else and in fewer lines.

So take for a very contrived example a function that needs to return a non boolean value for a boolean test. A use case could be if you need to figure out a string argument for another function, such as you need to determine if you need to keep the "first" or "last" duplicate in a dataframe (I'm thinking about pandas's df.drop_dupliactes method).

Continuing with the drop_duplicate thing let's say we have a dataframe we need to de-duplicate but for some reason if the overall length of the dataframe is even, we need to keep the first duplicate and if the dataframe length is odd we keep the last. I don't know why we would, but that was a very particular request from the customer and we need the money to buy more Warhammer figurines.

import pandas as pd

# With else statement
def foo(x: int) -> str:
    if x%2>0:
        return "last"
    else:
        return "first"

# No else statement, shorter.
def foo(x: int) -> str:
    if x%2>0:
        return "last"
    return "first"


#import dataframe, deduplicate
df = pd.read_csv("c:\\path\\to\\data.csv")
dedup = df.drop_duplicates(keep=foo(len(df))

Here's an essay that goes into more detail

[-] christopher@programming.dev 2 points 4 weeks ago
# No else statement, shorter.
def foo(x: int) -> str:
    if x%2:
        return "first"
    return "last"

This is easier to think about for me: am I weird? Numbers can be interpreted as boolean in C but not in Go, which came later and is presumably an improvement.

load more comments (2 replies)
load more comments (2 replies)
this post was submitted on 06 Aug 2024
41 points (97.7% liked)

Learn Programming

1616 readers
1 users here now

Posting Etiquette

  1. Ask the main part of your question in the title. This should be concise but informative.

  2. Provide everything up front. Don't make people fish for more details in the comments. Provide background information and examples.

  3. Be present for follow up questions. Don't ask for help and run away. Stick around to answer questions and provide more details.

  4. Ask about the problem you're trying to solve. Don't focus too much on debugging your exact solution, as you may be going down the wrong path. Include as much information as you can about what you ultimately are trying to achieve. See more on this here: https://xyproblem.info/

Icon base by Delapouite under CC BY 3.0 with modifications to add a gradient

founded 1 year ago
MODERATORS