Click here to Skip to main content
13,089,690 members (47,030 online)

JavaScript

 
AnswerRe: either my css or Javscript is in error, another set of eyes Pin
BobJanova15-Aug-12 23:36
memberBobJanova15-Aug-12 23:36 
QuestionDisplay Values Pin
lakshmiblp15-Aug-12 10:29
memberlakshmiblp15-Aug-12 10:29 
Questionhow to get default signature in outlook using Javascript Pin
anurag34879-Aug-12 2:20
memberanurag34879-Aug-12 2:20 
AnswerRe: how to get default signature in outlook using Javascript Pin
Pete O'Hanlon9-Aug-12 3:52
protectorPete O'Hanlon9-Aug-12 3:52 
AnswerRe: how to get default signature in outlook using Javascript Pin
BobJanova9-Aug-12 3:56
memberBobJanova9-Aug-12 3:56 
Newsurdu Pin
sania habib8-Aug-12 15:47
membersania habib8-Aug-12 15:47 
GeneralRe: urdu Pin
Richard MacCutchan8-Aug-12 21:28
mvpRichard MacCutchan8-Aug-12 21:28 
AnswerRe: urdu Pin
Rahul Rajat Singh22-Aug-12 1:24
memberRahul Rajat Singh22-Aug-12 1:24 
GeneralRe: urdu Pin
Sandip.Nascar22-Aug-12 11:03
memberSandip.Nascar22-Aug-12 11:03 
QuestionLooping through elements, to find textbox, and get values Pin
jkirkerx8-Aug-12 13:12
memberjkirkerx8-Aug-12 13:12 
AnswerRe: Looping through elements, to find textbox, and get values Pin
jkirkerx8-Aug-12 18:13
memberjkirkerx8-Aug-12 18:13 
GeneralRe: Looping through elements, to find textbox, and get values Pin
Andrei Straut9-Aug-12 4:41
memberAndrei Straut9-Aug-12 4:41 
GeneralRe: Looping through elements, to find textbox, and get values Pin
jkirkerx9-Aug-12 6:27
memberjkirkerx9-Aug-12 6:27 
AnswerRe: Looping through elements, to find textbox, and get values Pin
Andrei Straut9-Aug-12 9:37
memberAndrei Straut9-Aug-12 9:37 
Now, I'm no JavaScript guru, but...

if ((nameArray[i] != "") && (codeArray[i] != ""))

Here I would also add a jQuery.trim() on nameArray[i]. You may have values that are simply spaces, and checking for empty string will return false

Also, in the for you could dump nameCount and codeCount. Maybe you didn't post the full function, but nameCount is used only for upper bound limit (which could be replaced with parseInt($("._name_Field").length)), and codeCount isn't used at all.

Other than that, seems to be fine. Also, it's good that you are commenting your code. I've been through some situations in my (relatively short) career when I banged my hand on a wall for not commenting my code a few months earlier.

And if it works for you the way you want it, it's all good. I may get burned for this, but I have a saying:
Make it work. Then do it better

Wrapping it all up, the final function would be:
var elements = [];
 
// Create Arrays to hold the values
var nameArray = [];
$("._name_Field").each(function () { nameArray.push($(this).val()) });
 
var codeArray = [];
$("._code_Field").each(function () { codeArray.push($(this).val()) });
 
// Gather the Data from the Element Containers from valid elements
var jdx = 0;
for (i = 0; i <= parseInt($("._name_Field").length) - 1; i++) {
  if ($.trim(nameArray[i]) != ""
	&& $.trim(codeArray[i]) != "") {
     elements[jdx] = [];
     elements[jdx]["name"] = nameArray[i];
     elements[jdx]["code"] = codeArray[i].toUpperCase();
     
     jdx++;
  }
}

Full-fledged Java/.NET lover, full-fledged PHP hater.
Full-fledged Google/Microsoft lover, full-fledged Apple hater.
Full-fledged Skype lover, full-fledged YM hater.

