Javascript med PHP: array melder undefined

Du skal nok ikke have et komma after den sidste værdi. Det er kommaet i

            echo "[\"".$row_model['navn']."\"],\n";

der giver dig problemet, men kun sidste gang.

Nå, ja, også kommaet i: echo "]],\n\n";

Du kan nok nemmest løse det ved at have dine data i PHP-arrays inden du laver det til Javascript. Det vil også give mere overskuelig kode - syn's jeg

Det vil det uden tvivl. Men jeg har ingen annelse om hvordan jeg rykker php'en ind i min java, slet ikke når det er et array :)

Utestet - fixer kun det ene, men vigtigtse, komma:

var arr = [
<?php
    $sql = mysql_query("SELECT * FROM fabrikat ORDER BY navn");
    while($row = mysql_fetch_array($sql)) { 
    $id = $row['id'];
 
 
        $sql_model = mysql_query("SELECT * FROM model WHERE fabrikat = $id ORDER BY navn");
        $a=Array();
        while($row_model = mysql_fetch_array($sql_model)) {
          $a[]='["'.$row_model['navn'].'"]';
        }
     
    echo "[\"".$row['navn']."\",[\n\n";
    echo join(',',$a);
    echo "]],\n\n";
    }
?> 
];

Fjerner undefined i select box nummer to, men "udføres med fejl på siden".

Hvad er den javascript kode, der laves? Og hvad er fejlen?

Okay jeg smækker lige det hele herind :)

<-------------------JAVA------------------------->
<script language="javascript" type="text/javascript">
var arr = [
<?php
    $sql = mysql_query("SELECT * FROM fabrikat ORDER BY navn");
    while($row = mysql_fetch_array($sql)) {
    $id = $row['id'];


        $sql_model = mysql_query("SELECT * FROM model WHERE fabrikat = $id ORDER BY navn");
        $a=Array();
        while($row_model = mysql_fetch_array($sql_model)) {
          $a[]='["'.$row_model['navn'].'"]';
        }
   
    echo "[\"".$row['navn']."\",[\n\n";
    echo join(',',$a);
    echo "]],\n\n";
    }
?>
];
//alert(arr1[1][1])
function initSels(){
    createOpts(document.getElementById('sel1'));
    createOpts(document.getElementById('sel3'));
}

function createOpts(sel){
    var s = new Array();
    var num = 0, txt = "arr";
    var ar = opt = null;
    var n = sel.selectedIndex;
    var args = createOpts.arguments.length;
    for(var i=1;args>i;i++){
        s[i-1] = document.getElementById(createOpts.arguments[i]);
        if(s[i-1]==sel)num=i;
    }
    if(args>1){
        if(s[num])sel = s[num];
        else return;
    }
    if(n==0&&args>1){
        for(i=num;args>i+1;i++){
            s[i].length = 0;
            opt = document.createElement('OPTION');
            s[i].appendChild(opt);
            opt.value = "";
            opt.text = "\74-- Vælg --";
        }
        return true
    };
    for(i=0;num>i;i++){
        txt+="["+(s[i].selectedIndex-1)+"][1]";
    }
    ar = eval(txt);
    sel.length = 0;
    opt = document.createElement('OPTION');
    sel.appendChild(opt);
  opt.value = "";
  opt.text = "-- Vælg --";
    for(i=0;ar.length>i;i++){
        opt = document.createElement('OPTION');
        sel.appendChild(opt);
        if(typeof ar[i] == 'object'){
          opt.value = ar[i][0];
          opt.text = ar[i][0];
        }else{
          opt.value = ar[i];
          opt.text = ar[i];
      }
    }
    if(args>num+2){
        for(i=num+1;args-0>i;i++){
            s[i].length = 0;
            opt = document.createElement('OPTION');
            s[i].appendChild(opt);
            opt.value = "";
            opt.text = "\74-- Vælg --";
        }
    }
}
</script>
<------------------------JAVA------------------------------>

<------------------------HTML------------------------------>
<b>Forsøg 2</b>
<form name="sels" action="" onsubmit="return false;">

