|
[No.65 Ｋｅｎ３、土曜日工場でいろんな汗をかく、、、] [No.66 **************************************] [No.67 Ｋｅｎ３、久々Ｃ言語の質問にあわてる] [No.68 Ｋｅｎ３、ソースのコメントについて考える [No.69 Ｋｅｎ３、指摘メールに考えさせられる] |
| No.65 | 2000/05/29 Ｋｅｎ３、土曜日工場でいろんな汗をかく、、、 |
[ページTOPへ戻る] |
＜Ｋｅｎ３、土曜日工場でいろんな汗をかく、、、＞
目次 1.はじめの挨拶 2.1.44ＦＤでハマル、、、、 3.くたばれ、電気関係の頑固オヤジ、、、 4.おわりの挨拶 ---------------------------------------------------------------------------- /* * １.はじめの挨拶 */ こんにちは。 とある会社に勤める三流ＰＧのＫｅｎ３です。 このメールマガジンは脚色されているので、 ご注意を、、、（実際と違う内容も含まれているので、） なんか、日刊の愚痴日記っぽくなっている、 毎日愚痴ばっかなので、 みなさん、そろそろあきてきたかなぁ？ ＊みなさんの立場で読んで、笑ってください。 疑問・感想、、指摘、、あったら、メール下さい。 /* * ２．1.44ＦＤでハマル、、、、 */ ９年前、Ｃ言語で作成したハカリから重量データ取り込みのシステム、 土曜日、工場のラインが止まっているので現地インストール確認に行きました。 修正した、ソース、ＥＸＥをＦＤに保存し、 念のため、ＦＤが読めないといけないから、 ２枚複製を取って、、、持って行きました。 ところが（題名でネタバレの人は、ＮＥＣ９８を知ってるベテランさん？） 持って行ったＦＤが２枚とも読めない、、、 あれ、、ヤバイ、、と小パニック、、、 冷静になってみてみると、 ＮＥＣ９８０１ＤＸ？あっ、１．４４のＦＤ読めないジャン、、 ヤバイよ。 落着いて、1.44ＦＤを720に変換しようと思い、 工場長にＤＯＳ／Ｖの機械貸してもらいたいのですが？ と問い合わせたところ、うちにはＩＢＭのオフコンしか無い？ とのこと？ おかしいなぁ、、、さっきの世間話で、 本社発注はインターネットの画面からって言ってたジャン。。。 かま かけて、メールはどのマシーンで毎日見ているのですか？ とさりげなく問い合わせ、ノートパソコンの存在をつきとめる。 これでＯＫかなぁ、、と思ったら、 ２ＤＤのＦＤが無いよ、、、 昔の知恵？で、２ＨＤの右側の穴をつぶして、２ＤＤの変わりとする。 なんとか、１．４４ＦＤに保存されていた、ソース、ＥＸＥを ＮＥＣ９８に持っていくことができました。。。 枠外解説・愚痴 --- 世間話は必要？ 現地調整・デバッグなどに行くと、 興味を持ってるのか？ダマって後ろにハリツク人が居る。 何か質問ですか？と言うと、 イヤ、見てるだけです。。。と 心の中では、緊張するから、後ろに居ないでといいたいが、、、 ソースをスクロールさせながら見ているのをチェックしていて、 よくそんなに速く画面の文字読めますね とボソッと一言。 あのね、、、 全て読んでるんじゃなくて、 修正場所へ移動しているだけなんだよ、、、 速読のプロでもそんなに速く読めないって、、 英語ばかりでよくわかりますね、、、 って 英語知らなくてもプログラム組めますよ（知ってたほうがいいけど） この質問も、答えにくい、、、 英語は知らないので、記号だと思ってやってます、、といつも答えてるが、、、 こんな時、いても居なくてもいい、上司が同行していると、 おじさんの相手をしてくれて、楽でいい。 ＊上司のありがたみが、よくわかった、、、なんてね。 （私も、後輩から先輩が素人の相手してくれて助かったって 言われてるんだろうなぁ〜） /* * ３．くたばれ、電気関係の頑固オヤジ、、、 */ コンベアラインの入替えも同時に行っていたので、 空調が止まっていて、ものすごい暑かった。。。 太っていて、汗をかく私は見ていて、見苦しかった？ って、話は、置いといて、 くたばれ、電気関係の頑固オヤジ、、、と今回だけは言いたい。 システム概要を話すと、 配送センターの梱包ラインのシステムで、 出荷時バーコードで出荷ナンバーを読み取り、 その後、荷物の重量をハカリ、ナンバーと重量を保存、 ホストにＦＤでデータを返すシステムです。 で、３Ｋｇ以下は、バンドがけしないって信号があります。 今回修正したのは、ハカリの重量データ取り込み部分なので、 信号系の処理は、昔のままでした。 テスト稼動時、３Ｋｇ以下の信号と３Ｋｇ以上の信号が 逆になっている、、、とクレームが来る。 そんなはずないですよ、、と答えるが信じてくれない。。。 しかたなく、私がソースを変更し（逆にして）出力する。 テスト稼動ＯＫになる。。。 修正したソースをＦＤに保存して、さっ帰りますか、、、 と準備中に、シーケンサプログラムのなかで、交差していた？ ってわけわかんないこと急に言い出して、 やっぱり、元に戻してくれと、、、 表に出ている仕様書と実際が違ったらしい、、、 で、ソースをまた、元に戻す。。。 自分が間違っていない、、と頑固になるのもいいけどねぇ。 /* * ４.終りの挨拶 */ 今回も失敗談でしたね、、、 今度は、自慢できるような話を？得意気にしたいですね。 三流ＰＧ Ｋｅｎ３でした。 ＊一部の読者の心の中？ 毎日、いろいろあって、あきないねぇ、 三流のＫｅｎ３らしいよねぇ？ サクっと行った、自慢話は無いの？ ---> 体型同様、スマートじゃなくてねぇ（笑） ソースも無意味に肥大化しちゃって、ダイエット必要？ ＊プログラマーの性格がソースに現れるってホントですね。
| No.66 | 2003/00/00 *************** |
[ページTOPへ戻る] |
＜----＞
| No.67 | 2003/06/08 Ｋｅｎ３、久々Ｃ言語の質問にあわてる |
[ページTOPへ戻る] |
＜Ｋｅｎ３、久々Ｃ言語の質問にあわてる＞
サブタイトル、、 ご教授下さいより、、解説できるもんならやってみろのほうが好 目次 1.はじめの挨拶 2.ご教授下さいより、、、解説できるもんならやってみろのほうが好き 3.本題の質問にやっとこたえる 4.おわりの挨拶 ---------------------------------------------------------------------------- /* * １.はじめの挨拶 */ こんにちは。 三流ＰＧのＫｅｎ３です。 このメールマガジンは脚色されているので、 ご注意を、、、（実際と違う内容も含まれているので、） 今回は、なぜかＫｅｎ３に来た、 ＣからＶＢへの移植の質問について、、書きたいと思います。 ＊ほんとは、愚痴以外は別に分けたＶＢＡの方に書くんだけど、 今回は、こっちに載せて、反応見てからまとめてみたくて。。。 （なんか、あっちのＶＢＡ系は落ちつけなくて、、、） ＊みなさんの立場で読んで、笑ってください。 疑問・感想、、指摘、、あったら、メール下さい。 /* * ２．ご教授下さいより、、、解説できるもんならやってみろのほうが好き */ まだ、前置き、続きます。 私、変わり者なので（そんなん、いまさら言わなくてもわかってるって？） ご教授下さい、よろしくお願いします の質問より 解説できるもんならやってみろ のほうがなんとなく好きなんだよなぁ、、、 /* * ３.本題の質問にやっとこたえる */ 読者から来た、質問に下記のように答えました。 ＊足りないところや、ごまかし、あるので、 ご指摘、アドバイスください。 どうしたんですか？まじめな質問くるなんて、、、大丈夫ですか？ って、前置きは、これくらいで、 私が今、わかる範囲での回答、書きます。 In message "移植。", >こんちは。ken3。 > >チョット必要に迫られて、C言語で書いたソースを解析してんだけど。 >ワカラン所教えて。（VBに移植する為） >（全てを送るわけには、行かないんで、コードが部分的だけど （笑）） > >1) > > #includeこのヘッダーが有れば、ＬＳＩコンパイラでも使える？ ＬＳＩコンパイラでも使える？ これは、厳しいかも？ > #includeこのヘッダーは、何？ ｇｅｔｃｈ（）とか、コンソール系の関数だと思う。 SEEK_CURはｓｔｄｉｏ．ｈだと思うけど、 定数とかが入っている、、、 > #include 同上 なに？ > #pragma hdrstop これは、何でしょ？ えっ、、 > #define BLOCK_SIZE 32768 これの意味は？VBで言うCONST？ だと思って、いいと。 みなさんは、違うっていいそうだけど、、、 >-------------------------------------------- >2) > > #pragma argsused > int main(int argc, char **argv) > { >＊って、ポインタだよね？ だけど「＊＊」とは？ >この場合、exeにコマンドラインを渡せるって事？ >実際のexeでは、1つしか渡せないみたいだけど。 >なんで、引数が2つ？ In message "[三流PG No.020] C,VB,Access,Excel ここはどこ？私は誰？", >/* > * ５．C ここはどこ？私は誰？ >*/ >最後に、Ｃです（Ｍｓ−ＤＯＳコンソール） >----- >#include > >int main(int argc, char *argv[]) >{ > > printf("argv[0] = %s\n", argv[0]); > >} >----- >まだ、講座でやってませんが、Ｍａｉｎ関数の引数の、 >argvの0番目に、位置が入っていると思います。 > argcってのに、スペースで区切られたＥｘｅに渡された引数の数が入るんですよ kutabare.exe new1.txt outdata.txt なんてやると、 argcに２か３*忘れた、、 printf("argv[0] = %s\n", argv[0]); printf("argv[1] = %s\n", argv[1]); printf("argv[2] = %s\n", argv[2]); あと、argvにパラメーターが入ります。 char **argvはかっこいい書き方で、 char *argv[]のほうがＫｅｎ３流 > if(!(chk_str = findFile(argv[1]))) 上の「**argv」は、これと関係ある？なぜに[1]か？ argv[0]にＥＸＥの起動位置が入っていて、 実際の引数は２番目[1]からなんですよたぶん。 予想、、、ファイル名をＥＸＥの引数で渡して、なんかやってんの？ >-------------------------------------------- >3) > > LPSTR findFile(const char *filename) LPSTRとは？ LPSTR?なんだっけ？そんなのあった？ if(!(chk_str = findFile(argv[1]))) でやってるから、 リターンでLPSTR型を返している？ chk_strの型宣言、見てください。 > FILE *file; > char *buff = 0 これは、ポインタ宣言と、初期化を同時実行？ > size_t rc; これは、なに？ まぁ、このへんは、あまり気にしないで、、 > prot_file = fopen(filename, "rb"); これは、バイナリでオープンだよね？ だと思う。 > buff = (char *)malloc(BLOCK_SIZE); これの意味は？ メモリを動的に取っているんですよ。 #define BLOCK_SIZE 32768 分、バッファを取っているんじゃない？ 32768？１６進に直すときりいいのかなぁ？ > rc = fread(buff, 1, BLOCK_SIZE, file); 引数のそれぞれの意味は、なに？ バッファへファイルから、読んでいると思えばいいと、、 1 * BLOCK_SIZE 分 file から読んで、ｂｕｆｆにセット > fseek(file, -32, SEEK_CUR); 同上。でも、「SEEK_CUR」の宣言は、ないんだけど（汗）。 SEEK_CURはｓｔｄｉｏ．ｈとか、ヘッダファイルに定義されていると思う。 で、現在位置（ファイルの）から、32バイト分戻ってるみたいですね。 -32Ｌだと思うけど、、まぁ、いっか。 > continue; これはなに？ これ、ループの頭に戻る命令です。 例）１〜１０までだけど、５だけ後処理したくない for(i=1 ; i <= 10; i++) { 前処理 if (i == 5) { continue; これで、５の時は、下の処理しないでｆｏｒに戻る たしか、ＶＢＡには無くて、いろいろなとこで話題になってる？ break; は exit doなどあるから、、、わかりやすい？ } 後処理 } > s = strchr(buff, '\"'); これの意味は？ ＶＢＡのｉｎｓｔｒと同じじゃないかなぁ？ ”の位置を探して、何かしているとか？ >-------------------------------------------- >4) > > va = strtol(sa, 0, 16); この関数は、なに？ strtol?知らないなぁ、、 予想ですが、文字列をＬｏｎｇ型にしているとか、、、 でも、それなら、ａｔｏｌがあるしなぁ、、 strtolって、関数、自作されてないか（標準関数以外？）みてください。 また、Ｖａの型宣言や、使われているところから、判断？ ｓａの変数は？0，16って？ 16進の文字列を直しているとか？ > CharToOem(str, s); もしかして、Winの関数？ これも、、、わからない、、、 またまた、予想？ 文字列の連結とか置き換えとか、、自作関数では？ >ちゅう訳で、たのんます。 まぁ、こんな感じで、、、簡単な回答です。。。 /* * ４.終りの挨拶 */ なんか、こっちの愚痴マガに書くほうが、 落ち着くなぁ、、、 もう一つのＶＢＡ系のほうは、なんかノリが悪くって、、、 ＊２つ（両方）読んでる人、、、重なってすみません。。。 ｃｏｎｔｉｎｕｅの話とか、 ネタもらったし、まじめにＶＢＡ系も書くかなぁ、、、 三流ＰＧ Ｋｅｎ３でした。
| No.68 | 2000/06/09 Ｋｅｎ３、ソースのコメントについて考える |
[ページTOPへ戻る] |
＜Ｋｅｎ３、ソースのコメントについて考える＞
目次 1.はじめの挨拶 2.と、その前に、ご教授下さいファンに一言？ 3.ソースのコメントについて 4.おわりの挨拶 枠外：読者からの補足メールいただきました（感謝です） ---------------------------------------------------------------------------- /* * １.はじめの挨拶 */ こんにちは。 三流ＰＧのＫｅｎ３です。 このメールマガジンは脚色されているので、 ご注意を、、、（実際と違う内容も含まれているので、） バーチャルカンパニーです。（←それにしては、リアル？） 今回は、 前回、ＣからＶＢへの移植？の質問を受けた時、 なんとなくソースファイルのコメントについて書きたくなったので、 書きます。 ＊みなさんの立場で読んで、笑ってください。 疑問・感想、、指摘、、あったら、メール下さい。 /* * ２．と、その前に、ご教授下さいファンに一言？ */ ソースのコメントのこと書く前に、前置き、続きます。 >私、変わり者なので（そんなん、いまさら言わなくてもわかってるって？） >ご教授下さい、よろしくお願いします >の質問より >解説できるもんならやってみろ >のほうがなんとなく好きなんだよなぁ、、、 と 前回書いたら、 教えてもらうのでご教授下さいがいいのでは？（妥当では？） と ご教授ファンの人からや、 初対面の作者に馴れ馴れしく質問書く人なんか居る？ と 一般常識さんから メールをいただきました。 確かに、馴れ馴れしく 、 軽いノリの質問 を 初対面の人に書けない気持ちは、わかりますが、 ご教授下さい、、、まで言わなくてもいいでしょ？が私の素直な気持ちです。 インターネットのマナー？に質問時は、ご教授下さいって載ってるのなぁ？ >解説できるもんならやってみろ は、Ken3流の言いすぎですが、 私は軽いノリの質問のほうが好きです。 まぁ、三流君Ｋｅｎ３の感じ方なので、あまり気にしないでください。 でも、ご教授下さい、、、は、好きじゃないなぁ（オイオイしつこいって、、） /* * ３.ソースのコメントについて */ 人が作成したプログラムや自分が過去作成したシステムを 移植する時や解析時、 仕様書がしっかりも大切ですが、 ソースファイルにキレイナコメントが載っていると 後々わかりやすいと思います。 （＊ニセのうそつきコメントにだまされた、、って逆作用もあったけど、、） 下記、Ｃで昔作成したソースの一部です。（土曜日ハマッタ工場の修正） /* テンポラリファイル作成 */ #include#include #include #include void sk_fsave(fname) char *fname; { FILE *ifp; /* 入力ファイルポインタ */ FILE *ofp; /* 出力ファイルポインタ */ int i, n, c; int ret, sw; long r_cnt; char in_buf[40]; /* 入力バッファ */ char out_buf[40]; /* 出力バッファ */ static char cr_lf[2] = { 0x0d, 0x0a }; memset(&in_buf[0], 0x00, 40); ifp = fopen(fname, "r+b"); /* ファイルオープン */ ofp = fopen("temp", "w+b"); r_cnt = 0L; ret = fread(&in_buf[0], 32, 1, ifp); /* データ１件読み込み */ while(ret == 1) { /* データがなくなるまで */ if(in_buf[19] == '0') { /* 転送チェック */ memset(&out_buf[0], ' ', 32); memcpy(&out_buf[0], &in_buf[0], 11); n = fwrite(&out_buf[0], 30, 1, ofp);/* テンポラリへ出力 */ n = fwrite(&cr_lf[0], 2, 1, ofp); /*ファイルポインタを戻す */ sw = fseek(ifp, r_cnt * 32L, SEEK_SET); in_buf[19] = '0'; n = fwrite(&in_buf[0], 30, 1, ifp); /* フラグの書き込み */ n = fwrite(&cr_lf[0], 2, 1, ifp); sw = fseek(ifp, r_cnt * 32L + 32L, SEEK_SET); } memset(&in_buf[0], 0x00, 32); ret = fread(&in_buf[0], 32, 1, ifp); /* データ１件読み込み */ r_cnt = r_cnt + 1L; } fclose(ifp); /* ファイルクローズ */ fclose(ofp); } みなさんからも、参考意見いただきたいのですが、 私が感じたことを書きます。 （＊例がＣ言語ですが、ＶＢＡなどでも同様だと思います） ア．関数の情報として必要な項目は？ 頭に記載すべき？情報は？ void sk_fsave(fname) char *fname; { としか、書いてない。 a.処理概要が必要？（フラグの立っていないデータを抽出するなど？） b.入力引数の説明(char *fname;まぁ、みればなんとなくわかるけど) c.リターン値（このサンプルでは、無いけど） イ．あたりまえのコメントはいらない？ 私がそうなのですが、 簡単にコメント付けられる部分にだけ、コメントつけてます。 FILE *ifp; /* 入力ファイルポインタ */ FILE *ofp; /* 出力ファイルポインタ */ ifp = fopen(fname, "r+b"); /* ファイルオープン */ ofp = fopen("temp", "w+b"); など、、、 このへんは、みればそのままなので、コメントはいらない？ まぁ、さびしいので、あってもいい？って考えもありますが、、、 極論だと、 i++; i = i + 1; // iを１増やす なんてコメントはイラナイよね。。。 ウ．素直な日本語を書こう？ ret = fread(&in_buf[0], 32, 1, ifp); /* データ１件読み込み */ while(ret == 1) { /* データがなくなるまで */ さすがにここを、retが1の間ループとは、書かなかったが、 /* データがなくなるまで */ と書いてます。。。これでもいいのだが、 while( fread(&in_buf[0], 32, 1, ifp) == 1 ) { データが読めている間と書いた方が、わかりやすかったかも？ 現在の自分から未来の自分自身へ ただ、動くだけじゃなく、キレイナソース書こうね。。。 またまた、言うは容易く行い難し、、、って言葉しってる？ あと、仕様書もセットでね。。（ハイハイ、、、） ソース、コメント関係の参考意見、待ってます。 ｅｔｃ.バグ見っけ、、（過去のナマイキなＫｅｎ３さんへ） sw = fseek(ifp, r_cnt * 32L, SEEK_SET); in_buf[19] = '0'; n = fwrite(&in_buf[0], 30, 1, ifp); /* フラグの書き込み */ 昔の自分へオイオイ、、、、 in_buf[19] = '0'; じゃ、フラグ立ってなくて、次来た時もｔｅｍｐファイルに書いちゃうじゃん in_buf[19] = '1'; じゃないの、、、まぁ、いっか。。。 クレームないし、中間チェック機能を使っていないのでしょう。 ＊このファイル作成タイミングは、作業終了時に（１直２直交代時？） チェックリスト出力ようのファイルを生成、 （出力済みのデータを出さないためにフラグを立てているんだけど。。。） /* * ４.終りの挨拶 */ ヤッパ、こっちの愚痴マガに書くほうが、落ち着く。 なんて言ってないで、、、、 三流ＰＧ Ｋｅｎ３でした。
枠外通信？ 愚痴マガ読者のＴさんより、アドバイス、補足メールいただきました '----------------------------------------- >こんにちは、愚痴マガ読者の＊＊です。 > >いつも楽しく読ませてもらっています。 >説明入れておきます。 > >> > #includeこのヘッダーが有れば、ＬＳＩコンパイラでも使 >える？ >> >> ＬＳＩコンパイラでも使える？ >> これは、厳しいかも？ > >コード生成の方法が違うので、16/32ビットに関わらず、LSI-Cでは使えません。 > >> > #include 同上 >> >> なに？ > >これは解りません。少なくとも私の持っているコンパイラにはそのような >ヘッダファイルはありません。 > >> > #pragma hdrstop これは、何でしょ？ >> >> えっ、、 > >このプラグマは、プリコンパイル済みヘッダファイルを作成／使用するときに、 >ここまでプリコンパイルする(されている)ことをコンパイラに知らせるための >ものです。 > >プリコンパイル済みヘッダファイルとは、ヘッダファイルを途中まで >コンパイルした中間コードのダンプファイルです。 >これを作成して使用することにより、何度も同じヘッダファイルを >コンパイルするのに対して、コンパイル時間を大幅に短縮することが >可能になります。 > >VBにはそのような機能は無いので、移植の際に気にする必要は無いでしょう。 > >> >-------------------------------------------- >> >3) >> > >> > LPSTR findFile(const char *filename) LPSTRとは？ >> >> LPSTR?なんだっけ？そんなのあった？ >> if(!(chk_str = findFile(argv[1]))) >> でやってるから、 >> リターンでLPSTR型を返している？ >> chk_strの型宣言、見てください。 > >LPSTRはwindows.hで定義されている型です。char *(16ビットなら >char __far *)のtypedefです。 > >> >-------------------------------------------- >> >4) >> > >> > va = strtol(sa, 0, 16); この関数は、なに？ >> >> strtol?知らないなぁ、、 >> 予想ですが、文字列をＬｏｎｇ型にしているとか、、、 >> でも、それなら、ａｔｏｌがあるしなぁ、、 >> strtolって、関数、自作されてないか（標準関数以外？）みてください。 >> また、Ｖａの型宣言や、使われているところから、判断？ >> ｓａの変数は？0，16って？ >> 16進の文字列を直しているとか？ > >atolは10進数のみ変換可能です。strtolは16進数なども変換可能です。 > >> > CharToOem(str, s); もしかして、Winの関数？ >> >> これも、、、わからない、、、 >> またまた、予想？ >> 文字列の連結とか置き換えとか、、自作関数では？ > >WindowsAPI関数です。OEM定義の文字列に変換します。 > >それでは、メルマガ頑張って下さい。 strtol,CharToOem、、、あれ、標準関数だったのね、、 てっきり、自作関数かなぁ？なんて書いちゃって、 また、無知がバレちゃったけど、まぁ、Ｋｅｎ３らしくっていいかな？
| No.69 | 2003/06/10 Ｋｅｎ３、指摘メールに考えさせられる |
[ページTOPへ戻る] |