現在のコードのどこかに、不備があり、改善して、指定された出力を出したい。

Question

すみません、dodoxさんの翻訳内容であっています。訂正します。
現在、ユーザーからランダムな整数入りのリストを取って、そレを一旦２進数に変換し、新しい１０進数へと変換する、プログラムを作っています。例えば[11, 24] という　リストの箱で受け取った場合は最終的に
424896という１０進数に変換されて、出力されます。
受け取ったリストは最初に[11, 24] から　[1011, 11000]と変換されます。

そちら1つずつの整数要素を連続させます。例えば　1は11に0は00になります。上の例だとこうなります。
[1011, 11000] → [11001111, 1111000000]
それから、要素が区切られている部分に０を挿入して要素を増やします。０の挿入は連続の作業が終了した後に行います。今は、2つの要素だけで行っていますが、要素数が1つの場合は０は挿入しません、
また　要素数3つや4つの場合などはその要素の前ごとに挿入します。最後の要素の後には０は挿入しません。

例[1100] （要素が1つの場合）
[1011,0 ,11110000, 0 ,100111100] （要素が3つある時）

上の[11,24]だとこうなります、[11001111, 0 , 1111000000]。そしてそれらを結合すると、[1100111101111000000]となります。
それを１０進数に変換すると、424896となります。

ですが、私の書いたコードですと、[11, 24]の１０進数の出力が　51230208　となり、
本来の　424896　と出力されません。解決方法が見当たらず、質問させて頂きました。
すみませんが、よろしくお願いします。

def encode(numbers):
    numbers= ''.join(bin(e)[2: ] for e in the_input)#受けとったリストの中身を２真数へ
    combination= '' #0が結合した後の空のstr
    index = 0
    doubled_numbers = []　
    for number in numbers:
        double_number = ''
        number_string_rep = str(number) #int to str　
        for letter in number_string_rep:
            
            double_number = double_number + letter + letter #iterateしたletterを足して、２連続させる
            doubled_numbers.append(double_number)　#空のliistへappend
            combination= "0".join(doubled_numbers) #0の結合。
    return int(combination, 2)　#0が結合された後の文字列を１０進数へ変換
input: [11, 24]
out: 51230208
#out should be 424896

Comments

[toast-uz]

例がわかりません。
[10, 20, 30]はどこへ消えたのでしょうか？
2連続させる、という言葉の定義はなんでしょうか？ どうして、[1011,11000] → [11001111, 1111000000] となるのか、法則が全くわかりません。

[y_waiwai]

違う出力というのはどういう出力なんでしょうか。

[dodox86]

質問中の説明は少し分かりづらいようです。

> 例えば、[1011,11000] → [11001111, 1111000000]
「例えば」で唐突に取りあげた元の整数リストが分かりません。これは、[11, 24]を示したものですね？　10進数11の2進数表現は1011なので、このそれぞれの桁を同じ値で2つにする。それで 11001111 です。

更に「index」が2つ以上あったら」という表現も分かりづらく、読んだ者は悩みます。
これは「整数リストの要素が2つ以上あったら」と言うことで合っていますね？
[11,24]のリストは2つあるので、"0"で結合します。
"11001111" + "0" + "11001111"で、"11001111011001111" ということなのだと思います。

また、質問文中にあげたPythonスクリプトは全角文字が混ざっていたり、未定義の変数を使っていたりでエラーになります。文中の誤字を含め、気をつかっていただけると閲覧者、回答者は助かります

[hentaiman]

最初のコメントと同じで色々と意味不明だったんですが、dodoxさんの翻訳で合っているのなら質問文を修正してくださいね。

Answer 1

既存の書き方から離れて、よさそうなコードをゼロから書いてみました。

• 「連続させる」という独自の「演算pile()」を持つ2進数クラスMyBinaryを新規に作成
• numbersの各要素を、順に、MyBinary型へ変換、pile()演算し、できた文字列リストを'0'で連結して数値に戻す
• リストの各要素に同じ操作をするところはmap関数を活用

こういった独自演算を定義して繰り返し使うようなケースでは、クラスを作った方が、見通しが良くなります。

class MyBinary:
    def __init__(self, num):
        self.digit = f'{num:b}'
    def __repr__(self):
        return self.digit
    def pile(self):
        return ''.join([s*2 for s in self.digit])

numbers = [11, 24]
result = int('0'.join(map(lambda x: MyBinary(x).pile(), numbers)), 2)
print(result)

Comments

[toast-uz]

ちょっと遅かった泣

Answer 2

現在のコードのどこかに、不備があり、改善して、指定された出力を出したい。

残念ですが、ご提示のもとのコードは色々と間違っています。要望どおりに動きにならない場合はprintを使うなどしてプログラムの途中で自分が意図したとおりに結果になっているかを確認しましょう。質問文中のコードを尊重し、添削（？）して修正したものが以下になります。要所でprintを使い、動作状況を確認しやすくしているつもりです。コメント部分も併せて読み、質問者さんのもとのコードとの違いを確認してみてください。

# NG1: numbersを引数の名前にしている時点で間違いのはず。
#def encode(numbers):
def encode(the_input):
    # NG2: ここでjoinしてはダメでしょう。元の数リストの区切りが分からなくなります。
    #numbers= ''.join(bin(e)[2: ] for e in the_input)
    numbers = [bin(e)[2: ] for e in the_input]
    print('numbers={}'.format(numbers))
    combination = ''

    # NG3: 使っていないのでいらない
    # index = 0
    doubled_numbers = []
    for number in numbers:
        double_number = ''
        number_string_rep = str(number)
        for letter in number_string_rep:
            double_number = double_number + letter + letter
            # NG4: 2連続にするたびにappend()しているのでダメ

            # NG5: ここは要素の区切りではないので、"0"でjoin()してはダメ
            # doubled_numbers.append(double_number)
            #combination= "0".join(doubled_numbers)

        # ここで2連続にした2進数文字列をリストに追加
        doubled_numbers.append(double_number)
        print('doubled_numbers={}'.format(doubled_numbers))

    # 最後に"0"でjoin()
    combination= "0".join(doubled_numbers)
    print('combination={}'.format(combination))

    # 10進数値に変換
    result = int(combination, 2)
    return result

#input: [11, 24]
#out: should be 424896

l = [11, 24]
r = encode(l)
print('result={}'.format(r))

これを実行した結果です。

$ python3 t1.py
numbers=['1011', '11000']
doubled_numbers=['11001111']
doubled_numbers=['11001111', '1111000000']
combination=1100111101111000000
result=424896

Comments

[M_Nishi]

dodox86様、お忙しい中コーディングの修正の方をありがとうございました。
コメントや出力を照らし合わせながら、実装の方に移りたいと思います。

[dodox86]

Pythonの機能をフルに使えば恐らくもっと短く、効率的に書けると思いますが、最初はあまり難しいことをやらない方が良いです。まず頭で考えて、手で紙に書けるほどに具体的にやることをイメージできてから、実直にコードに書いていくことをお勧めします。Pythonを使うと魔法のようにやりたいことができるわけではないです。

Answer 3

for文のインデントがいけないのかと思います。
やりたい事は以下のようなコードではないでしょうか。

    for letter in number_string_rep:
        double_number = double_number + letter + letter

    doubled_numbers.append(double_number)
    combination= "0".join(doubled_numbers)＃０を代入し、結合