質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
連結リスト

連結リストとは、データ構造のひとつであるリストの中で、要素が前後の要素の情報を持つことで、要素が連結(リンク)しているリストの事を呼びます。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Raspberry Pi

Raspberry Piは、ラズベリーパイ財団が開発した、名刺サイズのLinuxコンピュータです。 学校で基本的なコンピュータ科学の教育を促進することを意図しています。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

553閲覧

上書きではなくdelを使うべきですか?

Third_Kei

総合スコア65

連結リスト

連結リストとは、データ構造のひとつであるリストの中で、要素が前後の要素の情報を持つことで、要素が連結(リンク)しているリストの事を呼びます。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Raspberry Pi

Raspberry Piは、ラズベリーパイ財団が開発した、名刺サイズのLinuxコンピュータです。 学校で基本的なコンピュータ科学の教育を促進することを意図しています。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

0クリップ

投稿2021/06/22 04:51

編集2021/06/22 04:55

前提・実現したいこと

Raspberry piで使うpythonコードを作成しています
リストに入れたクラスのインスタンスを、try-catch文を使って通信可能か調べ、通信できない場合はNoneで上書き処理しました
その後、filter関数を使って、Noneで上書き処理したリストを再代入?する形でリストを整えています
コードを書いた後、del文の存在を知りました

Noneで処理をしたときに、del文と比べてどのような不具合が起きるのでしょうか?
似たような方法で削除している箇所がいくつかあり、コードを置き換えるリスクとコードを放置したときのリスクを天秤にかけたいです

よろしくお願いします

該当のソースコード

addresses = [i for i in range(0x60,0x60+8)] sensor_addresses = [j for j in range(16,24)] modules = [controler.module(m,n) for m,n in zip(addresses,sensor_addresses)] for x,i in modules,range(8): try: x.changer(0) print("Connected:" + str(x.address)) except Exception as e: print("ignored:" + str(x.address)) modules[i] = None continue try: bus.write_byte_data(x.address_sensor, 0b00000110, 0b00100000) print("Connected:" + str(x.address_sensor)) except Exception as e: print("ignored:" + str(x.address_sensor)) x.address_sensor = None modules = [filter(None,modules)]

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

同じロジックで処理するのであれば、delでないほうがいいでしょう。

forループの中で、modules[i] = Noneのタイミングでそれをdelしてしまうと、リストの構造そのものが変ってしまい、予期しない状態になるでしょう。

リストに入っているは、インスタンスそのものでなく、リファレンスなので、delしてもNoneに置き換えてもインスタンスそのものへの影響は変わりません。

別件ですが、for x,i in modules,range(8):はenumerateを使ったほうがきれいだと思います。

投稿2021/06/22 05:21

TakaiY

総合スコア12765

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Third_Kei

2021/06/22 05:31

ありがとうございます、リストの中に物が入っているわけではないのですね enumerateについてもお教えいただきありがとうございます 物自体を消さないといけない(例えばメモリ?とかの関係で)、といったことはありますか? もしよろしければお教えいただけると嬉しいです
TakaiY

2021/06/22 09:10

インスタンスの削除はpythonのガベージコレクタがコントロールしていて、ユーザ側からメモリを直接解放をする手段はありません。 そういうことを気にせずに使えるのがいいところですね。 なので、少ないメモリをやりくりしながら動かすというようなアプリケーションは得意ではなってことですね。
Third_Kei

2021/06/23 02:21

勉強になりました、ありがとうございます
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問