comparison OrthancStone/Sources/Toolbox/DicomInstanceParameters.cpp @ 1817:385c268e8b56

cppcheck
author Sebastien Jodogne <s.jodogne@gmail.com>
date Tue, 25 May 2021 19:51:45 +0200
parents 6c246f862b00
children 36430d73e36c
comparison
equal deleted inserted replaced
1816:dccdc7e59929 1817:385c268e8b56
145 if (!sliceThicknessPresent) 145 if (!sliceThicknessPresent)
146 { 146 {
147 if (frameOffsets_.size() >= 2) 147 if (frameOffsets_.size() >= 2)
148 { 148 {
149 double sliceThickness = frameOffsets_[1] - frameOffsets_[0]; 149 double sliceThickness = frameOffsets_[1] - frameOffsets_[0];
150 bool sameSized = true; 150
151 if (sliceThickness > 0) 151 if (sliceThickness > 0)
152 { 152 {
153 bool sameSized = true;
154
153 for (size_t i = 2; i < frameOffsets_.size(); ++i) 155 for (size_t i = 2; i < frameOffsets_.size(); ++i)
154 { 156 {
155 double currentThickness = frameOffsets_[i] - frameOffsets_[i-1]; 157 double currentThickness = frameOffsets_[i] - frameOffsets_[i-1];
156 if (!LinearAlgebra::IsNear(sliceThickness, currentThickness)) 158 if (!LinearAlgebra::IsNear(sliceThickness, currentThickness))
157 { 159 {
158 LOG(ERROR) << "Unable to extract slice thickness from GridFrameOffsetVector (3004,000C) (reason: varying spacing)"; 160 LOG(ERROR) << "Unable to extract slice thickness from GridFrameOffsetVector (3004,000C) (reason: varying spacing)";
159 sameSized = false; 161 sameSized = false;
160 break; 162 break;
161 } 163 }
162 } 164 }
165
163 if (sameSized) 166 if (sameSized)
164 { 167 {
165 sliceThickness_ = sliceThickness; 168 sliceThickness_ = sliceThickness;
166 LOG(INFO) << "SliceThickness was not specified in the Dicom but was inferred from GridFrameOffsetVector (3004,000C)."; 169 LOG(INFO) << "SliceThickness was not specified in the Dicom but was inferred from GridFrameOffsetVector (3004,000C).";
167 } 170 }