guccisquad / webdev-learn

0 stars 0 forks source link

Замечания по lw1 #1

Closed ivklgn closed 7 years ago

ivklgn commented 7 years ago

Git:

  1. Использовать разбиение задачи на подзадачи с промежуточными фиксациями (коммитами)

HTML:

  1. Добавить обязательный тег title

JS:

  1. Имя файла с js кодом не отражает смысла программа
  2. Не соблюдено форматирование в файле (2 пробела отступов)
  3. Имена переменных не отражают смысла:
    const N = 10;
    var k;
    var i;
    var arr = [];
    var z = 1;
    var write = "";
  4. Нарушено форматирование
    var k = prompt("Введите искомое число: ");
    if ((k == "") || (isNaN(k))) {
    alert("Error!");
    }
  5. Заменить переменную на булеву (+ переименовать)
    var z = 1;
  6. Переименовать переменную в соответствии с ее назначением
    var write = "";

kT=1.1 kQ=0.4

ivklgn commented 7 years ago

Выполнение: randarray.js:16 Uncaught ReferenceError: arr is not defined at randarray.js:16

JS:

  1. Переписать код в соответствии с Паскаль программой (сначала рандомятся числа в массив, потом ищется введенное число)
  2. Поправить форматирование
    if ( (find == "") || (isNaN (find) ) ) {
    ...
ivklgn commented 7 years ago

Выполнение

  1. Остановить сценарий выполнения при ошибке

JS

  1. Проверить форматирование
  2. Индексация в массиве начинается с 0
    for(i = 1; i <= N; i++)
    ...