Poprawność kodu - prośba o pomoc

0

Drodzy developerzy,

piszę do Was jako osoba nie mająca wiele wspólnego z programowaniem jako umiejętnością techniczną. W pracy zawodowej zajmuję się co najwyżej wyszukiwaniem talentów umiejących programować :)

Chciałabym sprawić osobie kodującej prezent - kubek na kawę z nadrukiem w c#. Proszę o sprawdzenie poprawności poniższego kodu. Nie chiałabym się ośmieszyć, wykonując nadruk, który jest niepoprawny. Wiem, że nadruk powinien zawierać odpowiednią czcionkę i kolory, które by imitowały platformę. Czy ktoś mógłby pomóc?

Coffee coffee = new Coffee();

if (coffee.Empty)
{
coffee.Refill();
}
else
{
coffee.Drink();
}

Będę wdzięczna za pomoc,
K.

PS. Jeśli temat nie pasuje to kategorii forum, proszę o pouczenie i przeniesienie go do bardziej odpowiedniego miejsca.

1

Jest poprawny, chociaż ja bym zamienił coffee.Empty na coffee.IsEmpty
Kolorowanie według Visual Studio, czcionka Consolas, rozmiar 10:

coffee.PNG

2

wiecie ze jest cos takiego jak znaczniki code?

if (coffee.Empty)
{
    coffee.Refill();
}
else
{
    coffee.Drink();
} 

i w razie w to jeszcze to
http://hilite.me/

2

Ten kod jest bez sensu. Kawa nie może się sama pić, ani napełniać. I co to w ogóle za "new"? Obiekty tworzy się przez fabrykę, albo przez kontener IoC.

Trzeba iść w tę stronę, tylko oczywiście dodać obsługę błędów:

Coffee coffee = _coffeeFactory.CreateNew(CoffeType.Espresso);
if (coffee.IsEmpty)
{
    var request = new CoffeRefillRequest(coffee, _userProvider.CurrentUser);
    _cofeeRefillService.Process(request);
}
else
{
    _userProvider.CurrentUser.Drink(coffee);
}
1

Jakbym zmienił "coffee" na "cup", kubek kawy brzmi fajnie po polsku i po angielsku i żaden złośliwy programista nie przyczepi się nie można go wypić ;)

 
Coffee cup = new Coffee();

if (cup.IsEmpty)
{
    cup.Refill();
}
else
{
    cup.Drink();
}

Albo dodał "whie(true)" jako taki żarcik :P

 
Coffee cup = new Coffee();

while(true)
{
     if (cup.IsEmpty)
     {
         cup.Refill();
     }
     else
     {
         cup.Drink();
     }
}

1 użytkowników online, w tym zalogowanych: 0, gości: 1