Skip to content

Add files via upload#1

Open
nadya12082012 wants to merge 1 commit into
mainfrom
Zadanie_5
Open

Add files via upload#1
nadya12082012 wants to merge 1 commit into
mainfrom
Zadanie_5

Conversation

@nadya12082012

Copy link
Copy Markdown
Owner

Выполнила задание 5

Выполнила задание 5

@halavasty halavasty left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Отлично! хороший старт. 🥇
Теперь когда известно про конструкцию switch case я буду ожидать увидеть её в подобных ситуациях

Comment thread lesson1.js
@@ -0,0 +1,88 @@


let a = prompt("чтобы вычислить объем куба выберите - 1, параллелипипеда - 2, прямоугольного паралеллипипеда - 3, призмы -4, пирамиды -5")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Отлично! именно такой подход я и ожидал увидеть 👍

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

стоит использовать более читаемые имена переменных.
это сильно упростить читаемость кода через месяц для тебя и другим программистам.
так же тут использовалось let я бы предпочёл const для неизменяемых переменных

Comment thread lesson1.js
let a = prompt("чтобы вычислить объем куба выберите - 1, параллелипипеда - 2, прямоугольного паралеллипипеда - 3, призмы -4, пирамиды -5")

if (a == 1) {
function cube(){

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

объявлять функции не нужно внутри условного ветвления.
читать такое будет трудно.
правильнее будет вынести функции на самый верх. можно даже перед объявлением
let a
кстати интерпретатор сделает это в любом случае

Comment thread lesson1.js

if (a == 1) {
function cube(){
let a = prompt ( "Чтобы узнать объем куба - Введите значение стороны, в см" );

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

всегда используй const если переменная не будет меняться

Comment thread lesson1.js
function pPov() {
let a = prompt ( "Чтобы узнать площадь куба - Введите значение стороны, в см" );
let result = 6*a*a
return result;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

в это и других функциях ты создаёшь let result и сразу после этого возвращаешь его.
если присмотреться, то можно появляется вопрос - А точно ли нам нужно заводить переменную result

ничего против не имею если так для обучения нагляднее, но умные ide пометят это как ненужный блок кода и преобразуют к return 6 * (a * a);

Comment thread lesson1.js
let result = sOsn * h;
return result;
}
function sPP() {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Понимаю что задание математического характера, но даже в таких случаях старайся создавать читаемые имена переменных и функций

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants