Ошибка при запуске проекта в xCode 5 после подключения MessageUI?

Question

[mix2000]

Обучаюсь разработке под iOs по книге. Делал проект, в какой-то момент понадобилось подключить MessageUI и проект перестал запускаться. Собирается успешно, но не запускается. MessageUI отключаю и опять все хорошо.

При запуске такая ошибка:
2013-11-19 23:34:10.464 UIPickerViewExample[28419:70b] *** Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[ setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key myPicker.

В чем может быть дело?

Проект можно посмотреть здесь: https://github.com/pchelnikov/UIPickerViewCrashApp

Answer 1

[Ан]

Вы не поверите!
Банальное переименовывание ViewController'a с MPViewController в, к примеру, BBViewController (контекстное меню на объявлении класса, Refactoring, Rename)
решает проблему. Почему? не знаю?
Доброжо пожаловать в суровый мир objective-c

Comments

[Ан]

Большая просьба проверить мое решение. Если это так, то тогда проблема в недрах MessageUI. Скорей всего там есть такой класс MPViewController, и оно конфликтует.

[mix2000]

Я проверю, но вечером. Отпишу.

[Ан]

ок, с нетерпением жду. MPViewController существует в private API, но в другом Framework'e http://searchcode.com/codesearch/view/7523104

[mix2000]

Да, я тоже нашел эту ссылку. Правда там упоминается iOs 3.х.

[mix2000]

@Flanker_4 подтверждаю, работает! Переименовываю назад и перестает работать. Фантастика. :-) Чем эта информация может быть полезна?

[Ан]

Ну там явно в нутри messageUI неонка:), а если точнее, то объявлен класс MPViewController, с точно таким же названием как и у Вас. При загрузке сторибоарда, в рантайме берется не тот класс (с messageUI), у которого нет свойства myPicker - получаем падение. Но все выше сказанное - всего лишь предположение. Мораль проста, использовать уникальные имена классов (поменьше банальных ViewController) и не менее уникальные префиксы для классов. Можно переходить на 3х буквенные. плата за отсутсвие пространства имен :)

[Ан]

Просьба к Вам, оставить проект в текущем виде на гит-хаб (ну или перезалить в другой репозиторий и опубликовать тут), что бы можно было продемонстрировать проблему

[mix2000]

@Flanker_4 проект я оставил там, переименовал его и добавил ридми. Кстати, когда я переписал проект с использованием программных контролов, я как раз использовал трехбуквенный префикс (не знаю почему решил так сделать :-)), поэтому все прошло гладко.

Answer 2

[gooddy]

У тебя в storyboard висит привязка Picker к свойству, которое ты закомментил в контроллере, следовательно при "распаковке" контроллера оттуда и попытке присвоить этой переменной этот объект пикера, ты получаешь эту ошибку. Вот единственное странно, при каких обстоятельствах это вылазит. В общем правило такое, что никаких левых/мёртвых связей не должно торчать в storyboard/xib. Вот в правой панельке myPicker просто удали http://d.pr/i/7TVD

Comments

[mix2000]

Да вроде бы нет мертвых связей. Что имеешь ввиду закомментировано?

[gooddy]

//IBOutlet UIPickerView *myPicker;

[Alexander]

вот поэтому я и не пользуюсь interface builder'ом

[gooddy]

@alexyat ты просто не умеешь его готовить =)

[Alexander]

@gooddy надо как-нибудь написать статью на хабр, почему не стоит использовать IB. Как только появится время обязательно займусь этим.

[gooddy]

@alexyat так никто и не говорит, что их надо или не надо юзать, всему своё место и время, просто надо знать как этим пользоваться. А статья будет в стиле "почему не стоит использовать Singleton". Так что лучше название подобрать другое аля - бока вылзающие при использовании storyboard/xib =)

[mix2000]

@alexyat можно в двух словах, почему не стоит использовать IB? В этой книге как раз говориться о том, что конечно же

[mix2000]

ой, ...что конечно же можно не использовать IB, но лexit использовать, потому что это удобно. :-)

[mix2000]

@gooddy вечером проверю все еще раз.

[Alexander]

@mix2000 ооо в 2х словах не получится... начиная от простого дискомфорта, когда ждешь 1-2 секунды, когда откроется его окошко переходя из окна с кодом, или вот такие приколы как у вас, со связывание, бывает поставишь вью и забудешь связать... напишешь функцию которая с ним(вьювом) работает, а она нифига не работает, и все из-за ссылки... + размер приложений увеличивается очень быстро, увеличивается загрузка каждого вьюв-контрола из-за того что нужно загрузить и распарсить xib ... ну и еще куча всяких мелочей.

[mix2000]

@alexyat а объем кода в обратном случае? Как с этим быть?

[Alexander]

@mix2000 это удобно пока у вас проект из 3х экранов в каждом из которых до 10 вьювов, мне как-то попался проект на доработку, детская рисовалка и так были кнопки с цветами, их нужно было совсем по другому расставить.... 12 кнопок... это был реальный гемор, а в коде я бы просто поменял бы цикл который их выстраивает за пару минут.

