Pergunta

Esta questão já tem uma resposta aqui:

Eu tenho o código PHP como a seguir.

<?php
    echo "<SCRIPT LANGUAGE='javascript'>add dd_Employee('".$id1."','".$fname1."',".$sal1.");</SCRIPT>";
    echo "<SCRIPT LANGUAGE='javascript'>add dd_Employee('".$id2."','".$fname2."',".$sal2.");</SCRIPT>";
    ....

?>
...

E o meu código JavaScript contém

var details=new Array();

function addEmployee(id,fname,salary){
    var temp=new Array(id,fname,salary);
    details[details.length]=temp;
    alert(temp);
}

function displayEmployee(){
    for (var i=0;i<details.length;i++)
        var temp_details=details[i];

    //Display the id,name,sal
    var id=temp_details[0];
    var name=temp_details[1];
    var sal=temp_details[2];

    alert(id);
    alert(name);
    alert(sal);
}

Agora o problema é que quer atribuído pelo PHP irá visble apenas em AddEmployee. O que significa que está funcionando bem. Mas dentro displayEmployee o valor atribuído pelo PHP não vai funcionar. Existe alguma falha na minha lógica?

Foi útil?

Solução

Apenas preencher sua estrutura global de dados diretamente em vez de passá-lo através de uma função JavaScript. Você provavelmente estão correndo em um problema escopo de variáveis ??desde que você está alocando memória em addEmployee.

Exemplo:

<?php
print "details[details.length] = new Array('$id1', '$name1', '$salary1');\n";
?>

Outras dicas

Parece que você está perdendo algumas chaves em seu loop for. Agora, você tem um loop for 1-line que não faz nada, mas atribuir um valor para a variável temp, que é re-declarada em cada iteração. Eu não consigo lembrar o que as regras de escopo são para JavaScript nesta situação, mas geralmente é uma prática ruim para fazer isso (se é intencional). Tente algo como isso para o seu método displayEmployee().

function displayEmployee(){
    for(var i=0; i<details.length; i++) {

        var temp_details = details[i];

        //Display the id, name, and sal.
        var id = temp_details[0];
        var name = temp_details[1];
        var sal = temp_details[2];

        alert(id);
        alert(name);
        alert(sal);
    }
}

Isso deve resolver o seu problema imediato. Se você tem o controle sobre todo o seu código, eu recomendo transformando os dados de funcionários em um objeto por isso é mais administrável. Para fazer isso, você deve usar este código PHP:

<?php
    echo '<script>
              addEmployee('.json_encode(array(
                  'id'     => 'id1',
                  'name'   => 'fname1',
                  'salary' => 1024,
              )).');
              addEmployee('.json_encode(array(
                  'id'     => 'id2',
                  'name'   => 'fname2',
                  'salary' => 2048,
              )).');
         </script>';
?>

E, em seguida, alterar o seu código JavaScript para ficar assim:

var details = [];

function addEmployee(employeeObj){
    details[details.length] = employeeObj;

    alert(employeeObj);
}
function displayEmployee(){
    for(var i=0; i<details.length; i++) {
        alert(details[i].id);
        alert(details[i].name);
        alert(details[i].salary);
    }
}

Eu não testei este código, mas ele deve ser executado.

Você tem um erro em sua chamada JavaScript gerado pelo PHP

echo "<SCRIPT LANGUAGE='javascript'>add dd_Employee('".$id1."','".$fname1."',".$sal1.");</SCRIPT>";

deve ser

echo "<SCRIPT LANGUAGE='javascript'>addEmployee('".$id1."','".$fname1."',".$sal1.");</SCRIPT>";

E se você quiser mudá-lo para OOP?

<?php
    $employeeList[] = array('sdf3434', 'Bob', 123.34);
    $employeeList[] = array('wer335dg', 'Joe', 223);
?>
<html>
    <head>
        <script>
            function Employees() {
                this.list = new Array();
            }
            Employees.prototype.add = function (id, fname, sal) {
                this.list.push(new Array(id, fname, sal));
            };
            Employees.prototype.display = function () {
                for(var i=0, t = this.list.length;i<t;i++) {
                    var emp = this.list[i];
                    alert(emp[0]);
                    alert(emp[1]);
                    alert(emp[2]);
                }
            };
            var employees = new Employees();
            <?php
                foreach ($employeeList as $employee) {
                    list($id, $fname, $sal) = $employee;
                    printf("employees.add('%s', '%s', %f);\n", $id, $fname, $sal);
                }
           ?>
        </script>
    </head>
    <body>
        <a href="#" onclick="employees.display();">click</a>
    </body>
</html>
Licenciado em: CC-BY-SA com atribuição
Não afiliado a StackOverflow
scroll top