Уведомления

Группа в Telegram: @pythonsu

#1 Сен. 18, 2015 08:07:33

Shaman
Зарегистрирован: 2013-03-15
Сообщения: 1369
Репутация: +  88  -
Профиль   Отправить e-mail  

Есть рабочий код, нужна критика и дельные советы опытных.

Извращенцы… Наверное стоило или опубликовать код, или обсуждать его в привате.

Офлайн

#2 Сен. 18, 2015 08:16:24

J.R.
Зарегистрирован: 2015-07-18
Сообщения: 147
Репутация: +  16  -
Профиль  

Есть рабочий код, нужна критика и дельные советы опытных.

.

Отредактировано J.R. (Янв. 27, 2016 08:50:13)

Офлайн

#3 Сен. 18, 2015 09:12:12

J.R.
Зарегистрирован: 2015-07-18
Сообщения: 147
Репутация: +  16  -
Профиль  

Есть рабочий код, нужна критика и дельные советы опытных.

.

Отредактировано J.R. (Янв. 27, 2016 08:50:32)

Офлайн

#4 Сен. 18, 2015 10:31:11

py.user.next
От:
Зарегистрирован: 2010-04-29
Сообщения: 9895
Репутация: +  855  -
Профиль   Отправить e-mail  

Есть рабочий код, нужна критика и дельные советы опытных.

Shaman
Наверное стоило или опубликовать код, или обсуждать его в привате.
Не, надо привыкать работать по-нормальному.

J.R.
Если не против, я ссылку на гист опубликую. А то правда нехорошо выходит.
Да можешь публиковать, главное, чтобы каши-малаши не было. Участвовать-то могут многие.

J.R.
TypeError: if you give only one argument to maketrans it must be a dict
Я отменил там. Это я перепутал с list. Ты тогда там сделал list для передачи в dict, тогда как его делать не надо, потому что dict() берёт любой итерабл.

Вот теперь, прежде чем выгружать свои изменения, загружаешь мои через fetch и смотришь, изменилось ли что-нибудь. Если изменилось, свои припрятываешь (stash), мои вливаешь и свои вытаскиваешь (stash apply).

Там, кстати, ветки ещё есть, через github их не видно, но их можно делать и выгружать. Это если идея какая-нибудь возникла, в ветке её делаешь и выгружаешь для оценки, а там можно и влить её, если нормальная, либо просто хорошие комиты выдернуть через черипик.

Второе, что надо рассказать: у тебя слишком много изменений в одном коммите.
Надо делать изменения мелкими, чтобы их можно было отменять либо перебрасывать из ветки в ветку.
Так что если делаешь много изменений, то разделяй их на коммиты путём
git add -p
по такому принципу, когда каждое изменение - это самостоятельный элемент. (Можно добавить, можно убрать, но программа остаётся рабочей.)


J.R.
Вот смотри, склонировал этот гист к себе. Теперь буду с этим колоном работать.
По поводу организации:
У тебя один серверный репозиторий, один локальный репозиторий.
У меня один серверный репозиторий, один локальный репозиторий.
Когда ты хочешь сделать изменения, втягиваешь в свой локальный коммиты из моего серверного, а потом выталкиваешь мои коммиты и свои изменения в свой серверный.
Когда я хочу сделать изменения, втягиваю в свой локальный коммиты из твоего серверного, а потом выталкиваю твои коммиты и свои изменения в свой серверный.



Отредактировано py.user.next (Сен. 18, 2015 10:39:01)

Офлайн

#5 Сен. 18, 2015 10:36:46

Shaman
Зарегистрирован: 2013-03-15
Сообщения: 1369
Репутация: +  88  -
Профиль   Отправить e-mail  

Есть рабочий код, нужна критика и дельные советы опытных.

https://github.com/mrprint
Чуть-чуть возни с гитхабом и visualstudio. Можно критиковать!

Офлайн

#6 Сен. 18, 2015 11:10:04

