Q&A
printf("%d\n", strcmp("555", "55")); //適当に選んだ数字です
だと1
printf("%d\n", strcmp("10", "2"));
だと-1と判定されますね
そこらへんちょっとよくわからないです
2018/2/11
区切りの為に書きます
この投稿にもう私が投稿するころはありません
皆さん質問ありがとうございました
回答者の方々どうしで議論するために使ってください
よろしければこちらのアイデアについて
ご意見がうかがえると嬉しいです
https://togetter.com/li/1198124
学業を収めることが卒業するために必要なんだから
趣味よりもそちらを最優先したほうがいいのではないかと思いました。
なので、こちらでの質問の返信は一時停止します。
せっかく答えてくださったのに、すぐに返信できずすみません
前提・実現したいこと
プログラミング作法
問題1-2(21p、新版)の解法
発生している問題・エラーメッセージ
問題の解答がこれで正しいのか自信がないです。
該当のソースコード
次の関数を改良せよという問題です
c
10#include <string.h> 21 int smaller(char *s, char *t){ 32 if (strcmp(s, t) < 1) 43 return 1; 54 else 65 return 0;}
試したこと・解答例
この問題は関数名と実装を適切に作成しようという内容の章で出題された問題です
関数名がsmallerでstrcmpを使用しているから数字の文字列二つ(例("12345","12345678"))を引数にとって、1番目の引数が2番目の引数より小さいかを比較する関数であることがわかります。
http://hitorilife.com/strcmp.php
によれば
int strcmp( const char *str1 , const char *str2 ); の場合
■戻り値：
str1とstr2が等しいならば0、
str1>str2ならば正の値、
str1<str2ならば負の値を返す。
なので前述のコードだとstr1とstr2が等しい場合にも1が返ってくる
"smaller than"は「より小さい」と訳す、これは「未満」とも言い換えられるが、比較対象と自分自身が等しい場合も含める「以下」とは訳せない。
よって二つの引数が等しい場合に0を返すようにに2行目だけ次のように書き換える。
c
1 if (strcmp(s, t) < 0)
補足情報
cのif文の条件分岐では数値の0以外が真になるので1が真、0が偽であることを自明として扱ったが
c
1#define TRUE 1 2#define FALSE 0
みたいな文を付け加えて真偽を明示すべきかということと
関数名smallerの名前の変え方、「引数1番目は引数2番目より短いか」と「引数2番目は引数1番目より短いか」の二重の解釈ができるが、これは引数1番目を主語とするのが普通(?)で、文脈も読み取った解釈だと普通(?)は前者を選択するからsmallerのままでいいのではないかと思えて、自分はあまりこの関数の名前を修正する必要を感じない
それともそのことを明示した関数名にしたほうがいいか。
例えば
「最初の変数が次の変数より小さいか」をgoogle翻訳した
「Whether the first variable is smaller than the next variable」
を圧縮したような変数名すべきか。
その場合どのような変数名にしたらわかりやすく短くできるか
それ以外に自分の気が付いてない改善点や解答にミスがないか
初投稿で勢いで投稿しているところがあるので
おかしいところがあったら指摘してください
回答6件
0
関数名と実装を適切に作成しよう
やり取りの中で、次の仕様が明らかでないことが分かったかと思います。
- どちらの値が小さいときに1を返す？
- 『小さい』とはどんな尺度で？
smallerの命名を変えることによって、これらの曖昧さを除去する必要があります。
######どちらの値が小さいときに1を返す？
真面目に命名しようとすると、first_arg_is_smallerのように煩雑な命名になります。
既にstrcmpという前例があるのですから、これに倣えば良いでしょう。
ただし、何度も悩んでしまうのも良くないので、適切にコメントを付けると良いかと思います。
######『小さい』とはどんな尺度で？
これに関しましては、名前に意味を含めた方が良いかと思います。
なぜなら、今後の拡張によっては、その複数を実装する場合も考えられるからです。
辞書順の場合、???_in_lexical_order (これはただのstrcmpと同じ)
数値順の場合、???_in_numerical_orderなどが考えられるでしょう。
訂正：strcmp = 辞書順 は誤り。詳しくはa_saitohさんの回答をご覧ください。
すごく頭でっかちですけど、ここまで書けばかなり誤解は減るでしょう。
C
1/** 2@brief 数字列を数値に変換し、数値の大小を比較する。 3 4@param (s1) 数値に変換できる文字列。変換できない可能性は考慮しない。 5@param (s2) 数値に変換できる文字列。変換できない可能性は考慮しない。 6 7@return 次の凡例に従って戻り値を返す。 8 - strcmp_in_numerical_order("123", "12" ) ⇒ 正 9 - strcmp_in_numerical_order("123", "123") ⇒ 0 10 - strcmp_in_numerical_order("12", "123") ⇒ 負 11*/ 12int strcmp_in_numerical_order(const char* s1, const char* s2) { 13 return atoi(s1) - atoi(s2); 14}
不正な入力に対して脆弱(というかスルー)ですが、それはまた別の話です。
あと面倒なので桁あふれの可能性も無視してます。
cのif文の条件分岐では ... 真偽を明示すべきか
C99以降では、stdbool.hに定義されているtrue/falseを使えば良いかと思います。
コメントを受けて
おそらく関数の定義を自分が誤解していたので関数は辞書順に見て前か後ろ(または同じ)かという実装の問題ではないかと思います
私もそんな気がします。
C
bool strBefore(char *s, char *t) {
if (strcmp(s, t) <0)
return true;
else
return false;
}
bool strBefore2(char *s, char *t) {
return strcmp("1", "2")<0?true:false;
}
次のように書いた方が簡潔かと思います。
C
1bool strBefore3(char *s, char *t) { 2 return strcmp(s, t) < 0; 3}
変数名には 「より前」の翻訳で出てきたbeforeを使い
cmpよりは彩りがある言葉ですね。
しかしs.before(t)という形式で記述出来ないので、コメントは添えた方が丁寧に思います。
bool型に変えられる部分は変えてみました。
falseが0、trueが非0というのは信頼してコードを書いていいかと。
厳密には規格書に当たらないといけないですし、それに反する実装も存在し得ますが、
私にはそれを判断するだけの知見が現状無いです。すみません。
マジックナンバーはあまりよくないらしいですが、この関数では、多くの場合実装を変更する人は、strcmp()の仕様を既に理解しているか、知らなくても理解してから変更すると思われるので、「if (strcmp(s, t) <= 0)」の0をローカルだからと言って短い名前の変数にしても、strcmp()の返り値がわかっている人が見た場合、逆にコードの理解の妨げになるかと思ってつけませんでした。
その判断で正しいように思います。
返り値を三つ以上に分岐させたい場合は、列挙体の利用も視野に入れると良いでしょう。
余談ですがC言語の名前のつけ方でよりよく使われるのは次の二つのどちらでしょうか、またそれ以上に使われる名前の付け方はあるでしょうか。最もオーソドックスなやり方を覚えておきたいんです
xxxYyy(文字の区切りに大文字)、xxx_yyy(文字の区切りに_)
xxx_yyyが一般的ですね。『C言語 命名規則』でググると色々な意見が見られます。
| 名称 | Cでの用途 | 別名 |
|---|---|---|
| PascalCase | UpperCamelCase | |
| camelCase | lowerCamelCase | |
| snake_case | 変数、関数 | snail_case |
| UPPER_SNAKE_CASE | マクロ | UPPER_SNAIL_CASE |
他にもプレフィックス(アプリケーションハンガリアン)を知っておくと彩りが増えます。
今見てもピンとこないかもしれませんが： 間違ったコードは間違って見えるようにする
あとは、動詞形と名詞形の使い分けですね。
関数に動詞、変数に名詞、配列には複数形名詞を付けると見通しが良くなります。
また、論理値を返す関数名をis, can, hasなどで始める人が多いです。
命名規則は状況によって変わり得ます。
チームで開発をしているのならチームの規則に合わせるべきです。
個人で開発をする際には、充分なメリットが説明できれば特殊な規則に従っても良いでしょう。
投稿2018/02/01 15:40
編集2018/02/07 06:07
総合スコア35660
0
C言語、ほとんどさわってないですが・・・
strcmpを使った場合、"10"と"2"だと、どっちが小さいって判定されますか？
投稿2018/02/01 10:03
総合スコア4820
・・・上の例だと
'5'-'5' →'5'-'5'→'5'-'\0'・・・ ＞
'1'-'2'・・・ ＜
※この辺は実装依存と思いますが、文字列長が違う場合strcmp()は、!=0あるいは==0での判定に適していると思います。
0
ベストアンサー
まずstrcmpは辞書式比較ではなくバイトごと比較です。
辞書式なら(大文字が先に出てくるとするなら)
"A"< "a" < "Aa" < "B" < "b"
という大小関係にならなければいけません。
strcmpでは( ASCIIコードを使ってるマシンでは)
"A" < "Aa" <"B" < "a" < "b"
という大小関係になります。
まじめに辞書式比較をするならば、英語なら大文字小文字、日本語なら撥音拗音音引きあたりの処理がややこしいです。
あと作法の中に書いてあったかどうか忘れましたが､一般に
#define TRUE 1 #define FALSE 0 #define ZERO 0 #define BEGIN ｛ #define END }
などは悪手とされています。たとえばZEROといっても、0,0L, 0.0, '0', "0"などなどいろいろ有るので、下手にマクロにされると逆にわかりにくくなります。あと、Cでの真偽は0と0以外 です。比較式の値が0/1になりますが。なので1だけTRUEであるかのように思える(＝＝で真であることが判定できる)ようなまマクロはやめた方が良い、ってのもあったようなきがします。このあたり、Cのコーディングスタイるン本には詳しく載っていると思いますよ。
if(式＝＝TRUE)と書く必要はなくてif(式)でいいとおもいますが。
余談ですが,smallerはis_smaller的な関数にも思えますし、min的(二つの引数のうち小さい方を返す)関数の名前のようにも思えますね。
#include <string.h> char *smaller(char *s, char *t) { if (strcmp(s, t) < 0) return s; else return t; }
投稿2018/02/06 02:11
編集2018/02/07 05:03
総合スコア702
0
strcmpは二つの文字列を比較し、
どちらの文字列の方が辞書順でみて若いかによって返り値が決定します。数字としての大小ではありませんし、文字の長さでもありません。
投稿2018/02/01 13:09
総合スコア4830
0
コメントだけして逃げるのもアレだったので自分なりの考えを。
辞書順の結果を返すstrcmpが不適切と考え、
関数名通り整数数字を比較する関数にするならば、簡単な実装としてatoiを使って書き直せばいいのかなと思いました。(整数に変換できない場合などのエラー処理は考慮していません。)
私もC言語はあまりやらないので、自信はないですけど…(^ ^;
C
1#include <stdlib.h> 2 3int smaller(char *s, char *t){ 4 return atoi(s) < atoi(t); 5}
投稿2018/02/01 14:37
編集2018/02/01 14:52
総合スコア2043
0
投稿してから関数名smallerについて文字列の数値を読み込むという趣旨のことを追加したほうがいいのではないかと思ったが文字列であることは引数に明示されているのでそこは省略して
「numSmaller」に変更したほうがいいのかもしれないと思いました
投稿2018/02/01 10:03
退会済みユーザー
総合スコア0
『momon-ga様の回答は、「数値の比較ではなく文字列の比較ということを強調するべきでは？」という意味だと思います。
そう考えるとむしろ
"strSmaller"もしくは"shorter"の方がいいような…確かに引数で強調されてはいますが、numSmallerにするとむしろ混乱を招く気がします。』
ごめんなさい私も勘違いしている部分がありました。このコメントは無視していただけると幸いです。お騒がせいたしました。m(_ _)m
調べた感じだとstrcmpは辞書引きした時にどちらが先に来るかを返すみたいですね。
この問題の関数はそもそも文字列を比較する関数で自分はそれを誤解していた、ということでしょうか？
そうなると 自分が変更した"if (strcmp(s, t) < 0)"という修正が適切だったかも不安になってくるんですが
変更点とそれ以外の実装には問題はないんでしょうか
strcmpはstring compare(文字列比較)の略です。
smallerは文字列の文字数str1がstr2より短ければ真、そうでなければ偽を返す関数で、
だからnamnium1125さんが述べたように関数名としては適切でない。
それとは別にsmaller()を修正するにはstrcmp()の挙動を理解し、
必要によっては別の関数に変える必要がある、ということでしょうか
明日よく調べてみます
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2018/02/02 07:08
2018/02/02 07:21
退会済みユーザー
2018/02/02 11:35 編集
2018/02/02 09:15
退会済みユーザー
2018/02/03 06:29 編集
2018/02/05 01:43
2018/02/07 05:28