Ö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
Ayrılmış bir geliştiriciden kod devraldığınızda ilk iş onu anlamaktır: her parçanın ne yaptığı, hangi parçaların birbirine bağlı olduğu ve hangi satırlara dikkat etmeden dokunulmaması gerektiği. Yapıştırdığınız kodu okur, bölüm bölüm sade bir dille açıklar ve dosyalara ekleyebileceğiniz yorumlar yazarız. Ayrıca ölü kodu, gizli yan etkileri ve riskli noktaları işaretleriz; böylece bir şeyi değiştirmeden önce nerede dikkatli olmanız gerektiğini bilirsiniz.
Neler dahil
- Neler alırsınızSipariş sayfasında yazılı bir inceleme: ana fonksiyonların ve nasıl bağlandıklarının haritası, sade açıklamalar, yapıştırmaya hazır yorumlar ve önerilen düzeltmelerle riskli noktalar listesi. Premium paket, kodu adım adım temizlemek için güvenli bir sıra ekler.
- Kod haritasıAna fonksiyonlar ve nasıl bağlandıkları
- Sade açıklamalarBölüm bölüm
- Hazır yorumlarDosyalara yapıştırmak için
- Riskli noktalarYan etkiler, ölü kod, sihirli sayılar
- Önerilen düzeltmelerGerekli yerlerde düzeltilmiş parçalar
Nasıl çalışır
- 1BrifKodu ve hedefinizi yapıştırırsınız
- 2OkumaKodu haritalar ve açıklarız
- 3İncelemeAçıklamaları ve yorumları alırsınız
Sizden neler gerekiyor
Metin veya ekran görüntüsü olarak yapıştırılmış devralınan kod, dil ve sürüm, sistemin ne için kullanıldığı ve yakında içinde neyi değiştirmeniz gerektiği.
Size soracaklarımız:
- Kod hangi dilde ve sürümde?
- Sistem işletmeniz için ne yapıyor?
- Yakında neyi değiştirmeniz veya eklemeniz gerekiyor?
- Önceki geliştiriciden kalma belgeler veya notlar var mı?
Paketleri karşılaştırın
| Paketler | Temel100 satıra kadar kod$15 | Standart400 satıra kadar kod$35 | PremiumTemizlik sıralı 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 |
| Adım adım temizlik sırası | 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
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.
Modern bir dilde yeniden yazabilir misiniz?
Bu hizmet kodunuzu açıklar, not düşer ve düzeltmeler önerir. Tam bir yeniden yazım ayrı bir geliştirme projesidir.
Kod çok eski. Bu bir sorun mu?
Hayır. Eski PHP, VBA makroları, klasik ASP ve benzeri kod tam da bu incelemenin amacıdır.
Yanıtınızı bulamadınız mı?