親要素を指定できていない？んでしょうか？

javascript
1function ($td) {
2var time = new Date();
3var thisyear = time.getFullYear();
4for (var i = thisyear; i >= 1900; i--) {
5$td.find(' .year').append('<option value="' + i + '">' + i + '</option>');
6}
7for (var i = 1; i <= 12; i++) {
8$td.find(' .month').append('<option value="' + i + '">' + i + '</option>');
9}
10for (var i = 1; i <= 31; i++) {
11$td.find(' .day').append('<option value="' + i + '">' + i + '</option>');
12}
13};

HTMLのテーブルの<td>を要素で取ってきて、その子要素のyear,month,dayクラスに日付を入れたいのですがこれでは起動しません
いったいなにがだめなんでしょう？

HTML
1<td>
2<select class='year' name='year'></select>
3<select class='month' name='month'></select>
4<select class='day' name='day'></select>
5</td>

Lhankor_Mhy

2017/08/02 01:23

補足願います。関数呼び出し部分のコードも書いてください。また、関数名が抜けてたり、かっこの対応がずれてたりしてますが、実際に動かしてるコードがこれなんでしょうか？

subaru_10

2017/08/02 01:28

すみません　functionの閉じる前の部分に大量にだめだったものをコメントとして書いていたので閉じるところが抜けていました。呼び出しは同一ファイルに書いているので関数名はいらないと思い書いていませんでした。関数名つけて呼び出すというとfunction CDT ($td~ とつけて、呼び出す際CDT();とするとかでしょうか

m.ts10806

2017/08/02 01:43 編集

htmlもなるべく全体をご提示ください。またいきなりソースコードから入っていますが、「こういうことがしたくてこういう風にやってみた」と「こういうことがしたくて」の部分が抜けているため適切な回答が得られない可能性があります。

行動規範の内容に同意します

回答2件

function内部には特に不審な点はありませんので、
functionの呼び出し自体はされているのか？
引数の$tdはjQueryオブジェクトなのか？
引数の$tdは、本当に想定のタグを指しているのか？
このあたりを調べてください。
ブラウザでF12を押して出てくるデベロッパーツールを使えば、特に問題なく調査できるはずです。

投稿2017/08/02 01:41

zohnam

総合スコア1441

自己解決

すみません自己解決しました。
えっと、色々ずっと試していたら

javascript
1function() {
2var y = $("td").children(' .year');
3var m = $("td").children(' .month');
4var d = $("td").children(' .day');
5
6var time = new Date();
7var thisyear = time.getFullYear();
8for (var i = thisyear; i >= 1900; i--) {
9y.append('<option value="' + i + '">' + i + '</option>');
10}
11for (var i = 1; i <= 12; i++) {
12m.append('<option value="' + i + '">' + i + '</option>');
13}
14for (var i = 1; i <= 31; i++) {
15d.append('<option value="' + i + '">' + i + '</option>');
16}
17};