Flatlogic Bot 6bd90312de Initial import

2026-03-04 18:25:09 +00:00

13 KiB

Raw Permalink Blame History

🔧 Kapadokya Kuralları - Kritik Düzeltme (Görsel)

🎯 Sorun: SMART RESTAURANT Bypass

❌ ÖNCE (Hatalı Akış)

┌─────────────────────────────────────────────────────────────┐
│                    GÜN 1 - HATA SENARYOSU                   │
└─────────────────────────────────────────────────────────────┘

1️⃣ FLEXIBLE PLACES
   ┌─────────────────────────────────────────────┐
   │ Göreme Açık Hava Müzesi (museum)            │
   │ ✅ isValidForDay() → FLEXIBLE → İZİN        │
   └─────────────────────────────────────────────┘
   
   ┌─────────────────────────────────────────────┐
   │ Cafe Safak (cafe)                           │
   │ ✅ isValidForDay() → LIMITED → İZİN         │
   │    (günde henüz LIMITED yok)                │
   └─────────────────────────────────────────────┘
   
   ┌─────────────────────────────────────────────┐
   │ Paşabağ Vadisi (valley)                     │
   │ ✅ isValidForDay() → FLEXIBLE → İZİN        │
   └─────────────────────────────────────────────┘

   dayPlaces = [Göreme, Cafe Safak, Paşabağ]

2️⃣ SMART RESTAURANT
   hasRestaurant = dayPlaces.some(isRestaurant)
                 = dayPlaces.some(p => p.type === 'restaurant' || p.type === 'cafe')
                 = true (Cafe Safak var)
   
   ❌ SORUN: hasRestaurant kontrolü yanlış!
      - Cafe var ama hasRestaurant = false olarak hesaplanıyor
      - Çünkü isRestaurant fonksiyonu cafe'yi de kontrol ediyor
      - AMA eğer cafe FLEXIBLE PLACES'te eklenmişse...
   
   ❌ ASIL SORUN: isValidForDay() kontrolü YOK!
   
   ┌─────────────────────────────────────────────┐
   │ Seten Restaurant (restaurant)               │
   │ ❌ DOĞRUDAN EKLENDİ (kural kontrolü yok!)   │
   │ ❌ LIMITED kuralı bypass edildi!            │
   └─────────────────────────────────────────────┘

   dayPlaces = [Göreme, Seten Restaurant, Cafe Safak, Paşabağ]
                         ↑ Ortaya eklendi (splice)

❌ SONUÇ: Aynı günde hem cafe hem restaurant var!

✅ SONRA (Düzeltilmiş Akış)

┌─────────────────────────────────────────────────────────────┐
│                    GÜN 1 - DOĞRU SENARYO                    │
└─────────────────────────────────────────────────────────────┘

1️⃣ FLEXIBLE PLACES
   ┌─────────────────────────────────────────────┐
   │ Göreme Açık Hava Müzesi (museum)            │
   │ ✅ isValidForDay() → FLEXIBLE → İZİN        │
   └─────────────────────────────────────────────┘
   
   ┌─────────────────────────────────────────────┐
   │ Cafe Safak (cafe)                           │
   │ ✅ isValidForDay() → LIMITED → İZİN         │
   │    (günde henüz LIMITED yok)                │
   └─────────────────────────────────────────────┘
   
   ┌─────────────────────────────────────────────┐
   │ Paşabağ Vadisi (valley)                     │
   │ ✅ isValidForDay() → FLEXIBLE → İZİN        │
   └─────────────────────────────────────────────┘

   dayPlaces = [Göreme, Cafe Safak, Paşabağ]

2️⃣ SMART RESTAURANT
   hasRestaurant = dayPlaces.some(isRestaurant)
                 = true (Cafe Safak var)
   
   ✅ hasRestaurant = true → SMART RESTAURANT atlandı
   
   VEYA (eğer cafe yoksa):
   
   hasRestaurant = false
   
   ┌─────────────────────────────────────────────┐
   │ Seten Restaurant (restaurant)               │
   │ ✅ isValidForDay() kontrolü:                │
   │    - dayPlaces'te LIMITED var mı?           │
   │    - EVET (Cafe Safak)                      │
   │    - return false                           │
   │ ❌ REDDEDİLDİ (LIMITED kuralı)              │
   └─────────────────────────────────────────────┘

   dayPlaces = [Göreme, Cafe Safak, Paşabağ]
               (değişmedi)

