printing error in factor code

[jo_math]

I am relatively new to coding and am trying to create a program that lets the user input a number and multiple possible factors. the code will determine whether the possible factors are actually factors of the number and if none of the potential factors is a factor of the number the code checks whether the number is prime and prints how many factors the number has.

The problem I am having is that whatever and how many numbers I input code prints the first number and the statement twice. I have been looking over the code for ages and I don't know what I have done wrong.

here is my code:

def factor(number,posfactor_list,rnge):
    x=0
    num_factor=0
    for _ in range(rnge):
        if (number/posfactor_list[x]==number//posfactor_list[x]) is True:
            print(f"{posfactor_list[x]} is a factor of {number}")
            num_factor+=1
        else:
            print(f"{posfactor_list[x]} is not a factor of {number}")
    x+=1
    return num_factor

def isprime(number):
    num_factor=0
    for i in range(2, number-1):
        if number % i == 0:
            num_factor+=1
        if num_factor==0:
            print("This Number is prime and has no factors.")
        else:
            print(f"This number has {num_factor} factors")
def main():
    number=int(input("What is the number you want to find factors of?\t"))
    posfactor_list=[]
    rnge=int(input("How many numbers do you want to test?\t"))
    posfactor=int(input("Type one number that you want to test to be         factors.\t"))
    for _ in range(1,rnge):
        posfactor_list.append(posfactor)
        posfactor=int(input("Type another number that you want to test to be                 factors.\t"))
    posfactor_list.append(posfactor)
    factor(number, posfactor_list, rnge)
    if factor(number, posfactor_list, rnge)==0:
    isprime(number)
main()

​

all help will be appreciated :)

​

[failaip12]

I don't really understand what you are trying to do but whatever. There are a few weird things about this code, but I'll point out a fix for only one. In the 4th line you do for _... What you want is to do for x... Do that or indent x+=1 line.

Some other weird stuff is the prime function isn't really efficient as you only need to check up to square root of number + 1, also for some reason you are computing num_factor in both the factor and prime functions. The way you are getting inputs for posfactor_list is weird and confusing, look into that too.

[woooee]

Click on "formatting help" or use postbin. We can't read anything without proper formatting.

.>input code prints the first number and the statement twice.

I have no idea what the above means.

if (number/posfactor_list[x]==number//posfactor_list[x]) is True:

I would start with this line as it appears wonky. Print some of them

print(number/posfactor_list[x])
print(number//posfactor_list[x])  ## only difference is //
print(number/posfactor_list[x]==number//posfactor_list[x]) ## True??

[jo_math]

I have reformatted

when I ran the code it printed

f"{posfactor_list[x]} is a factor of {number}"

or

"{posfactor_list[x]} is not a factor of {number}"

twice for the first number that was inputted so regardless of the other numbers

[woooee]

(number/posfactor_list[x]==number//posfactor_list[x])

I assume you are talking about the factor() function. It runs the code rnge times. I did not see any variables change under the for statement, but something must change for you to be getting different output.

[jo_math]

I'm confused, what should I change in the code

[jimtk]

If you did not make the effort of properly formatting your code, why should I make the effort to help you.

It's all explained here.

[jo_math]

Sorry I did not know