質問編集履歴

4

加筆

2019/11/21 09:57

投稿

annderber
annderber

スコア98

test CHANGED
File without changes
test CHANGED
@@ -152,6 +152,6 @@
152
152
 
153
153
 
154
154
 
155
- 皆様どのような設計で作っておられるのか、ご意見いただければと思います。
155
+ 皆様どのような設計で作っておられるのか、理由も含めてご意見いただければと思います。
156
156
 
157
157
  よろしくお願いいたします。

3

修正

2019/11/21 09:57

投稿

annderber
annderber

スコア98

test CHANGED
File without changes
test CHANGED
@@ -128,7 +128,7 @@
128
128
 
129
129
 
130
130
 
131
- if($this->is_butyou_check($report) && $this->is_katyou_check($report) && $this->is_not_expired($report)) {
131
+ if($this->is_katyou_butyou_check($report) && $this->is_not_expired($report)) {
132
132
 
133
133
  return true;
134
134
 

2

修正

2019/11/21 09:29

投稿

annderber
annderber

スコア98

test CHANGED
File without changes
test CHANGED
@@ -124,11 +124,11 @@
124
124
 
125
125
  public function check_report($id) {
126
126
 
127
- $report = $this->Report->fetch($id);
127
+ $report = $this->fetch($id);
128
128
 
129
129
 
130
130
 
131
- if($this->Report->is_butyou_check($report) && $this->Report->is_not_expired($report)) {
131
+ if($this->is_butyou_check($report) && $this->is_katyou_check($report) && $this->is_not_expired($report)) {
132
132
 
133
133
  return true;
134
134
 

1

syuusei

2019/11/21 09:28

投稿

annderber
annderber

スコア98

test CHANGED
File without changes
test CHANGED
@@ -8,9 +8,7 @@
8
8
 
9
9
 
10
10
 
11
- ここでそれぞれのレコードのチェック状態を書くとき、それぞれのチェック状態をほしいので、
12
-
13
- それぞれのチェック処理を作って、さらにその両方ともチェックされているかの処理も書くと
11
+ ここでレコードのチェック状態を書くとき、それぞれのチェック状態をほしいのでチェック処理を作って、さらにその両方ともチェックされているかの処理も書くと
14
12
 
15
13
  チェックロジックが分散してしまう気がしてしまいます。
16
14