Simplificar la funcion

[Italico76]

Algo asi:

Código Javascript:

<div id="obj_nombre" value="dd">A</div>
<div id="obj_departamento" value="">B</div>
<div id="obj_ci" value="">C</div>
 
<script>
vacio = "1px #F00 solid";
// ...
 
var campovacio=0;
var objetos = new Array;
var objetos = ["nombre","departamento","ci"];
 
for (var i=0;i<objetos.length;i++)
{
    var obj = document.getElementById("obj_"+objetos[i]);   
    if (obj.getAttribute('value') == "undefined" || obj.getAttribute('value') == "")
    {       
        obj.style.border = vacio; 
        campovacio++;
    }else
        obj.style.border = ""; 
}    
 
</script>
 

Si son requeridos... yo les pondria el atribute "required" ...

A proposito... para los SELECT OPTIONS... hice algo:

Código Javascript:

<!--
    Validar si los SELECT..OPTIONs tienen opciones seleccionadas
    es una implementacion de required
    
    Si hay una primera opcion del tipo "Seleccione su ciudad",etc
    debe llevar la clase 'ignored'
 
-->
 
<script>
/*
    @author Pablo Bozzolo
    @param null
    @return boolean 
*/
function selectRequiredValidate()
{
    selects = document.getElementsByTagName('SELECT');
    
    selected=0; 
    required_selects = 0;
    for (var k=0;k<selects.length;k++)
    {       
        sel = selects[k];
        if (sel.getAttribute('class')=='required')
        {
            required_selects++;
            this_selected=false;
            for (ix=0;ix<sel.length;ix++)       
            {               
                option = sel[ix];                                               
                    if ((option.selected) && (option.getAttribute('class')!='ignored'))
                        this_selected=true;
                        
            }   
            
            if (this_selected)  
                selected++;
        }                   
    }
    
    return (selected==required_selects);                
}   
 
</script>
 
    <input type="text" name="nombres" value="Juan" class="required"/><p/> 
 
    <select name="departamento[]" class="required" >
        <option value="" class="ignored">Selecciona depto</option"> 
        <option value="Bogota DC">Bogota DC</option"> 
        <option value="Antioquia">Antioquia</option"> 
        <option value="Caldas" selected>Caldas </option"> 
        <option value="Quindio" >Quindio</option">  
    </select> 
    
    <label>Inquietudes medio-ambientales:</label><p/>
    <!-- SELECT...OPTIONs -->   
    <select name="departamento[]" multiple class="required">
        <option value="" class="ignored">Selecciona</option"> 
        <option value="Hg" selected="selected">Mercurio en las aguas</option"> 
        <option value="CO2">Emision de CO2</option"> 
        <option value="deforestacion" selected="selected">Deforestacion</option"> 
        <option value="otras" selected="selected">Otras</option">   
    </select> 
    <p/>   
    
<script>    
    console.log(selectRequiredValidate());
</script>