コマンドからから入力した文字列を数値に直したいです

Question

コマンドライン引数に最初に 1 か 2 を，次に 1 なら10進数， 2 なら2進数の数字の文字列を複数指定し，以下のように10 進数の数値にして印字したいです。
＜ 実行例＞

./sample 2 1100 111⏎
1100B = 12
111B = 7
./sample 1 1234 56789⏎
1234 = 1234
56789 = 56789

自分のコードで以下のように実行すると、以下のようになってしまいます。

./sample 1 1234 5678
1234 = 6562
5678 = 11006
./sample 2 1100 111
1100B = 348
111B = 151

色々試した結果str_to_dec,bin_to_decには問題がないと思われます。
訂正箇所を教えでいただけるとありがたいです。

コード
#include <stdio.h>
 int str_to_dec(const char s[]){
   x1 = s[0] - '0'; 
   for( i = 1; s[i] != '\0'; i++ ){
     x2 = s[i] - '0';
     x1 = 10 * x1 + x2;
   }
   return x1;
 }
 int bin_to_dec(const char s[]){
   x1 = s[0] - '0'; 
   for( i = 1; s[i] != '\0'; i++ ){
     x2 = s[i] - '0';
     x1 = 2* x1 + x2;
   }
    return x1;
 }
 int main(int argc, char *argv[]) {
   int i;
  /*引数が2つ以上指定されていなかった場合 */
   if(argc <= 2){
     fprintf( stderr, "usage: %s [1 or 2] [binary numbers]¥n", argv[0] );        
     return 1;
   }
   switch( argv[1][0] ){ 
     case '1':
       for(i = 2;i<argc;i++)
         printf("%s = %d\n",argv[i],str_to_dec(argv[i]));
         break;
     case '2';
       for(i = 2;i<argc;i++)
         printf("%sB= %d\n",argv[i],bin_to_dec(argv[i]));
         break;     
     default :
         break;
    }
    return 0;
}
  
 

Comments

[int32_t]

質問文のコードではコンパイルできないはずです。古いソースから作った実行ファイルを試してたりしませんか。

[earth1362]

古いソースから作ってはいないと思います。
コンパイル出来ない理由を教えていただけますでしょうか。

[int32_t]

y_waiwai さんの回答にあるとおりいくつか変数が宣言されていないのと、case '2' の後のセミコロンです。
コンパイルできるコードがあるなら、それをコピペしてくださいね。

[earth1362]

わかりました。
ありがとうございます。

[int32_t]

あれ、この質問は解決したのですか? コンパイルエラーを除くと何が問題でした?

Answer 1

int bin_to_dec(const char s[]){

2進数の後尾のBが来たときの処理が入ってませんね

---

ああ、B入れてるのは表示側なのね。
ってことで、まじめに動かしてみました。

Main.c:4:4: error: use of undeclared identifier 'x1'
   x1 = s[0] - '0'; 
   ^

コンパイルエラーでてますやん。
未定義の変数x1使ってるとおっしゃってますぜ

Comments

[earth1362]

すみません。
元のコードではきちんと定義されていました。
ここに移す際に書き忘れてしまいました。

[y_waiwai]

ちゃんとコピペでコードを提示しよう。
コードが間違ってるってことなら答えようないですぜ

[earth1362]

すみません。
linuxからwindowsにコピペができずそのまま移す形になってしまいました。