Solution

[OleksiukStepan]

Added and implemented:

1. Added the Cell class for filling empty cells and as a parent for Deck.
2. Added a special attribute to the Deck class to customize its symbol on the field ("□")
3. class Ship methods:

• get_deck (Return the deck located at the coordinates)
• fire (Match Deck as damaged ("*") or drowned ("x") if all ship Decks are damaged)
• if_all_deck_is_damaged (Verifies if all decks are damaged)
• create_decs (Create Decks based on ship length)
• create_ship (Instantiate Ship)

4. class Battleship methods:

• getter, setter ship (Also for deploying ships to the field)
• fire (Return information about the shot. Also can display field changes)
• add_ships_to_the_field (Placing ships on the field based on their coordinates)
• print_field (Printing field to the console)
• get_symbol_from_cell (Retrieving symbol ("□", "~", "*", "x") from the cell)
• create_empty_field (Create a field with the required coordinates and fill it with Cells ("~"))

[MNDonut]

use cls instead of Ship, it's the purpose of classmethod

[MNDonut]

no need for else statement because if statement has return

[MNDonut]

a bit don't get it. In case deck is not found, get_deck returns None, but you set deck.is_alive = False anyway. Shouldn't it be missed?

[OleksiukStepan]

You're correct. The method get_deck cannot return None, as each ship consists solely of decks; thus, it will always return a deck. I've made some adjustments to ensure it functions more accurately.

[OleksiukStepan]

Small changes:

• Ship -> cls
• removed else

Updated get_deck logic:

• Now get_deck returns only the deck.
• Added a check for duplicate coordinates: if the deck is already hit, print a message.