Bazı bağlamlarda min ve max işlevleri konusunda kafam karışıyor.

Bir bağlamda, iki değerden daha büyük veya daha azını almak için işlevleri kullanırken, sorun yoktur. Örneğin,

//how many autographed CD's can I give out?
int howManyAutographs(int CDs, int Cases, int Pens)
{
    //if no pens, then I cannot sign any autographs
    if (Pens == 0)
        return 0;

    //I cannot give away a CD without a case or a case without a CD
    return min(CDs, Cases);
}

Kolay. Ama başka bir bağlamda kafam karıştı. Bir maksimum veya minimum ayarlamaya çalışıyorsam, geriye doğru alıyorum.

//return the sum, with a maximum of 255
int cappedSumWRONG(int x, int y)
{
    return max(x + y, 255); //nope, this is wrong
}

//return the sum, with a maximum of 255
int cappedSumCORRECT(int x, int y)
{
    return min(x + y, 255); //much better, but counter-intuitive to my mind
}

Kendi işlevlerimi yapmam aşağıdaki gibi olmaz mı?

//return x, with a maximum of max
int maximize(int x, int max)
{
    return min(x, max);
}

//return x, with a minimum of min
int minimize(int x, int min)
{
    return max(x, min)
}

Açıkçası, yerleşikleri kullanmak daha hızlı olacak ama bu bana gereksiz bir mikro-optimizasyon gibi görünüyor. Bunun tavsiye edilemez olmasının başka bir nedeni var mı? Peki ya bir grup projesinde?

Başkalarının daha önce de söylediği gibi: yerleşik, standart kütüphane veya genel olarak kullanılan bir fonksiyona benzer bir isme sahip bir fonksiyon yaratmayın, ancak davranışını değiştirin. İlk bakışta size pek mantıklı gelmese bile bir adlandırma kuralına alışmak mümkündür, ancak aynı şeyi yapan ancak diğer işlevleri yerine getirdikten sonra kodunuzun işleyişini düşünmek imkansız olacaktır. isimleri değiştirildi.

Standart kütüphane tarafından kullanılan isimleri "aşırı yüklemek" yerine, tam olarak ne demek istediğinizi ileten yeni isimler kullanın. Senin durumunda, “asgari” bir şeyle gerçekten ilgilenmiyorsun. Aksine, bir değeri sınırlamak istersiniz . Matematiksel olarak, bu aynı işlemdir ancak anlamsal olarak, tam değildir. Öyleyse neden sadece bir işlev değil

int cap(int value, int limit) { return (value > limit) ? limit : value; }

ihtiyaç duyulan şeyi yapar ve ismini söyler. (Ayrıca uygulamak capaçısından mingösterildiği gibi timster 'ın cevabı ).

Sık kullanılan başka bir işlev ismi de clamp. Üç argüman alır ve verilen bir değeri diğer iki değer tarafından tanımlanan aralığa “kelepçeler”.

int clamp(int value, int lower, int upper) {
    assert(lower <= upper);  // precondition check
    if (value < lower) return lower;
    else if (value > upper) return upper;
    else return value;
}

Böyle genel olarak bilinen bir işlev adı kullanıyorsanız, ekibinize katılan herhangi bir yeni kişi (bir süre sonra tekrar koda döneceğiniz gelecek de dahil olmak üzere), sizi kırıp karıştırarak sizi şaşırttığı için küfretmek yerine ne olduğunu hızla anlayacaktır. bildiklerini düşündükleri fonksiyon isimleri hakkında beklentiler.

Böyle bir işlevi 10minimize(4, 10) döndüren bir işlev yaparsanız , bunun tavsiye edilemez olduğunu söylerim çünkü programcı arkadaşlarınız sizi boğabilir.

(Tamam, belki de tam anlamıyla seni ölümüne boğmayacaklar, ama cidden ... Bunu yapma.)

Bir işlevin diğer adı aynıdır, ancak mevcut terimlerin anlamlarını değiştirmeye çalışmayın.

İşlevin bir takma adı oluşturmak tamamdır - ortak kütüphaneler bunu her zaman yapar .

Bununla birlikte, terimleri genel kullanıma aykırı bir şekilde kullanmak, örneğin aklınıza maksimum ve minimumun çevrilmesi gereken örnek gibi kötü bir fikirdir. Diğer programcılar için kafa karıştırıcıdır ve bu terimleri standart dışı bir şekilde yorumlamaya devam etmek için kendinizi eğiterek kendinize bir kötülük yapacaksınız.

Bu yüzden, sizin durumunuzda, kafa karıştırıcı bulduğunuz ve anlaması kolay kodunuzu yarattığınız "mininum / maximum" parlance öğesinden vazgeçmeyin.

Örneğinizi yeniden düzenleme:

int apply_upper_bound(int x, int y)
{
    return min(x, y);
}


int apply_lower_bound(int x, int y)
{
    return max(x, y)
}

Ek bir bonus olarak, bu koda her baktığınızda, programlama dilinizde en az ve en çok nasıl kullanıldığını kendinize hatırlatacaksınız . Sonunda, kafanda mantıklı gelecektir.

Bu soruyu seviyorum. Gerçi onu kıralım.

1: Tek bir kod satırı kaydırmanız gerekir mi?

Evet, bunu yapabileceğiniz birçok örnek düşünebilirim. Belki de yazılan parametreleri uyguluyorsunuz veya bir arayüzün arkasında somut bir uygulamayı gizliyorsunuz. Örneğinizde, aslında statik bir yöntem çağrısı gizliyorsunuz.

Ek olarak, bugünlerde tek bir satırda birçok şey yapabilirsiniz.

2: 'Min' ve 'Max' isimleri kafa karıştırıcı mı

Evet! Onlar kesinlikle! Temiz bir kodlama gurusu, onları "FunctionWhichReturnsTheLargestOfItsParameters" veya başka bir şey olarak değiştirir. Neyse ki bizim belgelerimiz var (eğer şanslıysanız) IntelliSense ve bize yardım edecek yorumlar, böylece isimlerle karıştıran herkes ne yapması gerektiğini okuyabilir.

3: Bunları başka bir şeye yeniden adlandırmanız gerekir.

Evet, onun için git. Örneğin, şunları yapabilirdiniz:

class Employee
{
    int NumberOfHolidayDaysIShouldHave(int daysInLue, int maxAllowableHolidayDays)
    {
         // Return the number of days in lue, but keep the value under the max allowable holiday days!
         // Don't use max, you fool!!
         return Math.Max(daysInLue, maxAllowableHolidayDays)
    }
}

Anlam ekler ve arayan kişi değeri nasıl hesaplayacağını bilmek istemez.

4: "min" i "maksimize" olarak yeniden adlandırmanız gerekiyorsa

Hayır!! sen deli misin?! Fakat evet, soru, farklı insanların fonksiyon ve nesne adlarına farklı anlamlar okuduklarının altını çiziyor. Bir kişinin net ve geleneksel bir şey bulduğu şey opak ve kafa karıştırıcı bulur. Bu yüzden yorumlarımız var. Bunun yerine şunu yazmalısınız:

// Add x and y, but don't let it go over 255
s = min(x + y, 255);

Sonra biri okuduğunda

// Add x and y, but don't let it go over 255
s = max(x + y, 255);

senin bir hata yaptığını biliyorlar.

Hayır . Yerleşik işlevlere çok benzeyen adlara sahip işlevler yapmayın, ancak bunun tersini yapar . Sizin için sezgisel görünebilir, ancak daha fazla deneyime sahip olduğunuz zaman gelecekte diğer geliştiricilere ve hatta kendinize bile biraz kafa karıştırıcı olacak.

Anlamı max"maksimum", ama "sezgisel" anlayışınız "maksimum" gibi bir şey. Ama bu sadece bir yanlış fonksiyonun anlaşılması ve adını değişiyor maxiçin maximum, farklı bir yorumunu iletişim kurmaz. Dil tasarımcılarının bir hata yaptıklarına şiddetle inanıyor olsanız bile, böyle bir şey yapmayın.

Ancak, söylendiği cap(x, limit)gibi söylenecek olan ismi değiştirmek iyi olacaktır, çünkü niyeti açıkça sarsa bile, açıkça ifade eder min.

Kafanızı karıştıracak olan, ya fonksiyon adınızdaki Capped'i kullanmak ya da kapağın ne anlama geldiğini anlamaktır. Bu bir sınırlayıcıdır ve maksimum bir şey gerektirmez.

En düşük, en küçük veya en erken istenirse, Max'in uygun işlev olduğunu düşünüyor musunuz?

Min ve max'i yalnız bırakın. Testleri yaz, böylece en azından ikinci seferinde düzeltebilirsin.

Bu işlevleri projenizde çok fazla kullanmanız gerekirse, hangisini kullanacağınızı netleştirmenize yardımcı olacak bir ipucu bulursunuz. <Veya> gibi sırala, ağzın geniş kısmı daha büyük değere bakar.

Sorunuzu cevaplamak için: Bunun tavsiye edilemez olmasının başka bir nedeni var mı? Peki ya bir grup projesinde? Sorun değil kendi fonksiyonlarını istediğin anlaşılıyor. Sadece kendi yardımcı sınıfınızda olduklarından ve içe aktarmadıkları sürece başkaları için kolayca aranamadığından emin olun. (Joes.Utilities.)

Ama senin problemine tekrar bakmak için, temelde onun yerine düşünürdüm:

return (input >= 255) ? 255 : input;

Kafanız karışıyor, çünkü beyin mantığınızı bu min / maks işlevlerine uygulamaya çalışıyorsunuz. Bunun yerine sadece İngilizce konuşun. olduğunu aksi .ifinputgreater than or equal to 255 then return 255returninput

Hangisi:

if (input >= 255) {
   255 
} else {
   input
}

Benim fikrim. Max \ min işlevlerine yanlış nedenlerle gidiyorsunuz, bu şeylerin hızı ihmal edilebilir. Mantıklı olanı yap.

Sorunu anlasam da, bunu yapmak konusunda isteksizdim. Sadece kafatasına, min () ve max () yaptıklarını delmek daha iyi olur.

Çoğu programcı, min () ve max () işlevlerinin ne yaptığını bilir - sizin gibi, bazen herhangi bir zamanda kullanılacak olan sezgileri ile mücadele etseler bile. Bir programı okuyup maksimum (x, y) görüyorsam, ne yaptığını hemen biliyorum. Kendi "diğer ad" işlevinizi oluşturursanız, kodunuzu okuyan başka biri bu diğer adın ne yaptığını bilmez. İşlevini bulmak zorundalar. Gereksiz yere okuma akışını keser ve okuyucuyu, programınızı anlaması için fazladan bir düşünme yapmaya zorlar.

Bir noktada hangisini kullanacağınızı bulmakta sorun yaşıyorsanız, bunu açıklayan bir yorum ekleyelim. Öyleyse, gelecekteki bir okuyucunun benzer şekilde kafası karışırsa, yorumunuz açıklığa kavuşmalıdır. Ya da yanlış yaparsanız ancak yorum ne yapmaya çalıştığınızı açıklarsa, hata ayıklamaya çalışan kişi bir ipucuna sahip olacaktır.

Bir işlevi takma ad kullandığınızda, adınız sezginizle çakıştığı için ... sorunların olduğu tek durum bu mu? Yoksa diğer fonksiyonlara takma ad mı vereceksiniz? Belki "oku" ile kafanız karışır ve "kabul" olarak düşünmeyi daha kolay bulursunuz, "ekleme" yi "StringTogether" e, "yuvarlak" ı "DropDecimals" vb. İle değiştirirsiniz. ve programlarınız anlaşılmaz olacak.

Aslında, yıllar önce C'deki noktalama işaretlerini beğenmeyen bir programcı ile çalıştım. Bu yüzden "{" yerine "SON" yazıp "}" yerine "END-IF" yazmasına izin verecek bir sürü makro yazdı. ve düzinelerce başka tür ikameler. O zaman programlarını okumayı denediğinde, artık C'ye bile benzemiyordu, yepyeni bir dil öğrenmek zorundaydı. Şimdi "VE" nin "&" veya "&&" olarak çevrilip çevrilmediğini hatırlamıyorum - ve mesele bu. İnsanların dili ve kütüphaneyi öğrenmeye yaptığı yatırımları baltalıyorsunuz.

Bu, standart bir kütüphane işlevini çağırmak dışında hiçbir şey yapmayan bir işlevin mutlaka kötü olduğunu söylemem. İşlevinizin amacı bir takma ad oluşturmak değil, yalnızca tek bir işlev olan davranışı kapsıyorsa, bu iyi ve uygun olabilir. Demek istediğim, eğer mantıklı ve kaçınılmaz olarak, programın bu noktasında bir max yapmak zorundaysanız, o zaman doğrudan doğrudan max'i arayın. Ancak bugün bir maks için çağrı yapan, ancak gelecekte başka bir şey yapmak için değiştirilebilecek bir hesaplama yapmanız gerekiyorsa, bir ara işlev uygundur.

Yeni isimler kodunuzu çok net bir hale getirir ve hiç kimse tarafından anlaşılmayacaksa, yerleşik işlevlerin isimlerini değiştirmek sorun değildir. (C / C ++ kullanıyorsanız, ne olduğunu görmeyi zorlaştırdığı için #define kullanmayın.) Bir işlevin adı, çağıran kodun ne yaptığını ve neden ne yaptığını açıklayan bir yorum gibi davranmalıdır. .

Min ve max ile bu sorunu yaşayan tek kişi siz değilsiniz, ancak henüz tüm alanlarda çalışan iyi bir genel çözüm görmedim. Bence bu işlevlerin isimlendirilmesindeki bir problem, iki argümanın farklı mantıksal anlamlarına sahip olduğu, ancak aynı anlama geldiğidir.

Diliniz izin veriyorsa deneyebilirsiniz

return  calculatedDiscount.ButNoMoreThen(maxAllowedDiscount)

return  CDsInStocked.ButNoMoreThen(CasesInStock)

Hayır.

Paketleyicilerini yazmıyorsun. Bu paketleyicilerin isimleri çok önemli değil.

Yapmaya çalıştığın şey bir tür kod gizliliği. 2 amaca hizmet eden ekstra bir katman icat ediyorsunuz:

1. Diğer insanlar kodunuzu anlayamıyor.
2. Başkalarının kodunu anlamayı asla öğrenemezsin.

Rahat olmadığınız şeyleri gizleyerek, yalnızca kodunuzu şimdi ve kendiniz gelecekte yaralayacaksınız. Konfor bölgesinde kalarak büyüyemezsiniz. Ne gerek öğrenmek olduğunu minve maxçalışması.

Tamam, ve Min ve Max'i yeniden dizginlemek ve aşmak için kullanmak pek de mantıklı değil. Bu da kullanılarak yapılır:

• döşeme () ve tavan ()
• Kelepçe, sınır, sınır
• ve tabii ki yüksek dereceli kesmeli mod.

Bellenimde, bu genişletmeye dayanan modern 3D grafikleri daha önce belirleyen MMX'ten daha ileri uzanır.

Endüstri standardı bir fonksiyonu yerel olarak bile değiştirmek beni endişelendiriyor, türev bir isim daha iyi olabilirdi. C ++ öğrencileri belirsiz sınıfları için aşırı yüklenebilir.

Bazı durumlarda sorun değil, ama değil : o kelimeye çok daha iyi yollar vardır, çünkü örnekte
saturate, clamp, clipvb

'Sınırlı' adında genel bir işlev oluşturmayı tercih ederim

//Assumes lower_bound <= upper_bound
template <typename T>
T bounded(T value, T lower_bound, T upper_bound){
    if (value < lower_bound)
        return lower_bound;
    if (value > upper_bound)
        return upper_bound;
    return value;
}

//Checks an upper (by default) or lower bound
template <typename T>
T bounded(T value, T bound, bool is_upper_bound = true){
    if (is_upper_bound){
        if (value > bound)
            return bound;
    }
    else {
        if (value < bound)
            return bound;
    }
    return value;
}

veya 'min' ve 'max' kullanımıyla

//Assumes lower_bound <= upper_bound
template <typename T>
T bounded(T value, T lower_bound, T upper_bound){
    return max(min(value, upper_bound), lower_bound);
}

//Checks an upper (by default) or lower bound
template <typename T>
T bounded(T value, T bound, bool is_upper_bound = true){
    if (is_upper_bound)
        return min(value, bound);
    else
        return max(value, bound);
}

İşlevlerinizi çağırmaya ne dersiniz:

atmost(x,255): en fazla x veya 255'in altını döndürür.

atleast(10,x): x'in en yükseğe veya en az 10 değerini döndürün.

min(x+y, MAX_VALUE); çok daha fazla anlam taşıyacak myCustomFunction(x, y);

Yani cevabınız EVET, tavsiye edilemez . Sadece beyin dilinize bir takma ad olarak hizmet eder.