<select name="sel1" id="sel1" onchange="createOpts(this,'sel1','sel2');">
<option value="" selected>Vis alle</option>
</select>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<select name="sel2" id="sel2">
<option value="" selected
>&lt;-- Vælg --</option>

</select>


</form>

<------------------------HTML------------------------------>

Uh, Fejlen er "Et object var ventet" på linje 377 i html'en det er slutningen på arrayet så vidt jeg kunne nuske mig frem til.

Det var nu mere den javascript-kode, der bliver genereret jeg skal se - der er jo nok en lille fejl ;)

Ah beklager dybt... den fylder dog en del :)

var arr = [
["Alfa Romeo",[

["145"],["146"],["147"],["147 RST"],["155"],["156"],["156 MY"],["156 RST"],["156 RST2"],["159"],["164"],["166"],["166 MY"],["166 RST"],["175"],["1900"],["33"],["75"],["90"],["Alfasud"],["Alfetta"],["Brera"],["Giulietta"],["GT"],["GTV"],["RZ Roadster"],["Spider"],["Sprint"],["SZ"]]],

["Aston Martin",[

["DB7"],["DB7 Vantage"],["DB7 Vantage Volante"],["DB7 Volante"],["Lagonda"],["V8"],["V8 Coupé"],["V8 Vantage"],["V8 Vantage Coupé"],["Vanquish"],["Virage"],["Volante Cabriolet"],["Volante Zagato"],["Zagato"],["Zagato Cabriolet"]]],

["Audi",[

["100"],["200"],["80"],["90"],["A2"],["A3"],["A4"],["A6"],["A8"],["Allroad"],["Cabriolet"],["Coupé"],["Coupé GT"],["Q7"],["Quattro"],["RS 4"],["RS 6"],["S3"],["S4"],["S6"],["S8"],["TT"],["TT Coupé"],["TT Roadster"],["TTS Coupé"],["V8"]]],

["Austin",[

["Healey"],["Maestro"],["Metro"],["Mini"],["Montego"]]],

["Bentley",[

["Anage"],["Brookelands"],["Continental"],["Eight"],["Mulsanne Eight"],["Mulsanne R"],["Mulsanne S"],["Mulsanne Turbo R"],["Touring Limousine"],["Turbo R"],["Turbo R Sport"]]],

["BMW",[

["116"],["316"],["316 II"],["316i"],["316i coupé"],["316ti"],["318"],["318i"],["318i cabriolet"],["318i coupé"],["318is"],["318is coupé"],["318ti"],["320"],["320 ci cabriolet"],["320ci"],["320i"],["320i cabriolet"],["320i coupé"],["323ci"],["323i"],["323i cabriolet"],["323ti"],["325 cabriolet"],["325 i cabriolet"],["325ci"],["325e"],["325i"],["325i coupé"],["325ix"],["325ix"],["328ci"],["328ci cabriolet"],["328i"],["328i cabriolet"],["328i coupé"],["330i"],["330i coupé"],["330xi"],["330xi"],["518i"],["518i"],["520i"],["520i"],["520ia"],["520ia"],["523i"],["525 ix"],["525e"],["525i"],["525i"],["525ia"],["525ia"],["525ix"],["528i"],["528i"],["530i"],["530i"],["530ia"],["530ia"],["535i"],["535ia"],["540i"],["540i"],["540ia"],["545i"],["545i"],["628ci"],["628csi"],["630ci"],["630ci cabriolet"],["635csi"],["645ci"],["645ci cabriolet"],["728i"],["728ia"],["730i"],["735i"],["735ia"],["740i"],["740ia"],["745i"],["745ia"],["745li"],["750i"],["750ia"],["760li"],["840ci"],["850ci"],["850csi"],["850i"],["M3"],["M3 cabriolet"],["M3 coupé"],["M3 csl"],["M3 Sport Evolution"],["M5"],["M5"],["M535i"],["M635csi"],["X3"],["X5"],["Z1"],["Z3"],["Z3 coupé"],["Z3 M"],["Z3 M coupé"],["Z3 M Roadster"],["Z3 Roadster cabriolet"],["Z4 cabriolet"],["Z8 Roadster cabriolet"]]],

["Buick",[

["Century Custom Coupe "],["Elektra"],["Le Sabre Limited "],["Park Avenue"],["Riviera Coupe "],["Riviera T-type Coupé "],["Skylark Limited Coupe "],["Super"]]],

["Cadilac",[

["Allante"],["Broughham"],["CTS"],["De Ville"],["Eldorado"],["Eldorado coupé"],["Fleetwood"],["Limosine"],["Seville"],["Seville STS"],["SRX"],["STS"],["XLR"]]],

["Chevrolet",[

["Alero"],["Apache Pick-up"],["Astro"],["Astro Van"],["Beretta"],["Beretta coupe"],["Blazer"],["Camaro"],["Camaro cabriolet"],["Caprice"],["Celebrity"],["Chevelle"],["Corsica"],["Corsica hatchback"],["Corvette"],["Corvette cabriolet"],["Corvette convertible"],["El Camino"],["Evanda"],["Kalos"],["Lacetti"],["Lumina"],["Malibu"],["Matiz"],["Nubira"],["Pick-up"],["Sedan"],["Starcraft"],["SX3"],["Tacuma"],["Tahoe"],["TrailBlazer"],["Trans Sport"],["Z750S"]]],

["Chrysler",[

["300C"],["300M"],["Baretta"],["Crossfire"],["Daytona Shelby"],["Dodge"],["Grand Voyager"],["GS Turbo"],["Laser Turbo"],["Le Baron"],["Le Baron cabriolet"],["Le Baron coupé"],["Maserati"],["Neon"],["New Yorker"],["PT Cruiser"],["PT Cruiser cabriolet"],["Saratoga"],["Sebring"],["Stratus"],["Stratus cabriolet"],["Viper "],["Vision"],["Voyager"]]],

["Citroën",[

["2CV"],["AX"],["AX 10"],["AX 11"],["AX 14"],["Berlingo"],["BX"],["BX 14"],["BX 16"],["BX 19"],["C1"],["C2"],["C25"],["C3"],["C4"],["C4 coupé"],["C5"],["C5"],["C6"],["C8"],["CX"],["CX 20"],["CX 22"],["CX 22"],["CX 24"],["CX 25"],["CX 25"],["D-Special"],["Dyane"],["Evasion"],["GS"],["GSA"],["Jumper"],["Mehari"],["Picasso"],["Saxo"],["Saxo VTS"],["Visa"],["Xantia"],["Xantia"],["XM"],["XM 110"],["XM 122"],["XM 145"],["Xsara"],["Xsara coupé"],["Xsara Picasso"],["Xsara2"],["ZX"]]],

["Daewoo",[

["Evanda"],["Kalos"],["Lacetti"],["Lanos"],["Leganza"],["Matiz"],["Nexia"],["Nubira"],["Tacuma"]]],

["Daihatsu",[

["Applause"],["Charade"],["Charmant"],["Coure"],["Sirion"],["Terios"],["YRV"]]],

["DeTomaso",[

["Pantera"]]],

["Dodge",[

["Durango"],["Grand Caravan"],["Ramcharger"]]],

["Farrari",[

["308"],["328"],["348"],["348TS"],["355"],["360"],["412"],["456"],["512"],["512TR"],["550"],["575"],["575M"],["612"],["Enzo Farrari"],["F355"],["F40"],["F430"],["F50"],["F512"],["GTO"],["Mondial"],["Mondial Quat. Val."],["Testarossa"]]],

["Fiat",[

["124"],["126"],["127"],["128"],["131"],["132"],["500"],["600"],["Agenta"],["Brachetta"],["Brava"],["Bravo"],["Cinquecento"],["Coupe"],["Croma"],["Doblò "],["Ducato"],["Duna"],["Grande Punto"],["Idea"],["Marea"],["Marea Weekend"],["Multipla"],["Multiplay MY"],["Palio"],["Panda"],["Punto"],["Punto FL"],["Regata"],["Ritmo"],["Scudo"],["Scuto FL"],["Seicento"],["Stilo"],["Stilo MY"],["Tempra"],["Tipo"],["Ulysse"],["Uno"],["X 1/9"]]],

["Ford",[

["A"],["Bronco"],["Capri"],["Consul"],["Cortina"],["Cougar"],["Escort "],["Explorer"],["Fiesta "],["Focus"],["Fusion"],["Galaxy"],["Granada"],["GT"],["Ka"],["Linconl"],["M17"],["Maverick"],["Mercury Cougar XR 7"],["Mondeo"],["Mustang"],["Orion"],["Pobelar"],["Probe"],["Puma"],["S-max"],["Scorpio"],["Sierra"],["Taurus"],["Thunderbird"],["Transit"],["Windstar"]]],

["GMC",[

["Cyclone"],["Savanna"]]],

["Goggomobil",[

["TS"]]],

["Honda",[

["Accord"],["Acura"],["Civic"],["Concerto"],["CR-V "],["CRX "],["FR-V"],["HR-V"],["Integra"],["Legend"],["Logo"],["NSX"],["Prelude"],["S2000"],["S800"],["Shuttle"],["Stream"]]],

["Hummer",[

["H2"]]],

["Hyundai",[

["Accent "],["Atos"],["Coupé "],["Elantra"],["Galloper"],["Getz"],["H100"],["Lantra"],["Matrix"],["Pony"],["S-coupe"],["Santa Fe"],["Scoope"],["Sonata"],["Terracan"],["Trajet"],["Tuscon"]]],

["Jaguar",[

["Daimler"],["S-Type"],["Sovereign"],["Type E"],["X-Type"],["XJ"],["XJ12"],["XJ6"],["XJ8"],["XJS"],["XK"],["XK120"],["XK140"],["XK150"],["XK8"],["XKR"]]],

["Jeep",[

["151"],["Cherokee"],["Grand Cherokee"],["Wrangler"]]],

["Kia",[

["Carens"],["Carnival"],["Cerato"],["Clarus"],["Magentis"],["Opirus"],["Picanto"],["Pride"],["Rio"],["Sephia"],["Shuma"],["Sorento"],["Sportage"]]],

["Lada",[

["110"],["111"],["112"],["1200"],["1300"],["1500"],["1600"],["2104"],["2105"],["2107"],["21214"],["Easy"],["Niva"],["Samara"]]],

["Lancia",[

["A112"],["Dedra"],["Delta"],["K"],["Kappa"],["Lybre"],["Prisma"],["Thema"],["Thesis"],["Y10"],["Ypsilon"],["Zeta"]]],

["Land Rover",[

["110"],["90"],["Defender"],["Discorvery"],["Discovery 3"],["Freelander"],["Land Rover"],["Range Rover"],["Range Rover Voque"]]],

["Lexus",[

["GS"],["GS 300"],["GS 430"],["IS 200"],["IS 300"],["LS 400"],["LS 400i"],["LS 430"],["RX 300"],["RX 400h"],["SC 430"]]],

["Mazda",[

["121"],["2"],["3"],["323"],["323 C"],["323 F"],["323 S"],["5"],["6"],["616"],["626"],["818"],["929"],["Demio"],["MPV"],["MX-3"],["MX-5"],["MX-6"],["Premacy"],["RX-7"],["RX-8"],["Xedos 6"],["Xedos 9"]]],

["Mercedes",[

["170 V"],["180 B"],["190"],["190 E"],["190 SL"],["200"],["230"],["260"],["300"],["400"],["420"],["500"],["560"],["600"],["A"],["C"],["CL"],["CLK"],["E 200"],["E 220"],["E 230"],["E 240"],["E 280"],["E 300"],["E 320"],["E 420"],["E 430"],["E 50"],["E 500"],["E 55"],["G 320"],["G 50"],["G 500"],["ML 320"],["ML 350"],["ML 500"],["ML 55"],["S"],["S 280"],["S 320"],["S 350"],["S 420"],["S 430"],["S 500"],["S 55"],["S 600"],["SL"],["SL 320"],["SL 350"],["SL 500"],["SL 55"],["SL 60"],["SL 600"],["SLK"],["SLK 200"],["SLK 230"],["SLK 32"],["SLK 600"],["Sprinter"],["V 230"],["V 280"],["Viano"],["Vito"]]],

["MG",[

["A"],["B"],["F"],["Maestro EFI"],["Metro"],["MGF"],["Montego"],["TD"],["TF"],["ZR"],["ZS"],["ZT"],["ZT-T"]]],

["Mini",[

["Cooper"],["Cooper S"],["Mini"],["One"]]],

["Mitsubishi",[

["3000"],["Carisma"],["Colt"],["Eclipse"],["Galant"],["Galloper"],["Grandis"],["Grant Pajero"],["Lancer"],["Outlander"],["Pajero"],["Pajero Pinin"],["Sapporo"],["Sigma"],["Space Gear"],["Space Runner"],["Space Star"],["Space Wagon"],["Starion"]]],

["Nissan",[

["100 NX"],["120Y"],["140Y"],["160J"],["200 SX"],["280 ZX"],["300 ZX Targa"],["350Z"],["350Z"],["Almera"],["Almera Tino"],["Bluebird"],["Cherry"],["Infiniti"],["Laurel"],["Maxima"],["Micra"],["Mureno"],["Note"],["Pathfinder"],["Patrol"],["Patrol DR"],["Prairie"],["Primera"],["Serena"],["Silvia"],["Skyline"],["Stanza"],["Sunny"],["Terrano"],["X-Trail"]]],

["Opel",[

["Agila"],["Ascona"],["Astra"],["Calibera"],["Combo"],["Commodore"],["Corsa"],["Frontera"],["GT"],["Kadett"],["Kaptajn"],["Manta"],["Meriva"],["Monza"],["Olympia"],["Omega"],["Reccord"],["Senator"],["Signum"],["Speedster"],["Super"],["Tigra"],["Vectra"],["Vivaro"],["Zafira"]]],

["Peugoet",[

["1007"],["104"],["106"],["107"],["203"],["205"],["206"],["305"],["306"],["307"],["309"],["404"],["405"],["406"],["407"],["504"],["505"],["604"],["605"],["607"],["806"],["807"],["Boxer"],["Expert"],["Partner"]]],

["Pontiac",[

["Bonneville"],["Fiero"],["Firebird"],["Grand Prix"],["Pontiac"],["Trans Am"],["Trans Sport"]]],

["Porsche",[

["356"],["911"],["914"],["924"],["928"],["944"],["968"],["Boxster"],["Carrera GT"],["Cayenne"],["Cayman"]]],

["Renault",[

["11"],["12"],["14"],["16"],["19"],["20"],["21"],["25"],["30"],["5"],["Alpine"],["Clio"],["Clio III"],["Espace"],["Grand Espace"],["Grand Scénic "],["Kangoo"],["Laguna"],["Master"],["Master T28"],["Mégane"],["Mégane II"],["Modus"],["Safrane"],["Scénie"],["Trafic"],["Twingo"]]],

["Rover",[

["111"],["114"],["214"],["216"],["218"],["220"],["25"],["3500"],["414"],["416"],["420"],["45"],["620"],["623"],["75"],["800"],["820"],["825"],["827"],["MG"],["MINI"]]],

["Saab",[

["9-3"],["9-5"],["90"],["900"],["9000"],["900i"],["96"],["99"]]],

["Seat",[

["Alhambra"],["Altea"],["Arosa"],["Cordoba"],["Ibiza"],["Leon"],["Malaga"],["Marbella"],["Ronda"],["Toledo"]]],

["Skoda",[

["105"],["120"],["125"],["130"],["135i"],["136"],["Fabia"],["Favorit"],["Felicia"],["Octavia"],["Superb"]]],

["Smart",[

["Cabrio"],["City Coupé"],["ForFour"],["ForTwo"],["Roadster"]]],

["Ssang Yong",[

["Kyron"],["Musso"],["Rexton"]]],

["Subaru",[

["Forester"],["Impreza"],["Lagacy"],["Outback"],["SVX"],["Vivio"],["XT"]]],

["Suzuki",[

[" Samurai "],["Alto"],["Baleno"],["Grand Vitara"],["Ignis"],["Liana"],["SJ 410"],["SJ 413"],["Swift"],["Vitara"],["Wagon R"]]],

["Toyota",[

["Avensis"],["Camry"],["Carina"],["Celica"],["Corolla"],["HiAce"],["Landcruiser"],["MR2"],["Picnic"],["Previa"],["RAV4"],["Supra"],["Yaris"],["Yaris Verso"]]],

["Volvo",[

["240"],["340"],["360"],["440"],["460"],["480"],["740"],["760"],["780"],["850"],["940"],["950"],["960"],["C70"],["S40"],["S50"],["S60"],["S70"],["S80"],["V40"],["V60"],["V70"],["XC70"],["XC90"]]],

["VW",[

["1300"],["1302"],["1303"],["1600 TL"],["Beetle"],["Bora"],["Caddy"],["Caravelle"],["Fox"],["Golf"],["Jetta"],["Lupo"],["Multivan"],["Passat"],["Polo"],["Sharan"],["Touran"],["Vento"]]],

["Yugo",[

["45A"],["511"],["55"],["Florida"]]],

];


