関数がうまく動いてくれません

Question

voidの関数を使って10進数を2進数にするプログラムを作りたいのですが関数に入った後の計算結果が表示がされず、そのまま次のコマンドが入力できるようになってしまいます。
自分としては関数の中で無限ループを回して2で割ったあまりを出力すればいいと考えました。また負の値が入力されたら抜けるようにしました。

./a.out
10進数から2進数にした数値を入力してください
66

みたいな感じになってしまいます。

#include<stdio.h>
#include<stdlib.h>
void d2btrans(int);
int main(){
    int n,f;
    printf("10進数から2進数にした数値を入力してください\n");
    scanf("%d",&n);

printf("\n");

return 0;
}

void d2btrans(int i){
  
if(i<0){
    exit(2);
}
    while(i>0){
        i%=2;
        printf("%d",i);
    }
}

Answer 1

.do~while文を使って負の数値が入力されたときの対処をしました。

また、「ある程度大きな値でも対応できること」とそのままだと「表示が逆になること」を考慮して、
配列にデータを格納して表示しました。

#include <stdio.h>
#include <stdlib.h>

#define BUFSIZE 256

void d2btrans(int);

int main(void){
    
  char buf[BUFSIZE];
  int n;

  printf("10進数から2進数にした数値を入力してください\n");
  do{
      fgets(buf,sizeof(buf),stdin);
      n = strtol(buf,NULL,10);
      if(n < 0){
          printf("再入力してください。");
      }
  }while(n < 0);
    
  d2btrans(n);
  return 0;
    
}

void d2btrans(int n)
{
  int binary[32];
  int i = 0;

  while(n > 0){
    binary[i] = n % 2;
    n /= 2;
    i++;
  }
    
  while(i > 0){
  	printf("%d",binary[--i]);
  }
}

Comments

[Teemro_431265]

ありがとうございます。参考にさせていただきます。

Answer 2

出力が無いのはd2btransを呼び出してないからですね。
d2btrans自体ぼろぼろなので、呼び出しても期待通りに動作はしませんが。

void d2btrans(int i){
    if(i<0){
        exit(2);
    }
    while(i>0){
        printf("%d",i%2);
        i/=2;
    }
}

Comments

[Teemro_431265]

関数の中で出力ができると思ったのですができませか？
あと「ぼろぼろ」とはどういうことですか？

[y_waiwai]

修正後のコードでも、d2btrans 関数は呼び出されてませんし、
d2btrans 関数自体も、まともには動作しなさそうです

[退会済みユーザー]

@otnさんへ
提示されたコードだと2進数の表示が逆になってしまいます。
例えば, 8を入力したときに1000になるはずですが0001になってしまいます。

[otn]

> 関数の中で出力ができると思ったのですができませか？
出来ますよ。

> あと「ぼろぼろ」とはどういうことですか？
私のコードと見比べてください。

> 提示されたコードだと2進数の表示が逆になってしまいます。
元の関数の仕様？に合わせています。

Answer 3

とりあえず気が付いたことです。

• "10進数から2進数にした数値を入力してください\n"ってユーザーが１０進数から２進数に変換した数値を入力するってことですか？
• f=void d2btrans(int n);は構文エラーになります。
• 結果が格納されているであろうｆを表示していません。
• d2btrans関数のwhileループの中でiに0を代入しちゃってます。

あと10進数を２進数に変換するのにループのなかで２の余剰を表示するというのは
一見よさそうですが、そのままだと下の桁から表示していきますので問題があります。

Comments

[Teemro_431265]

一つ目の質問は「したい」にしたつもりでただの脱字です。
二つ目はvoidが不要ということですよね？
三つ目は最初fはmainの中で出力しようと思ったのですが関数の中で出力できると思いやめたので表示されないようにしています。
四つ目は代入する値を初期化していないと思いそうしただけですが投稿した後に直しました。
最後のご指摘は2進数の表示が逆さまになってしまうという解釈でいいですか？