Estou criando uma função com um .fetch pra pegar dados do openweathermap.org, que por definição retorna em fahrenheit. É aceitável do ponto de vista Clean Code, que diz que a função deve fazer apenas uma coisa e ter poucas linhas, fazer um bloco If-Else pra controlar isso OU é mais apropriado fazer outra função com uma flag e que retorne a url no formado desejado e usa-la na requestCurrentWeather()? Abaixo um exemplo de como fica a extensa função com If-Else.
let flagUnitsFormat= true;
const requestCurrentWeather = cityName => {
let URL_CURRENT_WEATHER = `http://api.openweathermap.org/data/2.5/weather?q=${cityName}&appid=${keyApiCurrent}`
if(flagUnitsFormat) {
fetch(`${URL_CURRENT_WEATHER}&units=metric`, {method:'get'})
.then(response => {
response.json()
.then(result => { console.log(result)});
})
.catch((error) => { console.error(error); });
} else {
fetch(URL_CURRENT_WEATHER, {method:'get'})
.then(response => {
response.json()
.then(result => { console.log(result)});
})
.catch((error) => { console.error(error); });
}
}