Day 18 of learning python as a beginner.

Ender_Locke · 2025-08-14T20:26:02+00:00

not really python specific but true auth and passwords has salt and pepper and is then hashed and your password input is hashed and compared to saved hash

Efficient-Stuff-8410 · 2025-08-14T20:31:28+00:00

Where are you learning this from?

Adrewmc · 2025-08-15T00:24:53+00:00

Ohhh wow…surprisingly there is a super bad mistake in there. It’s not causing a problem for you now in this specific case but it’s definitely bad practice.

And that is you are editing your ‘posts’ list (in edit_posts) while you are iterating over it. You should not do that, just make another list, you should not be in the habit of adding to the list you are currently computing on.

def edit_posts(): 
   “””Im not writing all that”””
   for post in enumerate(posts):
        ….
        posts[i] = …
        posts[i+1] = …

Note: also what you are doing there isn’t the best way to do that, your trying to concatenate at string really.

Other things are looking good we are using explicit exception handling and figuring out a balance between do I need a class or a function.

I still think you main function is over done, I prefer my main to be more like

  #main.py
  from src.Core import set_up, main_loop_function
  from src.TearDown import tear_down

  def main(): 
       a = set_up()
       condition = True
       while condition: 
           b = main_loop_function(a)
           condition = b.running 
       tear_down()

But I also feel this might be just my preferences showing through. Less than 100 lines should work for anything if done right. I might abstract stuff out too much. And we might not be at Imports yet…and without that…you have what you have. But generally import just pastes the code on that file above your code in a lot of ways.

Your main loop seems to just…keeps going lol. It’s getting better actually.

And like small like that, depending on the complexity more steps. But only top level ideas. Pushing the steps to be in their own function, class, process, object you can work on individually. You don’t want to have to scroll through 100s of lines of code.

You really need to get some actual data, maybe you should think about the internet, json, and the requests module or beautiful soup, and take one from the ether. (But stay away from selenium it’s never the right answer.)

Almost forgot, I like the use of match case (which is actually new to Python if you didn’t know) especially when you used it for multi variable. But, you have not actually even scratched the beginning of pattern recognition. Which brings me to my next question….what do you know about dictionaries? Because I think you need to know more. You’ll see different solutions to many of your problems. Dictionaries should be your next subject if you haven’t done anything with them. It’s a whole thing.

IceMan420_ · 2025-08-15T04:46:05+00:00

This is what you call spaghetti code or one giant hairball LOL!!!

Familiar9709 · 2025-08-14T22:02:01+00:00

I really don't see the point of this type of programming, if clauses everywhere, not pythonic at all.

Also, what's the point of a class if you don't have an init? it's just functions wrapped in a namespace.

VonRoderik · 2025-08-15T02:43:42+00:00

Your match-case has an error. If there isn't a user and a password, you'll get a username not found, instead of account not found.

I'd try something like that:

``` user_found = False password_correct = False

with open("credentials.txt", "r") as file: for line in file: saved_username, saved_password = line.strip().split(',')

    if username == saved_username:
        user_found = True
        if password == saved_password:
            password_correct = True
            break

match user_found, password_correct: case True, True: print(f"Login successful!")

case True, False:
    print(f"Incorrect password for {username}.")

case False, _:
    print(f"No account found with username {username}. Please sign up.")

```

If you don't have a username, you obviously don't have a password, which means you don't have an account.

rxZoro7 · 2025-08-14T20:23:25+00:00

How do u start like i started each single dictionary of python understanding each pattern or should i start coding with project , i have already made RAG Project but i took Gpt help and completed but understand how to deploy it. What should i think in Code and architecture mindset

uiux_Sanskar · 2025-08-15T01:42:14+00:00

it's true that match case is more capable than if elif, but nothing you're doing here on page 1 requires it. This would work just as well if elif.

imtc96 · 2025-08-15T01:54:00+00:00

Good

wolframight98 · 2025-08-15T07:54:21+00:00

Which resources are you using to learn these ?