GeneralRe: Looping through elements, to find textbox, and get values Pin
jkirkerx9-Aug-12 10:18
memberjkirkerx9-Aug-12 10:18 
GeneralRe: Looping through elements, to find textbox, and get values Pin
Andrei Straut9-Aug-12 10:35
memberAndrei Straut9-Aug-12 10:35 
AnswerRe: Looping through elements, to find textbox, and get values Pin
Andy Brummer9-Aug-12 10:26
sitebuilderAndy Brummer9-Aug-12 10:26 
GeneralRe: Looping through elements, to find textbox, and get values Pin
jkirkerx9-Aug-12 16:54
memberjkirkerx9-Aug-12 16:54 
GeneralRe: Looping through elements, to find textbox, and get values Pin
Andy Brummer10-Aug-12 4:56
sitebuilderAndy Brummer10-Aug-12 4:56 
GeneralRe: Looping through elements, to find textbox, and get values Pin
jkirkerx10-Aug-12 6:31
memberjkirkerx10-Aug-12 6:31 
QuestionGet value from textbox created with createElement Pin
jkirkerx6-Aug-12 15:48
memberjkirkerx6-Aug-12 15:48 
AnswerRe: Get value from textbox created with createElement Pin
BobJanova7-Aug-12 0:12
memberBobJanova7-Aug-12 0:12 
GeneralRe: Get value from textbox created with createElement Pin
jkirkerx7-Aug-12 6:24
memberjkirkerx7-Aug-12 6:24 
AnswerRe: Get value from textbox created with createElement Pin
twseitex7-Aug-12 8:25
membertwseitex7-Aug-12 8:25 
GeneralRe: Get value from textbox created with createElement Pin
jkirkerx8-Aug-12 10:54
memberjkirkerx8-Aug-12 10:54 
Questionadapt datatable jquery to jee application Pin
ahmadiss4-Aug-12 23:34
memberahmadiss4-Aug-12 23:34 
AnswerRe: adapt datatable jquery to jee application Pin
Richard MacCutchan5-Aug-12 1:24
mvpRichard MacCutchan5-Aug-12 1:24 
Questionshowmodaldialog return value undefined across domain Pin
Amr M. K.4-Aug-12 23:24
memberAmr M. K.4-Aug-12 23:24 
AnswerRe: showmodaldialog return value undefined across domain Pin
jkirkerx5-Aug-12 11:05
memberjkirkerx5-Aug-12 11:05 
GeneralRe: showmodaldialog return value undefined across domain Pin
Amr M. K.6-Aug-12 22:05
memberAmr M. K.6-Aug-12 22:05 
AnswerRe: showmodaldialog return value undefined across domain Pin
BobJanova5-Aug-12 23:43
memberBobJanova5-Aug-12 23:43 
GeneralRe: showmodaldialog return value undefined across domain Pin
Amr M. K.6-Aug-12 21:26
memberAmr M. K.6-Aug-12 21:26 
QuestionEvalulating and passing RGB values from keydown Pin
jkirkerx2-Aug-12 8:12
memberjkirkerx2-Aug-12 8:12 
AnswerRe: Evalulating and passing RGB values from keydown Pin
enhzflep2-Aug-12 9:58
memberenhzflep2-Aug-12 9:58 
GeneralRe: Evalulating and passing RGB values from keydown Pin
jkirkerx2-Aug-12 11:54
memberjkirkerx2-Aug-12 11:54 
AnswerRe: Evalulating and passing RGB values from keydown Pin
BobJanova5-Aug-12 23:31
memberBobJanova5-Aug-12 23:31 
GeneralRe: Evalulating and passing RGB values from keydown Pin
jkirkerx6-Aug-12 7:22
memberjkirkerx6-Aug-12 7:22 
GeneralRe: Evalulating and passing RGB values from keydown Pin
BobJanova7-Aug-12 0:11
memberBobJanova7-Aug-12 0:11 
GeneralInstalling Pygame Pin
johtnkucz30-Jul-12 4:36
memberjohtnkucz30-Jul-12 4:36 
GeneralRe: Installing Pygame Pin
johtnkucz30-Jul-12 4:45
memberjohtnkucz30-Jul-12 4:45 
QuestionRe: Installing Pygame Pin
Richard MacCutchan30-Jul-12 6:14
mvpRichard MacCutchan30-Jul-12 6:14 
AnswerRe: Installing Pygame Pin
Sandeep Mewara31-Jul-12 8:56
mvpSandeep Mewara31-Jul-12 8:56 
GeneralRe: Installing Pygame Pin
Richard MacCutchan31-Jul-12 22:32
mvpRichard MacCutchan31-Jul-12 22:32 
QuestionAnalysing an obfuscated malware script Pin
Bernhard Hiller29-Jul-12 21:48
memberBernhard Hiller29-Jul-12 21:48 
AnswerRe: Analysing an obfuscated malware script Pin
Richard MacCutchan29-Jul-12 22:00
mvpRichard MacCutchan29-Jul-12 22:00 
GeneralRe: Analysing an obfuscated malware script Pin
enhzflep30-Jul-12 1:43
memberenhzflep30-Jul-12 1:43 
GeneralRe: Analysing an obfuscated malware script Pin
Bernhard Hiller30-Jul-12 4:48
memberBernhard Hiller30-Jul-12 4:48 
GeneralRe: Analysing an obfuscated malware script Pin
enhzflep30-Jul-12 23:41
memberenhzflep30-Jul-12 23:41 
GeneralRe: Analysing an obfuscated malware script Pin
Bernhard Hiller31-Jul-12 2:26
memberBernhard Hiller31-Jul-12 2:26 
GeneralRe: Analysing an obfuscated malware script [edit] Pin
Jacek Gajek1-Jun-13 4:10
memberJacek Gajek1-Jun-13 4:10 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.


Advertise | Privacy |
Web04 | 2.8.170813.1 | Last Updated 16 Aug 2017
Copyright © CodeProject, 1999-2017
All Rights Reserved. Terms of Service
Layout: fixed | fluid