Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #11 (permalink)  
Старый 10.10.2014, 17:35
Новичок на форуме
Отправить личное сообщение для REMOX Посмотреть профиль Найти все сообщения от REMOX
 
Регистрация: 27.09.2014
Сообщений: 7

Сообщение от danik.js Посмотреть сообщение
А че вобще за задача стоит? Код все равно не читаемый, ибо имена переменных обфусцированы. Пожалуйста, не пости здесь офусцированный код. Пости исходник.
благодарю вас, мистер-учитель-который-убывает-все-знания-и-желания-учеников))) я справился сам, спасибо, что попинали немного))

чем вам код не нравится мой? то вам надо обьязательно цыкл, то имена переменных не нравятся))
нет, никакой они не обфусцированы, это я их так обозвал, и так они у меня в проэкте, это исходник))

А "А че вобще за задача стоит?"=== написано в теме))
так вы не только любитель попинать почем зря, вы еще и невнимательный

получилось вот так:
Код:
var divdeffo=rez[0].split('-');
var divdeffc=divdeffo.length;
var rezdmin = [];
var rezdpl = [];
for(i=0;i<divdeffc;i++){
var divrezhide5=document.getElementsByClassName(divdeffo[i])[0].getElementsByClassName("rez")[0].style="display: none;";
var divrezhide6=document.getElementsByClassName(divdeffo[i])[0].getElementsByClassName("rez")[1].style="display: none;";

var ddff=divdeffo[i];
var n = i+3;
var armrez=rez[n].split('-');

var l=0;
rezdmin[i] = [];
rezdpl[i] = [];
for (var t = 0; t <= 11; t++){
l=t-1;
if(t==0){
rezdmin[i][0]=ddff;
rezdpl[i][0]=ddff;
}else{
rezdmin[i][t]=window['defarm_'+ddff+t]-armrez[l];
rezdpl[i][t]=armrez[l];
}
}}
спасибо еще раз и пускай вы не мучаетесь так долго, как я))
Ответить с цитированием
  #12 (permalink)  
Старый 10.10.2014, 17:49
Аватар для danik.js
Профессор
Отправить личное сообщение для danik.js Посмотреть профиль Найти все сообщения от danik.js
 
Регистрация: 11.09.2010
Сообщений: 8,804

Сообщение от REMOX
так вы не только любитель попинать почем зря, вы еще и невнимательный
Ты не описывал задачу. Че делает этот код - загадка. Хотя обычно по коду (нормальному) ясно что он делает.
Сообщение от REMOX
пускай вы не мучаетесь так долго, как я))
Если бы с самого начала писал нормальный код, то и ошибку бы быстрей нашел, или подсказали.

А вобще, наверняка твою задачу можно решить в три-четыре простых строчки.
__________________
В личку только с интересными предложениями
Ответить с цитированием
  #13 (permalink)  
Старый 10.10.2014, 18:13
Новичок на форуме
Отправить личное сообщение для REMOX Посмотреть профиль Найти все сообщения от REMOX
 
Регистрация: 27.09.2014
Сообщений: 7

Сообщение от danik.js Посмотреть сообщение
Ты не описывал задачу. Че делает этот код - загадка. Хотя обычно по коду (нормальному) ясно что он делает.
Если бы с самого начала писал нормальный код, то и ошибку бы быстрей нашел, или подсказали.

А вобще, наверняка твою задачу можно решить в три-четыре простых строчки.
в теме четко пишет "задать новый элемент массива"))
термин "нормальный" подразумевает под собой "такой, как большинство",,
применимо к коду - "нормальный" - лично я понимаю как "работающий",,
можно еще разделять на "оптимизированно работающый" и "сяк-так работающый"))
(первое - кто пишет на чистом js, второе - кто пишет на jQuery и других костилях, которые в своем большинстве дают лишний вес кода и нагрузку на проц)

насчет вашего последнего - это- вряд ли))
я, понимаете ли, не решаю задачки со школы, как вы, возможно, подумали, нет, я пишу один свой проэкт))

спасибо за беседу, извините, если мои орфографические ошыбки режут глаз - я русский только по сети немножко знаю и все))
Ответить с цитированием
  #14 (permalink)  
Старый 11.10.2014, 20:07
Профессор
Отправить личное сообщение для kostyanet Посмотреть профиль Найти все сообщения от kostyanet
 
Регистрация: 23.10.2010
Сообщений: 2,718

ТС, на эту борду постите свой нормальный код - http://govnokod.ru/
Ответить с цитированием
  #15 (permalink)  
Старый 11.10.2014, 20:17
Аватар для danik.js
Профессор
Отправить личное сообщение для danik.js Посмотреть профиль Найти все сообщения от danik.js
 
Регистрация: 11.09.2010
Сообщений: 8,804

Сообщение от REMOX
в теме четко пишет "задать новый элемент массива"))
Хе. Представляю прям картину: приходит с утра менеджер и говорит: "На сегодня у тебя задача - задать новый элемент массива". И уходит.
__________________
В личку только с интересными предложениями
Ответить с цитированием
  #16 (permalink)  
Старый 11.10.2014, 20:21
Профессор
Отправить личное сообщение для kostyanet Посмотреть профиль Найти все сообщения от kostyanet
 
Регистрация: 23.10.2010
Сообщений: 2,718

Сообщение от REMOX
применимо к коду - "нормальный" - лично я понимаю как "работающий",,
Следовательно и по-вашему этот код не нормальный, потому что не работающий. Или вы уж задали новый элемент массива героически?
Ответить с цитированием
  #17 (permalink)  
Старый 11.10.2014, 20:23
Профессор
Отправить личное сообщение для kostyanet Посмотреть профиль Найти все сообщения от kostyanet
 
Регистрация: 23.10.2010
Сообщений: 2,718

А, я понял, он работающий, но не совсем, где-то чуть-чуть не работает, а в целом - работающий, но где-то не работает, где - хз, надо на форуме спросить... Это как в том анекдота: зарплата маленькая - но хорошая.
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Не могу настроить выборку элементов из массива menscool Events/DOM/Window 4 11.07.2014 21:19
не могу выбрать элемент alex2012 AJAX и COMET 2 05.11.2012 02:15
Массив из картинок. Не могу изменить свойство элемента массива Володя Шишкин Ваши сайты и скрипты 7 17.02.2012 11:21
Удалить элемент из массива, не убирая с экрана KamalovRadik Javascript под браузер 0 15.10.2011 12:48
Не могу определить элемент AlexMak Общие вопросы Javascript 23 21.01.2009 17:50