Задача для первого класса, но я только учусь!! 
условие:1. 
Создайте новый класс и назовите его TemperatureConverter. 
Добавьте в него метод для преобразования температур, с такой cигнатурой: public String convertTemp(int temperature, char convertTo) 
Если аргумент convertTo равен F, то температура должна быть преобразована в Фаренгейты, если C, то в Цельсии. 
когда вы будете вызывать этот метод, поместите значение аргумента 
типа char в одинарные кавычки. 


Плз покритикуйте !!!! Что так а что не так , я просто хочу с самого начала писать более менее грамотный код=) 
P S Всем спасибо!

Сори не тот код : 

что-то я особой разницы в коде не заметил, хотя не присматривался  .
ИМХО, написано норм, только можно бы вместо банального

сгенерировать какой-нибудь эксэпшен.

1) не помешали бы скобки... если код в case вырастет, будет сложно читать.
2) convertTo можно изначально делать toLowerCase и писать только одну проверку.

блин, не смог удержаться...

V0R0N91, ну что за издевательство?! 
нормальный код ты списал откуда то...
и что?...

Ну чтобы не палиться, что вы только начали

http://www.oracle.com/technetwork/java/jav...141855.html#277

А к чему это?

выбрасывай IllegalArgumentException в случае кривого символа, Сharacter.toUpperCase() перед свитчом
опять же странный какой то метод - принимает целочисленную температуру - а возвращает строковую
public int convertTemp(int temperature, char convertTo) - было бы логичнее для метода с названием конверт,
также я бы зафигачил внутренний енум - но это моя слабость - дело то хозяйское) 

V0R0N91, 
Ну, если уж учиться ...

Что сделал бы я:
1. Метод "должен" быть static final. 

2. Название пакета не должно совпадать с названием класса. Ну некрасиво это. И, вообще, посмотрите как принято называть пакеты.

3.  char convertTo некрасиво ужасно, лучше ENUM, но хотя бы делайте перед сравнением приведение к одному регистру.

4.  Метод должен преобразовывать, а не печатать, или что-то там округлять. 
Поэтому возвращайте double, пользователь сам решит, что с этим делать.

5. System.out.println("Error"); - еще раз, дело метода преобразовывать, а не печатать, если преобразовать не получается, то
надо выбросить ошибку, а не писать в консоль "караул".

6. Где описание метода? 
Как по мне - уже одно это говорит о неграмотном коде.
Может название метода и самодокументированное, но вот что делает, например, char convertTo ? понимает только автор метода и то только первую неделю ...
А еще, например, как мне догадаться, что метод не проверяет переданные числа на допустимость и может мне вернуть -456С?
Интуитивно?

7. 0K = −273.15 °C, а не -273!

8.  "Если аргумент convertTo равен F, то температура должна быть преобразована в Фаренгейты, если C, то в Цельсии."

Какая прелесть! Т.е. во что температура должна быть преобразована догадались, а из чего нет?
Т.е. сижу это я в интернет, читаю книжку, ну того же Дюма, а он там, вот сюрприз-то температуру в Реомюрах дает.
Ввожу я ее в ваш калькулятор и получаю "хрень разноцветная 1 штука"! 

Кстати, сюрприз! - Вики знает еще минимум 4 шкалы : Ранкин    Делиль    Ньютон    Рёмер!
Что это за штуки я, кстати, сам не знаю, но есть же 

Ага, я уже догадался, как хочется сказать, что для учебной задачи это не важно, так обломитесь!  даже для учебной задачи это самое важное, надо " с пеленок" понимать, что писать хрень нельзя!
Медленный код можно, не очень грамотный технически можно, не документированный можно, а вот имеющий на выходе "хрень зеленая 1 штука" нельзя!

9. Вообще-то, в нашем учебном примере хорошо бы перейти к замене switch на полиморфизм, интерфейсы, фабрики и т.п. (всякие там паттерны типа стратегии).
Т.е. есть куда расти с этим же самым примером от 1 до 10 класса! 

Мне вот подумалось, что будет, если я выложу свой любимый из текущего проекта класс, который занимает, прости господи, 6000 строк, если на такую милую задачку 9 пунктов нашлось  

Можешь ничего не выкладывать, 6000 строк это уже приговор - все переписать.  

Дааа, больше стараюсь такую абракадабру не писать) Остальное пока за 600 строк не вылазило   Ну кто ж мне, нубу, сказал, что не стоит так делать... Теперь, когда у меня 30+ классов развелось, задумалась еще о пользе пакетов) А еще книжку почитала "идеальный код", и вообще все захотелось выкинуть и переписать. А нельзя 

Добавлено через 3 минуты и 3 секунды
Пересчитала. 50+ классов  Пакеты определенно полезны(

афторы книжек разных


во-во. только она не единственная. нужна архитектура на всех уровнях: приложение, модуль, класс, функция. тогда программа будет короче, понятнее, и, возможно, быстрее.

Ну вот, наблюдаем ООП головного мозга.

почему не

?

+1