Problem zu vereinfachen sich wiederholende - mehrere wenn elif-Anweisungen und mehrere Schleifen in einer Funktion in Python

Question

Ich bin sehr neu in Python.

Ich bin ein überzeugter Anhänger des einfachen, prägnanten und effizienten algorithmischen Designs sowie des Kodierungsstils. Während ich Python lerne, merke ich, dass Python eine Menge Dinge hinter der Szene macht, so dass die Sprache selbst für Programmierer sehr freundlich ist. Das ist schön, aber ich wollte genau darüber lernen, welche Optimierungen ich beim Codieren machen oder beibehalten kann. Und heute geriet ich in Schwierigkeiten, meinen Code zu vereinfachen.

Die folgende Funktion wird verwendet, um leere Punkte auf einem Sudoku-Board basierend auf dem gewählten Schwierigkeitsgrad zu erstellen.

Hier ist mein Code:

class normalSudoku(Board): 
    def __init__(self,difficulties): 
     super.__init__() 
     self.Create_Empty_Entries(difficulties) 


    def Create_Empty_Entries(self,difficulties): 
     numbers = list(range(0,9)) 
     if difficulties == "Easy": 
      for x in range(25): 
       a,b = choice(numbers),choice(numbers) 
       if self.sudoku[a][b] != None: 
        self.sudoku[a][b] = None 
        self.holes += 1 
       self.holes += 1 
      return None 

     elif difficulties == "Medium": 
      for x in range(35): 
       a,b = choice(numbers),choice(numbers) 
       if self.sudoku[a][b] != None: 
        self.sudoku[a][b] = None 
        self.holes += 1 
      return None 

     elif difficulties == "Hard": 
      for x in range(45): 
       a,b = choice(numbers),choice(numbers) 
       if self.sudoku[a][b] != None: 
        self.sudoku[a][b] = None 
        self.holes += 1 
      return None 

     else: 
      for x in range(65): 
       a,b = choice(numbers),choice(numbers) 
       if self.sudoku[a][b] != None: 
        self.sudoku[a][b] = None 
        self.holes += 1 
      return None 

Wie Sie es ist sehr repetitiv sehen. Jede Idee zur Vereinfachung oder ein effizienterer Codierungsstil wird geschätzt.

Gibt es auch eine bessere Möglichkeit, eine Klasse in Python zu initialisieren, anstatt __init__() in Bezug auf Leistung und Speichernutzung aufzurufen? Genau wie in C++ gibt es eine Initialisierungsliste, wo es sauberer und schneller ist.

Bitte zögern Sie nicht, auf die Fehler, die ich gemacht habe, hinzuweisen. Jeder Rat wird sehr geschätzt. Danke

Answer 1

Da die einzige Sache, die sich ändert, ist die Auswahl der Zahlen, von denen ich empfehle, ein Diktat zu schaffen, wo Schwierigkeit zu dieser Nummer dann das Verwenden in einer einzelnen Funktion bildet, die die Zahlen festlegt.

class normalSudoku(Board): 
    def __init__(self,difficulties): 
     super.__init__() 
     self.Create_Empty_Entries(difficulties) 


    def Create_Empty_Entries(self,difficulties): 
     numbers = list(range(0,9)) 
     difficulty_values = {'Easy':25,'Medium':35, 'Hard':45, 'Default':65} 

     # check the difficulty level exists in the dict. 
     # If it does, use that value, if it doesn't then use the default value 
      difficulty = difficulty_values.get(difficulties, difficulty_values['Default']) 

      # now use that difficulty to set the numbers once. 
      for x in range(difficulty): 
      a,b = choice(numbers),choice(numbers) 
      if self.sudoku[a][b] != None: 
       self.sudoku[a][b] = None 
       self.holes += 1 
      self.holes += 1 
    return None 

Answer 2

Sie könnten eine Prüfmethode Ihnen Klasse hinzufügen:

# add this to the class body 
def auto_increment(self, a, b): 
    if self.sudoku[a][b] != None: 
     self.sudoku[a][b] = None 
     self.holes += 1 
    self.holes += 1 
    return 

Dann können Sie einfach Sie Parameter an Ihre Methode übergeben werden:

self.auto_increment(choices(number), choices(number)) 

Slots sind ein effektiver Weg zur Reduzierung Speichernutzung Usage of __slots__?

Answer 3

Eine Option ist das Verschieben die Parameter von Code zu Daten und arbeiten dann mit den Daten:

# You could source these two dicts from elsewhere, like a JSON/YAML/config file 
difficulties = { 
    Easy: { 
    size: 25 
    }, 
    Medium: { 
    size: 35 
    }, 
    Hard: { 
    size: 45 
    } 
} 

defaultDifficulty = { 
    size: 65 
} 

# ... 

def Create_Empty_Entries(self, difficultyName): 
    if difficultyName in difficulties: 
    difficulty = difficulties[difficultyName] 
    else: 
    difficulty = defaultDifficulty 

    numbers = list(range(0,9)) 
    for x in range(difficulty.size): 
    a,b = choice(numbers),choice(numbers) 
    if self.sudoku[a][b] != None: 
     self.sudoku[a][b] = None 
     self.holes += 1