How to become an author
Search
Write a publication
Pull to refresh

Comments 35

А где же отечественные соц-сети? В пролёте?
Total votes 4: ↑2 and ↓20
меняем фейсбук на вконтакте — получаем русскую версию
Total votes 9: ↑8 and ↓1+7
Отечественные социальные сети — это «вконтакте», а «впролёте» — это буржуйский стартап-аналог! Ж:-)
Total votes 12: ↑8 and ↓4+4
Хреновый код, на php не пошло. Шутка, конечно же на java запускать надо.
Total votes 7: ↑5 and ↓2+3
Хреновая шутка, этот код можно запустить и на РНР
Total votes 6: ↑3 and ↓30
я ошибаюсь, или для PHP там должно быть $message, а не message?
Total votes 4: ↑1 and ↓3-2
message === «message» + Notice: Use of undefined constant
А код не выполнится(на php 5) из-за ключевого слова public — GooglePlus.PUBLIC
Total votes 2: ↑1 and ↓10
можно оформить константой
Total votes 2: ↑2 and ↓0+2
это правильный ответ
Total votes 2: ↑2 and ↓0+2
И на Java не надо запускать, потому что тип переменной circles нигде объявлен не был.
Это называется псевдокод.
Total votes 2: ↑2 and ↓0+2
UFO just landed and posted this here
В Adobe не на одном языке пишут.
Да и когда я последний раз писал на AS там тоже надо было объявлять переменную перед использованием.
Total votes 1: ↑1 and ↓0+1
UFO just landed and posted this here
С учетом «неполноты» он будет валиден еще на нескольких языках.
Total votes 3: ↑3 and ↓0+3
Там не хватает пары оберток над API соц сетей изкоробки для запуска этого кода))
Total votes 2: ↑2 and ↓0+2
Думаю тому кто это напишет — присудят Нобелевскую премию.
Total votes 7: ↑2 and ↓5-3
Да не, там в принципе не так уж и сложно, даже ненативные классы можно сделать =) Будет что то вроде:

GooglePlus googlePlus = new GooglePlus('username', 'password');
Facebook facebook = new Facebook('username', 'password');
Twitter twitter = new Twitter('username', 'password');

Только вот в твиттере могут быть проблемы с авторизацией, там кажется по токену авторизировались приложения)
Total votes 3: ↑3 and ↓0+3
if / elseif избыточно построен. нужно упростить
Total votes 7: ↑6 and ↓1+5
Я конечно не уверен что именно хотел сказать PQR, но думаю проблема в том что message.isPublic проверяется дважды.

Например, я бы написал следующим образом:
if (message.isPublic) {
  if (message.length <= 140) {
    Twitter.tweet(message.text);
  } else if (!intersection(message.recipients, Contacts.facebookOnlyUsers).isEmpty) {
    FaceBook.setStatus(message.text);
  } else {
    GooglePlus.share(message.text, GooglePlus.PUBLIC);
  }
} else {
  circles = GooglePlusUtilities.computeRelevantCircles(message.recipients);
  GooglePlus.share(message.text, circles);
}
Total votes 12: ↑11 and ↓1+10
12 строк было, 12 строк осталось. чем вы упростили?
Total votes 6: ↑0 and ↓6-6
Он упростил условия и оптимизировал читаемость кода — один из важнейших принципов программирования.
Total votes 8: ↑7 and ↓1+6
на вкус и цвет все фломастеры разные.
Total votes 7: ↑0 and ↓7-7
Да, именно это имел ввиду
Total votes 1: ↑1 and ↓0+1
Честно — я не въехал. Мне нужно пойти в лес плакать?
Total votes 13: ↑11 and ↓2+9
Я бы вот так оптимизировал:

if (message.isPublic) {
GooglePlus.share(message.text, GooglePlus.PUBLIC);
} else {
circles = GooglePlusUtilities.computeRelevantCircles(message.recipients);
GooglePlus.share(message.text, circles);
}

Кажется суть в этом.
Total votes 3: ↑1 and ↓2-1
ГуглПлюс очень красиво раскачивается, тонкий пиар от Гугла, много авторитетных личностей уже ведут там странички и делятся впечатлениями. Такими темпами Фейсбук отойдет в сторону на перекур.
Total votes 2: ↑1 and ↓10
Sign up to leave a comment.

Articles

Show the best of all time
Change theme settings

Your account

Sections

Information

Services

Support
© 2006–2024, Habr