Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

weather-api #35

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions submissions/eoneoff/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
{
"name": "eoneoff",
"version": "1.0.0",
"description": "kottans nodejs",
"main": "''",
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1"
},
"repository": {
"type": "git",
"url": "git+https://github.com/eoneoff/nodejs-2019-homeworks.git"
},
"author": "eoneoff",
"license": "ISC",
"bugs": {
"url": "https://github.com/eoneoff/nodejs-2019-homeworks/issues"
},
"homepage": "https://github.com/eoneoff/nodejs-2019-homeworks#readme",
"devDependencies": {
"semistandard": "^14.2.0"
}
}
45 changes: 45 additions & 0 deletions submissions/eoneoff/weather-api/getWeather.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
'use strict';

const axios = require('axios');

const weather = axios.create({
baseURL: 'http://api.openweathermap.org/data/2.5/',
params: {
APPID: 'c4ad7974f3977f8f388a60b5c0267caa'
}
});

module.exports.getWeather = async function (mode, location, units) {
const data = (await weather.get(mode, {
params: {
q: location,
units: units
}
})).data;
const wData = [];
if (mode === 'weather') {
wData.push(makeWeatherData(data, data.name));
} else {
for (const day of data.list) {
wData.push(makeWeatherData(day, data.city.name));
}
}
return wData;
};

function makeWeatherData (data, name) {
const day = {
icon: '_' + data.weather[0].icon,
city: name,
date: new Date(data.dt * 1000),
description: data.weather[0].description,
temp: data.main.temp,
maxTemp: data.main.temp_max,
minTemp: data.main.temp_min,
wind: {
deg: data.wind.deg,
speed: data.wind.speed
}
};
return day;
}
22 changes: 22 additions & 0 deletions submissions/eoneoff/weather-api/main.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
'use strict';

const { showWeather } = require('./showWeather');

const argv = require('yargs')
.option('range', {
alias: 'r',
type: 'string'
})
.option('location', {
alias: 'l',
type: 'string'
})
.option('units', {
alias: 'u',
type: 'string'
}).argv;

const mode = argv.range === 'week' ? 'forecast' : 'weather';
const units = argv.units === 'farenheit' ? 'imperial' : 'metric';

showWeather(mode, argv.location || '', units);
62 changes: 62 additions & 0 deletions submissions/eoneoff/weather-api/manageCities.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
'use strict';

const fs = require('fs');