J.R.
Зарегистрирован: 2015-07-18
Сообщения: 147
Репутация: +  16  -
Профиль  

Есть рабочий код, нужна критика и дельные советы опытных.

.

Отредактировано J.R. (Янв. 27, 2016 08:50:50)

Офлайн

#7 Сен. 18, 2015 12:35:08

py.user.next
От:
Зарегистрирован: 2010-04-29
Сообщения: 9895
Репутация: +  855  -
Профиль   Отправить e-mail  

Есть рабочий код, нужна критика и дельные советы опытных.

J.R.
Вот смотри, я сделал git fetch… а где и как смотреть, что изменилось у тебя?
Если она загрузила что-нибудь, то это находится в удалённой (remote) ветке.
git branch -a
Обращение к удалённой ветке делается через источник/ветка.
К примеру, чтобы влить удалённую ветку в текущую:
git merge fork/master
где fork - имя источника, а master - ветка мастер в том источнике.

А так можно посмотреть, что входит в правую ветку, но не входит в левую:
git log fork/master..origin/master
У меня сейчас моя мастер ветка содержит три коммита, которых нет в твоей мастер ветке, поэтому для просмотра этих коммитов я твою ставлю слева, а свою - справа.

[guest@localhost cuetool-concept-my]$ git log fork/master..origin/master
commit 714baac47bcc8d2c0a330c7cf010c04d166a2828
Author: guest <guest@localhost>
Date: Fri Sep 18 17:58:30 2015 +1000

Make static methods dynamic

Use static method when you are going to use it
without object.

>>> class A:
... @staticmethod
... def f(arg):
... print(arg)
...
>>> a = A()
>>> a.f(1)
1
>>> A.f(2)
2
>>>

In other cases use a simple dynamic method.

commit abaee69506c00ed511ba402430373af4305a676c
Merge: 4f5064b 0f58dc2
Author: guest <guest@localhost>
Date: Fri Sep 18 17:16:59 2015 +1000

Merge remote-tracking branch 'fork/master'

commit 4f5064b6ce0c601d9517889fcb86764522970eb6
Author: guest <guest@localhost>
Date: Fri Sep 18 17:16:01 2015 +1000

Cancel c3ea852
(END)

J.R.
Как мне теперь отменить?
Ты просто вливаешь, я отменил - это значит, что я исправил обратно и закоммитил это исправление.

J.R.
С гитом вопросов больше чем с Питоном…
Это с непривычки, но изучить его надо, есть одна книга бесплатная, её прочитаешь - всё будешь знать, а дальше там только хелп останется читать.

J.R.
По коду замечания есть?
Ну, я там закоммитил уже одну вещь. :)
Сейчас разберёшься, вольёшь и посмотришь.



Отредактировано py.user.next (Сен. 18, 2015 12:39:58)

Офлайн

#8 Сен. 18, 2015 12:47:01

4kpt_III
Зарегистрирован: 2014-12-22
Сообщения: 999
Репутация: +  39  -
Профиль   Отправить e-mail  

Есть рабочий код, нужна критика и дельные советы опытных.

1. awersome_slugify.
2. docstring.

Без 2 даже читать не стал

Офлайн

#9 Сен. 18, 2015 13:15:00

J.R.
Зарегистрирован: 2015-07-18
Сообщения: 147
Репутация: +  16  -
Профиль  

Есть рабочий код, нужна критика и дельные советы опытных.

.

Отредактировано J.R. (Янв. 27, 2016 08:51:08)

Офлайн

#10 Сен. 18, 2015 13:18:46

J.R.
Зарегистрирован: 2015-07-18
Сообщения: 147
Репутация: +  16  -
Профиль  

Есть рабочий код, нужна критика и дельные советы опытных.

.

Отредактировано J.R. (Янв. 27, 2016 08:51:21)

Офлайн

Board footer

Модераторировать

Powered by DjangoBB

Lo-Fi Version