NULLまたはオブジェクトではありませんのエラーについて

質問

いつも大変勉強させていただいています。初心者です。
さて、現在自動計算を作成中ですが、
現在、オプションメニューで計算させる事には成功したのですが、出来ればラジオボタンで選択したものを計算できるようにと考えて色々試したのですが、上手く設定が出来ません。
皆さんのお知恵を拝借できないでしょうか？
宜しくお願いいたします

＜＜外部リンクファイルです＞＞
function KeiCalc(KC) {
//オーダー見積計算式
function KeiCalc(KC) {
var item1,no1;
//表生地
item1 = document.Calc.X1.options[document.Calc.X1.selectedIndex].value;
if(item1 == 1){Z1 = 8000}
if(item1 == 2){Z1 = 10000}
document.Calc.Z1.value =Z1;
no1 = item1;
//裏生地
item2 = document.Calc.X2.options[document.Calc.X2.selectedIndex].value;
if(item2 == 1){Z2 = 8000}
if(item2 == 2){Z2 = 10000}
document.Calc.Z2.value =Z2;
no2 = item2;
（省略）
//合計計算
document.Calc.PRC.value = Math.round((eval(document.Calc.Z1.value)+eval(document.Calc.Z2.value)+(eval(document.Calc.Z3.value)*eval(Z6))+eval(document.Calc.Z4.value))*eval(Z5));
（省略）
}

ホームページのフォームの内容
【この場合は動作するのですが、ラジオボタンに変更すると動作しません】
<SELECT name="X1" onchange="KeiCalc(this.form)">
<OPTION value="1">ノヴェル・ピンク</OPTION>
<OPTION value="2">ノヴェル・ブル</OPTION></SELECT>

途中のフォームの内容

<INPUT type="radio" name="X1" value="1">ノヴェル
<INPUT type="radio" name="X1" value="2">ツインベリー
これで動作させると以下のようなエラーが出てしまいます
document.Calc.X1.optionsはNULLまたはオブジェクトではありません・・・と

宜しくお願いいたします

回答No.3

mekerock · ベストアンサー

それでは、以下のようにしてみたら、如何でしょう？

ここでは、
item1(no1) -> ラジオボタンで選択された値
にするという前提です。

ラジオボタンやチェックボックスの値の取り方は以下のようにすると、よいでしょう。
if( document.form.button.length ){
for( var i = 0; i < document.form.button.length; i++ ){
if( document.form.button[i].checked ){
val = document.form.button[i].value;
}
}
}else{
if( document.form.button.checked ){
val = document.form.button.value;
}
}

document.form.button　の部分は、使用しているフォームにあわせて名前を変えてくださいね。
ここでのval が　item1(no1)にしたい値となります。
後は、今までと同じように、
if( item1 == 1 ){略}

とするとよいでしょう。

お礼

早速の回答ありがとうございます。おかげ様でできるようになりました(^^♪・・・少し悩んだのですが
これで色々な物に応用できそうです
ありがとうございました。

回答No.4

encountor

省略せずに、一度全文を見せて頂けますか？
これだけでは、エラーがあった箇所しか指摘ができません。

お礼

全文を追加で書こうとしたのですが、お礼や補足の文章が1000文字以下の為文字数がオーバーして書けませんでした。再度投稿しなおしてと思ってた矢先にN0.3さんのアドバイスのおかげでなんとか出来るようになりましたので、誠にありがとうございました
次回からは全文掲載させていただきますのでまた分からないことがありましたら教えてください

回答No.2

mekerock

radio に　option のプロパティはないですよ

フォームのデータを参照するときは、value　プロパティを検証するのが、汎用性があります。

お試しを。

参考URL：

http://www.openspc2.org/

お礼

お返事ありがとうございます
item1 = document.Calc.X1.options[document.Calc.X1.selectedIndex].value;
if(item1 == 1){Z1 = 8000}
if(item1 == 2){Z1 = 10000}
document.Calc.Z1.value =Z1;
no1 = item1;
この部分のoptionsがダメなんですか？
No.1さんの通りにすると動作はするのですが、
Item1（NO1）とZ１と両方に値を代入しないと他の動作がしません。No.1さんの方法で両方に値を入れるにはどうしたらよいのでしょうか？
基本からあまりよく分らずにやっている初心者ですので誠に申し訳ございませんが教えてもらえると幸いです

回答No.1

encountor

if (document.Calc.X1[0].checked) {
Z1=8000;
} else {
Z1=10000;
}

こんな感じかな

お礼

早速のご回答誠にありがとうございます。たしかにこの部分を変更すると計算はされるようになるのですが、先ほどは省略した部分でエラーが出てしまいます
//表生地
item1 = document.Calc.X1.options[document.Calc.X1.selectedIndex].value;
if(item1 == 1){Z1 = 8000}
if(item1 == 2){Z1 = 10000}

//オーダー見積計算式
function KeiCalc(KC) {
var item1,no1;

//表生地
item1 = document.Calc.X1.options[document.Calc.X1.selectedIndex].value;
if(item1 == 1){Z1 = 8000}
if(item1 == 2){Z1 = 10000}

document.Calc.Z1.value =Z1;
no1 = item1;
省略
var item1 = new Array()
var item2 = new Array()
var item3 = new Array()
var item4 = new Array()
var item5 = new Array()
var item6 = new Array()

//表生地
item1[1] = "【表生地：ロアン生地】";
item1[2] = "【表生地：ノヴェル生地】";

//裏生地
item2[1] = "【裏生地：ロアン生地】";
item2[2] = "【裏生地：ノヴェルピンク】";

省略
document.Calc['OPT-01'].value = item1[no1]+":"+item2[no2]+":"+item3[no3]+":"+item6[no6]+":"+item4[no4]+":"+item5[no5];
}