Il mio collega di sviluppo sta producendo numerosi blocchi di codice che sono un po 'preoccupato in realtà non rende facile lettura e potenziali problemi in futuro. Ho provato a dare piccoli consigli lungo la strada senza dire in modo blando ciò che penso, ma la sua tecnica sembra non cambiare mai.
Normalmente non sarei preoccupato, ma all'inizio del nostro progetto non abbiamo mai identificato alcun standard di programmazione specifico e ora sto davvero cominciando a comportarmi un po 'perché devo passare attraverso il suo codice e provare a eseguirne il debug.
Un piccolo frammento di una piccola parte del codice è come tale.
foreach(DistributionRecord record in _manager.Distributions)
{
foreach (Engine.Model.DistributionModel.LossFromAtmosphericConditionsEnum NLossAt
in Enum.GetValues(typeof(Engine.Model.DistributionModel.LossFromAtmosphericStructureEnum)))
if (record.NlossAt.ContainsKey(NLossAt))
if (record.NlossAt[NLossAt].ContainsKey(DistributionModel.LossFromAtmosphericConditionsEnum.Atvolat))
_indirect[GetIndirectSourceEnumFromEnterpriseType(record)] += record.NlossAt[NLossAt][DistributionModel.LossFromAtmosphericConditionsEnum.Atvolat]
* _manager.Scenario.Management.EmissionFactors.EmissionFactorIndirectBackgroundVolatisation;
foreach (Engine.Model.DistributionModel.LossOffEnum lossOff in Enum.GetValues(typeof(Engine.Model.DistributionModel.LossOffEnum)))
if (enterprise.NlossOff.ContainsKey(lossOff))
if (enterprise.NlossOff[lossOff].ContainsKey(DistributionModel.LossTypesEnum.lossleach))
_indirect[GetIndirectSourceEnumFromEnterpriseType(record)] += record.NlossOff[lossOff][DistributionModel.LossTypesEnum.lossleach][NutrientEnum.N].Value
* _manager.Scenario.Management.EmissionFactors.EmissionFactorN2OIndirectBackgroundLeach;
}
Ho parlato con il capo e senza dire esplicitamente che ha spiegato le mie preoccupazioni ecc. Non so cos'altro fare.
Non sono sicuro se questo è appropriato per questo forum, ma qualsiasi consiglio sarebbe apprezzato in sostanza su come posso affrontare questo con il mio collega. O forse sto solo diventando irrealistico e devo entrare nel mondo reale nel senso che alcune persone scrivono il codice ed è ok?