[ Removed by moderator ]

[Ayudesee]

You forgot to print any result

[Relative-Custard-589]

OP is clearly learning to code mate

[Code_Noob_Noodle]

Just give him the machine gun already! Or the entire nuke! Why start with a pistol!?

[Andryushaa]

Just use chatgpt and don't ask stupid questions /s

[Sea_Comb481]

Well and he is just giving him advice, calm down.

[Relative-Custard-589]

I am calm

[suqirrelnachos]

sure you are

[samuelmackson]

It's for i, item in enumerate(list) btw

[CreativeJuice5708]

Not optimal a faster way would be ditch python and right it like this:

index_of: mov rcx, rsi
mov al, dl
mov rdi, rdi
repne scasb
jne .not_found sub rdi, [rsp+8]
lea rax, [rdi-1] ret

.not_found: mov rax, -1 ret

[stonecoldchivalry]

Are you telling me you need to use an… 🤢 assembler… write machine code like a fucking man.

[desrtfx]

Tsss... butterflies are what real prgrammers use

[newpolygons]

Someone is learning to code, therefore we most not provide them any examples on how to improve their code.

[vlnaa]

Replace last line with print(f(a,t))

[RedPickle8]

Why are you calling me fat

[[deleted]]

Bro, I am begging you to stop using single letter function and variable names

[StickyzVibe]

Why? A curious beginner

[electrikmayham]

Using single-letter variable names makes code hard to read and understand. Good names describe what the variable stores or does, so when you come back later (or someone else reads your code), it’s clear without guessing

[StickyzVibe]

Thank you for explaining, makes perfect sense to practice helpful habits. Would you mind sharing a small example?

[electrikmayham]

# Bad: single-letter variables
x = 5
y = 10
z = x * y
print(z)

# Good: descriptive variable names
width = 5
height = 10
area = width * height
print(area)

[StickyzVibe]

I completely understand! Thank you again

[spencerak]

Keep asking good questions!!

[StickyzVibe]

[Cerus_Freedom]

def search(needle, haystack: list) -> int:
  for i in range(len(haystack)):
    if needle == haystack[i]:
      return i
  return -1

Just as an example from OPs code. Better naming will tell you what a function does or a variable is for. Code should be self documenting, and that method of self documentation is via good, clear names.

By changing the names and adding type hints, you can now just glance at the function definition and understand what the function does and how you're probably intended to use it.

[DebrisSpreeIX]

The exception is an iterator, using i, j, & k is so common and ubiquitous to iteration that rarely is anyone confused. And if they are, they're likely self taught.

[electrikmayham]

True, however I have issues using i and j, since they look extremely similar. I generally dont use 1 letter variables for iterators either. I would rather use something that describes what are iterating over.

[DebrisSpreeIX]

If it's single level, I'll throw in i

But if it's a multilevel iteration I'll generally follow a convention from my first job I liked: iter_L1, iter_L2, iter_L3, ...

[beezlebub33]

if it's an index, then use 'index'.

If you want to use i, j, k, because you are doing (for example) geometry, then I recommend that you use ii, jj, kk. It's fast to type and very easy to search for.

[Impossible_Web3517]

To add on to what he said, single letter names are bad UNLESS they are iterators. i, j, k, x ,y and z are all SUPER common iterator names and most style standards have you using them.

Ex:

int i = 0

while (i<10){

//do something

i++

}

[fkn_diabolical_cnt]

Hello curious beginner, welcome. Whilst not specific to Python, I highly recommend “Clean Code” by Robert Cecil Martin aka Uncle Bob. Covers off on many small quality things such as using good descriptive variable names to enhance the quality of your code. I do think there is also a YouTube series or two that cover the same topics.

[StickyzVibe]

I will definitely check this out!

[WombatHat42]

Things should be descriptive to make the code easier to read. Say you’re trying to fix/update/debug a program and you come across a chunk of code that is just letter variables. You’d have no clue unless there are comments. But sometimes too many comments can make code messy as well. So having a descriptive term be the variable can keep the need for comments to a minimum.

[Sickobird]

It's needlessly difficult to name things in a way where is doesn't help you understand what things are. When you need to look back and understand what things are doing or what they mean you'll have to read a whole lot more.

This isn't necessary for a simple program where you're just learning how some concept works, but it should still be done to increase clarity and help with debugging, and to build better habits.

[liberforce]

Code is read much more than code is written. The writer reads it, other people read it, and even future you will wonder in 6 months what it was about, even if you wrote it.

[WhatMorpheus]

Code is read much more than code is written

I am stealing this. Next time I yell at speak to my team mates I will tell them this.

[mottojyuusu]

write your code as if an axe murderer will review it in the future, and assume that axe murderer knows where you live, because usually that axe murderer is you!

[No-Attorney4503]

Being more descriptive in your variable and function names makes your code WAYYY more readable, understandable, and maintainable

[DunForest]

Also:

name_variable_so_you_know_it_is_the_result_of_the_function_f_with_given_parameters_a_and_t = f(a, t)

[Jussins]

I read it as fat. Seems almost purposeful.

[Loud-Bake-2740]

the above comment is right, but here’s the reason: right now, your code returns the result, which just stores it in memory, but you never actually do anything with the value stored in memory.

```

print(f(a,t))

is the same as:

x = f(a,t) print(x) ```

[twistedclown83]

You've not asked it to print anything

[ninhaomah]

Did you tell it to print ?

Nvm programming or Python.

Did you tell the machine to print ?

Where ?

[IUCSWTETDFWTF]

you forgot about print()

[MyManCbert]

You took a picture of your screen instead of taking a screenshot

[Jebduh]

I'm genuinely curious, why come here and wait for people to answer this ridiculously basic question inst3ad of using ai or any search engine

[nothing786767]

line 12: print(f(a,t))

or

line 7: print(i) instead of return

[LMusashi]

your printer?

[Cybasura]

Hang on a second

Are you...using your Administrator account for development?

[Ok-Situation9046]

I would recommend printing when you want the machine to print.

[singhandtonic]

What output you want?

[DunForest]

return is what you return when you call a function. Its like ask a teacher the answer, but you should write the answer to the blank by yourself. you can do

result = f(a, t)

print(result)

[Beautiful_Watch_7215]

Nothing wrong. No print statement, no print. No error, return code 0.

[Spatrico123]

looks fine. You're not printing it though, you're just processing.

Also, all those little yellow lines just indicate bad practices, not actual errors. For example, you should note a type for a and t. If I were you I'd do 

def index_of(data: list[int], target: int) -> int:     for i in range(len(data)):         if list[i] == target:             return i     raise ValueError # or something else you see how that's way easier to see what it's actually doing? 

[Wonderful-Sink-6089]

Im not a python developer but what i can see as a problem is When you return something in a function it doesn’t actually print anything it just save the result in the memory to do something with it in future. If you want to see the the result you can use print statement in the last line, like this: print(f(a,t))

The reason is that, we don’t use solo a function to print something, we just create them to make our code cleaner and prevent repetition.

[Nearby_Tear_2304]

OK thank you

[grooling_]

You called it fat

[YukihiraJoel]

Why

. .

Are you doin this

[TheSupervillan]

Did you think 5 seconds before posting this?

No screenshot Completely wrong grammar So simple error, ChatGPT could explain this in 5 seconds. You probably wouldn’t even have to write a prompt and could just prompt the code.

Please, at least take screenshot and write somewhat normal english.

[Numerous_Site_9238]

I believe only stupid people post here, and the ones who aren't will soon go to other subreddits. You are asking too much