Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop #142

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Develop #142

wants to merge 4 commits into from

Conversation

wndzlf
Copy link
Contributor

@wndzlf wndzlf commented Apr 10, 2019

No description provided.

@@ -703,6 +605,30 @@ extension ItemViewController: UICollectionViewDelegate, UICollectionViewDataSour
navigationController?.pushViewController(SeeMoreRestVC, animated: true)
}
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing Whitespace Violation: Lines should not have trailing whitespace. (trailing_whitespace)


func sendDataToStoreVC(tableViewSection: TableViewSection, indexPath: IndexPath) {

guard let storeViewController = UIStoryboard.main.instantiateViewController(withIdentifier: "CollectionViewController")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line Length Violation: Line should be 120 characters or less: currently 127 characters (line_length)

return .init()
}

private func setTableViewSectionCell(_ tableviewSection: TableViewSection, indexPath: IndexPath) -> UITableViewCell {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cyclomatic Complexity Violation: Function should have complexity 10 or less: currently complexity equals 14 (cyclomatic_complexity)
Function Body Length Violation: Function body should span 40 lines or less excluding comments and whitespace: currently spans 55 lines (function_body_length)
Line Length Violation: Line should be 120 characters or less: currently 121 characters (line_length)

return UIEdgeInsets(top: 30, left: 30, bottom: 30, right: 30)
}
}
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

File Line Length Violation: File should contain 400 lines or less: currently contains 412 (file_length)

}
}

func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, insetForSectionAt section: Int) -> UIEdgeInsets {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line Length Violation: Line should be 120 characters or less: currently 162 characters (line_length)

tablecell.collectionView.reloadData()
}

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function Body Length Violation: Function body should span 40 lines or less excluding comments and whitespace: currently spans 47 lines (function_body_length)

@IBOutlet var scrollView: UIScrollView!
private var images: [String] = ["1_1", "2_1", "3_1", "4_1", "5_1", "6_1"]
private var frame = CGRect(x: 0, y: 0, width: 0, height: 0)
private var labelString: [String] = ["추천요리", "가까운 인기 레스토랑", "예상 시간 30분 이하", "Uber Eats 신규 레스토랑", "주문시 5천원 할인 받기", "가나다라", "마바사", "아자차카", "타파하", "아아아앙아", "집에", "가고", "싶다"]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line Length Violation: Line should be 120 characters or less: currently 174 characters (line_length)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants