Показать сообщение отдельно
  #1 (permalink)  
Старый 27.09.2014, 08:00
Новичок на форуме
Отправить личное сообщение для 4ipset Посмотреть профиль Найти все сообщения от 4ipset
 
Регистрация: 27.09.2014
Сообщений: 5

Оптимизация кода
Здравствуйте дорогие форумчани,
Вот решил заняться изучением Джавы. Долго как то руки не доходили или терпение не хватало а сейчас решил заняться )). Можно сказать первый день и вот такой набросок. Подскажите если можно как то оптимизировать код (сократить) я думаю как то это можно сделать,да и вообще взглянити глазами бывалых скриптовщиков и скажите есть смысл продолжать )))). Всем заранее спасибо. калькулятор в виде задачи для построения (выполнения) я взял тут , можно сказать что первое попалось под руки http://doma-fitnes.ru/sutochnaya-norma-kalorij
function calculate(){

    // var age = document.getElementById('age');
    // var ageError = document.getElementById('ageError');
    // var sexM = document.getElementById('sexM');
    // var sexF =document.getElementById('sexF');
    // var sexErorr = document.getElementById('sexError');
    // var weightKg = document.getElementById('weightKg');
    // var weightErorr = document.getElementById('weightError');
    // var heightSm =document.getElementById('heightSm');
    // var heightError = document.getElementById('heightError');
    // var idNone = document.getElementById('idNone');
    // var threeTimes = document.getElementById('threeTimes');
    // var fiveTimes = document.getElementById('fiveTimes');
    // var fiveTimesH = document.getElementById('fiveTimesH');
    // var everyday = document.getElementById('everyday');
    // var everydayH = document.getElementById('everydayH');
    // var workoutError = document.getElementById('workoutError');
    // var submitSpan =document.getElementById('submitSpan');
    var submitLw = document.getElementById('losingWeight')
 
    var x=  parseInt (age.value);
    var y = parseInt (weightKg.value);
    var z = parseInt (heightSm.value);
    var c = (threeTimes.value);
    var q = (fiveTimes.value);
    var w = (fiveTimesH.value);
    var e = (everyday.value);
    var r = (everydayH.value);  
    var f = (sexF.value);
    var m = (sexM.value);
  *!* 
var totalM = parseInt (10*y+6.25*z-5*x+5);//не берет переменную "m" пришлось ставить цифру  
*/!* 
    var totalF = parseInt (10*y+6.25*z-5*x-f);// а тут взял переменную f та к и не понял почему 
   
 
    if (document.getElementById('sexF').selected ){submitSpan.innerHTML = totalF} 

        else if (document.getElementById('sexM').selected) {submitSpan.innerHTML = totalM}

if (document.getElementById('sexF').selected && document.getElementById('threeTimes').selected)
    {submitSpan.innerHTML = parseInt (totalF*c)} 
else if (document.getElementById('sexM').selected && document.getElementById('threeTimes').selected)
    {submitSpan.innerHTML = parseInt (totalM*c)}
if (document.getElementById('sexF').selected && document.getElementById('fiveTimes').selected)
    {submitSpan.innerHTML = parseInt (totalF*q)} 
else if (document.getElementById('sexM').selected && document.getElementById('fiveTimes').selected)
    {submitSpan.innerHTML = parseInt (totalM*q)}
if (document.getElementById('sexF').selected && document.getElementById('fiveTimesH').selected)
    {submitSpan.innerHTML = parseInt (totalF*w)} 
else if (document.getElementById('sexM').selected && document.getElementById('fiveTimesH').selected)
    {submitSpan.innerHTML = parseInt (totalM*w)}
if (document.getElementById('sexF').selected && document.getElementById('everyday').selected)
    {submitSpan.innerHTML = parseInt (totalF*e)} 
else if (document.getElementById('sexM').selected && document.getElementById('everyday').selected)
    {submitSpan.innerHTML = parseInt (totalM*e)}
if (document.getElementById('sexF').selected && document.getElementById('everydayH').selected)
    {submitSpan.innerHTML = parseInt (totalF*r)} 
else if (document.getElementById('sexM').selected && document.getElementById('everydayH').selected)
    {submitSpan.innerHTML = parseInt (totalM*r)}


if (document.getElementById('sexF').selected && document.getElementById('threeTimes').selected)
    {submitLw.innerHTML = parseInt (totalF*c-0.2*(totalF*c))} 
else if (document.getElementById('sexM').selected && document.getElementById('threeTimes').selected)
    {submitLw.innerHTML = parseInt (totalM*c-0.2*(totalM*c))}
if (document.getElementById('sexF').selected && document.getElementById('fiveTimes').selected)
    {submitLw.innerHTML = parseInt (totalF*q-0.2*(totalF*q))} 
else if (document.getElementById('sexM').selected && document.getElementById('fiveTimes').selected)
    {submitLw.innerHTML = parseInt (totalM*q-0.2*(totalM*q))}
if (document.getElementById('sexF').selected && document.getElementById('fiveTimesH').selected)
    {submitLw.innerHTML = parseInt (totalF*w-0.2*(totalF*w))} 
else if (document.getElementById('sexM').selected && document.getElementById('fiveTimesH').selected)
    {submitLw.innerHTML = parseInt (totalM*w-0.2*(totalM*w))}
if (document.getElementById('sexF').selected && document.getElementById('everyday').selected)
    {submitLw.innerHTML = parseInt (totalF*e-0.2*(totalF*e))} 
else if (document.getElementById('sexM').selected && document.getElementById('everyday').selected)
    {submitLw.innerHTML = parseInt (totalM*e-0.2*(totalm*e))}
if (document.getElementById('sexF').selected && document.getElementById('everydayH').selected)
    {submitLw.innerHTML = parseInt (totalF*r-0.2*(totalF*r))} 
else if (document.getElementById('sexM').selected && document.getElementById('everydayH').selected)
    {submitLw.innerHTML = parseInt (totalM*r-0.2*(totalM*r))}



  
     
}
   window.onload = function(){
    var submit = document.getElementById('submit');
    submit.onclick = calculate;
    
}

Последний раз редактировалось 4ipset, 28.09.2014 в 13:29.
Ответить с цитированием