Есть ли причина, по которой мы просто не объявляем cellForRowAtIndexPath следующим образом:

Как указывает rmaddy, вы не можете использовать первую версию, потому что -[UITableView cellForRowAtIndexPath:] может привести к тому, что табличное представление снова отправит вам tableView:heightForRowAtIndexPath:, что приведет к бесконечной рекурсии.

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

Если вы динамически создаете ячейки для строк, вторая версия в конечном итоге истощит очередь повторного использования вашего табличного представления, а затем создаст новую ячейку для каждой строки, потому что tableView:cellForRowAtIndexPath: возвращает автоматически освобожденный объект. Ни одна из этих ячеек не будет освобождена до конца цикла выполнения, поэтому в дополнение к временным затратам на создание и уничтожение всех этих ячеек вы также используете память, пропорциональную количеству строк в вашей таблице. Если вы хотите сделать это таким образом и у вас много строк, вы можете использовать явный пул автоматического освобождения:

-(CGFloat)tableView:(UITableView *)tableView heightForRowAtIndexPath:(NSIndexPath *)indexPath{
    @autoreleasepool {
        UITableViewCell *cell = [self tableView:tableView cellForRowAtIndexPath:indexPath];
        return cell.bounds.size.height;
    }
}

Обычно вы хотите получить ячейку из таблицы, как в 1-м бите кода. Но в этом случае нельзя. Если вы попытаетесь, вы получите рекурсивный вызов между cellForRowAtIndexPath и heightForRowAtIndexPath.

Если вы должны получить ячейку из метода heightForRowAtIndexPath, вы НЕ должны запрашивать ячейку у таблицы.