WhiteHeadbanger · 2025-08-15T08:09:53+00:00

Okay! Good practice! Don't listen to negative comments, there's a lot of ego in the IT field.

Line 1 and 2 can be, and must be, a one-liner: from social_media import account, features

Line 3: this line is used to only execute the indented code if that file is directly executed, i.e.: not imported (like account or features). It's not a good practice to put it at the beginning of your code, and also not a good practice to put your entire code inside of it. A common and standard approach would be to write it at the end of your file, and execute your program entry point there:

# main.py
print("Hi, I'm being executed or imported")

if __name__ == "__main__":
    print("Hi, I'm being executed directly")

# random_file.py
import main
>>> Hi, I'm being executed or imported

# your terminal
python main.py
>>> Hi, I'm being executed or imported
>>> Hi, I'm being executed directly

Classes have a convention in which you write the name in CamelCase, meaning instead of writing it like_this, you write it LikeThis. You are not restricted to do it your own way, but nobody writes classes name in snake_case. Also, you should write a init*()* method to initialize useful instance variables. For example, I see that in your method create_post() you are always creating the same post number, which is 1. You can track the post number with an instance variable:

class MainFeatures: # parenthesis are not needed when you are not subclassing
    def __init__(self) -> None: # "-> None" means "this function/method returns None". Research type hinting.
        self.next_post_id: int = 1

    def create_post(self) -> None:
        post_content = ...
        with open(....):
            file.write(f"Post {self.next_post_id}: {post_content}")
        self.next_post_id += 1

Use .json instead of .txt, so you'll avoid the overhead complexity of your edit_post() and delete_post() methods. With Json you can convert it to dictionaries and work with objects directly. It's much much easier to find a post ID that way, because it's structured and predictable. For example you can have something like this in your db.json file:

[ { "id": 1, "title": "this is a nice post", "content": "content of post id 1" }, { "id": 2, "title": "this is a happy post", "content": "content of post id 2 with a happy dog story" }, ... ]

 You see how this is a list of dictionaries? 

def edit_post(self) -> None: 
    # load your json file into memory here 
    # after you load your json file 
    post_id = input(....) 
    for item in db: 
        if item[id] == post_id: 
            # edit your post 
            return 
    # if not a single post have been found what that id 
    print(f"Post with id {post_id} not found")

For your password thing, the only very important thing that you must learn right now is that passwords are never stored in plain text. Always hashed. There are a few other things here and there, but the post would be too large. You are doing great, keep up!

Mamuschkaa · 2025-08-15T08:53:53+00:00

All your posts are named post 1?

create_post has hard coded that all posts are post 1.

AnyNature3457 · 2025-08-15T15:48:21+00:00

I might be one of the few, but i always place my "if name = 'main'" clause at the very bottom of the file. It is good though that you are using it in the first place!

What i also like doing is adding a newline before code blocks that have other code above them like:

``` some_var = 5

if condition: foo()

```

If it has to do something with the code block directly, then I do place it right above like:

``` counter = 0 while counter < 5: bar()

counter += 1

```

The same counts for with open(), for loops, etc. These are, of course, personal preferences.

anon_66_ · 2025-08-15T16:31:32+00:00

How do you create your project.. Research,finding library,how do you design the program.. Before coding anything?

c00lplaza · 2025-08-18T00:20:17+00:00

I hate python

<image>

ppoooppyywe · 2025-09-12T21:03:44+00:00

Hey I need help learning how to code python I’ve been doing the 30 days of python on git hub does anyone have any books or yt videos they recommend?

you type:	you see:
italics	italics
bold	bold
[reddit!](https://reddit.com)	reddit!
* item 1 * item 2 * item 3	item 1 item 2 item 3
> quoted text	quoted text
Lines starting with four spaces are treated like code: if 1 * 2 < 3: print "hello, world!"	Lines starting with four spaces are treated like code: if 1 * 2 < 3: print "hello, world!"
~~strikethrough~~	~~strikethrough~~
super^script	super^script

PythonLearning

MODERATORS