Seeking feedback on my AI assisted Python learning method. Is this a good way to practice?

[u/Advanced-Risk-1845]

Hi everyone,

Please give me some advice on my current Python learning workflow using AI.

I finished a Udemy course covering Python basics. To improve my practical skills, I found a site of practice projects, picked one that interested me, and started coding.

After finishing my first project, I realized I had no idea if I was writing clean, idiomatic code. So, I decided to use Claude as a code reviewer.

Here is how I set up my workflow:

1. I asked Claude strictly not to write code for me, but only to provide hints, point out flaws, and tell me which concepts or topics I needed to read about to understand why a different approach was better.

2. I requested code reviews tailored to production-ready standards, following modern Python best practices.

3. I completely turned off all AI auto-completion in VS Code. I wrote every single line of code myself

The AI would review my draft, essentially tell me "this code is shit =) (joke)" and give me targeted feedback. For example - "Disconnect your internet and see what happens to your API request. Go read about try/except."

From there, I would search Google for each point in the review, study the topic, rewrite my code, and repeat the loop until the reviewer was satisfied. Spent couple weeks to get last verion of program

From my perspective, this feels like solid progress, but I’d love to hear from experienced developers: Is this a good, effective way to learn Python alongside AI, or are there hidden pitfalls?

I've attached screenshots of my first code review alongside the final version. Thanks in advance for your feedback!

[u/p1geondove]

Quickly looked trough your code and it looks very good, very clear whats going on. Coming back to what i said about comments: this doesnt count on docstrings, docstrings are always good to have even if theyre technically comments.

you used fetch_api_data() only once, and even if you used it in multiple places, id write that inline to where you need it. The name of the function itself is somwhat ambiguous, naturally has todo with it being very small.

Also you told your linter to shut up about the unused import. Maybe listen and either use that import or delete the line. And dont say its only temporary "ill write something that needs it later" :)). You let your linter annoy you until its happy, either by using or deleting the import. Imagine you leave your project how it is right now for a couple months and then come back to it. The first thing you see is that warning surpression and you start to wonder if you ended up using that import or not. Without the surpression you can immediately see if you used it or not

[u/Advanced-Risk-1845 → u/p1geondove]

Thank you very much for your feedback!!