Python Maze Generator

lain iwakura · 18 Nis 2019

Daniel Shiffman'in Coding Challenges adlı video serisinde JavaScript ile yazdığı ettiği Maze Generator'ü Python kullanarak yeniden yazdım. Nasıl çalıştığını merak ediyorsanız, direk videoları izleyebilirsiniz ya da burayı okuyabilirsiniz.

dpaste: dpaste.com/066XDFN

[ame]https://www.youtube.com/watch?v=HyK_Q5rrcr4[/ame]

Gbmdpof · 20 Nis 2019

Elinize sağlık, merak ettiğim şey bu kodu videoyu izleyip mi yazdınız yoksa izlemeden mi? Her iki durumda da oldukça güzel olmuş kodunuz, tekrardan elinize sağlık.

lain iwakura · 22 Nis 2019

Gbmdpof' Alıntı:
Elinize sağlık, merak ettiğim şey bu kodu videoyu izleyip mi yazdınız yoksa izlemeden mi? Her iki durumda da oldukça güzel olmuş kodunuz, tekrardan elinize sağlık.

Konuda da belirttiğim üzere, video üzerine yaptığım bir şey. O yüzden, evet, videoyu izledikten sonra yazdım. Ama elinde sonunda implement edeceğim şey aşağıda yer alan algoritma olduğundan, izleyip izlememiş olmamın nasıl bir etkisi olacağını anlayamadım.

Teşekkür ederim.

Anonim6 · 22 Nis 2019

Ben pek de beğendim. Birkaç öneride bulunacağım,

Kod:

is_valid = lambda i, j: True if not any([
            i < 0, 
            j < 0, 
            i > self.grid.rows - 1, 
            j > self.grid.cols - 1
        ]) else False

Bu kod parçacığında yer alan if-else satırı sadeleştirilebilir,

Kod:

is_valid = lambda i, j: not any([
            i < 0, 
            j < 0, 
            i > self.grid.rows - 1, 
            j > self.grid.cols - 1
        ])

şeklinde. Ne de olsa dönen Boolean değeri direkt olarak fonksiyon dönütüne yerleştirebiliriz. Sınıf hiyerarşisinde ayrı ayrı __repr__ ve __str__ metodlarına aynı işlem namına yer vermek yerine, yalnızca __repr__'ın üstüne yazman kafi. Sonuçta gördüğüm kadarıyla __str__ ile de nesnenin dizgisini yazdırıyorsun. Hücreleme mantığıyla her bir hücreyi ele alarak şekillendirme gayet hoş olmuş. Değişken ve fonksiyon isimlerinde tekrara düştüğünden, misal neighbours, hele de yorum satırına yer vermediğinden okunabilirlikten taviz vermişsin.

Kod:

neighbours = [self.grid[i][j] for i, j in self.neighbours(self.curr_c)]
        neighbours = [n for n in neighbours if n.visited == False]

Satırı, zaten neighbours isimli fonksiyonun uzantısı niteliğinde olduğundan, tekrara düşmeksizin fonksiyon içerisinde barındırmak daha temiz olacaktır, çünkü herhangi bir başka yerde ilgili fonksiyonu kullanmamışsın.

Ve bunun gibi 1-2 minik, önemsiz not daha. Bunların tamamının zorlama, kuru laf olduğunu belirterek ellerine sağlık diyorum. Gayet güzel bir algoritmayı bir dile en olası haliyle uyarlamışsın.

Python Maze Generator

lain iwakura

Yeni üye

Gbmdpof

Kıdemli Üye

lain iwakura

Yeni üye

Anonim6

Yeni üye

Sosyal medya sayfalarımız