Node.js API incelemesiProgramlama Eğitimi

Node.js ve Express API kod incelemesi

Genellikle 15 dakika içinde hazırRevizyonlar dahilKalite kontrolü
Paketler
Ödeme korumasıAyrıntılar
  • Paket kapsamındaki revizyonlar ücretsizdir.
  • Teslimattan sonraki 3 gün içinde yanıt vermezseniz iş otomatik olarak onaylanır.

Para, ödeme yaptığınızda bloke edilir ve yalnızca işi onayladığınızda aktarılır.

Node.js ve Express API kod incelemesi

Örnek sonuç

Standart paket için örnek

Code review: Python order-import script for a small online shop

What the code does

  1. Reads orders.csv from the shop export.
  2. For each order line, finds the product by SKU in stock.xlsx and subtracts the quantity.
  3. Writes the updated stock back and logs totals to import_log.txt.

The structure is clear and functions are well named. Most issues are about input that does not look the way the script expects.

Bugs and weak spots

#LineSeverityIssue
148HighThe same CSV can be imported twice; there is no check for already processed order IDs. This is the most likely cause of negative stock.
272Highint(row["qty"]) crashes on empty or "2.0" values from refunded orders.
395MediumSKU lookup is case-sensitive: "gt-101" and "GT-101" are treated as different products.
4130MediumThe file is opened without encoding="utf-8", so product names with accents break on Windows.
5201LowBare except: hides real errors and makes the log useless.

Corrected fragments

Issue 1: skip orders already imported

processed = load_processed_ids("processed_orders.txt")
for row in reader:
    if row["order_id"] in processed:
        continue
    # ... existing logic ...
    processed.add(row["order_id"])
save_processed_ids("processed_orders.txt", processed)

Why: each order is applied once, even if you run the script on the same file again.

Issue 2: read quantity safely

raw = (row.get("qty") or "").strip()
try:
    qty = int(float(raw))
except ValueError:
    log.warning("Order %s: bad quantity %r, skipped", row["order_id"], raw)
    continue

Why: refunded or partial lines are skipped with a clear note instead of stopping the whole import.

Issue 3: normalise SKUs

sku = row["sku"].strip().upper()

Apply the same .strip().upper() when building the stock dictionary.

Style and readability

  • process() is 90 lines long; split it into read_orders, apply_to_stock, write_report.
  • Replace magic column numbers (row[7]) with header names.

Please test each fix on a copy of your files: we read the code but did not run it.

Client: owner of a tea shop who wrote a 310-line Python script that imports orders from a CSV export into the stock spreadsheet Problem reported: "Sometimes stock goes negative, and the script crashes on some files."

Müşteri ve tüm ayrıntılar kurgusaldır.

Bu hizmet hakkında

Express API'leri aynı noktalarda bozulma eğilimindedir: hata fırlatıp isteği asılı bırakan bir async rota, yanlış sıradaki middleware, doğrulanmadan kullanılan istek gövdeleri ve diğer tüm kullanıcıları bloklayan senkron dosya çağrıları. Rotalarınızı, middleware'inizi ve veritabanı çağrılarınızı okur, bir isteğin bunlar arasında nasıl ilerlediğini açıklar, çökmeleri, güvenlik açıklarını ve yavaş noktaları işaretleriz. Düzeltilmiş parçalar, merkezi hata yönetimi ve net durum kodlarıyla async ve await kullanır.

Neler dahil

  • Neler alırsınızSipariş sayfasında yazılı bir inceleme: bir isteğin API'nizdeki yolu, önem sırasına göre sorunlar ve düzeltilmiş rota ile middleware kodu. Premium paket, rotalar, servisler ve veri erişimi için önerilen bir klasör yapısı ekler.
  • İstek akışıMiddleware sırası ve yönlendirme
  • Hata yönetimiAsılı kalan istek veya çökme yok
  • Girdi doğrulamaGövdeler, parametreler ve sorgu dizeleri
  • Olay döngüsüBloklayan çağrılar bulunur ve değiştirilir
  • Gizli bilgiler ve yapılandırmaAnahtarlar koddan çıkar, ortam değişkenleri kullanılır

Nasıl çalışır

  1. 1
    BrifRotaları, middleware'i ve günlükleri yapıştırırsınız
  2. 2
    OkumaBir isteği baştan sona izleriz
  3. 3
    İncelemeSorunları ve düzeltilmiş kodu alırsınız

Sizden neler gerekiyor

Metin veya ekran görüntüsü olarak yapıştırılmış rotalarınız, middleware'iniz ve ana uygulama dosyanız, Node.js ve Express sürümleri, kullandığınız veritabanı ve sunucu günlüğündeki hatalar.

Size soracaklarımız:

  • Hangi Node.js ve Express sürümleri?
  • Hangi veritabanını ve kütüphaneyi kullanıyorsunuz?
  • Ne ters gidiyor ve sunucu günlüğü ne diyor?
  • API herkese açık mı yoksa yalnızca kendi ön yüzünüz tarafından mı kullanılıyor?

Paketleri karşılaştırın

PaketlerTemel100 satıra kadar API kodu$15Standart400 satıra kadar API kodu$35PremiumYapı planlı 1.000 satıra kadar$69
Teslim süresi1 gün2 gün3 gün
Revizyonlar123
Kod satırıen fazla 100en fazla 400en fazla 1.000
Düzeltilmiş parçalarEvetEvetEvet
Klasör yapısı planıHayırHayırEvet
Sipariş ver TemelSipariş ver StandartSipariş ver Premium

Ödeme koruması

Sipariş verirken ödersiniz, ancak para yalnızca onaylanan iş için aktarılır.

  • Sabit fiyatPaket fiyatı, ödeme yaptıktan sonra değişmez.
  • Para bloke edilirÖdeme yaptığınızda tutar yalnızca bloke edilir. İşi onayladığınızda hizmete aktarılır.
  • Teslimattan önce kontrolHer sonuç size ulaşmadan önce kalite kontrolünden geçer ve onaylamadan önce sipariş sayfasında görürsünüz.
  • Ücretsiz revizyonlarBrifiniz kapsamında, paketin izin verdiği sayıda değişiklik isteyin.
  • 3 gün sonra onayTeslimattan sonraki 3 gün içinde işi onaylamaz veya değişiklik istemezseniz iş otomatik olarak onaylanır. Teslimattan sonra sipariş sohbetine yazarsanız sipariş bekletilir; yanıt verdikten sonra 3 günlük süreyi yeniden başlatabiliriz ve bunu sohbette bildiririz.
  • Yeniden yapım veya tam iadeSonuç brifinize uymuyorsa ve revizyonlar sorunu çözmediyse ya da iş teslim süresinin iki katından uzun sürerse sipariş sohbetine yazın: işi yeniden yaparız veya ödemenin tamamını iade ederiz.
Nasıl çalışır →

Sorular ve yanıtlar

Kodumu çalıştırıyor musunuz?

Hayır. Dikkatle okur ve nasıl davranacağını akıl yürüterek değerlendiririz, ancak çalıştırmayız; bu yüzden önerilen her düzeltmeyi kendi projenizde test edin.

Göndermeden önce API anahtarlarımı kaldırmalı mıyım?

Evet. Anahtarları, parolaları ve bağlantı dizelerini yer tutucularla değiştirin. Kodu incelemek için onlara ihtiyacımız yok.

API'me yük testi yapabilir misiniz?

Hayır, sunucunuza çağrı yapmayız. Yük altında yavaşlayacak kodu gösteririz, testi siz yaparsınız.

Yanıtınızı bulamadınız mı?

Paketler