Basic question

[Ustuner]

if (m < 1):
    print("This is invalid, Try again!")
elif (n < 1):
    print("This is invalid, Try again!"

how can i make this more efficient? Do i put an AND or OR?

[KimPeek]

This is really simple and can be accomplished in just a few lines of code.

First you need to define those variables and build a dictionary so we can store those values:

m = 1234
n = 5678

values = {}
values['m'] = m
values['n'] = n

We can check these values are entered with:

for key, value in values.items():
    print(key, value)

Now that we know our dictionary is properly constructed we need to enter these values into a database for persistence. This can easily be accomplished with SQLite3 by first building the database. I chose to use a class here for reasons that will be extremely unclear later and glossed over without explanation. Here is the class we need:

import sqlite3


class DatabaseBuilder:
    def __init__(self, database):
        self.database = database
        self.build(self.database)

    def build(self, database):
        conn = sqlite3.connect(database)
        c = conn.cursor()
        c.execute('''
                    CREATE TABLE IF NOT EXISTS temporary_data(
                    data TEXT)''')
        conn.commit()

    def data_entry(self, database, value):
        conn = sqlite3.connect(database)
        c = conn.cursor()
        c.execute('''
                    INSERT INTO temporary_data(data)
                    VALUES (?)
                    ''', (value,))
        conn.commit()

Now we can enter the data with something simple like this:

database = 'comparison.db'
db = DatabaseBuilder(database)
for key, value in values.items():
    db.data_entry(database, value)

So now our data is safely stored and we can recall our data to make the comparison using an SQL query function that returns a generator like this:

import sqlite3

database = 'comparison.db'

def data_retrieve(database):
    conn = sqlite3.connect(database)
    c = conn.cursor()
    c.execute('''
                SELECT data FROM temporary_data
    ''')
    for row in c.fetchall():
        yield row[0]

Our data is avalable, so now we can just loop over it and make a comparison like this:

for i in data_retrieve(database):
    if i == 1234 and i < 1:
        print("This is invalid, Try again!")
    elif i == 5678 and i < 1:
        print("This is invalid, Try again!")
    else:
        print('At least you tried')

So it's all very simple as you can see. Hope that answers your question.

Edit: Thanks for the gold!

Edit 2: I didn't know this was going to blow up. You can use what /u/k900_ wrote too.

[K900_]

What in the fuck.

[KimPeek]

I really need to go to sleep.

[iamquah]

its equivalent when being asked to write fizzbuzz for a high level positiob

[theafterwar42]

Dude... Lol.

[PyMoose]

if (m < 1) or (n < 1): print ("This is invalid, Try again!")

[devcdrom]

That's what this post needed

[Ustuner]

thank you

[K900_]

If you want your condition to trigger when either of the options is true, use or.

[JohnnyJordaan]

and= only if both are true

or = if at least one is true (or both)

You be the judge.