✅ SONUÇ: Günde sadece 1 LIMITED tip (cafe)

📊 Kod Karşılaştırması

❌ ÖNCE (Satır 1066-1069)

if (restaurants.length > 0) {
  dayPlaces.splice(1, 0, restaurants[0]);        // ❌ Doğrudan ekleniyor
  usedPlaceIds.add(restaurants[0].id);           // ❌ Kural kontrolü yok
}

Sorun:

Restaurant doğrudan ekleniyor
isValidForDay() kontrolü yok
LIMITED kuralı bypass ediliyor

✅ SONRA (Satır 1066-1073)

if (restaurants.length > 0) {
  const restaurant = restaurants[0];
  
  // ✨ KURAL KONTROLÜ: Type-based validation (LIMITED rule)
  if (isValidForDay(restaurant, dayPlaces, usedPlaceIds, { balloonAdded })) {
    dayPlaces.splice(1, 0, restaurant);          // ✅ Kural kontrolünden sonra
    usedPlaceIds.add(restaurant.id);             // ✅ Sadece geçerliyse ekle
  }
}

Düzeltme:

Restaurant önce değişkene atanıyor
isValidForDay() ile kontrol ediliyor
Sadece geçerliyse ekleniyor

🔍 isValidForDay() Detaylı Akış

┌─────────────────────────────────────────────────────────────┐
│         isValidForDay(restaurant, dayPlaces, ...)           │
└─────────────────────────────────────────────────────────────┘
                            │
                            ▼
                ┌───────────────────────┐
                │ 1️⃣ TEKRARLAMA KONTROLÜ│
                │ usedPlaceIds.has(id)? │
                └───────────────────────┘
                    │           │
                    │ HAYIR     │ EVET
                    ▼           ▼
                ┌──────┐    ┌──────┐
                │ Devam│    │ ❌   │
                └──────┘    │REDDET│
                    │       └──────┘
                    ▼
                ┌───────────────────────┐
                │ 2️⃣ KATEGORİ BELİRLE   │
                │ getPlaceCategory      │
                │ ('restaurant')        │
                │ → 'LIMITED'           │
                └───────────────────────┘
                            │
                            ▼
                ┌───────────────────────┐
                │ 3️⃣ LIMITED KONTROLÜ   │
                │ dayPlaces'te LIMITED  │
                │ var mı?               │
                └───────────────────────┘
                    │           │
                    │ HAYIR     │ EVET
                    ▼           ▼
                ┌──────┐    ┌──────┐
                │ ✅   │    │ ❌   │
                │İZİN  │    │REDDET│
                └──────┘    └──────┘

🎯 Gerçek Dünya Örneği

Senaryo: 2 Günlük Kapadokya Turu

❌ ÖNCE (Hatalı)

GÜN 1:
  09:00 - Göreme Açık Hava Müzesi (museum)
  12:00 - Cafe Safak (cafe)              ← FLEXIBLE PLACES'ten
  13:00 - Seten Restaurant (restaurant)  ← SMART RESTAURANT'tan (HATA!)
  15:00 - Paşabağ Vadisi (valley)

  ❌ SORUN: Aynı günde hem cafe hem restaurant!
  ❌ LIMITED kuralı ihlal edildi!

GÜN 2:
  09:00 - Zelve Açık Hava Müzesi (museum)
  12:00 - Dibek Restaurant (restaurant)  ← SMART RESTAURANT'tan
  15:00 - Devrent Vadisi (valley)

✅ SONRA (Doğru)

GÜN 1:
  09:00 - Göreme Açık Hava Müzesi (museum)
  12:00 - Cafe Safak (cafe)              ← FLEXIBLE PLACES'ten
  15:00 - Paşabağ Vadisi (valley)
  17:00 - Uçhisar Kalesi (viewpoint)

  ✅ DOĞRU: Günde sadece 1 LIMITED tip (cafe)
  ✅ Restaurant reddedildi (LIMITED kuralı)

