eCAPTCHA | CAPTCHA Oluşturucu [AR-GE]

Phone call

Video
​

Gbmdpof' Alıntı:

Sesli CAPTCHA'nın çözülmesini zorlaştımak için gürültü, bipleme gibi sesler eklenecek

Genişletmek için tıkla ...

• Her şeyden önce dikkatimi audio klasörü çekti, içerisinde farklı dillerde rakamların seslendirmeleri var. Özellikle de ileride daha fazla dil seçeneği geleceği için böyle bir yol kullanmak aptallık. Bunun yerine bir TTS API kullanabilirsin, dosyaları tek tek backend'te saklamaktan daha mantıklı.
• İleride farklı fontların geleceğini düşünürsek font dosyalarının da kendilerine ait bir klasörü olsa fena olmaz açıkçası.

• generate_text() ve generate_nums() fonksiyonları
  • ImageCaptcha sınıfının içerisinde olmalı, hem okuyucular hem de geliştiriciler için daha fazla rahatlık sağlar.
  • Sahip oldukları argüman adı built-in fonksiyon olan len() ile karışıyor. Gelecekte olabilecek çakışmaları önlemek amacıyla değiştirilmesi gerek.
  • İki fonksiyon da birer kere çağrılmakta, ayrı bir fonksiyon olmalarına gerek yok.
  • Fonksiyon isimleri çöp, daha esnek ve dinamik isimler seçilmeli.
  • ASCII karakterleri her CAPTCHA üretileceği zaman bir kütüphaneden çekip birleştirmek aptallık, hard-coded yazmak çok daha iyi bir seçim.
  • generate_text() ve generate_nums() fonksiyonunda random.choice() kullanmak ve her seferinde ASCII değerleri toplamak aptallık, random.sample() kullanılması daha iyi bir seçim:
    
    Kod:

    def text(length=5):
      return random.sample('abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789', length)
    
    def number(length=5):
      return random.sample('0123456789', length)

    Kod:

    >>> text()
    ['O', 'S', 'q', 'a', '1']
    >>> number()
    ['9', '5', '6', '1', '2']

• is_int() büyük ihtimalle koddaki en aptalca yazılmış fonksiyon. Python'ın esnekliğini tamamen ihlal ediyor. Böyle saçma bir şey yapmak yerine isinstance() built-in fonksiyonu kullanılmalı:
  

  PY3.7 Docs.' Alıntı:

  Return true if the object argument is an instance of the classinfo argument, or of a (direct, indirect or virtual) subclass thereof. If object is not an object of the given type, the function always returns false. If classinfo is a tuple of type objects (or recursively, other such tuples), return true if object is an instance of any of the types. If classinfo is not a type or tuple of types and such tuples, a TypeError exception is raised. - Source

  Genişletmek için tıkla ...

  Kod:

  >>> isinstance(2, int)
  True
  >>> isinstance('a', int)
  False

• AudioCaptcha sınıfı
  • Neden ses dosyaları ile ilgili değişkenleri private yaptığını anlamış değilim açıkçası. Hiçbir sebep göremiyorum.
  • _LANGS değişkeni tanımlanmış ancak kullanılmamış, harika.
  • _EN_NUMBERS ve _TR_NUMBERS değişkenleri sadece tek bir yerde kullanılmış, değişken olmalarına gerek yok.
  • Private olarak tanımladığın değişkenler ve dirs, __init__'in içerisinde bulunmalı. Her nasılsa zaten kendi yazdığına karşı gelip self kullanarak erişmişsin.
  • __init__() fonksiyonunda okunabilirlik için "Ask forgiveness, not permission." ilkesi kullanılmalıydı:
    
    Kod:

    def __init__(self, lang='en'):
        try: nums_dir = self.dirs[lang]
        except: raise LanguageNotAvailable(f'Language {lang} not available!')

  • generate fonksiyonu
    • kwargs sadece kullanılmak için kullanılmış. Buradaki kullanımı yersiz ve gereksiz olmakla kalmıyor aynı zamanda okunabilirliği düşürüyor.
    • output'un varsayılan değeri CAPTCHA değerine eşit olsaydı end-user için işler daha kolay olabilirdi.
    • Wave formatındaki dosyaları birleştirmek için neyi nasıl ve neden kullandığını anlamam yarım saatimi aldı. Benim yazdığım daha basit, ancak test etmedim:
      
      Kod:

      with wave.open(output, 'wb') as out:
          for f in files:
              with wave.open(f, 'rb') as ff:
                  if not out.getnframes(): out.setparams(ff.getparams())
                  out.writeframes(ff.readframes(ff.getnframes()))

      
      Daha okunabilir hale getirilebilirdi elbette ama uğraşmak istemedim.
• ImageCaptcha sınıfı
  • Renklerin RGB formatında hard-coded bir şekilde durması ve programın bu renkleri kullanması saldırganın işini çok kolaylaştırır. Bunun yerine RGB değerlerini random seçse daha mantıklı olurdu.
  • COLORS değişkeni __init__'in içerisinde olmalı yine, ancak tekrar kendi yazdığını çiğneyip self ile erişmişsin.
  • Neden width, height ve font'u private olarak belirlediğini de anlamadım.
  • generate fonksiyonu
    • kwargs sadece kullanılmak için kullanılmış. Buradaki kullanımı yersiz ve gereksiz olmakla kalmıyor aynı zamanda okunabilirliği düşürüyor.
    • Oluşturulan resmin arkaplan rengi de yine hard-coded olmak yerine rastgele bir değer seçilmeliydi.
    • Çizilen çizgilerin kalınlıkları ve sayıları rastgele seçilmeliydi.
• Kodu anlamak bu yazıyı yazmaktan daha uzun sürdü, nedeni ise kodda yorum sayısının tam olarak 0 (sıfır) olması. Yorumu bırak docstring bile yok.
• Kodda bahsettiğim yerler dışında kalan yerlerin neredeyse tamamında PEP8'e uyulmamış.
• Değişken, fonksiyon ve sınıf isimleri çöp, hepsinin tek tek daha açıklayıcı ve basit bir şekilde, tekrar etmeden yazılması gerek.
• OOP yapacağım diye projeyi öpmek iyi bir seçenek değil.