Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Оцените скрипт новичка. (https://javascript.ru/forum/project/34790-ocenite-skript-novichka.html)

Comcarus 19.01.2013 14:46

Оцените скрипт новичка.
 
Здравствуйте, я новичок в сфере JS. Я сделал скрипт для решения квадратных уравнений. Делал только с использованием блокнота! Всё работает. Скажите есть ли у меня какие-нибудь нелепости в коде. Вообщем, мне нужна критика. Вот код:
<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
</head>
<body>
<script>

//данные

var x = "null"; //не менять (неизвестное)
var a = "9"; //значение "А"
var b = "-12"; //значение "b"
var c = "4"; //значение "c"
 
//конец данных

// формула дискриминанта

var d = b * b - 4 * a * c;
 
// конец формула дискриминанта

// условие неимения корней

if (d < 0) {
 alert( "Корней нет!" )
} else {

// конец условия неимения корней

//расчёт корней

var ask1 = (-b + Math.sqrt(d)) / (2 * a);
var ask2 = (-b - Math.sqrt(d)) / (2 * a);

//конец расчёта корней

//условие количества корней

alert(d)
if (d == 0) {
  alert(ask1);
} else {
  alert(ask1);
  alert(ask2)
}
}

//конец условие количества корней


</script>
</body>
</html>

Tim 19.01.2013 14:59

Цитата:

var x = "null"; //не менять (неизвестное)
12
var a = "9"; //значение "А"
13
var b = "-12"; //значение "b"
14
var c = "4"; //значение "c"
всё это - строки. и даже null - строка. зачем?

kobezzza 19.01.2013 17:08

Цитата:

Делал только с использованием блокнота!
Это не повод для гордости :) Тяжёлую ИДЕ мб и нет смысла юзать, но без подсветки синтаксиса всё грустно, попробуйте например notepad++.

Comcarus 19.01.2013 20:15

kobezzza,
notepad++ мне не очень нравится, слишком много функций :)

Comcarus 19.01.2013 20:16

Дзен-трансгуманист,
Я вообще не понимаю о чём речь идёт.

dmitriymar 19.01.2013 20:27

А что вообще здесь собственно оценивать? 3 строки элементарных вычислений?

Tim 19.01.2013 22:19

Comcarus,
Усложни задачу. Сделай HTML-форму с полями для ввода исходных данных. Сделай какую то обработку на случай ввода не корректный исходных данных.

Цитата:

Тяжёлую ИДЕ мб и нет смысла юзать
По началу мб и нет

l-liava-l 19.01.2013 23:39

Comcarus,
Старайся не обьявлять глобальных переменных, все вычесления проводи внутри функций.
А вобще тут код маленький и элементарный, так что проанализировать его не получится)
Цитата:

Не будь так строг, вот это он еще даже краем глаза не видел.) Всему свое время.
Цитата:

Дзен-трансгуманист,
Я вообще не понимаю о чём речь идёт.

Это тоже самое, только на русском http://es5.javascript.ru/



Цитата:

notepad++ мне не очень нравится, слишком много функций
Там только подсветка синтаксиса и выбор кодировки.

Цитата:

Цитата:

Тяжёлую ИДЕ мб и нет смысла юзать
По началу мб и нет
Это кому как, я до сих пор gedit и notepad++ юзаю. Пробовал webstorm и еще парочку ide, но мне не понравилось

Tim 20.01.2013 00:38

Цитата:

Это кому как, я до сих пор gedit и notepad++ юзаю. Пробовал webstorm и еще парочку ide, но мне не понравилось
Согласен. Под линуксом как то так всё работает замечательно что кроме gedit и "кати" ни чем пользоваться не возможно.


Часовой пояс GMT +3, время: 00:04.