Domanda

Ho un menu di marchi di prodotti che voglio dividere su 4 colonne. Quindi, se ho 39 marchi, voglio che il numero massimo di articoli per ogni colonna sia 10 (con un unico spazio nell'ultima colonna. Ecco come sto calcolando il conteggio degli articoli per una colonna (usando C #):

int ItemCount = Convert.ToInt32(Math.Ceiling(Convert.ToDecimal(BrandCount) / 4m));

Tutta quella conversione mi sembra davvero brutta. C'è un modo migliore per fare matematica sugli interi in C #?

È stato utile?

Soluzione

Puoi lanciare:

int ItemCount = (int) Math.Ceiling( (decimal)BrandCount / 4m );

Inoltre, poiché int / decimal genera un decimale puoi rimuovere uno dei cast:

int ItemCount = (int) Math.Ceiling( BrandCount / 4m );

Altri suggerimenti

Perché stai persino usando un decimale?

int ItemCount = (BrandCount+3)/4;

Il +3 ti assicura di arrotondare piuttosto che verso il basso:

(37+3)/4 == 40/4 == 10
(38+3)/4 == 41/4 == 10
(39+3)/4 == 42/4 == 10
(40+3)/4 == 43/4 == 10

In generale:

public uint DivUp(uint num, uint denom)
{
    return (num + denom - 1) / denom;
}

Un'alternativa più lunga con Mod.

ItemCount = BrandCount / 4;
if (BrandCount%4 > 0) ItemCount++;

Forse prova qualcosa del genere ... Supponendo che BrandCount sia un numero intero. Hai ancora gli stessi cast, ma potrebbe essere più chiaro:

int ItemCount = (int)(Math.Ceiling(BrandCount / 4m));

Non sono un grande fan della classe Convert e lo evito quando possibile. Sembra sempre che il mio codice sia illeggibile.

Autorizzato sotto: CC-BY-SA insieme a attribuzione
Non affiliato a StackOverflow
scroll top