Ö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.
Örnek sonuç
Code review: Python order-import script for a small online shop
What the code does
- Reads
orders.csvfrom the shop export. - For each order line, finds the product by SKU in
stock.xlsxand subtracts the quantity. - 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
| # | Line | Severity | Issue |
|---|---|---|---|
| 1 | 48 | High | The same CSV can be imported twice; there is no check for already processed order IDs. This is the most likely cause of negative stock. |
| 2 | 72 | High | int(row["qty"]) crashes on empty or "2.0" values from refunded orders. |
| 3 | 95 | Medium | SKU lookup is case-sensitive: "gt-101" and "GT-101" are treated as different products. |
| 4 | 130 | Medium | The file is opened without encoding="utf-8", so product names with accents break on Windows. |
| 5 | 201 | Low | Bare 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 intoread_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
İşe alım ekibi bir junior'ın portfolyo projesini açtığında birkaç işarete göz gezdirir: net adlar, küçük fonksiyonlar, hata yönetimi, birkaç test ve projenin ne yaptığını ve nasıl çalıştırılacağını anlatan bir README. Projenizin ana dosyalarını yapıştırın, bir inceleyicinin yapacağı gibi inceleyelim; yarım kalmış görünenleri, eğitimden kopyalanmış görünenleri ve gerçek beceri gösterenleri işaretleriz. Düzeltilmiş parçalar ve ilk izlenimi en çok iyileştiren değişikliklerin kısa bir listesini alırsınız.
Neler dahil
- Neler alırsınızSipariş sayfasında yazılı bir inceleme: ilk izlenim notları, önem sırasına göre kod sorunları, düzeltilmiş parçalar ve önerilen bir README taslağı. Premium paket, bir inceleyicinin fark edeceği eksikler için bir çalışma planı ekler.
- İlk izlenimBir inceleyicinin dakikalar içinde fark ettikleri
- Kod kalitesiAdlar, fonksiyonlar ve hata yönetimi
- TestlerBirkaç testin en çok işe yarayacağı yerler
- README taslağıNe yaptığı ve nasıl çalıştırılacağı
- Öncelik listesiEtkiye göre sıralanmış değişiklikler
Nasıl çalışır
- 1BrifAna dosyaları ve hedef rolünüzü yapıştırırsınız
- 2Okumaİşe alım inceleyicisi gibi inceleriz
- 3İncelemeDüzeltmeleri ve öncelik listesini alırsınız
Sizden neler gerekiyor
Metin veya ekran görüntüsü olarak yapıştırılmış projenizin ana dosyaları, mevcut README'niz, teknoloji yığını, başvurduğunuz rol ve projenin neyi göstermesini istediğiniz.
Size soracaklarımız:
- Hangi rol için başvuruyorsunuz?
- Proje ne yapıyor ve hangi teknoloji yığınını kullanıyor?
- Kendi fikriniz mi yoksa bir kursa mı dayanıyor?
- Hangi dosyalar en iyi çalışmanızı gösteriyor?
Paketleri karşılaştırın
| Paketler | Temel100 satıra kadar ana dosyalar$15 | Standart400 satıra kadar proje kodu$35 | PremiumÇalışma planlı 1.000 satıra kadar$69 |
|---|---|---|---|
| Teslim süresi | 1 gün | 2 gün | 3 gün |
| Revizyonlar | 1 | 2 | 3 |
| Kod satırı | en fazla 100 | en fazla 400 | en fazla 1.000 |
| Düzeltilmiş parçalar | Evet | Evet | Evet |
| Çalışma planı | Hayır | Hayır | Evet |
| Sipariş ver Temel | Sipariş ver Standart | Sipariş 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.
Sorular ve yanıtlar
GitHub profilime bakabilir misiniz?
Buraya yapıştırdığınız kodu ve README'yi inceleriz. Depolarınıza göz atmayız, bu yüzden kapsanmasını istediğiniz dosyaları ekleyin.
Bu beni işe aldırır mı?
Projeyi okumayı kolaylaştırır ve inceleyicilerin beklediğine yaklaştırır, ancak işe alım kararları tek bir projeden çok daha fazlasına bağlıdır.
Projeyi benim için yazar mısınız?
Hayır. Proje sizin kendi işiniz olmalı; biz inceler ve nasıl geliştirileceğini gösteririz.
Yanıtınızı bulamadınız mı?