617. Merge Two Binary Trees

[TakayaShirai]

解いた問題：https://leetcode.com/problems/merge-two-binary-trees/description/
次に解く問題：https://leetcode.com/problems/convert-sorted-array-to-binary-search-tree/description/

[mamo3gr]

そのまま代入してしまっても通じそうです。

Suggested change

	final mergedLeft = mergeNodes(source1?.left, source2?.left);
	final mergedRight = mergeNodes(source1?.right, source2?.right);
	parent.left = mergedLeft;
	parent.right = mergedRight;
	parent.left = mergeNodes(source1?.left, source2?.left);
	parent.right = mergeNodes(source1?.right, source2?.right);

一旦束縛したいのであれば、空行は無くても良さそうです。

Suggested change

	final mergedLeft = mergeNodes(source1?.left, source2?.left);
	final mergedRight = mergeNodes(source1?.right, source2?.right);
	parent.left = mergedLeft;
	parent.right = mergedRight;
	final mergedLeft = mergeNodes(source1?.left, source2?.left);
	final mergedRight = mergeNodes(source1?.right, source2?.right);
	parent.left = mergedLeft;
	parent.right = mergedRight;

[mamo3gr]

dart的なお作法に逆らわないのであれば、rightと対称にしたいです。

Suggested change

	if (mergedLeft != null) {
	mergeSourcesWithParent.add((source1?.left, source2?.left, mergedLeft));
	}
	if (mergedLeft != null) {
	mergeSourcesWithParent.add((
	source1?.left,
	source2?.left,
	mergedLeft,
	));
	}

[TakayaShirai]

レビューありがとうございます。

完全に同意します。おそらくフォーマッターで非対称になってしまったと思われます。

[mamo3gr]

フォーマッターのオプションで固定できそうですかね。

ちなみに、Pythonのフォーマッターであるruffでは、末尾にカンマを入れると、取り除かれずにスレッド元のようなフォーマットになりました（つまりデフォルトの設定で、意図的に対称な形にできる）。dartではどうかわかりませんがご参考まで。