[Alexander]

@mix2000 переиспользовать код, разновидностей кнопок в одном проекте 1-3 вот и делаете класс у которого статические методы строят кнопки, по нескольким параметрам... CGRect и допустим title и просто кладете их сразу на нужный вам вью и везде вызов в 1 строку этой статической функции Например вот так: <code lang="objectivec"> +(UIButton*)greenButtonWithObject:(id)object andRect:(CGRect)rect fontSize:(float)size { UIButton *button = [UIButton buttonWithType:UIButtonTypeCustom]; UIImage *stretchBG = [[UIImage imageNamed:@"greenbutton.png"] stretchableImageWithLeftCapWidth:10.0 topCapHeight:10.0]; button.frame = rect; if([object isKindOfClass:[UIImage class]]) [button setImage:object forState:UIControlStateNormal]; else { [button setTitleColor:[UIColor whiteColor] forState:UIControlStateNormal]; [button.titleLabel setFont:[UIFont fontWithName:@"HelveticaNeue-Bold" size:size]]; [button setTitle:object forState:UIControlStateNormal]; } [button setBackgroundImage:stretchBG forState:UIControlStateNormal]; return button; } </code>

[Ан]

@alexyat "вот поэтому я и не пользуюсь interface builder'ом" "Как только появится время обязательно займусь этим." Вот по этому и нет времени:) А если серьезно, использование IB существенно ускоряет разработку (1) , как не крути , рисовать формочку быстрее, чем ее же кодить. И субьективно, в чужом xib'e(сделаном другим программистом) на порядок проще разобраться чем в чужом коде(2). Особенно если речь идет о frame'ах или autolayout'e. насчет генерации кнопочкек, никто не запрещает генерировать некоторые элементы вьюшки в коде, комбинируя оба подхода. Единственный плюс - это скорость загрузки. Иногда это критично.

[Alexander]

@Flanker_4 ускорение разработки? серьезно? я не спорю что для простых экранов можно использовать, там где 1-2 лабела и табличка. Но вот скажите что выбудете делать если у вас по 15 экранам разбросаны стилистически одинаковые кнопки, отличающиеся только размером CGRect'а, и вот заказчик решил их немного поменять.... и хорошо если это будет только что-то одно - например картинка, а если еще и шрифт, и выравнивание текста, вот что делать - будете мучаться, а еще он завтра попросит еще раз? я в своих проектах поменяю 1 статический метод - это меньше минуты.

[Ан]

Ну так а кто мешает переопределить awakeFromNib или сделать outlet collection(ну или просто outlet) и применить точно так же требуемый единый стиль к кнопкам? Вы так говорите, как будто использование xib. Но при этом не будет проблем с расположением кнопки на своей позиции+сразу же, не отходя от кассы, можно будет расположение пэлемнов на 3,5", 4", iOS7+выставить autolayout constraince.

[Ан]

Ах да, забыл, а еще можно создать кастомный класс, наследника UIButton, и в методе initWithCoder сделать то, о чем вы говорили. И не будет никаких левых статических методов. Само собой установить в IB для нужных кнопок этот класс. Имхо, это самое правильное решение

Answer 3

[Alexander]

В комментарии тег кода не работает, поэтому напишу тут и удалить комментарий нельзя (
переиспользовать код, разновидностей кнопок в одном проекте 1-3 вот и делаете класс у которого статические методы строят кнопки, по нескольким параметрам... CGRect и допустим title и просто кладете их сразу на нужный вам вью и везде вызов в 1 строку этой статической функции Например вот так:

+(UIButton*)greenButtonWithObject:(id)object andRect:(CGRect)rect fontSize:(float)size
{
    UIButton *button = [UIButton buttonWithType:UIButtonTypeCustom];
    UIImage *stretchBG = [[UIImage imageNamed:@"greenbutton.png"]
                          stretchableImageWithLeftCapWidth:10.0 topCapHeight:10.0];
    button.frame = rect;
    if([object isKindOfClass:[UIImage class]])
        [button setImage:object forState:UIControlStateNormal];
    else
    {
        [button setTitleColor:[UIColor whiteColor] forState:UIControlStateNormal];
        [button.titleLabel setFont:[UIFont fontWithName:@"HelveticaNeue-Bold" size:size]];
        [button setTitle:object forState:UIControlStateNormal];
    }
    [button setBackgroundImage:stretchBG forState:UIControlStateNormal];
    return button;
}

Answer 4

[mix2000]

Похоже, это все же глюк xCode. Как ни пытался, оно не заработало. В какой-то момент даже перестал вызываться метод viewDidLoad.

Пообщался с товарищем, который давно занимается разработкой, говорит, что использует IB только для Core Data, потому что, говорит, тупо крэшится приложение при использовании IB. Видимо, у меня такая же ситуация. :-)

Переписал приложение с использованием программного пикера и все стало ок.