Simple Custom Logs

[durda]

Sıralandırmalı daha iyidi sankiii ??

paylaşım için teşekkürler?

Aynen tek satırlık txt yığını?

 

[rebelnd]

Gordugum birkac eksiklikleri belirtecegim.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

CreateDirectoryA fonksiyonunun donus degerini kontrol etmiyorsun, eger birisi klasoru "x/logs" seklinde degistirirse ve x klasoru mevcut degilse fonksiyon basarisiz olur.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

Ayni sekilde donus degerini kontrol etmemissin fopen fonksiyonunun, eger basarisiz olursa null pointer doner.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

va_end cagirmamissin.


Kaynaklar

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

 

[durda]

Gordugum birkac eksiklikleri belirtecegim.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

CreateDirectoryA fonksiyonunun donus degerini kontrol etmiyorsun, eger birisi klasoru "x/logs" seklinde degistirirse ve x klasoru mevcut degilse fonksiyon basarisiz olur.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

Ayni sekilde donus degerini kontrol etmemissin fopen fonksiyonunun, eger basarisiz olursa null pointer doner.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

va_end cagirmamissin.


Kaynaklar

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

Eve gidince düzeltirim <3

 

[Koray']

yukardaki arkadaşa ek olarak bende gördüklerimi yazayım.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

constructor de vectör temizlenmesine gerek yok zaten boş olarak oluşturur.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

ve

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

make_unique ile default destructor tanımlayabilirsin, class da tekrar kendin kullanmana gerek kalmaz;
m_fp.emplace_back(std::make_unique<FILE*>(fopen((path + "/" + v).c_str(), "w"), ::fclose));

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

[] operatörü yerine .at kullanırsan daha güvenli olur şuanki halde kontrol olmadığı için indexin doğru değer verdiği belli değil
time, localtime, fprintf gibi c fonksiyonları yerine std::chrono ve std::cout kullanabilirsin

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

sıralı ve kontrollü bir liste kullanıyorsan std::vector yerine std::set daha uygun olur.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

_vsnprintf yerine string stream kullanılabilir.

 

[durda]

yukardaki arkadaşa ek olarak bende gördüklerimi yazayım.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

constructor de vectör temizlenmesine gerek yok zaten boş olarak oluşturur.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

ve

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

make_unique ile default destructor tanımlayabilirsin, class da tekrar kendin kullanmana gerek kalmaz;
m_fp.emplace_back(std::make_unique<FILE*>(fopen((path + "/" + v).c_str(), "w"), ::fclose));

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

[] operatörü yerine .at kullanırsan daha güvenli olur şuanki halde kontrol olmadığı için indexin doğru değer verdiği belli değil
time, localtime, fprintf gibi c fonksiyonları yerine std::chrono ve std::cout kullanabilirsin

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

sıralı ve kontrollü bir liste kullanıyorsan std::vector yerine std::set daha uygun olur.

Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.

_vsnprintf yerine string stream kullanılabilir.

tamam. yalnız ballı adamsın tam custom deleter bakıyordum üstüne sen yazdın