GÜN 2:
  09:00 - Zelve Açık Hava Müzesi (museum)
  12:00 - Dibek Restaurant (restaurant)  ← SMART RESTAURANT'tan
  15:00 - Devrent Vadisi (valley)
  17:00 - Avanos Seramik (workshop)

  ✅ DOĞRU: Günde sadece 1 LIMITED tip (restaurant)

📈 Etki Analizi

Önce (Hatalı)

Gün	Cafe	Restaurant	LIMITED Sayısı	Durum
1	✅	✅	2	❌ HATA
2	❌	✅	1	✅ Doğru

Sorun: Gün 1'de LIMITED kuralı ihlal ediliyor

Sonra (Doğru)

Gün	Cafe	Restaurant	LIMITED Sayısı	Durum
1	✅	❌	1	✅ Doğru
2	❌	✅	1	✅ Doğru

Sonuç: Her günde LIMITED kuralı uygulanıyor

🧪 Test Matrisi

Test	FLEXIBLE PLACES	SMART RESTAURANT	Sonuç
Cafe eklendi	✅ Cafe	❌ Restaurant reddedildi	✅ Doğru
Restaurant eklendi	✅ Restaurant	❌ Atlandı (hasRestaurant=true)	✅ Doğru
Hiçbiri eklenmedi	❌	✅ Restaurant eklendi	✅ Doğru
İkisi de aday	✅ Cafe (önce geldi)	❌ Restaurant reddedildi	✅ Doğru

🎓 Kritik Noktalar

1. hasRestaurant Kontrolü Yeterli Değil

const hasRestaurant = dayPlaces.some(isRestaurant);

Sorun:

Sadece restaurant/cafe var mı kontrol eder
LIMITED kategorisini kontrol etmez
Gelecekte yeni LIMITED tipler eklenirse çalışmaz

Çözüm:

isValidForDay() kategori bazlı kontrol yapar
Tüm LIMITED tipleri kapsar
Genişletilebilir yapı

2. Tüm Ekleme Noktalarında Kontrol Gerekli

✅ BALLOON        → shouldAddBalloon()
✅ FLEXIBLE       → isValidForDay()
✅ SMART REST     → isValidForDay() (YENİ!)
✅ MIN FILL       → isValidForDay()

Önemli:

Bir yer timeline'a eklendiği HER noktada kontrol gerekli
Hiçbir bypass noktası bırakılmamalı
Tutarlı kural uygulaması şart

3. Kod Değişikliği Minimal

  if (restaurants.length > 0) {
+   const restaurant = restaurants[0];
+   
+   // ✨ KURAL KONTROLÜ
+   if (isValidForDay(restaurant, dayPlaces, usedPlaceIds, { balloonAdded })) {
-     dayPlaces.splice(1, 0, restaurants[0]);
-     usedPlaceIds.add(restaurants[0].id);
+     dayPlaces.splice(1, 0, restaurant);
+     usedPlaceIds.add(restaurant.id);
+   }
  }

Avantajlar:

Minimal değişiklik (5 satır)
Mevcut yapıya uyumlu
Test edilmiş fonksiyon kullanımı

✅ Doğrulama

Tüm Ekleme Noktaları

$ grep -n "dayPlaces.push\|dayPlaces.splice" src/db/api.ts

1021:  dayPlaces.push(balloon);           # ✅ shouldAddBalloon()
1038:  dayPlaces.push(place);             # ✅ isValidForDay()
1071:  dayPlaces.splice(1, 0, restaurant);# ✅ isValidForDay() (YENİ!)
1090:  dayPlaces.push(p);                 # ✅ isValidForDay()

Tüm Kural Kontrolleri

$ grep -n "isValidForDay\|shouldAddBalloon" src/db/api.ts

1006:  shouldAddBalloon(...)              # ✅ BALLOON
1034:  isValidForDay(place, ...)          # ✅ FLEXIBLE PLACES
1070:  isValidForDay(restaurant, ...)     # ✅ SMART RESTAURANT (YENİ!)
1086:  isValidForDay(p, ...)              # ✅ MIN FILL

✅ Sonuç: Tüm ekleme noktalarında kural kontrolü var!

Tarih: 2025
Durum: ✅ Düzeltildi
Değişiklik: 1 dosya, 5 satır
Etki: Kritik - LIMITED kuralı artık tam olarak çalışıyor

13 KiB Raw Permalink Blame History Unescape Escape