⚡ Bolt: [Derleme hatası düzeltmesi ve gereksiz dizi kaldırma]

.jules/bolt.md

@@ -12,3 +12,7 @@
 ## 2026-05-12 - [Remove Duplicate Static Initialization & Unsafe String Operations]
 **Learning:** The previous optimization attempt introduced a bug where 'cached_nodes' was initialized twice as different types (one array of xml_node pointers, one array of const char*). Moreover, directly casting const char* returned from pugixml without assigning to std::string when used in ternary operations can cause operand type mismatches.
 **Action:** Removed the redundant array initialization block and cast the char* obtained via `pt.text().get()` from the single cached xml_node array to `std::string` inside the ternary conditional to prevent implicit conversion mismatches.
+
+## 2026-06-06 - [Avoid Premature Micro-optimizations]
+**Learning:** Modern C++ std::string implementations use Small String Optimization (SSO) for strings under ~15-22 characters, allocating them directly on the stack rather than the heap. Attempting to manually replace `std::to_string` with C-style character arrays and ASCII arithmetic for short strings (like "12:59") is a premature micro-optimization that severely harms readability while providing no real-world performance benefit, as heap allocations are already avoided by SSO.
+**Action:** Do not sacrifice code readability to avoid `std::to_string` for short formatted strings. Keep utilizing standard idiomatic C++ library features for string generation unless profiling proves it to be an actual bottleneck due to sizes exceeding the SSO limit.

src/src-class/Zaman.cpp

@@ -111,16 +111,7 @@ void zaman::vkt_h_v_d()
 		return nodes;
 	}();
 
-	static const char* cached_nodes[400] = {nullptr};
-	static bool cached_nodes_init = []() {
-		for (pugi::xml_node pt = cached_sehir.child("prayertimes"); pt; pt = pt.next_sibling("prayertimes")) {
-			int day = std::atoi(pt.attribute("dayofyear").value());
-			if (day >= 0 && day < 400) cached_nodes[day] = pt.text().get();
-		}
-		return true;
-	}();
-
-	zaman::xml_bu_gun = (zaman::h_rakam_gun_senenin >= 0 && zaman::h_rakam_gun_senenin < 400 && cached_nodes[zaman::h_rakam_gun_senenin]) ? cached_nodes[zaman::h_rakam_gun_senenin] : "";
+	zaman::xml_bu_gun = (zaman::h_rakam_gun_senenin >= 0 && zaman::h_rakam_gun_senenin < 400 && cached_nodes[zaman::h_rakam_gun_senenin]) ? std::string(cached_nodes[zaman::h_rakam_gun_senenin].text().get()) : "";
 
 	zaman::h_aksam         = zaman::xml_bu_gun.substr(50, 6);
 	zaman::h_istibak_nucum = zaman::xml_bu_gun.substr(56, 6);
@@ -130,7 +121,7 @@ void zaman::vkt_h_v_d()
 	//buradaka kodları yeniliyoruz çünkü bir sonraki gün kılacağız verileri:
 
 	int next_day = zaman::h_rakam_gun_senenin + 1;
-	zaman::xml_bu_gun = (next_day >= 0 && next_day < 400 && cached_nodes[next_day]) ? cached_nodes[next_day] : "";
+	zaman::xml_bu_gun = (next_day >= 0 && next_day < 400 && cached_nodes[next_day]) ? std::string(cached_nodes[next_day].text().get()) : "";
 
 	zaman::h_imsak          = zaman::xml_bu_gun.substr(0, 4) ;
 	zaman::h_sabah          = zaman::xml_bu_gun.substr(5, 5) ;