qualidade de código

e suas nuances

roberta arcoverde

@rla4

rla4.com

/whois

recifense
diretora de engenharia na stack overflow
co-host do hipsters.tech
microsoft mvp
alura stars
@rla4

essa palestra é sobre o quê?

qualidade de código

(minhas experiências, frustrações, ideias e opiniões sobre parte das coisas que influenciam a manutenibilidade de projetos de software em evolução)

🤷‍♀️

um pouco de história

α

manutenibilidade

code smells

consenso

"DRY - don't repeat yourself", Andy Hunt/Dave Thomas

The Pragmatic Programmer

"Duplication is far cheaper than the wrong abstraction" Sandi Metz

https://www.youtube.com/watch?v=x1wnI0AxpEU

mas como saber se a abstração está certa ou errada?

@aphyr, CraftConf 2015

código que aumenta o potencial de manutenibilidade do sistema

manutenibilidade

fácil de entender
- quanto tempo leva pra saber o que um trecho de código faz e pra alterá-lo com confiança
- quantos arquivos ou módulos diferentes são necessários (princípio da localidade)

fácil de explicar
- quanto tempo uma nova dev leva pra conseguir entender ou modificar o código

public static List<HelpPost> All()
{
  return Current
    .GlobalCache
    .GetSet<List<HelpPost>>(
        CacheKey,
        (old, ctx) => 
        {
            using (var sitesDb = ctx.NewDbContext())
            {
                return sitesDb.Query<HelpPost>(@"SELECT p.Id, p.Title FROM ...");
            }
        }, 24 * 60 * 60, 24 * 60 * 60, server: StackRedis.Server.PreferReplica);
}

fácil de entender
fácil de explicar
fácil de debugar
difícil de testar

não é "clean", mas não causa dor de cabeça

dev mantendo código

dev mantendo código

😢

difícil de entender
difícil de explicar
difícil de debugar
impossível de testar

"fácil de entender" e "simples" são conceitos subjetivos

function reduceTweet(users) {
    users.reduce((acc, curr) => {
        if (curr.active) {
            return acc;
        }
        return {...acc, [curr.id]: curr.name};
    }, {});
}

function reduceWithoutCopy(users) {
    users.reduce((acc, user) => {
        if (!user.active) {
            acc[user.id] = user.name;
        }
        return acc;
    }, {});
}

function forLoop(users) {
    const nonActiveUsers = {};
    for (let i = 0; i < users.length; i++) {
        if (!users[i].active) {
            nonActiveUsers[users[i].id] = users[i].name;
        }
    }
}

"cognitive refactoring"

names = [c.name for c in customers if c.age > 50]

names = []

for c in customers:
    if c.age > 50:
        names.append(c.name)

https://www.youtube.com/watch?v=GLicXcf0uUE

"write simple code"

código que cabe na sua cabeça
uso > reuso
composição > herança
fácil de mudar > difícil de estender

https://www.youtube.com/watch?v=rqw_Jnv6ZX4

https://alexandreaquiles.com.br/2022/09/11/clean-arch-morreu/

obrigada :)

@rla4

rla4.com

bônus de abstração errada!