function saveRecent (city) {
Copy link
Contributor

@Siusarna Siusarna Dec 21, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove nested in this function, if by using negative check and then return

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I can't see how. I need to remove the last record from recent if there are more then 9 records and save the changes anyway. If I return the saving will be skipped.
I have removed the nested if another way.

Copy link
Contributor

@Siusarna Siusarna Dec 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I may be writing not-working code, but if it works, then its solution would be better

Suggested change
function saveRecent (city) {
function saveRecent (city) {
const recent = readRecent();
if(!recent.includes(city)) return;
recent.unshift(city);
if (recent.lenght > 9) recent.pop();
writeRecent(recent);
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I have done. One thing: not
if(!recent.includes(city)) return
but
if(recent.includes(city)) return
because we add a new city to recent if it is not in the list

Actually I'm not sure, that it is correct to do so, because thus we make code somewhat shorter but, in my opinion less readable. This can make some sense in browser js where shorter code means less traffic, but in server node.js code it makes no sense. IMHO.

But there is another problem. Semistandard finds an error in my code, but the error message encoding is a mess and I can not get where the error is. The semistandard extension for my VS Code shows no linter errors.

const recent = readRecent();
if (recent.includes(city)) return;
recent.unshift(city);
writeRecent(recent.slice(0, 10));
}

function saveFavorite (city) {
const favorite = readFavorite();
const exists = favorite.find(c => c.name === city);
if (exists) exists.count++;
else {
favorite.push({
name: city,
count: 1
});
}
writeFavorite(favorite.sort((a, b) => a.count - b.count).reverse().slice(0, 10));
}

function readFavorite () {
if (!fs.existsSync('favorite.csv')) return [];

const output = fs.readFileSync('favorite.csv').toString()
.split('\n')
.map(c => {
const cData = c.split(';');
return {
name: cData[0],
count: cData[1]
};
});
return output;
}

function writeFavorite (favorite) {
fs.writeFile('favorite.csv', favorite.map(c => `${c.name};${c.count}`).join('\n'), () => {});
}

function readRecent () {
if (fs.existsSync('recent.csv')) {
return fs.readFileSync('recent.csv').toString().split('\n');
}
return [];
}

function writeRecent (recent) {
fs.writeFile('recent.csv', recent.join('\n'), () => {});
}

module.exports.saveCities = (city) => {
saveRecent(city);
saveFavorite(city);
};

module.exports.getRecent = readRecent;

module.exports.getFavourite = readFavorite;
21 changes: 21 additions & 0 deletions submissions/eoneoff/weather-api/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
{
"name": "weather-api",
"version": "1.0.0",
"description": "",
"main": "main.js",
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1"
},
"repository": {
"type": "git",
"url": "https://github.com/eoneoff/nodejs-2019-homeworks/tree/master/submissions/eoneoff/weather-api"
},
"author": "eoneoff",
"license": "ISC",
"dependencies": {
"axios": "^0.19.0",
"chalk": "^3.0.0",
"cli-table": "^0.3.1",
"readline-sync": "^1.4.10"
}
}
68 changes: 68 additions & 0 deletions submissions/eoneoff/weather-api/showWeather.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
'use strict';

const { getWeather } = require('./getWeather');
const { icons } = require('./weatherIcons');
const { saveCities } = require('./manageCities');
const { wrongCity } = require('./wrongCity');
const Table = require('cli-table3');
const { cyan: c } = require('chalk');

function showBlocks (data, tUnit, wUnit) {
for (const day of data) {
dayBlock(day, tUnit, wUnit);
}
}

function dayBlock (data, tUnit, wUnit) {
const block = new Table({
chars: {
'top-mid': '─',
'bottom-mid': '─',
'left-mid': '',
mid: '',
'mid-mid': '',
'right-mid': '',
middle: ' '
},
colWidths: [17, 40]
});
block.push([icons[data.icon], weatherData(data, tUnit, wUnit)]);
process.stdout.write(block.toString() + '\n');
}

function weatherData (data, tUnit, wUnit) {
return `\n${c('Location:')} ${data.city}\n` +
`${c('Date:')} ${data.date.toDateString()}\n` +
`${c('Time')} ${data.date.toLocaleTimeString()}\n` +
`${data.description}\n` +
`${c('Temperature:')} ${data.temp}${tUnit}\n` +
`${c('Maximum temperature:')} ${data.maxTemp}${tUnit}\n` +
`${c('Minimum temperature:')} ${data.minTemp}${tUnit}\n` +
`${c('Wind:')} ${windDirection(data.wind.deg)} ${data.wind.speed}${wUnit}\n`;
}

function windDirection (wind) {
if (wind > 338 || wind < 22) return 'N';
if (wind < 67) return 'NE';
if (wind < 112) return 'E';
if (wind < 157) return 'SE';
if (wind < 202) return 'S';
if (wind < 247) return 'SW';
if (wind < 292) return 'W';
return 'NW';
}

module.exports.showWeather = function showWeather (mode, location, units) {
getWeather(mode, location, units).then(data => {
showBlocks(data,
units === 'imperial' ? 'F°' : 'C°',
units === 'imperial' ? 'mph' : 'm/s',
data.name);
saveCities(location);
}).catch(error => {
const message = (error.response.data || {}).message;
if (message === 'city not found' || message === 'Nothing to geocode') {
wrongCity(location).then(c => showWeather(mode, c, units));
} else throw (error);
});
};
85 changes: 85 additions & 0 deletions submissions/eoneoff/weather-api/weatherIcons.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,85 @@
const { yellow: y, cyan: c, gray: g } = require('chalk');

const icons = {
_01d: y(`
OOO
OOOOO
(OOOOO)
OOOOO
OOO`),
_01n: `
OOO
OOOOO
(OOOOO)
OOOOO
OOO`,
_02d: `
${c('@@@')} ${y('OOO')}
${c('@ @@@')}${y('OOO')}
${c('@@ @@@')}${y('OO)')}
${c(`@@@ @@@@@@
@@@@@@@@@@@@@@`)}`,
_02n: `
${c('@@@')} OOO
${c('@ @@@')}OOO
${c('@@ @@@')}OO)
${c(`@@@ @@@@@@
@@@@@@@@@@@@@@`)}`,
_03d: c(`
@@@
@ @@@
@@ @@@
@@@ @@@@@@
@@@@@@@@@@@@@@`),
_04d: `
${c('@@@')}${g('##')}
${c(' @ @@@')}${g('##')}
${c('@@ @@@')}${g('##')}
${c('@@@ @@@@@')}${g('###')}
${c('@@@@@@@@@@@@@@')}`,
_09d: `
${c('@@@')}${g('##')}
${c('@ @@@')}${g('##')}
${c('@@ @@@')}${g('##')}
${c('@@@@@@@@@@@@')}
${g(',\',\',\',\',\',\',')}`,
_10d: `
${c('@@@')} ${y('OOO')}
${c('@@ @@@')}${y('OOO')}
${c('@@@ @@@@')}${y('O)')}
${c('@@@@@@@@@@@@')}
${g(',\',\',\',\',\',\'')}`,
_10n: `
${c('@@@')} OOO
${c('@@ @@@')}OOO
${c('@@@ @@@@')}O)
${c('@@@@@@@@@@@@')}
${g(',\',\',\',\',\',\'')}`,
_11d: `
${c('@@@')}${g('##')}
${c('@@@')}${y('/')}${c('@@')}${g('##')}
${c('@@@')}${y('/_')}${c('@@@')}${g('##')}
${y(`/
/`)}`,
_13d: `
*
*****
***
*****
*`,
_50d: g(`
--——
—--—
—---—--
—--—`)
};

icons._03n = icons._03d;
icons._04n = icons._04d;
icons._09n = icons._09d;
icons._11n = icons._11d;
icons._13n = icons._13d;
icons._50n = icons._50d;

module.exports.icons = icons;
32 changes: 32 additions & 0 deletions submissions/eoneoff/weather-api/wrongCity.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
const rl = require('readline-sync');
const { getRecent, getFavourite } = require('./manageCities');

function selectRecent () {
const recent = getRecent();
const option = rl.keyInSelect(recent, 'Select');
return option === -1 ? '' : recent[option];
}

function selectFavourite () {
const favourite = getFavourite().map(city => city.name);
const option = rl.keyInSelect(favourite, 'Select');
return option === -1 ? '' : favourite[option];
}

module.exports.wrongCity = async function (name) {
const options = ['Select city form recent', 'Select city from favourite', 'Enter city'];
process.stdout.write(name ? `There is no weather data for the city with the name ${name}\n`
: 'There is no city name provided\n');
process.stdout.write('Would you like to');
const option = rl.keyInSelect(options, 'Select');
switch (option) {
case 0:
return selectRecent();
case 1:
return selectFavourite();
case 2:
return rl.question('Enter city name: ');
case -1:
process.exit(0);
}
};