Det er det der bliver genereret :)

Ok - så tror jeg du skal tælle linier igen for at se præcis hvor det går galt. Jeg kan umiddelbart efter de linier du har pastet her, skrive

alert(arr[1][1][1]);

og få en værdi - helt uden fejl, endda i flere forskellige browsere.

Linje 377:     var n = sel.selectedIndex;

Hvilket giver hat mening hos mig

Den er fra function createOpts(sel)

Hvis
      <select name="sel1" id="sel1" onchange="createOpts(this,'sel1','sel2');">
er det eneste sted du kalder createOpts er jeg også forvirret.

Men der er jo et sted mere
    createOpts(document.getElementById('sel3'));
Har du en 'sel3' ?

Nej, men den kode jeg huggede havde. Så der var 3 'afhængige' selects

alert(sel) i funktionen giver hvad?

Ellers put alert("Her er jeg") før og efter alle createOpts-kald - med forskellig tekst, så du kan se hvor den bøffer.

Skal jeg forstå at du ikke har en 'sel3'? Så er det vel løsningen på problemet.

Dvs.: der er stadig en udefineret værdi efter komma i dit array. Skal det også fixes?

Jeg har ikke en sel3, og ja det må du meget gerne fixe :)

Men kører det nu, hvis du fjerner linien med 'sel3' - og hvis det kører, siger den så stadig "undefined".

Jeg får ikke lige tid til at fixe noget nu.

function initSels(){
    createOpts(document.getElementById('sel1'));
}

fjernede fejlen - slettede sel3. Så det eneste problem nu er at der står 'undefined' i bunden af sel1.

Ok, utestet igen

var arr = [
<?php
    $b=Array();
    $sql = mysql_query("SELECT * FROM fabrikat ORDER BY navn");
    while($row = mysql_fetch_array($sql)) {
    $id = $row['id'];


        $sql_model = mysql_query("SELECT * FROM model WHERE fabrikat = $id ORDER BY navn");
        $a=Array();
        while($row_model = mysql_fetch_array($sql_model)) {
          $a[]='["'.$row_model['navn'].'"]';
        }
   
    $b[]= "[\"".$row['navn']."\",[\n\n".join(',',$a)."]]";
    }
  echo join(',',$b);
?>
];

Den sad lige i øjet... TUSIND TAK

Jeg samler ikke på point. Svar selv, accepter eget svar.

Hmm hjælpe for at hjælpe... skrammende hvis alle gjorde det :)