linked-listリンクリストノードの削除このコードは、リンクリスト内の奇数値のノードを削除するためのものです。これはうまくいきますが、読みやすくするために微調整したいと思います。改善を提案してください。意味のない変数名を使用すると事が複雑になります。それに類似したものと呼ぶことは途方もない助けになるでしょう。Witvoet源トンピーターをありがとう!これはまさに私が知りたかったものです。私はOOPの抽象化の柱を見逃していたため、addとprintの両方の方法を混同していました。源'適切な'リンクリストインターフェースの論理的な利点を広げてくれてありがとう。簡単な解説MSDNでは、命名規則をいくつか提供しています。)を持つことが理想的です。古い(コメント付き)コードを削除するのが最善の方法です。それは維持されずにそこにあるコードと互換性がなくなるという習慣があります。それはまた混乱を追加します。'export'コードの場合、コンソールに明示的に書き込むメソッドがあると便利なことはめったにありません。)、または単に文字列を返します。これらは消費者にもっと多くの選択肢を与えます。deleteOdd(MyLinkedListを参照)は、空のリスト(渡されたり返されたりするリスト)に対処できない可能性があります。はすばらしい変数名ではありません。"LL"ビットは冗長です。印刷が左から右に行われるのか、それとも何かに関連するのであれば、しばらくの間試してみます。すばらしい変数名ではありません。パラメータがオプションであること、あるいは実際にそこにあることを認めないと思います。これは単にいくつかのオプションを説明するためのものです。両方を使用するほうが良いでしょう。詳細な回答VisualMelonをどうもありがとう!多分私ははっきりしなかったが、私が実際に尋ねるつもりだったのは、コードをフォーマットするだけではなく、ロジックをよりエレガントに見えるように調整することでした。基礎となる実装がコーディングした方法で機能するかどうかを理解したいのです。フレームワークのLinkedList型を使用できれば、カスタムコードを実装する意味がまったくありません。私はコンソールからの独立の良い例である改良された印刷機能が好きです。TLDR、インタビューでフレームワークのデータ型を使用することは最初からこの煩わしさを避けてもよいですか。ありがとうございます。源@ShreeHarsha許可されているかどうかはわかりませんが、許可されていない場合は驚き、それが明確に指定されていない場合はもっと驚きます。私は既存のデータ構造を扱うコードを書くことは実証するのに有用なスキルになるだろうと思いました(そして標準ライブラリの認識を示すことは決して悪いことではありえません)。短いほど、「より充実した」リンクリストAPIを使用してロジックをより明確にすることができることを示しています。変数の命名あなたのコードを見たことがない読者を考えてみてください(おそらく、このコードを6か月以内に見直す必要がある場合には、これがあなたの立場になるでしょう)。いう名前を付けました。それを言うのは非常に難しいですし、そうすることに対して私は助言するでしょう。次に、「LinkedList」という名前には特定の意味があります。それは実際にはリンクされたリストの派生クラスですか?私はクラスの命名はかなりあいまいさがあると言っているだけで、それは明快なコードを書くことの敵です。メソッドを呼び出すことができると期待することができますか?より良い名前はこれを反映するのに適切でしょう。listには特定の意味があります。私はおそらく「リスト」という言葉を避けたいと思います。list.上記のソースコードの内容に基づいて、次のリファクタリングをお勧めします。洞察力0x51baをどうもありがとうございました。これは私の質問の論理的な部分に答えます。